Spec URL: http://hicham.fedorapeople.org/PgsLookAndFeel.spec SRPM URL: http://hicham.fedorapeople.org/PgsLookAndFeel-1.1-1.20090721cvs.fc11.src.rpm Description: The PgsLookAndFeel is a nice looking LookAndFeel for Swing. It aims be a very modern cross-platform LookAndFeel with nice features and much interaction for users. rpmlint output : none
- The BuildRoot is , use a newer one https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag - You have BR: laf-plugin twice. Should the other be Requires:? - Preserve the time stamps of the sources with "install -p" and "cp -p".
Updated the spec file. Spec URL: http://hicham.fedorapeople.org/PgsLookAndFeel.spec SRPM URL: http://hicham.fedorapeople.org/PgsLookAndFeel-1.1-2.20090721cvs.fc11.src.rpm Description: The PgsLookAndFeel is a nice looking LookAndFeel for Swing. It aims be a very modern cross-platform LookAndFeel with nice features and much interaction for users. rpmlint output : none
- I suggest versioning the BR: and R: on java to >= 1:1.6.0 to make sure the project is built with openjdk, not gcj. rpmlint output is clean. MUST: The package does not yet exist in Fedora. The Review Request is not a duplicate. OK MUST: The spec file for the package is legible and macros are used consistently. OK MUST: The package must be named according to the Package Naming Guidelines. OK - Upstream clearly has a preference and PgsLookAndFeel is clearer than pgslookandfeel. MUST: The spec file name must match the base package %{name}. OK MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines. OK MUST: The License field in the package spec file must match the actual license. OK - Some files are BSD which should be compatible with ASL 2.0. MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. OK MUST: The package MUST successfully compile and build into binary rpms. OK MUST: The spec file MUST handle locales properly. N/A MUST: Optflags are used and time stamps preserved. OK MUST: A package must own all directories that it creates or require the package that owns the directory. OK MUST: Files only listed once in %files listings. OK MUST: Debuginfo package is complete. N/A MUST: Permissions on files must be set properly. OK MUST: Clean section exists. OK MUST: Java documentation must go in a -javadoc subpackage. NEEDSWORK - Javadoc files must go into %{_javadocdir}/%{name}. MUST: All relevant items are included in %doc. Items in %doc do not affect runtime of application. OK MUST: Desktop files are installed properly. N/A MUST: No file conflicts with other packages and no general names. OK MUST: Buildroot cleaned before install. OK SHOULD: %{?dist} tag is used in release. OK SHOULD: If the package does not include license text(s) as separate files from upstream, the packager should query upstream to include it. NEEDSWORK - Ask upstream to put license and README in cvs. SHOULD: The package builds in mock. OK
ping?
I have sent an email to upstream asking him to put license files in CVS, but I didn't get an answer till now. I am gonna resend an email to him, and wait.
(In reply to comment #6) > I have sent an email to upstream asking him to put license files in CVS, but I > didn't get an answer till now. > > I am gonna resend an email to him, and wait. That's a SHOULD, so the package can be approved once you have fixed the javadoc issue.
Updated the spec file. Spec URL: http://hicham.fedorapeople.org/PgsLookAndFeel.spec SRPM URL: http://hicham.fedorapeople.org/PgsLookAndFeel-1.1-3.20090805cvs.fc11.src.rpm Description: The PgsLookAndFeel is a nice looking LookAndFeel for Swing. It aims be a very modern cross-platform LookAndFeel with nice features and much interaction for users. rpmlint output : none Koji build page : http://koji.fedoraproject.org/koji/taskinfo?taskID=1584273
You might want to use a fully versioned dependency, Requires: %{name} = %{version}-%{release} ** I'd change %{_javadir}/* to %{_javadir}/%{name}*.jar Actually, http://fedoraproject.org/wiki/Packaging/Java#Jar_file_naming states that "If the package provides several JAR files, the filenames assigned by the build must be used" so I think you shouldn't add the version suffix to the jar files. ** You need to run find -name '*.jar' -o -name '*.class' -exec rm -f '{}' \; in %prep to remove any pre-existing jar and class files.
(In reply to comment #9) > You need to run > find -name '*.jar' -o -name '*.class' -exec rm -f '{}' \; > in %prep to remove any pre-existing jar and class files. Actually, there's something wrong with this command. Better to run find -name '*.jar' -exec rm -f '{}' \; find -name '*.class' -exec rm -f '{}' \; This actually reveals there was one jar file that you didn't remove, belonging to the JIDE SWING suite. It's probably the one at https://jide-oss.dev.java.net/ Needless to mention, this must be packaged separately as well.
I think that jide-oss have a very restrictive license to be included in Fedora : http://www.jidesoft.com/purchase/EULA.htm Correct me if i am wrong.
It's dual licensed as per the link in comment #10. However, I am unable to get the project to build, [javac] ^ [javac] jide-oss/src/com/jidesoft/plaf/aqua/AquaJidePopupMenuUI.java:9: package apple.laf does not exist [javac] import apple.laf.AquaPopupMenuUI; [javac] ^ [javac] jide-oss/src/com/jidesoft/plaf/aqua/AquaJidePopupMenuUI.java:15: cannot find symbol [javac] symbol: class AquaPopupMenuUI [javac] public class AquaJidePopupMenuUI extends AquaPopupMenuUI { [javac] ^ Seems like it's trying to compile some MacOS/X extensions.
true, u either compile with the provided MacOS jar, or patch it so that we don't have to use the MacOS extensions during compilation. but i am wondering if PgsLookAndFeel should depends on jide-oss as it provides a customized version of jide oss. if we want to compile PgsLookAndFeel with the original jide-oss, we have to patch it so that it doesn't check for a valid license. i am confused about what to do.
(In reply to comment #13) > true, u either compile with the provided MacOS jar, or patch it so that we > don't have to use the MacOS extensions during compilation. All pre-existing jar files MUST be removed, so it just has to build without it. > but i am wondering if PgsLookAndFeel should depends on jide-oss as it provides > a customized version of jide oss. You mean JIDE OSS is not the same thing PgsLookAndFeel depends on? > if we want to compile PgsLookAndFeel with the original jide-oss, we have to > patch it so that it doesn't check for a valid license. > > i am confused about what to do. What do you mean by checking for a valid license..?
if we try to build PgsLookAndFeel against jide-oss jar, we get an error message : [javac] Compiling 49 source files to /home/hicham/rpmbuild/BUILD/pgslookandfeel/classes [javac] /home/hicham/rpmbuild/BUILD/pgslookandfeel/src/com/pagosoft/plaf/PlafOptions.java:258: cannot find symbol [javac] symbol : class Lm [javac] location: package com.jidesoft.utils [javac] com.jidesoft.utils.Lm.verifyLicense("Patrick Gotthardt", "PgsLookAndFeel", "aW6dEcgCQMkW4e:OGYJCDdPATGYEiLo2"); That is what I meant by checking for a valid license. So the provided jar is not the same as the jide-oss jar, maybe it uses a different license. By removing that line, the code compile fine against jide-oss. What do u think I should do, patch PgsLookAndFeel to use the jide-oss jar?
(In reply to comment #15) > if we try to build PgsLookAndFeel against jide-oss jar, we get an error message > : > > [javac] Compiling 49 source files to > /home/hicham/rpmbuild/BUILD/pgslookandfeel/classes > [javac] > /home/hicham/rpmbuild/BUILD/pgslookandfeel/src/com/pagosoft/plaf/PlafOptions.java:258: > cannot find symbol > [javac] symbol : class Lm > [javac] location: package com.jidesoft.utils > [javac] com.jidesoft.utils.Lm.verifyLicense("Patrick Gotthardt", > "PgsLookAndFeel", "aW6dEcgCQMkW4e:OGYJCDdPATGYEiLo2"); > > That is what I meant by checking for a valid license. So the provided jar is > not the same as the jide-oss jar, maybe it uses a different license. > > By removing that line, the code compile fine against jide-oss. > > What do u think I should do, patch PgsLookAndFeel to use the jide-oss jar? Yes, if it works. However, jide-oss must be built from source as well, and currently I don't know how to do it since it includes a binary jar file from MacOS X that must be somehow replaced.
no problem in building jide-oss from source, I replaced the MacOS extensions with standard ones, and the code compiled fine. So is it OK to start a package review for jide-oss ?
(In reply to comment #17) > no problem in building jide-oss from source, I replaced the MacOS extensions > with standard ones, and the code compiled fine. So is it OK to start a package > review for jide-oss ? Oh OK. Yes, please submit it for review.
jide-oss have been submitted for review : https://bugzilla.redhat.com/show_bug.cgi?id=516074
ok, so now you can update this package to use jide-oss, and after that update MiniCopier to use this package (if you didn't do so already).
updated spec file Spec URL: http://hicham.fedorapeople.org/PgsLookAndFeel.spec SRPM URL: http://hicham.fedorapeople.org/PgsLookAndFeel-1.1-4.20090805cvs.fc11.src.rpm Description: The PgsLookAndFeel is a nice looking LookAndFeel for Swing. It aims be a very modern cross-platform LookAndFeel with nice features and much interaction for users. rpmlint output : none
- You are missing Requires: jide-oss. - Javadoc stuff has been fixed. - I'd drop the versioning since at least in this case it isn't done by upstream, as you install -D -p -m 644 jar/%{name}.jar \ %{buildroot}%{_javadir}/%{name}-%{version}.jar install -D -p -m 644 jar/%{name}-jide.jar \ %{buildroot}%{_javadir}/%{name}-jide-%{version}.jar - Change %{_javadir}/* to %{_javadir}/PgsLookAndFeel-jide.jar %{_javadir}/PgsLookAndFeel.jar as it avoids the unnecessary wildcard which is troublesome e.g. when files are missing. With these notes the package is APPROVED please fix the issues before CVS import.
updated the spec file Spec URL: http://hicham.fedorapeople.org/PgsLookAndFeel.spec
yes yes, you can request the cvs branch, I've approved the package already.
New Package CVS Request ======================= Package Name: PgsLookAndFeel Short Description: Nice looking LookAndFeel for Swing Owners: hicham Branches: F-10 F-11 InitialCC: hicham
cvs done.
PgsLookAndFeel-1.1-4.20090805cvs.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/PgsLookAndFeel-1.1-4.20090805cvs.fc11
PgsLookAndFeel-1.1-4.20090805cvs.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/PgsLookAndFeel-1.1-4.20090805cvs.fc10
PgsLookAndFeel-1.1-4.20090805cvs.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
PgsLookAndFeel-1.1-4.20090805cvs.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.