Bug 673658 - Review Request: R-Rcompression - In-memory decompression for GNU zip and bzip2 formats.
Summary: Review Request: R-Rcompression - In-memory decompression for GNU zip and bzip...
Keywords:
Status: CLOSED DUPLICATE of bug 753185
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Haïkel Guémar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-BUNDLEDLIBS 673660
TreeView+ depends on / blocked
 
Reported: 2011-01-29 11:02 UTC by Pierre-YvesChibon
Modified: 2011-11-11 17:40 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-11-11 17:40:38 UTC
Type: ---
Embargoed:
karlthered: fedora-review?


Attachments (Terms of Use)

Description Pierre-YvesChibon 2011-01-29 11:02:22 UTC
Spec URL: http://pingou.fedorapeople.org/RPMs/R-Rcompression.spec
SRPM URL: http://pingou.fedorapeople.org/RPMs/R-Rcompression-0.92.0-1.fc14.src.rpm
Description: 
 The package is a basic interface to some of the compression facilities in
the zlib and bzip2 libraries for uncompressing (and compressing) data in
memory that is not in a file. It handles bz2, gzip and regular compress
(.Z) content. It can work on files or with data in memory, e.g. downloaded
directly into memory via an HTTP request. It is used when we don't want to
write data to a file and then read it back into R. This is common when
performing HTTP requests via the RCurl package and dependent packages such
as SSOAP.

Comment 1 Haïkel Guémar 2011-02-12 18:43:33 UTC
R-Rcompression

all relevant MUST and SHOULD points are listed here.

Must: rplint on src.rpm and binary rpms   OK (nothing relevant or in violation to packaging guidelines
$ rpmlint -iv R-Rcompression-0.92.0-1.fc14.src.rpm
R-Rcompression.src: I: checking
R-Rcompression.src: W: spelling-error %description -l en_US zlib -> lib, glib, z lib
The value of this tag appears to be misspelled. Please double-check.

R-Rcompression.src: W: spelling-error %description -l en_US uncompressing -> uncompressed, compressing, uncomprehending
The value of this tag appears to be misspelled. Please double-check.

R-Rcompression.src: W: spelling-error %description -l en_US gzip -> zip, grip, g zip
The value of this tag appears to be misspelled. Please double-check.

R-Rcompression.src: I: checking-url http://www.omegahat.org/Rcompression/index.html (timeout 10 seconds)
R-Rcompression.src: I: checking-url http://www.omegahat.org/Rcompression/Rcompression_0.92-0.tar.gz (timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


$ rpmlint -iv R-Rcompression-0.92.0-1.fc14.x86_64.rpm
R-Rcompression.x86_64: I: checking
R-Rcompression.x86_64: W: spelling-error %description -l en_US zlib -> lib, glib, z lib
The value of this tag appears to be misspelled. Please double-check.

R-Rcompression.x86_64: W: spelling-error %description -l en_US uncompressing -> uncompressed, compressing, uncomprehending
The value of this tag appears to be misspelled. Please double-check.

R-Rcompression.x86_64: W: spelling-error %description -l en_US gzip -> zip, grip, g zip
The value of this tag appears to be misspelled. Please double-check.

R-Rcompression.x86_64: I: checking-url http://www.omegahat.org/Rcompression/index.html (timeout 10 seconds)
R-Rcompression.x86_64: W: unable-to-read-zip /usr/lib64/R/library/Rcompression/sampleData/tests.zip: File bzTar.R is encrypted, password required for extraction
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

MUST: package is named accordingly to general and R specific naming guidelines OK

MUST: spec file name matches %{name} OK

MUST: package meet packaging guidelines OK

MUST: package is licensed under a fedora compliant license (BSD) OK

Note: Description says BSD, but they have used zlib licensed code, both licenses are fedora-compliant and almost the same (but not interchangeable)
but i feel that upstream should fix this issue in relicensing the whole under zlib license. I have confidence that you'll sort that minor issue later. NOTABLOCKER.
No license file included, so no need to provide it.

MUST: spec is written in legible american english. OK

MUST: provided and upstream sources checksums match.  OK
provided sources: 8166316349e8e032fcedb9b4813a01286bdee711
upstream sources: 8166316349e8e032fcedb9b4813a01286bdee711

MUST: compilation tested on all supported platforms (x86 and x86_64) for F14 under mock. KO
Note: missing BR & R to zip, except that, it compiles fine.

MUST: don't bundle third party libraries though it borrowed some zlib code. OK

MUST: package owns created directories (and only them). OK

MUST: files are not listed twice. OK
 
MUST: files permissions are properly set. OK

MUST: consistent use of macros. OK

MUST: only ships permissible content. OK

MUST: doc is properly handled. OK

MUST: filenames are valid UTF-8. OK 

General and R specific guidelines are more or less respected, as soon as you fix the mock build issue, it will be approved (the minor license issue could be fixed later).

Comment 2 Haïkel Guémar 2011-02-12 19:30:07 UTC
one more thing, use your real name in changelog and not a nickname.

Comment 3 Pierre-YvesChibon 2011-02-13 10:47:27 UTC
Spec: http://pingou.fedorapeople.org/RPMs/R-Rcompression.spec
Srpm: http://pingou.fedorapeople.org/RPMs/R-Rcompression-0.92.0-2.fc13.src.rpm

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2836812

I have contacted upstream to ask his opinion regarding relicensing.
I prefer to hear back from him before getting this package approved.

I will also contact him about this encrypted zip file.

Comment 4 Haïkel Guémar 2011-02-16 07:55:27 UTC
About the zlib licensed code:
Some of the C source code in src/ comes from zlib-derived library minizip (you can check this by grep-ping minizip author's name like this  find src/ -exec grep -nH Volland {} \; ). 
I'm not aware of how R handle C library linking but either it's possible to dynamically link to minizip system library (and it's conflicts with our "don't ship bundled system library" rule), or it's not or use an heavily customized version of minizip and then it's a (partial ?) derivative work.

The main issue is that zlib and BSD are both non-copyleft licenses which requires to keep the original notice, mentionning original authors and if original sources were modified. It's feasible but differences between zlib/BSD are not worth shipping two notices, you'd better stick to zlib (the maintainer should not worry about compatibility with other third party-libraries).

As i said, the licensing issue by itself is not a blocker for inclusion in Fedora but it should be sorted out.

Comment 5 Jason Tibbitts 2011-02-16 20:31:36 UTC
The licensing isn't a blocker but the bundled library is.

Comment 6 Haïkel Guémar 2011-02-17 05:49:41 UTC
The point is that Rcompression main functionnalities lie in a semi-generated wrapper (as explained in tuNotes.xml) around minizip code. I'm not sure that it's possible to get rid of this code without rewrite the whole stuff.

Comment 7 Pierre-YvesChibon 2011-11-11 17:40:38 UTC

*** This bug has been marked as a duplicate of bug 753185 ***


Note You need to log in before you can comment on or make changes to this bug.