[Bug 226555] Merge Review: xerces-j2

[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 report.

Summary: Merge Review: xerces-j2


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226555





------- Additional Comments From mwringe@xxxxxxxxxx  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@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.

_______________________________________________
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]