Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 143374

Summary: ivtv-0.7.0.ebuild has too many requirements on kernel config
Product: Gentoo Linux Reporter: Sybren Harmsma <cpwins>
Component: New packagesAssignee: Gentoo Linux bug wranglers <bug-wranglers>
Status: RESOLVED DUPLICATE    
Severity: minor    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: ivtv-0.7.0-r1.ebuild

Description Sybren Harmsma 2006-08-09 12:07:51 UTC
From the ivtv-0.7.0.ebuild:

if kernel_is 2 6 17; then
  CONFIG_CHECK="EXPERIMENTAL VIDEO_DEV I2C VIDEO_V4L1 VIDEO_V4L2 FW_LOADER"
  CONFIG_CHECK="${CONFIG_CHECK} VIDEO_WM8775 VIDEO_MSP3400 VIDEO_CX25840 VIDEO_TUNER"
  CONFIG_CHECK="${CONFIG_CHECK} VIDEO_SAA711X VIDEO_SAA7127 VIDEO_TVEEPROM"
else
  die "This only works on 2.6.17 kernels"
fi

The requirement to have VIDEO_V4L1 and VIDEO_SAA711X are not necessary I think. I compiled a kernel with these options and all works fine:

# CONFIG_VIDEO_V4L1 is not set
# CONFIG_VIDEO_V4L1_COMPAT is not set
CONFIG_VIDEO_V4L2=y

Even more, the stable gentoo-sources-2.6.17-r4 doesn't have a VIDEO_SAA711X option at all:

less .config | grep SAA
# CONFIG_VIDEO_SAA6588 is not set
# CONFIG_VIDEO_SAA5249 is not set
# CONFIG_VIDEO_SAA7134 is not set
CONFIG_VIDEO_SAA7127=m

BTW I'm using a Hauppauge PVR-500 (dual tuner). Unless other hardware specifically requires these two options, I suggest dropping them from the ebuild. For your convenience, I have attached a ivtv-0.7.0-r1.ebuild, which I would like other users to test (i.e. ~x86)
Comment 1 Sybren Harmsma 2006-08-09 12:10:48 UTC
Created attachment 93859 [details]
ivtv-0.7.0-r1.ebuild

diff ivtv-0.7.0.ebuild ivtv-0.7.0-r1.ebuild

        if kernel_is 2 6 17; then
-               CONFIG_CHECK="EXPERIMENTAL VIDEO_DEV I2C VIDEO_V4L1 VIDEO_V4L2 FW_LOADER"
+               CONFIG_CHECK="EXPERIMENTAL VIDEO_DEV I2C VIDEO_V4L2 FW_LOADER"
                CONFIG_CHECK="${CONFIG_CHECK} VIDEO_WM8775 VIDEO_MSP3400 VIDEO_CX25840 VIDEO_TUNER"
-               CONFIG_CHECK="${CONFIG_CHECK} VIDEO_SAA711X VIDEO_SAA7127 VIDEO_TVEEPROM"
+               CONFIG_CHECK="${CONFIG_CHECK} VIDEO_SAA7127 VIDEO_TVEEPROM"
        else
                die "This only works on 2.6.17 kernels"
        fi
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2006-08-09 12:12:41 UTC
(In reply to comment #0)
> The requirement to have VIDEO_V4L1 and VIDEO_SAA711X are not necessary I think.
> Even more, the stable gentoo-sources-2.6.17-r4 doesn't have a VIDEO_SAA711X
> option at all:

Err, that's exactly because you are missing VIDEO_V4L1.


*** This bug has been marked as a duplicate of 141232 ***