Comment 13 for bug 242043

Revision history for this message
TJ (tj) wrote : Re: [Bug 242043] Re: [needs-packaging} read-edid-2.0.0 for amd64

On Wed, 2009-03-18 at 06:40 +0000, Iulian Udrea wrote:
> Are these rdepends affected by this upgrade?
>
> Reverse Depends:
> xdebconfigurator
> lm-sensors
>
> It looks like xdebconfigurator is not using read-edid by default. It
> just suggests read-edid now.

That's correct, and it detects the presence of the read-edid tools
using:

xdebconfigurator:74:chomp($getedid = `which get-edid`); # read-edid exists?

I couldn't find any usage of read-edid's executables (get-edid or
parse-edid) in lm-sensors. The only occurrence of the string "edid" at
all...

pwd

/home/all/SourceCode/lm-sensors-3-3.0.2

grep -rni edid *

debian/control:14:Suggests: sensord, read-edid, i2c-tools
debian/changelog:776: * Suggests: read-edid (closes: bug#265890).
prog/detect/sensors-detect:1212: # for SPD and EDID EEPROMs because some hardware monitoring chips
prog/detect/sensors-detect:1218: name => "EDID EEPROM",

...is in the Debian changelog where it says:

debian/changelog:776: * Suggests: read-edid (closes: bug#265890)

Looking at that Debian bug

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=265890

it seems as if the only reason for the Suggests was because:

"...decode-edid.pl is not present in the package..."

The output of get-edid | parse-edid is the same as it ever was so
packages relying on the output won't be affected. The only change is a
gain in the availability of the tools on 64-bit installations.