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@xxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review