Bug 504521

Summary: Review Request: SevenZip - Java SDK for LZMA
Product: [Fedora] Fedora Reporter: Orcan Ogetbil <oget.fedora>
Component: Package ReviewAssignee: Rakesh Pandit <rpandit>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, fkooman, kevin, notting
Target Milestone: ---Flags: rpandit: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 4.65-1.fc11 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-06-27 17:58:52 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:
Bug Depends On:    
Bug Blocks: 492203    

Description Orcan Ogetbil 2009-06-07 20:14:12 UTC
Spec URL: http://oget.fedorapeople.org/review/SevenZip.spec
SRPM URL: http://oget.fedorapeople.org/review/SevenZip-4.65-1.fc11.src.rpm
Description:
SevenZip is the Java version of LZMA compressing and decompressing SDK.

rpmlint:
SevenZip.src:112: W: libdir-macro-in-noarch-package (main package) %{_libdir}/gcj/%{name}/

This is a false warning since the package is not noarch by default.

Koji rawhide build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1397876

Comment 1 Rakesh Pandit 2009-06-08 04:59:39 UTC
will review shortly

Comment 2 Rakesh Pandit 2009-06-24 10:30:26 UTC
package naming -- done
api-subpackage -- done

[rakesh@dhcp7-204 SevenZip-4.65-1.fc11.x86_64]$ tree
.
`-- usr
    |-- lib64
    |   `-- gcj
    |       `-- SevenZip
    |           |-- SevenZip-4.65.jar.db
    |           `-- SevenZip-4.65.jar.so
    `-- share
        |-- doc
        |   `-- SevenZip-4.65
        |       |-- 7zFormat.txt
        |       |-- Methods.txt
        |       |-- history.txt
        |       `-- lzma.txt
        `-- java
            |-- SevenZip-4.65.jar
            `-- SevenZip.jar -> SevenZip-4.65.jar

8 directories, 8 files

Symlink between project name and commonly used jar file name done
jar files going to %{_javadir} - done
package naming - fails

I am confused about wether SevenZip would be right name for this packages .. will confirm in some time.

spec file language -- sane
spec file -  legible


[rakesh@dhcp7-204 SPECS]$ md5sum lzma465.tar.bz2 
29d5ffd03a5a3e51aef6a74e9eafb759  lzma465.tar.bz2

[rakesh@dhcp7-204 SPECS]$ md5sum SevenZip-4.65-1.fc11.src/lzma465.tar.bz2 
29d5ffd03a5a3e51aef6a74e9eafb759  SevenZip-4.65-1.fc11.src/lzma465.tar.bz2

md5sum matches -- done
Compiles -- successfully

Build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1433244

rpmlint output - mentioned
buildrequires - fine
%files - fine
-doc package - fine

Summary: naming .. I am in doubt will confirm in some time.

Comment 3 Orcan Ogetbil 2009-06-24 16:10:52 UTC
Thanks for the review! 

SevenZip source tree is being copied to other programs' tarballs. frinika (bug #492203) is one of them. I made it a separate package as a Fedora requirement.

On the other hand this source tree of lzma has the lzma framework for 
C/C++/C sharp/Java. The former two are already covered in the current lzma package we have in Fedora. I am not much interested in C sharp, I am sure only very few of us are. That leaves the Java, which is what this package is.

Comment 4 Rakesh Pandit 2009-06-25 07:27:56 UTC
Yeah .. I had posted it fedora-devel regarding .. "what would be most suited name for this" .. may you follow up there? I don't think SevenZip is apt.

May you check this .. I did not see any debuginfo file ?

[rakesh@dhcp7-204 new]$ rpmlint /home/rakesh/rpmbuild/RPMS/x86_64/SevenZip-4.65-1.fc11.x86_64.rpm 
SevenZip.x86_64: W: unstripped-binary-or-object /usr/lib64/gcj/SevenZip/SevenZip-4.65.jar.so
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Comment 6 Rakesh Pandit 2009-06-26 04:53:36 UTC
I am okay with SevenZip ... may you check about debuginfo file ? as suggested in comment #4

Comment 7 Orcan Ogetbil 2009-06-26 05:01:11 UTC
My system builds the debuginfo package. You need to install redhat-rpm-config or use mock.

Comment 8 Rakesh Pandit 2009-06-26 05:10:26 UTC
APPROVED

Comment 9 Orcan Ogetbil 2009-06-26 05:53:18 UTC
Thanks for the package review!

New Package CVS Request
=======================
Package Name: SevenZip
Short Description: Java SDK for LZMA
Owners: oget
Branches: F-10 F-11
InitialCC:

Comment 10 Jason Tibbitts 2009-06-27 00:34:30 UTC
CVS done.

Comment 11 Fedora Update System 2009-06-27 02:13:14 UTC
SevenZip-4.65-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/SevenZip-4.65-1.fc11

Comment 12 Fedora Update System 2009-06-27 02:14:06 UTC
SevenZip-4.65-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/SevenZip-4.65-1.fc10

Comment 13 Orcan Ogetbil 2009-06-27 17:58:52 UTC
Package in rawhide now. Thanks everyone.

Comment 14 Kevin Kofler 2009-06-28 00:10:23 UTC
The name really needs to be changed, see also the feedback on the mailing list. Rakesh, why did you first ask for feedback and then ignore it? It would have been trivial to fix this right at the beginning, now this needs to be reviewed again, and as it has been already built, the badly-named source package can't easily be removed. :-( Next time please decide the naming BEFORE approving a package!

Comment 15 Kevin Kofler 2009-06-28 00:15:24 UTC
Oh, and also no thanks for oget for filing the CVS request without sorting out the name.

Comment 16 Rakesh Pandit 2009-06-28 03:34:21 UTC
(In reply to comment #14)
> The name really needs to be changed, see also the feedback on the mailing list.
> Rakesh, why did you first ask for feedback and then ignore it? It would have
> been trivial to fix this right at the beginning, now this needs to be reviewed
> again, and as it has been already built, the badly-named source package can't
> easily be removed. :-( Next time please decide the naming BEFORE approving a
> package!  

I asked for views and saw the response yesterday and decided based on it that "SevenZip" is apt and it is.

Comment 17 Kevin Kofler 2009-06-28 13:23:29 UTC
Huh? In the mailing list thread, the submitter (Orcan Ogetbil) was the only one who thinks SevenZip is the correct name.

Comment 18 Fedora Update System 2009-06-30 21:21:58 UTC
SevenZip-4.65-1.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 19 Fedora Update System 2009-06-30 21:38:03 UTC
SevenZip-4.65-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.