[Bug 226555] Merge Review: xerces-j2
bugzilla at redhat.com
bugzilla at redhat.com
Wed Apr 25 23:27:34 UTC 2007
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: xerces-j2
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226555
------- Additional Comments From mwringe at redhat.com 2007-04-25 19:27 EST -------
(In reply to comment #2)
>
> * specfile name matches %{name}
> X verify source and patches (md5sum matches upstream, know what the patches do)
> - Source0 doesn't exist
Updated link location
> * skim the summary and description for typos, etc.
> * correct buildroot
> - should be:
> %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
> * if %{?dist} is used, it should be in that form (note the ? and %
> locations)
> * license text included in package and marked with %doc
> * keep old changelog entries; use judgement when removing (too old?
> useless?)
> * packages meets FHS (http://www.pathname.com/fhs/)
> X rpmlint on <this package>.srpm gives no output
> W: xerces-j2 non-standard-group Text Processing/Markup/XML
> W: xerces-j2 unversioned-explicit-provides jaxp_parser_impl
> The group one is OK, please fix the unversioned-explicit-provides
Fixed
> X specfile is legible
> - please get rid of section tag
Done
> - URL indicates 'Xerces has moved out of the XML project and is now a project
> in its own right.' , please update with the new URL
This is the proper URL, they are just saying welcome to the new project page
they are not part of the XML project anymore.
> - in this spec file, commands are usually just plain commands instead of
> macros except for %{__sed}
Fixed
> X consider using cp -p to preserve timestamps
> - please use cp -p on lines 164, 248-253
fixed (note -a is the same as -dpR)
> X run rpmlint on the binary RPMs
> [pcheung at to-fcjpp1 pcheung]$ rpmlint
> /var/lib/mock/fedora-development-x86_64-core-pcheung/result/xerces-j2-*x86*rpm
> W: xerces-j2 non-standard-group Text Processing/Markup/XML
> W: xerces-j2 dangling-symlink /usr/share/java/jaxp_parser_impl.jar
/etc/alternatives
> W: xerces-j2 symlink-should-be-relative /usr/share/java/jaxp_parser_impl.jar
> /etc/alternatives
> - Can these symlink warnings be fixed?
No, the package needs to use the external Alternatives system, therefore the
symlinks warnings can't be removed.
>
> W: xerces-j2-demo non-standard-group Development/Testing
> W: xerces-j2-demo no-documentation
> - Is there any doc for the -demo package?
No
> W: xerces-j2-javadoc-apis non-standard-group Development/Documentation
> W: xerces-j2-javadoc-impl non-standard-group Development/Documentation
> W: xerces-j2-javadoc-other non-standard-group Development/Documentation
> W: xerces-j2-javadoc-xni non-standard-group Development/Documentation
> W: xerces-j2-scripts non-standard-group Text Processing/Markup/XML
> W: xerces-j2-scripts no-documentation
> - Is there any doc for the -scripts package?
No
>
> SHOULD:
> * package should include license text in the package and mark it with %doc
> * package should build on i386
> * package should build in mock
>
Updated SRPM:
https://mwringe.108.redhat.com/files/documents/175/351/xerces-j2-2.7.1-9jpp.1.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the package-review
mailing list