[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #30 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #31 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |CURRENTRELEASE Last Closed||2012-11-27 00:18:50 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #29 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc18 has been pushed to the Fedora 18 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #28 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc18 has been pushed to the Fedora 18 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #24 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|RELEASE_PENDING |MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #25 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/csmith-2.1.0-6.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #26 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/csmith-2.1.0-6.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #27 from Fedora Update System upda...@fedoraproject.org --- csmith-2.1.0-6.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/csmith-2.1.0-6.fc18 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 Shakthi Kannan shakthim...@gmail.com changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #21 from Shakthi Kannan shakthim...@gmail.com --- All documentation have been moved to a single directory. Removed the BuildRoot tag in the .spec file, and have added a comment on multiple licenses. SPEC: http://shakthimaan.fedorapeople.org/SPECS/csmith.spec SRPM: http://shakthimaan.fedorapeople.org/SRPMS/csmith-2.1.0-6.fc16.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 Jerry James loganje...@gmail.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #22 from Jerry James loganje...@gmail.com --- That looks like everything to me. This package is APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=756321 Shakthi Kannan shakthim...@gmail.com changed: What|Removed |Added Status|MODIFIED|RELEASE_PENDING Flags||fedora-cvs? --- Comment #23 from Shakthi Kannan shakthim...@gmail.com --- New Package SCM Request === Package Name: csmith Short Description: Tool to generate random C programs for compiler testing Owners: shakthimaan Branches: f16 f17 f18 f19 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #20 from Jerry James loganje...@gmail.com --- Hmmm, that results in the main csmith package creating both /usr/share/doc/csmith and /usr/share/doc/csmith-2.1.0. Can you get all of the documentation files into one or the other? Also, the BuildRoot tag in the spec file is unnecessary. I recommend deleting it. This MUST item has not been addressedd: [!]: If the package is under multiple licenses, the licensing breakdown must be documented in the spec. Everything else looks good. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #19 from Shakthi Kannan shakthim...@gmail.com --- Thanks for the review. The --disable-static doesn't have any effect, and hence have used the find syntax. I have made the other necessary changes. SPEC: http://shakthimaan.fedorapeople.org/SPECS/csmith.spec SRPM: http://shakthimaan.fedorapeople.org/SRPMS/csmith-2.1.0-5.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
https://bugzilla.redhat.com/show_bug.cgi?id=756321
Jerry James loganje...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||loganje...@gmail.com
Assignee|nob...@fedoraproject.org|loganje...@gmail.com
Whiteboard|NotReady|
Flags||fedora-review?
--- Comment #18 from Jerry James loganje...@gmail.com ---
I will take this review. The duplicate file warning is because %files contains
this:
%{_docdir}/%{name}/probabilities.txt
%{_docdir}/%{name}/compiler_test.in
%{_docdir}/csmith
You probably need only the last of these. Also, COPYING needs to be included
in the documentation.
If you change %configure to %configure --disable-static, that should both
shorten the build time and make this line in %install unnecessary:
find %{buildroot} -name *.a -exec rm -f {} \;
Please see all of the boxes marked [!] below. Ignore the one in the Perl
section, though; that is for perl modules, and this package just contains perl
scripts.
Package Review
==
Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed
Issues:
===
[!]: Package does not contain duplicates in %files.
Note: warning: File listed twice: /usr/share/doc/csmith/compiler_test.in
See: http://fedoraproject.org/wiki/Packaging/Guidelines#DuplicateFiles
= MUST items =
C/C++:
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries.
[x]: Changelog in prescribed format.
[!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf %{buildroot} present but not required
[x]: Sources contain only permissible code or content.
[!]: Each %files section contains %defattr if rpm 4.4
Note: %defattr present but not needed
[x]: Macros in Summary, %description expandable at SRPM build time.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[!]: Package does not contain duplicates in %files.
Note: warning: File listed twice: /usr/share/doc/csmith/compiler_test.in
[x]: Permissions on files are set properly.
[x]: Fully versioned dependency in subpackages, if present.
[x]: Package complies to the Packaging Guidelines
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[-]: Large documentation files are in a -doc subpackage, if required.
[!]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
BSD (2 clause), LGPL (v2.1 or later) (with incorrect FSF address),
GPL (v2 or later), Unknown or generated. 4 files have unknown
license. Detailed output of licensecheck in
/home/jamesjer/756321-csmith/licensecheck.txt
[!]: License file installed when any subpackage combination is installed.
[x]: Package consistently uses macro is (instead of hard-coded directory
names).
[!]: If the package is under multiple licenses, the licensing breakdown must
be documented in the spec.
[x]: Package is named using only allowed ASCII characters.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
Note: Package contains no Conflicts: tag(s)
[x]: Package do not use a name that already exist
[x]: Package obeys FHS, except libexecdir and /usr/target.
[x]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
https://bugzilla.redhat.com/show_bug.cgi?id=756321 --- Comment #17 from Shakthi Kannan shakthim...@gmail.com --- Sorry for the delayed response. I have created a patch to remove the custom header files, so users can use the system header files. SPEC: http://shakthimaan.fedorapeople.org/SPECS/csmith.spec SRPM: shakthimaan.fedorapeople.org/SRPMS/csmith-2.1.0-4.fc16.src.rpm Successful Koji builds for F16, F17, F18 respectively: http://koji.fedoraproject.org/koji/taskinfo?taskID=4120706 http://koji.fedoraproject.org/koji/taskinfo?taskID=4120709 http://koji.fedoraproject.org/koji/taskinfo?taskID=4120714 $ rpmlint csmith.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint csmith-2.1.0-4.fc16.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint csmith-2.1.0-4.fc16.x86_64.rpm csmith.x86_64: W: no-manual-page-for-binary csmith csmith.x86_64: W: no-manual-page-for-binary compiler_test.pl csmith.x86_64: W: no-manual-page-for-binary launchn.pl 1 packages and 0 specfiles checked; 0 errors, 3 warnings. $ rpmlint csmith-devel-2.1.0-4.fc16.x86_64.rpm csmith-devel.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 Michael Scherer m...@zarb.org changed: What|Removed |Added CC||m...@zarb.org --- Comment #14 from Michael Scherer m...@zarb.org 2012-03-19 08:40:18 EDT --- Since the inclusion have been refused by FPC, what is the status of this package review ? ( ie, can it be marked as NotReady until the problem is fixed ? ) -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #15 from Jon Ciesla limburg...@gmail.com 2012-03-19 08:47:41 EDT --- It will either need to wait until the problem is fixed or use system headers. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 Michael Scherer m...@zarb.org changed: What|Removed |Added Status Whiteboard||NotReady --- Comment #16 from Michael Scherer m...@zarb.org 2012-03-19 14:25:48 EDT --- Let's mark it as NotReady until a decision have been made. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #12 from Jon Ciesla limburg...@gmail.com 2012-01-04 07:58:35 EST --- https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries Can you document that these headers meet the criteria for an exception? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #13 from Shakthi Kannan shakthim...@gmail.com 2012-01-04 23:36:51 EST --- I have made a FPC request for this package to include the headers at: https://fedorahosted.org/fpc/ticket/127 -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #11 from Shakthi Kannan shakthim...@gmail.com 2012-01-03 23:22:03 EST --- Upstream's reply on the headers: For the sake of everyone's sanity, I'd suggest that the Fedora-fied source package contain exactly what is in the output of `make dist'. If this isn't possible for some reason, maybe we can address the problem to make it possible. I'd rather not be in the situation of having the Fedora package be greatly different than what we distribute, which leads to questions about which distributions people are using when they report bugs, etc. We already have enough problems with that, asking people to specify git commit hashes. Since, this is a self-contained tool by itself, and they would like to keep track of the sources and bugs, and it is our best interest to stay close to upstream as possible, we will retain the headers as is. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #9 from Shakthi Kannan shakthim...@gmail.com 2011-12-06 07:18:43 EST --- I checked with upstream on using the header files. Their reply was: Csmith doesn't just target compilers that run on Unix/Linux platforms. In the past we've wanted to test compilers for embedded platforms, for example, and as I recall some of those had inadequate stdint headers. Basically, copying the stdint headers freed us from an external dependency on those headers, which was good for at least some cases. ~ Eric Eide -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #10 from Jon Ciesla limburg...@gmail.com 2011-12-06 08:16:24 EST --- That's fine for them, as long as the licensing allows them to redistribute the headers. But for Fedora, we must remove them and use the system versions. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #8 from Shakthi Kannan shakthim...@gmail.com 2011-12-01 07:13:14 EST --- I checked with upstream regarding test_csmith.pl and it is not required for end users. I have removed it from the shipped package. SPEC: http://shakthimaan.fedorapeople.org/SPECS/csmith.spec SRPM: http://shakthimaan.fedorapeople.org/SRPMS/csmith-2.1.0-3.src.rpm -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #6 from Tom Hughes t...@compton.nu 2011-11-28 04:41:40 EST --- Obviously there aren't any bundled libraries as such, but there are some header files which appear to have been copied from other libraries? One other thing I noticed, should test_csmith.pl be being installed as a binary? It seems it is intended for running self tests on csmith so maybe it should actually be run in %check? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #7 from Jon Ciesla l...@jcomserv.net 2011-11-28 09:22:32 EST --- Bundled headers are as much a problem as bundled solibs. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #5 from Shakthi Kannan shakthim...@gmail.com 2011-11-26 03:04:35 EST --- Thanks for your review and comments. * Added patch to build for powerpc64 for dist-6E-epel. * Moved compiler_test.in to documentation directory as it is to be used by developers for their own testing. * Added GPLv2+ and LGPLv2+ licenses. * Upstream has fixed the FSF address. Will reflect in the next release. * The shared libraries are useful for using csmith functions in runtime. What is the bundled library here? Please elaborate. SPEC: http://shakthimaan.fedorapeople.org/SPECS/csmith.spec SRPM: shakthimaan.fedorapeople.org/SRPMS/csmith-2.1.0-2.src.rpm Successful Koji builds for F15, F16, F17 and dist-6E-epel: http://koji.fedoraproject.org/koji/taskinfo?taskID=3542463 http://koji.fedoraproject.org/koji/taskinfo?taskID=3542464 http://koji.fedoraproject.org/koji/taskinfo?taskID=3542459 http://koji.fedoraproject.org/koji/taskinfo?taskID=3542462 rpmlint output: $ rpmlint SPECS/csmith.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint SRPMS/csmith-2.1.0-2.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint RPMS/x86_64/csmith-2.1.0-2.x86_64.rpm csmith.x86_64: W: no-manual-page-for-binary csmith csmith.x86_64: W: no-manual-page-for-binary compiler_test.pl csmith.x86_64: W: no-manual-page-for-binary test_csmith.pl csmith.x86_64: W: no-manual-page-for-binary launchn.pl 1 packages and 0 specfiles checked; 0 errors, 4 warnings. $ rpmlint RPMS/x86_64/csmith-devel-2.1.0-2.x86_64.rpm csmith-devel.x86_64: W: no-documentation csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_ia32.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_avr.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_msp430.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/custom_limits.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_ia64.h 1 packages and 0 specfiles checked; 5 errors, 1 warnings. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #1 from Shakthi Kannan shakthim...@gmail.com 2011-11-23 04:30:50 EST --- $ rpmlint SPECS/csmith.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint SRPMS/csmith-2.1.0-1.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint RPMS/x86_64/csmith-2.1.0-1.x86_64.rpm csmith.x86_64: E: script-without-shebang /usr/bin/compiler_test.in csmith.x86_64: W: no-manual-page-for-binary csmith csmith.x86_64: W: no-manual-page-for-binary launchn.pl csmith.x86_64: W: no-manual-page-for-binary compiler_test.pl csmith.x86_64: W: no-manual-page-for-binary test_csmith.pl csmith.x86_64: W: no-manual-page-for-binary compiler_test.in 1 packages and 0 specfiles checked; 1 errors, 5 warnings. $ rpmlint RPMS/x86_64/csmith-devel-2.1.0-1.x86_64.rpm csmith-devel.x86_64: W: no-documentation csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_ia32.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_avr.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_msp430.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/custom_limits.h csmith-devel.x86_64: E: incorrect-fsf-address /usr/include/csmith-2.1.0/stdint_ia64.h 1 packages and 0 specfiles checked; 5 errors, 1 warnings. I will inform upstream to fix the FSF address. Successful Koji builds for F15, F16 and F17: http://koji.fedoraproject.org/koji/taskinfo?taskID=3534326 http://koji.fedoraproject.org/koji/taskinfo?taskID=3534329 http://koji.fedoraproject.org/koji/taskinfo?taskID=3534330 -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 --- Comment #3 from Tom Hughes t...@compton.nu 2011-11-23 06:21:33 EST --- Oh, and some of the header files are GPL so license tag needs to reflect the BSD+GPL mix. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321
Tom Hughes t...@compton.nu changed:
What|Removed |Added
CC||t...@compton.nu
--- Comment #2 from Tom Hughes t...@compton.nu 2011-11-23 06:20:29 EST ---
An informal review as I'm not a sponsored packager yet:
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
C/C++
[x]: MUST Header files in -devel subpackage, if present.
[!]: MUST ldconfig called in %post and %postun if required.
/sbin/ldconfig not called in %post
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contains kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
[x]: MUST Development .so files in -devel subpackage, if present.
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at least
one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is correct (EPEL5 Fedora 10)
%{_tmppath}/%{name}-%{version}-%{release}-root
[!]: MUST Package contains no bundled libraries.
Not sure if the headers from the runtime directory are an issue here?
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).(EPEL6 Fedora 13)
[?]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm 4.4
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package requires other packages for directories it uses.
[-]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the
beginning of %install. (EPEL5)
[!]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[-]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[!]: MUST Package meets the Packaging Guidelines.
Should use %doc to install documentation
[x]: MUST Package is named according to the Package Naming Guidelines.
[?]: MUST Package does not generates any conflict.
[!]: MUST Package obeys FHS, except libexecdir and /usr/target.
I don't think compiler_test.in belongs in /usr/bin
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[!]: MUST Requires correct, justified where necessary.
Require of base package should use %{?_isa} qualification
[!]: MUST Rpmlint output is silent.
rpmlint csmith-2.1.0-1.x86_64.rpm
csmith.x86_64: E: script-without-shebang /usr/bin/compiler_test.in
csmith.x86_64: W: no-manual-page-for-binary csmith
csmith.x86_64: W: no-manual-page-for-binary launchn.pl
csmith.x86_64: W: no-manual-page-for-binary compiler_test.pl
csmith.x86_64: W: no-manual-page-for-binary test_csmith.pl
csmith.x86_64: W: no-manual-page-for-binary compiler_test.in
1 packages and 0 specfiles checked; 1 errors, 5 warnings.
rpmlint csmith-devel-2.1.0-1.x86_64.rpm
csmith-devel.x86_64: W: no-documentation
csmith-devel.x86_64: E: incorrect-fsf-address
/usr/include/csmith-2.1.0/stdint_ia32.h
csmith-devel.x86_64: E: incorrect-fsf-address
/usr/include/csmith-2.1.0/stdint_avr.h
csmith-devel.x86_64: E: incorrect-fsf-address
/usr/include/csmith-2.1.0/stdint_msp430.h
csmith-devel.x86_64: E: incorrect-fsf-address
/usr/include/csmith-2.1.0/custom_limits.h
csmith-devel.x86_64: E: incorrect-fsf-address
[Bug 756321] Review Request: csmith - Tool to generate random C programs for compiler testing
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=756321 Jon Ciesla l...@jcomserv.net changed: What|Removed |Added CC||l...@jcomserv.net --- Comment #4 from Jon Ciesla l...@jcomserv.net 2011-11-23 09:47:54 EST --- Bundled lib comment seconded, what's going on with those, are they actually being used? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
