Bug 249212
Summary: | Review Request: inchi - The IUPAC International Chemical Identifier library | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Dominik 'Rathann' Mierzejewski <dominik> |
Component: | Package Review | Assignee: | Warren Togami <wtogami> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, mtasaka, notting |
Target Milestone: | --- | Flags: | wtogami:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-11-08 19:01:08 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Dominik 'Rathann' Mierzejewski
2007-07-22 18:32:33 UTC
> License: LGPL Not necessary to do this in the -devel package. > If you are going to develop programs which will use this library > you should install inchi-devel. You'll also need to have the > inchi package installed. Is this part necessary? http://www.iupac.org/inchi/license.html Licensing is confusing... - It implies trademark integrity within the context of copyright terms. - It implies a "request" of copyright advertisement clause, but it seems non-binding by this language. Is this the intent? - None of the source files contain proper copyright notices. Recommendation: 1) Upstream should clear up this confusion by creating a clear separation between the copyright and trademark rights. Copyright explicitly LGPL, and have a separate page/file containing the trademark guidelines. http://www.mozilla.org/foundation/trademarks/policy.html As an example, Mozilla uses a trademark guideline to protect the integrity of their mark, without running afoul of the (L)GPL requirement of "no additional restrictions" on the copyright. 2) All source files must contain a proper and explicit copyright notice. To quote LGPL: " You must give prominent notice with each copy of the work that the Library is used in it and that the Library and its use are covered by this License." Under the terms of the LGPL, those copyright statements would of course not be removed by others. Will this reply: http://sf.net/mailarchive/message.php?msg_name=20070810112606.ZSMM17393.aamtaout02-winn.ispmail.ntl.com%40ALAN suffice to get this review a green light? Given that this package is fine split, I'd rather wait until the upstream source is fully fixed and explicitly clarified before approving this. IANAL, but upstream merely stating "go ahead" doesn't satisfy #2 above. http://sourceforge.net/mailarchive/message.php?msg_name=20070905185634.WDPK17393.aamtaout02-winn.ispmail.ntl.com%40ALAN Looks like they did exactly as we asked. Updated package here: http://rathann.fedorapeople.org/review/inchi-1.0.2-0.1.fc7.src.rpm http://rathann.fedorapeople.org/review/inchi.spec Excellent, the licensing is FAR less confusing and explicit now. Possibly a few more issues to deal with before approval: [builder1@newcaprica x86_64]$ rpmlint inchi-1.0.2-0.1.fc8.x86_64.rpm inchi.x86_64: W: wrong-file-end-of-line-encoding /usr/share/doc/inchi-1.0.2/LICENSE inchi.x86_64: W: wrong-file-end-of-line-encoding /usr/share/doc/inchi-1.0.2/readme.txt inchi.x86_64: W: dangling-relative-symlink /usr/lib64/libinchi.so.1 libinchi.so.1.01.00 inchi.x86_64: W: invalid-license LGPL inchi.x86_64: E: invalid-ldconfig-symlink /usr/lib64/libinchi.so.1.02.00 libinchi.so.1.01.00 http://fedoraproject.org/wiki/Licensing The invalid-license warning is a must fix. Please refer to the licensing guidelines to use a label that is more specific to the LGPL license and version. The symlink warning and errors are possibly suspect, please look into that. ping Fixed. http://rathann.fedorapeople.org/review/inchi-1.0.2-0.2.fc7.src.rpm http://rathann.fedorapeople.org/review/inchi.spec ping OK, everything is fixed and rpmlint is completely silent. Good job. +1 APPROVED New Package CVS Request ======================= Package Name: inchi Short Description: The IUPAC International Chemical Identifier library Owners: rathann Branches: F-8 F-7 InitialCC: Cvsextras Commits: yes Thanks for the review! cvs done. Built for devel. Builds for other branches will follow after I've updated openbabel. Package Change Request ====================== Package Name: inchi New Branches: EL-5 cvs done. |