[Bug 456281] Review Request: xerces-j2 - Java XML parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=456281





--- Comment #2 from Victor G. Vasilyev <victor.vasilyev@xxxxxxx>  2008-08-18 10:36:36 EDT ---
The second release is prepared for review.
Spec URL: http://nbi.netbeans.org/files/documents/210/2048/xerces-j2.spec
SRPM URL:
http://nbi.netbeans.org/files/documents/210/2126/xerces-j2-2.8.0-2.fc10.src.rpm

Changes:
- Appropriate values of Group Tags are chosen from the official list
- Remove Epoch tag
- Remove Obsoletes tag
- Clean up versionless javadoc dir symlinking
- Add LICENSE to both demo and scripts packages
- Avoid use of macros in %%changelog
- Remove dangerous command in %%pre: rm -f %%{_javadir}/xerces.jar
- Change priority for alternatives
- The archive.apache.org is used to obtain sources

rpmlint shows no error and no warnings against all binary RPMs, but it shows 3
warnings against SRPM:
-------------------
xerces-j2.src:226: W: libdir-macro-in-noarch-package %dir
%{_libdir}/gcj/%{name}
xerces-j2.src:227: W: libdir-macro-in-noarch-package %attr(-,root,root)
%{_libdir}/gcj/%{name}/%{name}-%{version}.jar.*
xerces-j2.src:251: W: libdir-macro-in-noarch-package %attr(-,root,root)
%{_libdir}/gcj/%{name}/%{name}-samples.jar.*
------------------

All warnings listed above are not issues, because all of them have relation to
the gcj support, but the gcj support is turned off by default in the package
specification. Note, all the libdir macros are enclosed within the macro %if
%{gcj_support} ... %endif  and there is the %define gcj_support 0 at the start
of the specification.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]