[Bug 744339] Review Request: dieharder - Random number generator tester and timer

[bugzilla at redhat.com]

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 at gmail.com> 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.