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=655601 --- Comment #7 from Ville Skyttà <ville.skytta@xxxxxx> 2010-11-29 13:41:13 EST --- > 1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from srpm All the packages I've worked with include it in the source rpm. It doesn't cost anything to have it there, and the upside is that people can actually verify the contents of the srpm as well as roll updates just with the information in it without having to hunt missing bits from elsewhere. > 2. simplify documentation installation by using %doc macro I intentionally didn't do that initially because for some reason I remembered that the special %doc macro would do a "cp --parents" and would thus include unwanted subdirs below the doc dirs. But that's incorrect, simplified in -5. > 3. remove %clean section and rm -rf from %install section These (and the Group tags) are intentionally left there because I want the package to be cleanly buildable as is for example on EL-5 (assuming dependencies are available). They don't cause any problems, and I can easily live with them around. > I must say I am impressed with the patch and bugs you filed upstream to fix > the build system a bit. Great work. Thanks, and thanks for the review which got quite a bit bigger than what you initially signed up for. Next revision with the %doc stuff simplified: http://scop.fedorapeople.org/packages/jing-trang.spec http://scop.fedorapeople.org/packages/jing-trang-20091111-5.fc14.src.rpm Some things I plan to look into whether they'd make sense and be feasible to implement in the future for this package, let me know if you'd like them or some of them done during the review, or have any opinions about them, in no particular order: - Try to port the missing test suite generation bits to some other XSLT processor than the old saxon, and if successful, run the whole test suite. - Install dtdinst's schemas and XSLs as non-%doc and register them with the system XML catalog. - Review jing's %docs; isorelax.copying.txt and xerces.copying.txt could/should maybe be dropped because they're not shipped in this package (and copying.html maybe modified not to link to them) and maybe datatype-sample.jar should be dropped as well. -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review