Re: Unable to build media_build (mk II)
On Saturday, January 08, 2011 13:56:27 Daniel O'Connor wrote: > > On 08/01/2011, at 23:14, Hans Verkuil wrote: > >> Looking at some other consumers of that function it would appear the last > >> argument (NULL in this case) is superfluous, however the file appears to > >> be replaced each time I run build.sh so I can't update it. > > > > Only run build.sh once. After that you can modify files and just run 'make'. > > > > build.sh will indeed overwrite the drivers every time you run it so you > > should > > that only if you want to get the latest source code. > > Ahh, I see. > > Any chance the README could be modified to say something about that? > > Currently it doesn't mention build.sh at all - I had to google to find > anything of use. > > Perhaps also rename build.sh to setup.sh. It was news to me as well :-) Can someone with access rights to that git tree update the README and rename the script? Regards, Hans > > Thanks :) > > -- > Daniel O'Connor software and network engineer > for Genesis Software - http://www.gsoft.com.au > "The nice thing about standards is that there > are so many of them to choose from." > -- Andrew Tanenbaum > GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C > > > > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- Hans Verkuil - video4linux developer - sponsored by Cisco -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Unable to build media_build (mk II)
On 08/01/2011, at 23:14, Hans Verkuil wrote: >> Looking at some other consumers of that function it would appear the last >> argument (NULL in this case) is superfluous, however the file appears to be >> replaced each time I run build.sh so I can't update it. > > Only run build.sh once. After that you can modify files and just run 'make'. > > build.sh will indeed overwrite the drivers every time you run it so you should > that only if you want to get the latest source code. Ahh, I see. Any chance the README could be modified to say something about that? Currently it doesn't mention build.sh at all - I had to google to find anything of use. Perhaps also rename build.sh to setup.sh. Thanks :) -- Daniel O'Connor software and network engineer for Genesis Software - http://www.gsoft.com.au "The nice thing about standards is that there are so many of them to choose from." -- Andrew Tanenbaum GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Unable to build media_build (mk II)
On Saturday, January 08, 2011 13:38:25 Daniel O'Connor wrote: > Hi again :) > I am still having trouble building unfortunately, I get the following: > CC [M] /home/myth/media_build/v4l/hdpvr-video.o > CC [M] /home/myth/media_build/v4l/hdpvr-i2c.o > /home/myth/media_build/v4l/hdpvr-i2c.c: In function 'hdpvr_new_i2c_ir': > /home/myth/media_build/v4l/hdpvr-i2c.c:62: error: too many arguments to > function 'i2c_new_probed_device' > make[3]: *** [/home/myth/media_build/v4l/hdpvr-i2c.o] Error 1 > make[2]: *** [_module_/home/myth/media_build/v4l] Error 2 > make[2]: Leaving directory `/usr/src/linux-headers-2.6.32-26-generic' > make[1]: *** [default] Error 2 > make[1]: Leaving directory `/home/myth/media_build/v4l' > make: *** [all] Error 2 > *** ERROR. Aborting *** > > Looking at some other consumers of that function it would appear the last > argument (NULL in this case) is superfluous, however the file appears to be > replaced each time I run build.sh so I can't update it. Only run build.sh once. After that you can modify files and just run 'make'. build.sh will indeed overwrite the drivers every time you run it so you should that only if you want to get the latest source code. Regards, Hans > > [mythtv 23:00] ~/media_build >uname -a > Linux mythtv 2.6.32-26-generic #48-Ubuntu SMP Wed Nov 24 10:14:11 UTC 2010 > x86_64 GNU/Linux > [mythtv 23:00] ~/media_build >cat /etc/lsb-release > DISTRIB_ID=Ubuntu > DISTRIB_RELEASE=10.04 > DISTRIB_CODENAME=lucid > DISTRIB_DESCRIPTION="Ubuntu 10.04.1 LTS" -- Hans Verkuil - video4linux developer - sponsored by Cisco -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
