Bug 491425
| Summary: | Review Request: R-RODBC - An ODBC database interface for R | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tom "spot" Callaway <tcallawa> | ||||
| Component: | Package Review | Assignee: | Mattias Ellert <mattias.ellert> | ||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | rawhide | CC: | fedora-package-review, mattias.ellert, notting | ||||
| Target Milestone: | --- | Flags: | mattias.ellert:
fedora-review+
tcallawa: fedora-cvs+ |
||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | 1.2-2.fc10 | Doc Type: | Bug Fix | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2009-03-31 20:28:17 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: | |||||||
| Attachments: |
|
||||||
|
Description
Tom "spot" Callaway
2009-03-20 21:45:40 UTC
Created attachment 336169 [details]
Review comments
I attach my review. Package looks good except for some minor license issues that need clarification.
Quoting from your review: The license statement in these files: RODBC/R/RODBC.R RODBC/R/TypeInfo.R RODBC/R/sql.R RODBC/R/win.R RODBC/inst/LICENCE RODBC/DESCRIPTION all say GPL2 or GPL3, which is the same as the spec file states, but RODBC/src/RODBC.c says GPL version 2 or any later version. and RODBC/COPYING only contains the GPL2 license text. ***** GPLv2 or GPLv3 is functionally equivalent to GPLv2 or any later version (as the only later version is GPLv3). (In reply to comment #2) > GPLv2 or GPLv3 is functionally equivalent to GPLv2 or any later version (as the > only later version is GPLv3). This is true right now. But it will stop being true the moment a new version of GPL is published, and you can not rely on that a new version of GPL will never ever be released. If upstream will not change the license so that all files are released under the same license, the Fedora package must state all licenses, which will be something like: License: GPLv2+ and (GPLv2 or GPLv3) see https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Combined_Dual_and_Multiple_Licensing_Scenario It will be a rather weird statement - preferably upstream should be asked about their inconsistent licensing, so that it can be resolved for something more reasonable. If upstream is unable or unwilling to change it, you should state all licenses as above. Yes, however, since those files are compiled together, the most restrictive license applies to the whole work, which is the GPLv2 or GPLv3 license. But they are not compiled together in a single binary. The C file under the GPLv2+ license is compiled into the RODBC.so library, and the R files under GPLv2 or GPLv3 are compiled into the RODBC.rdb and RODBC.rdx. So RODBC.so ends up with a different license than the rest of the package. Ahh, okay. Fixed in -2: New SRPM: http://www.auroralinux.org/people/spot/review/new/R-RODBC-1.2-2.fc11.src.rpm New SPEC: http://www.auroralinux.org/people/spot/review/new/R-RODBC.spec Package approved. New Package CVS Request ======================= Package Name: R-RODBC Short Description: An ODBC database interface for R Owners: spot Branches: F-9 F-10 devel InitialCC: ... and it's done. R-RODBC-1.2-2.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/R-RODBC-1.2-2.fc9 R-RODBC-1.2-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/R-RODBC-1.2-2.fc10 R-RODBC-1.2-2.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. R-RODBC-1.2-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. |