Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: jrosetta - JRosetta provides a common base for graphical component
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Summary: Review Request: jrosetta - JRosetta provides a common base for graphical component Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: kwizart@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://kwizart.fedorapeople.org/SPECS/jrosetta.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/jrosetta-1.0.1-1.fc8.kwizart.src.rpm Description: JRosetta provides a common base for graphical component
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Dominik 'Rathann' Mierzejewski rpm@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rpm@greysector.net AssignedTo|nobody@fedoraproject.org |rpm@greysector.net Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Dominik 'Rathann' Mierzejewski rpm@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
--- Comment #1 from Dominik 'Rathann' Mierzejewski rpm@greysector.net 2008-11-02 11:50:05 EDT --- Full review, relevant items only.
MUST Items:
- MUST: rpmlint must be run on every package. The output should be posted in the review. $ rpmlint /var/lib/mock//fedora-rawhide-i386/result 2 packages and 0 specfiles checked; 0 errors, 0 warnings.
- MUST: The package must meet the Packaging Guidelines .
Summary: JRosetta provides a common base for graphical component
No need to repeat package name in Summary. Suggestion:
Summary: A common base for graphical user interface components
ln -fs ${j}-%{version}.jar $RPM_BUILD_ROOT%{_javadir}/${j}.jar
Why does it need the symlinks?
All patches should have an upstream bug link or comment, so please add a comment before Patch0: jrosetta-1.0.1-rm_classpath.patch
- MUST: The License field in the package spec file must match the actual license.
Licence is GPLv2 only, not v2+. Please correct.
- MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. Reviewers should use md5sum for this task. If no upstream URL can be specified for this package, please see the Source URL Guidelines for how to deal with this.
Source matches upstream: 0f42365004065d01923bd55ca458fb47 jrosetta-1.0.1-GPL.zip
Please add a comment in the specfile with either the URL of the download page or the full URL of the zip archive, i.e.: # http://dev.artenum.com/projects/JRosetta/download/JRosetta-1-0-1/data/src-gp...
- MUST: The package must successfully compile and build into binary rpms on at least one supported architecture.
Builds fine in mock for rawhide(F-10)/i386
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Dominik 'Rathann' Mierzejewski rpm@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(kwizart@gmail.com | |)
--- Comment #2 from Dominik 'Rathann' Mierzejewski rpm@greysector.net 2008-11-21 17:40:55 EDT --- ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
D Haley mycae@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |472639
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
D Haley mycae@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(kwizart@gmail.com | |) |
--- Comment #3 from D Haley mycae@yahoo.com 2008-12-19 21:11:41 EDT --- I hope no-one minds, but I have taken the liberty of updating this package, as it is blocking a package that I am keen to complete.
SPEC URL: http://dhd.selfip.com/427e/jrosetta-2.spec SRPM URL: http://dhd.selfip.com/427e/jrosetta-1.0.1-2.fc9.src.rpm
*Sat Dec 20 2008 <mycae(a!t)yahoo.com> 1.0.1-2 - Fix build paths to prevent setup from asking for user input after multiple builds - Refresh source to match MD5. - Update patch to match path change - Corrected licence from GPL2+ to GPL2
I agree with the symlinking, as this is done in many java packages, where the version of interest is installed in %{name}-%{version}.jar, and then a symlink is made to this from %{name}.jar . Unless I have mis-interpreted the intent of the command.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: jrosetta - |Review Request: JRosetta - |JRosetta provides a common |A common base to build a |base for graphical |graphical console |component |
--- Comment #4 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-01-14 05:11:38 EDT --- Spec URL: http://kwizart.fedorapeople.org/SPECS/JRosetta.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/JRosetta-1.0.2-1.fc10.src.rpm Description: A common base to build a graphical console
Changelog - Fix License (GPLv2 only) (was confirmed by phone call with upstream, the shortname license can be seen in the MANIFEST of the jar files.) - Fix Summary - Update to 1.0.2 - previous patch merged upstream - Rename to JRosetta
Koji scratch build for Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=1051591
rpmlint is quiet on local build from F-10
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
--- Comment #5 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-01-14 10:19:06 EDT --- @Haley ----------- %setup -qc [...snip] pushd . [...snip] popd
#The zip file is a bit messy. #We reshuffle the top level dir mv %{name}-%{version}-GPL/* . mv %{name}-%{version}-GPL/.[A-z]* . ----------- This is weird to fix problem you have introduced with not using the correct %setup macro: %setup -q -n %{name}-%{version}-GPL
Along with your debuginfo tweak in the scilab package (that, again, isn't needed), You seem to have a strange buildsys. At this time, I don't know if you got inattentive or your buildsys is buggy. The result is that I cannot trust any packages that comes from your side. I hope you can solve this, because we really need hands for scilab and others...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
--- Comment #6 from D Haley mycae@yahoo.com 2009-01-14 18:21:14 EDT --- Nicholas, If you have specific problems with my packages, I would be most apprreciative if you could please comment on them one by one and where appropriate.
My packages are subject to review and can be rejected as required. Personal attacks are unprofessional, and finally it is not up to you to decide whether to accept packages or not. My personal policy is that others are allowed to modify my work, if I fail to do so in a timely fashion.]
A bug report forum is not a place to conduct such aggressive behaviour. If you dont like it, please say so in a review, or contact me personally.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
--- Comment #7 from Dominik 'Rathann' Mierzejewski rpm@greysector.net 2009-01-14 19:27:36 EDT --- (In reply to comment #4)
Spec URL: http://kwizart.fedorapeople.org/SPECS/JRosetta.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/JRosetta-1.0.2-1.fc10.src.rpm Description: A common base to build a graphical console
Changelog
- Fix License (GPLv2 only) (was confirmed by phone call with upstream, the
shortname license can be seen in the MANIFEST of the jar files.)
- Fix Summary
- Update to 1.0.2 - previous patch merged upstream
- Rename to JRosetta
Full review:
rpmlint clean: $ rpmlint /var/lib/mock/fedora-rawhide-i386/result 2 packages and 0 specfiles checked; 0 errors, 0 warnings.
Naming: Please rename it back to jrosetta. There is no good reason to have mixed-case name. In fact, no java package present in Fedora has mixed-case name.
Pre-built binaries: The 1.0.2 source tarball contains pre-built binaries in build/*, please remove them in %prep and ask upstream to provide a source-only tarball, if possible. 1.0.1 was shipped without those binaries and the tarball was half the size.
Source MD5 matches upstream: ef0f9208202762c93c8c415d8472aa76 jrosetta-1.0.2-GPL.zip ef0f9208202762c93c8c415d8472aa76 JRosetta-1.0.2-GPL.zip
Other than that, it seems fine.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
--- Comment #8 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-01-16 12:15:55 EDT --- Spec URL: http://kwizart.fedorapeople.org/SPECS/jrosetta.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/jrosetta-1.0.1-1.fc8.kwizart.src.rpm Description: A common base to build a graphical console
I'm reverting back to jrosetta as package named as the .jar files are named jrosetta*.jar anyway. Im' not bumping the release since they were not a package named jrosetta 1.0.2 previously.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
--- Comment #9 from Dominik 'Rathann' Mierzejewski rpm@greysector.net 2009-01-16 16:05:16 EDT --- (In reply to comment #8)
Spec URL: http://kwizart.fedorapeople.org/SPECS/jrosetta.spec
Specfile says 1.0.2, but...
SRPM URL: http://kwizart.fedorapeople.org/SRPMS/jrosetta-1.0.1-1.fc8.kwizart.src.rpm
this is 1.0.1.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: JRosetta - |Review Request: jrosetta - |A common base to build a |A common base to build a |graphical console |graphical console
--- Comment #10 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-01-16 18:02:10 EDT --- SRPM URL: http://kwizart.fedorapeople.org/SRPMS/jrosetta-1.0.2-1.fc10.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Dominik 'Rathann' Mierzejewski rpm@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #11 from Dominik 'Rathann' Mierzejewski rpm@greysector.net 2009-01-19 18:04:43 EDT --- Very nice, APPROVED.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #12 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-01-20 06:30:50 EDT --- New Package CVS Request ======================= Package Name: jrosetta Short Description: A common base to build a graphical console Owners: kwizart Branches: F-10 EL-5 Cvsextras Commits: yes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #13 from Kevin Fenzi kevin@tummy.com 2009-01-20 16:19:32 EDT --- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=468797
Peter Lemenkov lemenkov@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |496433(RussianFedora)
package-review@lists.fedoraproject.org