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=744339 --- Comment #17 from Jiri Hladky <hladky.jiri@xxxxxxxxx> 2011-11-16 18:16:56 EST --- Hi Richard, thanks a lot for all your effort! I do really appreciate that. Thanks also for uploading updated SPEC file. It feels like I'm now reviewing your changes:-) ad 1) Done. ad 2) I have checked difference between COPYING, NOTES and README files under libdieharder/ directory and the top level directory and it seems to me like that COPYING, NOTES and README files under libdieharder/ directory are just older versions of the files in the top level. I have contacted developer to check what is the current status of these files and what is his opinion on splitting the main package to library and program. I personally think that only COPYING, NOTES and README files are maintained. In such case I would vote to move the files COPYING ChangeLog Copyright NOTES README from the main package to the libs package as the libs package has to be always installed. But let's wait for the developer response first. ad 3) Thanks for that! It was on my TODO list. I have just moved manual/%{name}.pdf from the devel package to the main package. Or perhaps it belongs to the libs package? It's pretty general documentation of random number testing methodology and then it goes to the detailed description of the tests provided by the library. I have uploaded new version of SPEC file and source rpm to this location: Spec URL: http://jhladky.fedorapeople.org/dieharder.spec SRPM URL: http://jhladky.fedorapeople.org/dieharder-3.31.1-3.fc16.src.rpm I will post update here as soon as I get response from the author. Thanks Jirka -- 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