Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: <main package name here> - <short summary here>
https://bugzilla.redhat.com/show_bug.cgi?id=648053
Summary: Review Request: <main package name here> - <short summary here> Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: pcarrier@redhat.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: ---
Spec URL: http://pcarrier.fedorapeople.org/packages/libMicro/libMicro.spec SRPM URL: http://pcarrier.fedorapeople.org/packages/libMicro/libMicro-0.4.0-1.fc14.src... Description: --- 8< --- LibMicro is a portable set of micro-benchmarks that many Solaris engineers used during Solaris 10 development to measure the performance of various system and library calls. LibMicro was developed by Bart Smaalders and Phil Harman as part of their If Linux is faster it's a Solaris bug performance campaign.
The libMicro sources include an awk script that generates color coded HTML pages to easily compare various OS revs, processors and other differences in a quick manner. --- >8 ---
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=648053
Pierre Carrier pcarrier@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: <main |Review Request: libMicro - |package name here> - <short |Portable micro-benchmarks |summary here> |for various system and | |library calls
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=648053
--- Comment #1 from Pierre Carrier pcarrier@redhat.com 2010-10-31 22:21:10 EDT --- I don't think we should use %{optflags} on this one, we don't want to impact the benchmark. Opinions appreciated.
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=648053
--- Comment #2 from Michael Schwendt mschwendt@gmail.com 2010-11-21 14:54:58 EST --- Run rpmlint on the src.rpm and your built rpms. It's one of the things a packager ought to do. The ReviewGuidelines are not specific to the reviewer(s). ;)
%{_libexecdir}/%{name}/*
What package owns %{_libexecdir}/%{name} then? http://fedoraproject.org/wiki/Packaging:UnownedDirectories
I don't think we should use %{optflags} on this one, we don't want to impact the benchmark. Opinions appreciated.
Not using %optflags and hence also creating a bad -debuginfo package doesn't fly.
How much overhead do you expect from the code that does the measuring compared with the actual execution time of the system calls?
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=648053
--- Comment #3 from Pierre Carrier pcarrier@redhat.com 2010-11-21 17:49:22 EST --- Thanks Michael. Updated.
$ rpmlint -v RPMS/x86_64/libMicro-0.4.0-1.fc14.x86_64.rpm libMicro.x86_64: I: checking libMicro.x86_64: W: spelling-error %description -l en_US awk -> awl, aw, wk libMicro.x86_64: I: checking-url http://hub.opensolaris.org/bin/view/Project+libmicro/ (timeout 10 seconds) libMicro.x86_64: W: no-manual-page-for-binary multiview libMicro.x86_64: W: no-manual-page-for-binary bench 1 packages and 0 specfiles checked; 0 errors, 3 warnings.
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=648053
Jon Ciesla limb@jcomserv.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |limb@jcomserv.net AssignedTo|nobody@fedoraproject.org |limb@jcomserv.net Flag| |fedora-review?
--- Comment #4 from Jon Ciesla limb@jcomserv.net 2011-06-30 11:27:52 EDT --- - rpmlint checks return:
libMicro.spec: W: invalid-url Source0: http://hub.opensolaris.org/bin/download/Project+libmicro/Source+Archives/lib... <urlopen error [Errno 104] Connection reset by peer> The value should be a valid, public HTTP, HTTPS, or FTP URL.
libMicro.src: W: invalid-url URL: http://hub.opensolaris.org/bin/view/Project+libmicro/ <urlopen error [Errno 104] Connection reset by peer> The value should be a valid, public HTTP, HTTPS, or FTP URL.
Is this upstream still valid?
libMicro.x86_64: W: no-manual-page-for-binary multiview Each executable in standard binary directories should have a man page.
libMicro.x86_64: W: no-manual-page-for-binary bench Each executable in standard binary directories should have a man page.
If these exist, include them.
Other than that and the awk spelling, it's quiet.
??? package meets naming guidelines If upstream is libmicro, why libMicro? OK package meets packaging guidelines OK license ( ) OK, text in %doc, matches source OK spec file legible, in am. english See above, can't check! source matches upstream OK on x86_64 package compiles on devel (x86) - no missing BR - no unnecessary BR OK no locales OK not relocatable OK owns all directories that it creates OK no duplicate files OK permissions ok OK %clean ok OK macro use consistent OK code, not content OK no need for -docs OK nothing in %doc affects runtime OK no need for .desktop file
Mock build to chck BRs in progress.
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=648053
--- Comment #5 from Jon Ciesla limb@jcomserv.net 2011-06-30 11:41:47 EDT --- Mock build good, BRs are Ok.
So it's upstream URLs and name capitalization and that's it.
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=648053
Xose Vazquez Perez xose.vazquez@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |xose.vazquez@gmail.com
--- Comment #6 from Xose Vazquez Perez xose.vazquez@gmail.com 2011-11-01 11:46:30 EDT ---
new upstream release 0.4.1 : http://hub.opensolaris.org/bin/view/Project+libmicro/Source+Archives
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=648053
--- Comment #7 from Jon Ciesla limburgher@gmail.com 2011-12-06 13:09:15 EST --- Any updates to your SRPM so I can finish this up?
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=648053
--- Comment #8 from Jon Ciesla limburgher@gmail.com 2011-12-21 10:36:15 EST --- Ping?
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=648053
--- Comment #9 from Jon Ciesla limburgher@gmail.com 2012-01-26 13:50:17 EST --- Ping?
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=648053
Jon Ciesla limburgher@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |WONTFIX Last Closed| |2012-02-28 10:55:25
--- Comment #10 from Jon Ciesla limburgher@gmail.com 2012-02-28 10:55:25 EST --- Closing due to lack of submitter response, if you still wish to get this package into Fedora please submit a new review. Thanks!
package-review@lists.fedoraproject.org