[Bug 437691] Review Request: monitor-edid - Tool for probing and parsing EDID

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: monitor-edid - Tool for probing and parsing EDID


https://bugzilla.redhat.com/show_bug.cgi?id=437691





------- Additional Comments From ville.skytta at iki.fi  2008-04-05 16:58 EST -------
Created an attachment (id=301389)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=301389&action=view)
Use system lrmi

Using the private lrmi is not nice, something like the attached patch could do
the trick.  I have no way to check if the result actually works on i386, but it
does build.  If you apply this, also add %ifarch %{ix86} "BuildRequires:
lrmi-devel" to the specfile and "rm -f lrmi.h lrmi.c" in %prep to make sure the
bundled lrmi is not used.

MIT should probably be dropped from licenses even if system lrmi is not used;
lrmi gets built into an LGPLv2+ executable which I think makes the combined
work LGPLv2+ and we only list licenses for binaries.

%description could be improved, suggestion:
Monitor-edid is a tool for probing and parsing Extended display identification
data (EDID) from monitors.  For more information about EDID, see
http://en.wikipedia.org/wiki/EDID

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.