[Bug 754004] Review Request: lv2-abGate - an LV2 Noise Gate plugin
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=754004 --- Comment #8 from Fedora Update System 2012-01-12 02:05:35 EST --- lv2-abGate-1.1.3-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/lv2-abGate-1.1.3-2.fc15 -- 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 766154] Review Request: lv2-kn0ck0ut - An LV2 spectral subtraction plugin (linux audio)
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=766154 --- Comment #13 from Fedora Update System 2012-01-12 02:04:05 EST --- lv2-kn0ck0ut-1.1-0.3.git60421a3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/lv2-kn0ck0ut-1.1-0.3.git60421a3.fc16 -- 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 754004] Review Request: lv2-abGate - an LV2 Noise Gate plugin
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=754004 --- Comment #9 from Fedora Update System 2012-01-12 02:05:44 EST --- lv2-abGate-1.1.3-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/lv2-abGate-1.1.3-2.fc16 -- 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 766154] Review Request: lv2-kn0ck0ut - An LV2 spectral subtraction plugin (linux audio)
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=766154 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 754004] Review Request: lv2-abGate - an LV2 Noise Gate plugin
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=754004 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 766154] Review Request: lv2-kn0ck0ut - An LV2 spectral subtraction plugin (linux audio)
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=766154 --- Comment #12 from Fedora Update System 2012-01-12 02:03:57 EST --- lv2-kn0ck0ut-1.1-0.3.git60421a3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/lv2-kn0ck0ut-1.1-0.3.git60421a3.fc15 -- 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 734531] Review Request: unison240 - File synchronisation tool
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=734531
Brendan Jones changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #18 from Brendan Jones 2012-01-12
01:43:38 EST ---
The %optflags mentioned above are not required.
This is looking pretty good Gregor, but can you check here:
http://fedoraproject.org/wiki/Packaging:OCaml#Requires_and_provides
I believe this macro is newer than the previous version of unison and probably
should be included. Please include before updating
This package is APPROVED
Required
+ - OK
- - N/A
X - attention
? - comment please
[+] named according to the Package Naming Guidelines
[+] The spec file name must match the base package %{name}, in the format
%{name}.spec
[+] Meet the Packaging Guidelines
[+] Be licensed with a Fedora approved license and meet the Licensing
Guidelines
[+] The License field in the package spec file must match the actual license
[+] License file must be included in %doc
[+] The spec file must be written in American English
[+] The spec file for the package MUST be legible
[+] The sources used to build the package must match the upstream source
[+] Successfully compile and build into binary rpms on at least one primary
architecture
[+] Proper use of ExcludeArch
[+] All build dependencies must be listed in BuildRequires
[+] The spec file MUST handle locales properly
[+] Shared library files (not just symlinks) in any of the dynamic linker's
default paths, must call ldconfig in %post and %postun
[+] Packages must NOT bundle copies of system libraries
[+] If the package is designed to be relocatable, the packager must state this
fact in the request for review, along with the rationalization for relocation
of that specific package
[+] A package must own all directories that it creates
directories under this
[+] A Fedora package must not list a file more than once in the spec file's
%files listings
[+] Permissions on files must be set properly.
[+] Each package must consistently use macros
[+] The package must contain code, or permissable content
[-] Large documentation files must go in a -doc subpackage
[+] If a package includes something as %doc, it must not affect the runtime of
the application
[-] Header files must be in a -devel package
[-] Static libraries must be in a -static package
[-] library files that end in .so (without suffix) must go in a -devel package
You have separated the *_debug.so files out into a separate package?
[-] devel packages must require the base package using a fully versioned
dependency
[+] Packages must NOT contain any .la libtool archives
[+] GUI apps must include a %{name}.desktop file, properly installed with
desktop-file-install in the %install section
[+] Packages must not own files or directories already owned by other packages
[+] All filenames in rpm packages must be valid UTF-8
[+] Packaged according to Fedora OCAML packaging guidelines
Should Items
[-] the packager SHOULD query upstream for any missing license text files to
include it
[-] Non-English language support for description and summary sections in the
package spec if available
[+] The reviewer should test that the package builds in mock
[-] The package should compile and build into binary rpms on all supported
architectures
[+] The reviewer should test that the package functions as described
[+] If scriptlets are used, those scriptlets must be sane
[-] Usually, subpackages other than devel should require the base package using
a fully versioned dependency
[-] The placement of pkgconfig(.pc) should usually be placed in a -devel pkg
[-] If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin consider requiring the package which provides the file
instead of the file itself
[-] Should contain man pages for binaries/scripts
--
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 772217] Review Request: libyuv - YUV conversion and scaling functionality library
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=772217 Peter Lemenkov changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Peter Lemenkov 2012-01-12 01:28:09 EST --- Thanks! New Package SCM Request === Package Name: libyuv Short Description: YUV conversion and scaling functionality library Owners: peter Branches: el5 el6 f16 InitialCC: -- 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 773485] Review Request: ibutils - InfiniBand fabric management utilities
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=773485 --- Comment #4 from Doug Ledford 2012-01-12 01:22:20 EST --- Because upstream can be short sighted sometimes. That > 1.1.5 is needed, yes. That they chose to use == instead of >=, short sighted indeed. It's not the first time I've run across this in the InfiniBand packages. They work just fine in spite of the dire warning. -- 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 636581] Review Request: arduino-mk - A Makefile for compiling and uploading to Arduino micro-controllers
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=636581 Scott Baker changed: What|Removed |Added CC||sc...@perturb.org --- Comment #3 from Scott Baker 2012-01-12 01:13:02 EST --- I think it makes sense to include this in arduino-core -- 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 773485] Review Request: ibutils - InfiniBand fabric management utilities
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=773485 --- Comment #3 from Albert Strasheim 2012-01-12 00:52:56 EST --- Looks good. I did notice that configure prints: configure: WARNING: SWIG version == 1.1.5 is required. You have 2.0.4. You should look at http://www.swig.org Everything builds, but I wonder why upstream chose to put that warning in there? -- 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 773492] Review Request: ibsim - InfiniBand network simulator
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=773492 --- Comment #3 from Albert Strasheim 2012-01-12 00:52:13 EST --- Looks good. -- 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 772217] Review Request: libyuv - YUV conversion and scaling functionality library
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=772217
Brendan Jones changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #6 from Brendan Jones 2012-01-12
00:34:24 EST ---
This package is APPROVED
+ OK
- N/A
X Needs attention
Required
[+] named according to the Package Naming Guidelines
[+] The spec file name must match the base package %{name}, in the format
%{name}.spec
[+] Meet the Packaging Guidelines
unless building for F12 and below or EPEL
[+] Be licensed with a Fedora approved license and meet the Licensing
Guidelines
[+] The License field in the package spec file must match the actual license
[+] License file must be included in %doc
[+] The spec file must be written in American English
[+] The spec file for the package MUST be legible
[+] The sources used to build the package must match the upstream source
[+] Successfully compile and build into binary rpms on at least one primary
architecture
[+] Proper use of ExcludeArch
[+] All build dependencies must be listed in BuildRequires
[+] The spec file MUST handle locales properly
[+] Shared library files (not just symlinks) in any of the dynamic linker's
default paths, must call ldconfig in %post and %postun
[+] Packages must NOT bundle copies of system libraries
[-] If the package is designed to be relocatable, the packager must state this
fact in the request for review, along with the rationalization for relocation
of that specific package
[+] A package must own all directories that it creates
directories under this
[+] A Fedora package must not list a file more than once in the spec file's
%files listings
[+] Permissions on files must be set properly
[+] Each package must consistently use macros
[+] The package must contain code, or permissable content
[-] Large documentation files must go in a -doc subpackage
[+] If a package includes something as %doc, it must not affect the runtime of
the application
[+] Header files must be in a -devel package
[-] Static libraries must be in a -static package
[+] library files that end in .so (without suffix) must go in a -devel package
[+] devel packages must require the base package using a fully versioned
dependency
[+] Packages must NOT contain any .la libtool archives
[-] GUI apps must include a %{name}.desktop file, properly installed with
desktop-file-install in the %install section
[+] Packages must not own files or directories already owned by other packages
[+] All filenames in rpm packages must be valid UTF-8
Should Items
[+] the packager SHOULD query upstream for any missing license text files to
include it
[-] Non-English language support for description and summary sections in the
package spec if available
[+] The reviewer should test that the package builds in mock
[+] The package should compile and build into binary rpms on all supported
architectures
[+] The reviewer should test that the package functions as described
[+] If scriptlets are used, those scriptlets must be sane
[+] Usually, subpackages other than devel should require the base package using
a fully versioned dependency
[+] The placement of pkgconfig(.pc) should usually be placed in a -devel pkg
[-] If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin consider requiring the package which provides the file
instead of the file itself
[-] Should contain man pages for binaries/scripts
--
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 773511] Review Request: linuxdcpp - A file-sharing client for the Direct Connect network
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=773511 --- Comment #3 from Steven Sheehy 2012-01-12 00:01:26 EST --- linuxdcpp was previously in Fedora but was obsoleted due to not being maintained. The previous package review was in bug #234331, but that bug did not allow me to set fedora-cvs flag to ? as mentioned in: https://fedoraproject.org/wiki/Package_SCM_admin_requests#Package_Change_Requests_for_existing_packages Since it was not recommended to reopen that bug, I am opening this new bug and attaching the updated spec and SRPM files (I don't have a public URL to upload them to). These changes upgrade the linuxdcpp package to 1.1.0 and drop a patch that has been merged into upstream. A couple of patches were added to stop rpmlint from complaining about some issues that did not keep the package from building and running. Since I am a developer of linuxdcpp as well I will make sure these get merged into upstream before the next release. -- 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 768545] Review Request: blktap - blktap user space utilities
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=768545 --- Comment #11 from Fedora Update System 2012-01-11 23:53:12 EST --- blktap-2.0.90-3.git20111216.62de90d.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/blktap-2.0.90-3.git20111216.62de90d.fc16 -- 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 773511] Review Request: linuxdcpp - A file-sharing client for the Direct Connect network
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=773511 --- Comment #2 from Steven Sheehy 2012-01-11 23:52:18 EST --- Created attachment 552288 --> https://bugzilla.redhat.com/attachment.cgi?id=552288 linuxdcpp-1.1.0-1.fc16.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 768545] Review Request: blktap - blktap user space utilities
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=768545 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 773511] New: Review Request: linuxdcpp - A file-sharing client for the Direct Connect network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: linuxdcpp - A file-sharing client for the Direct Connect network https://bugzilla.redhat.com/show_bug.cgi?id=773511 Summary: Review Request: linuxdcpp - A file-sharing client for the Direct Connect network Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: steven.she...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: Attached to bug SRPM URL: Attached to bug Description: A file-sharing client for the Direct Connect network LinuxDC++ is a Direct Connect client based on DC++. Utilizing the latest DC++ core, LinuxDC++ offers similar functionality to the Windows client like segmented downloading, TTH based file integrity, etc. with a GTK+ user interface. LinuxDC++ is free and open source software licensed under the GPL. Direct Connect (DC) is a peer-to-peer (P2P) file-sharing protocol. Clients connect to a central hub where they can chat or share files with one another. Users can view other users' list of shared files or search the hub for files. -- 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 773511] Review Request: linuxdcpp - A file-sharing client for the Direct Connect network
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=773511 Steven Sheehy changed: What|Removed |Added Attachment #552287|Spec file |linuxdcpp.spec description|| -- 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 773511] Review Request: linuxdcpp - A file-sharing client for the Direct Connect network
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=773511 --- Comment #1 from Steven Sheehy 2012-01-11 23:51:30 EST --- Created attachment 552287 --> https://bugzilla.redhat.com/attachment.cgi?id=552287 Spec file -- 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 756635] Review Request: gap - Computational discrete algebra
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=756635 Jerry James changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Jerry James 2012-01-11 22:35:00 EST --- New Package SCM Request === Package Name: gap Short Description: Computational discrete algebra Owners: jjames Branches: f16 InitialCC: -- 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 756635] Review Request: gap - Computational discrete algebra
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=756635 Scott Tsai changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #7 from Scott Tsai 2012-01-11 22:16:36 EST --- (In reply to comment #6) Bravo, this was one tough package! APPROVED. -- 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 756635] Review Request: gap - Computational discrete algebra
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=756635
--- Comment #6 from Jerry James 2012-01-11 22:05:35 EST
---
(In reply to comment #5)
> Sorry for the delay for this review:
No problem. I've been quite busy with the mass rebuild for F17 anyway. You've
done a lot of work on this review, and I appreciate it a lot.
> 1. gap-core should own /usr/lib/gap/ (%{gaparchdir}) since it owns
> /usr/lib/gap/sysinfo.gap
Done.
> 2. For gap-vim, since you're not shipping README.vim-utils I recommend you
> ship
> the attached gap.vim as /usr/share/vimfiles/ftdetect/gap.vim. Otherwise the
> user would have to know to manually ":set filetype=vim" to activate the VIM
> sytnax hightlighting and indentation plugins.
> Hopefully the *.g, *.gi, *.gd file extensions are not used by other file
> types.
Thank you. I've added this.
> 3. Placing "README.fedora" in gap-libs and placing "description4r4p10" in
> gap-core seems a bit strange to me. Maybe move "README" and "README.fedora" to
> gap-core? (not critical)
Good point. I have moved the READMEs to gap-core.
> 4. Since Fedora packaging lack Debian's "recommand" and "suggest"
> functionality, I'm not sure splitting out the prim, small and trans group
> subpackages is really a good idea but I'll leave it to your disgression.
I think this is still a good idea considering the size of those packages.
However, to address the point that we have no way to recommend them, I made the
gap metapackage Require all 3 of these. That way, somebody you does "yum
install gap" will get them all, and others can be more selective.
> 5. I was able to build and load the "Example" package from
> http://www.gap-system.org/Packages/packages.html sucessfully.
Great!
[snip]
> I'll approve this as soon as you fix points one and two above.
New URLs:
http://jjames.fedorapeople.org/gap/gap.spec
http://jjames.fedorapeople.org/gap/gap-4.4.12-3.fc16.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 773021] Review Request: piglit - Collection of automated tests for OpenGL implementations
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=773021
--- Comment #1 from Scott Tsai 2012-01-11 21:43:37 EST ---
1. I believe the license should be "MIT and GPLv2+ and GPLv3 and LGPLv2".
Fro the COPYING file:
"The following is the 'standard copyright' agreed upon by most
contributors, and is currently the canonical license preferred by the
piglit project." and the "Copyright @ 2010 Intel" license that follows is the
"Modern Style without sublicense" variant of the MIT license from:
http://fedoraproject.org/wiki/Licensing:MIT. Grepping for "Intel" shows that a
lot of files carry this license.
2. If you add a BuildRequires on mesa-libGLES-devel then the
OPENGL_gles{1,2}_LIBRARY cmake variables would be set and the tests in the
corrpesponding directories would get built.
3. When you invoke cmake, you're not passing %{optflags} down like the "%cmake"
macro in /etc/rpm/macros.cmake thus piglit is compiled at the default
optimization level. From the README file, I see that's probably what upstream
developers do as well, but I recommend you add a comment in the spec file if
it's intentional.
4. When you're creating the /usr/bin/piglit* symlinks with:
for srcfile in piglit*.py ; do
ln -sf ../..%{_libdir}/%{name}/${srcfile} \
$RPM_BUILD_ROOT%{_bindir}/$(basename ${srcfile} .py)
done
Since "../../%{_libdir}" hard codes the assumption that %{_bindir}
is only two levels from '/' you might as well make the symlink paths absolute:
-ln -sf ../..%{_libdir}/%{name}/${srcfile} \
+ln -sf %{_libdir}/%{name}/${srcfile} \
5. Also in the above shell snippet, I believe the ".py" at the end of
"$(basename ${srcfile} .py)" is superfluous.
Note that tests/util/glew.{ch} and friends in the source is an embedded copy of
the OpenGL Extension Wrangler Library ("glew" in Fedora). I frankly think that
a device driver testsuite embedding a copy of a small utility library causes no
harm and would approve this package regardless.
--
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 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #21 from Jon Ciesla 2012-01-11 20:35:17 EST --- Git done (by process-git-requests). -- 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 588428] Review Request: rubygem-addressable - Improved URI/URL handling
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=588428 --- Comment #24 from Jon Ciesla 2012-01-11 20:34:07 EST --- Git done (by process-git-requests). -- 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 773502] Review Request: perl-Spreadsheet-read - universal API to read any spreadsheet
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=773502 Mark Goodwin changed: What|Removed |Added CC||f...@redhat.com Blocks||754678 -- 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 773502] New: Review Request: perl-Spreadsheet-read - universal API to read any spreadsheet
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: perl-Spreadsheet-read - universal API to read any
spreadsheet
https://bugzilla.redhat.com/show_bug.cgi?id=773502
Summary: Review Request: perl-Spreadsheet-read - universal API
to read any spreadsheet
Product: Fedora
Version: rawhide
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: medium
Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgood...@redhat.com
QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
Story Points: ---
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Spec URL:
http://people.redhat.com/mgoodwin/cpan/perl-Spreadsheet-read/perl-Spreadsheet-Read.spec
SRPM URL:
http://people.redhat.com/mgoodwin/cpan/perl-Spreadsheet-read/perl-Spreadsheet-Read-0.45-1.fc15.src.rpm
Description: Spreadsheet::Read tries to transparently read *any* spreadsheet
and return its content in a universal manner independent of the parsing
module that does the actual spreadsheet scanning. It supports spreadsheets
handled by Text::CSV_XS, Spreadsheet::ParseExcel, Spreadsheet::XLSX and
Spreadsheet::ReadSXC (note: XLSX and ReadSXC are not yet in Fedora. Depending
how this review request goes, I can add them too).
The cpanspec utility was used to download the tarball from cpan and
generate the spec. This request to add Spreadsheet:read is to satisfy
a missing dependency for pcp-import-sheet2pcp, the new version of which
is currently in updates-testing (see BZ 754678).
Have built this for {el4,el5,el6,f15,f16,f17}-candidate and tested on f15.
--
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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
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=674008 --- Comment #79 from Rosen Diankov 2012-01-11 19:53:06 EST --- hi tim, ok, the two features you requested are now in r2949 #define OPENRAVE_PLUGINS_INSTALL_DIR "@OPENRAVE_PLUGINS_INSTALL_ABSOLUTE_DIR@" #define OPENRAVE_DATA_INSTALL_DIR "@OPENRAVE_DATA_INSTALL_ABSOLUTE_DIR@" #define OPENRAVE_PYTHON_INSTALL_DIR "@OPENRAVE_PYTHON_INSTALL_ABSOLUTE_DIR@" As for sympy, i think you'll have an easier time including a sympy 0.6.x version into fc15 rather than getting ikfast to work with 0.7.1. of course, we'd be more than happy to get patches for it, as long as 0.6.x compat is preserved. -- 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 588428] Review Request: rubygem-addressable - Improved URI/URL handling
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=588428 Shawn Starr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #23 from Shawn Starr 2012-01-11 19:25:25 EST --- ah cvs flag... done -- 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 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Shawn Starr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #20 from Shawn Starr 2012-01-11 19:26:52 EST --- n/m fedora-cvs is still used ;p -- 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 768545] Review Request: blktap - blktap user space utilities
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=768545 --- Comment #10 from Jon Ciesla 2012-01-11 18:46:05 EST --- Git done (by process-git-requests). -- 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 716384] Review Request: filo - Useful FILe and stream Operations
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=716384 --- Comment #5 from Adam Huffman 2012-01-11 18:33:27 EST --- I've had time to work on this again and have made progress on a gzstream package but it's not ready for review yet. -- 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 773492] Review Request: ibsim - InfiniBand network simulator
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=773492 --- Comment #2 from Doug Ledford 2012-01-11 18:28:38 EST --- Created attachment 552254 --> https://bugzilla.redhat.com/attachment.cgi?id=552254 ibsim spec file -- 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 773492] New: Review Request: ibsim - InfiniBand network simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: ibsim - InfiniBand network simulator https://bugzilla.redhat.com/show_bug.cgi?id=773492 Summary: Review Request: ibsim - InfiniBand network simulator Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dledf...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fenla...@redhat.com, full...@gmail.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- ibsim allows you to create a topology file that represents your InfiniBand fabric and then hook the ibsim application in between your end user application and the fabric itself and then use ibsim to create fake fabric events that your application will then see (such as link up/link down/packet loss/etc). If you aren't well versed in InfiniBand fabric management or know what a MAD packet is, this application is not for you. It is a very low level, get your hands dirty program. However, to someone who wants to see exactly how an application will perform under specific circumstances it will simulate those circumstances without the application knowing the difference between the simulation or the real event. Packages can be found at: http://people.redhat.com/dledford/Package%20Review/ -- 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 773492] Review Request: ibsim - InfiniBand network simulator
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=773492 --- Comment #1 from Doug Ledford 2012-01-11 18:28:13 EST --- [dledford@schwoop SPECS]$ rpmlint ibsim.spec ../SRPMS/ibsim-0.5-5.fc15.src.rpm ../RPMS/x86_64/ibsim-* ibsim.src: W: spelling-error %description -l en_US infiniband -> infinitude ibsim.x86_64: W: spelling-error %description -l en_US infiniband -> infinitude ibsim.x86_64: W: no-manual-page-for-binary ibsim 3 packages and 1 specfiles checked; 0 errors, 3 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 700814] Review Request: din - A musical instrument using multiple Bezier curves
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=700814 --- Comment #11 from Adam Huffman 2012-01-11 18:12:52 EST --- Update to 1.9.2: http://verdurin.fedorapeople.org/reviews/din/din.spec http://verdurin.fedorapeople.org/reviews/din/din-1.9.2-1.fc16.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 773487] New: Review Request: trac-batchmodify-plugin - Batch modification of tickets
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: trac-batchmodify-plugin - Batch modification of tickets https://bugzilla.redhat.com/show_bug.cgi?id=773487 Summary: Review Request: trac-batchmodify-plugin - Batch modification of tickets Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: andrea.v...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://averi.fedorapeople.org/RPMs/trac-batchmodify-plugin/trac-batchmodify-plugin.spec SRPM URL: http://averi.fedorapeople.org/RPMs/trac-batchmodify-plugin/trac-batchmodify-plugin-0.8.0-20120111svn11133.fc16.src.rpm Description: A trac plugin that allows users to modify several tickets together in one shot. CCing Kevin as requested. -- 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 773485] Review Request: ibutils - InfiniBand fabric management utilities
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=773485 --- Comment #1 from Doug Ledford 2012-01-11 18:10:29 EST --- [dledford@schwoop x86_64]$ rpmlint ../../SPECS/ibutils.spec ../../SRPMS/ib* * ibutils.x86_64: W: no-manual-page-for-binary ibnlparse ibutils.x86_64: W: no-manual-page-for-binary dump2psl.pl ibutils.x86_64: W: no-manual-page-for-binary dump2slvl.pl ibutils-devel.x86_64: W: no-documentation ibutils-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libibmscli.so.1.0.0 exit@GLIBC_2.2.5 ibutils-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libibdm.so.1.1.1 exit@GLIBC_2.2.5 ibutils-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libibdmcom.so.1.1.1 exit@GLIBC_2.2.5 ibutils-libs.x86_64: W: no-documentation 5 packages and 1 specfiles checked; 0 errors, 8 warnings. [dledford@schwoop x86_64]$ -- 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 773485] Review Request: ibutils - InfiniBand fabric management utilities
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=773485 --- Comment #2 from Doug Ledford 2012-01-11 18:12:56 EST --- Created attachment 552252 --> https://bugzilla.redhat.com/attachment.cgi?id=552252 Specfile for ibutils package -- 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 773485] New: Review Request: ibutils - InfiniBand fabric management utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: ibutils - InfiniBand fabric management utilities https://bugzilla.redhat.com/show_bug.cgi?id=773485 Summary: Review Request: ibutils - InfiniBand fabric management utilities Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dledf...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fenla...@redhat.com, full...@gmail.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- The ibutils package is intended to allow people building out large scale infiniband networks to be able to properly map their network with various tools for troubleshooting of communications bottlenecks and the like. Packages and spec file can be found at this site: http://people.redhat.com/dledford/Package%20Review/ -- 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 700818] Review Request: libircclient - Library implementing client-server IRC protocol
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=700818 --- Comment #8 from Adam Huffman 2012-01-11 17:30:31 EST --- The warnings are fixed now. New version at: http://verdurin.fedorapeople.org/reviews/libircclient/libircclient.spec http://verdurin.fedorapeople.org/reviews/libircclient/libircclient-1.3-8.fc16.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 773470] Review Request: muffin - Window and compositing manager based on Clutter
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=773470 --- Comment #2 from Bill Nottingham 2012-01-11 17:17:58 EST --- If muffin has been changed enough that it *can't* work with gnome-shell or Moblin, that part of the description should be removed. -- 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 513541] Review Request: cpulimit - CPU Usage Limiter for Linux
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=513541 Thomas Spura changed: What|Removed |Added CC||toms...@fedoraproject.org Resolution|NOTABUG |DUPLICATE --- Comment #37 from Thomas Spura 2012-01-11 17:02:41 EST --- *** This bug has been marked as a duplicate of bug 772406 *** -- 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 772406] Review Request: cpulimit - CPU Usage Limiter for Linux
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=772406 Thomas Spura changed: What|Removed |Added CC||ashay.hum...@gmail.com --- Comment #6 from Thomas Spura 2012-01-11 17:02:41 EST --- *** Bug 513541 has been marked as a duplicate of this bug. *** -- 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 771252] Review Request: cinnamon - Window management and application launching for GNOME
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=771252 --- Comment #2 from leigh scott 2012-01-11 17:05:35 EST --- More info: The next cinnamon release will use muffin so I have packaged it and submitted a review request https://bugzilla.redhat.com/show_bug.cgi?id=773470 -- 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 771252] Review Request: cinnamon - Window management and application launching for GNOME
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=771252 leigh scott changed: What|Removed |Added Depends on||773470 -- 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 773470] Review Request: muffin - Window and compositing manager based on Clutter
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=773470 leigh scott changed: What|Removed |Added Blocks|706599 |771252 -- 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 773470] Review Request: muffin - Window and compositing manager based on Clutter
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=773470 --- Comment #1 from leigh scott 2012-01-11 17:01:41 EST --- I meant to post a link to my cinnamon review https://bugzilla.redhat.com/show_bug.cgi?id=771252 -- 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 513541] Review Request: cpulimit - CPU Usage Limiter for Linux
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=513541 Thomas Spura changed: What|Removed |Added Blocks|201449(FE-DEADREVIEW) | -- 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 773470] New: Review Request: muffin - Window and compositing manager based on Clutter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: muffin - Window and compositing manager based on Clutter https://bugzilla.redhat.com/show_bug.cgi?id=773470 Summary: Review Request: muffin - Window and compositing manager based on Clutter Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: leigh123li...@googlemail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://leigh123linux.fedorapeople.org/pub/review/muffin/1/muffin.spec SRPM URL: http://leigh123linux.fedorapeople.org/pub/review/muffin/1/muffin-3.2.1-1.fc16.src.rpm Description: Muffin is a fork of mutter need for cinnamon https://github.com/linuxmint/muffin https://bugzilla.redhat.com/show_bug.cgi?id=706599 Muffin is a window and compositing manager that displays and manages your desktop via OpenGL. Muffin combines a sophisticated display engine using the Clutter toolkit with solid window-management logic inherited from the Metacity window manager. While Muffin can be used stand-alone, it is primarily intended to be used as the display core of a larger system such as gnome-shell or Moblin. For this reason, Muffin is very extensible via plugins, which are used both to add fancy visual effects and to rework the window management behaviors to meet the needs of the environment. -- 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 773470] Review Request: muffin - Window and compositing manager based on Clutter
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=773470 leigh scott changed: What|Removed |Added Blocks||706599 -- 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 768545] Review Request: blktap - blktap user space utilities
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=768545 David Nalley changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #9 from David Nalley 2012-01-11 16:53:58 EST --- Package Change Request == Package Name: blktap New Branches: f16 Owners: ke4qqq [typo excluded f16, sorry about that] -- 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 755104] Review Request: condor-wallaby - condor related tools for interacting with wallaby
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=755104 --- Comment #8 from Fedora Update System 2012-01-11 16:44:06 EST --- condor-wallaby-4.1.2-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/condor-wallaby-4.1.2-3.fc15 -- 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 755104] Review Request: condor-wallaby - condor related tools for interacting with wallaby
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=755104 --- Comment #9 from Fedora Update System 2012-01-11 16:44:56 EST --- condor-wallaby-4.1.2-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/condor-wallaby-4.1.2-3.fc16 -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 --- Comment #7 from Fedora Update System 2012-01-11 15:52:27 EST --- libcxgb4-1.2.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/libcxgb4-1.2.0-1.fc15 -- 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 772709] Review Request: TeXamator - Helping you making your exercise sheets
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=772709 --- Comment #4 from Mario Santagiuliana 2012-01-11 15:50:27 EST --- Thank you Mohamed! I update the spec file and the srpm :) Spec URL: http://marionline.fedorapeople.org/packages/SPECS/TeXamator.spec SRPM URL: http://marionline.fedorapeople.org/packages/SRPMS/TeXamator-1.7.4-1.fc16.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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 --- Comment #8 from Fedora Update System 2012-01-11 15:52:36 EST --- libcxgb4-1.2.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/libcxgb4-1.2.0-1.fc16 -- 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 772243] Review Request: rds-tools - utilities for testing rds protocol networking
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=772243 --- Comment #8 from Doug Ledford 2012-01-11 15:50:33 EST --- Upstream has been notified of the license issues. I'll ping in this review bug again when a package with a new tarball that fixes the issue is available. -- 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 773442] New: Review Request: libvirt-sandbox - libvirt application sandbox framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: libvirt-sandbox - libvirt application sandbox framework https://bugzilla.redhat.com/show_bug.cgi?id=773442 Summary: Review Request: libvirt-sandbox - libvirt application sandbox framework Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: berra...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://berrange.fedorapeople.org/review/libvirt-sandbox/libvirt-sandbox.spec SRPM URL: http://berrange.fedorapeople.org/review/libvirt-sandbox/libvirt-sandbox-0.0.1-1.fc16.src.rpm Description: The libvirt-sandbox package provides an API for building application sandboxes using libvirt. Sandboxes can be based on either container or machine based virtualization technology. Also included is a simple command line tool for launching sandboxes for arbitrary commands. This package is the basis for 2 Fedora 17 features https://fedoraproject.org/wiki/Features/VirtSandbox https://fedoraproject.org/wiki/Features/SecureContainers -- 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 773442] Review Request: libvirt-sandbox - libvirt application sandbox framework
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=773442 --- Comment #1 from Daniel Berrange 2012-01-11 15:44:04 EST --- I have a F17 scratch build on x8_64: http://koji.fedoraproject.org/koji/taskinfo?taskID=3640418 rpmlint is expected to show the following libvirt-sandbox.x86_64: W: summary-not-capitalized C libvirt application sandbox framework libvirt-sandbox.src: W: summary-not-capitalized C libvirt application sandbox framework libvirt-sandbox-libs.x86_64: W: summary-not-capitalized C libvirt application sandbox framework libraries libvirt-sandbox-libs.x86_64: E: statically-linked-binary /usr/libexec/libvirt-sandbox-init-qemu libvirt-sandbox-devel.x86_64: W: summary-not-capitalized C libvirt application sandbox framework development files 4 packages and 0 specfiles checked; 1 errors, 4 warnings. The warnings are all bogus. The error can be ignored, because this particular binary is intentionally static. It must run inside an initrd where there are no dynamic libraries available. Fortunately this binary is tiny, has no external deps except glibc, and always runs inside a KVM guest under sVirt so is protected against any security issues -- 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 768545] Review Request: blktap - blktap user space utilities
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=768545 --- Comment #8 from Jon Ciesla 2012-01-11 15:43:43 EST --- Git done (by process-git-requests). -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 --- Comment #6 from Jon Ciesla 2012-01-11 15:36:16 EST --- Git done (by process-git-requests). -- 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 768545] Review Request: blktap - blktap user space utilities
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=768545 David Nalley changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from David Nalley 2012-01-11 15:20:07 EST --- New Package SCM Request === Package Name: blktap Short Description: blktap user space utilities Owners: ke4qqq Branches: f15 el6 InitialCC: -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 --- Comment #5 from Doug Ledford 2012-01-11 15:10:53 EST --- New Package SCM Request === Package Name: libcxgb4 Short Description: Driver for Chelsio T4 hardware Owners: dledford Branches: f15 f16 InitialCC: -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 Doug Ledford changed: What|Removed |Added Flag||fedora-cvs? -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 Jay Fenlason changed: What|Removed |Added Flag|fedora-review? |fedora-review+ -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 Doug Ledford changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 Doug Ledford changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|full...@gmail.com -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 Doug Ledford changed: What|Removed |Added Flag||fedora-review? -- 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 773419] New: Review Request: warmux - 2D turn-based artillery game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: warmux - 2D turn-based artillery game https://bugzilla.redhat.com/show_bug.cgi?id=773419 Summary: Review Request: warmux - 2D turn-based artillery game Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jpope...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://jpopelka.fedorapeople.org/warmux.spec SRPM URL: http://jpopelka.fedorapeople.org/warmux-11.04.1-1.fc17.src.rpm Description: WarMUX is a game in the genre of "turn-based artillery games" like Scorched Earth or Worms. The project was started in December 2002 as Wormux, and was renamed to Warmux in November 2010. This is actually only a renaming review because the wormux package already exists (I'll retire it once the review is complete). Review ticket of wormux is bug #194436. -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 --- Comment #4 from Albert Strasheim 2012-01-11 14:23:24 EST --- Looks good to me. -- 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 771123] Review Request: fex - Field split/extraction like cut/awk
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=771123 Mattia Verga changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Mattia Verga 2012-01-11 14:13:15 EST --- All issues are fixed, looks good now: APPROVED. -- 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 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
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=771485 --- Comment #3 from Doug Ledford 2012-01-11 14:08:11 EST --- Albert, are you good with signing off on this package at this point? -- 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 772288] Review Request: infiniband-diags - various InfiniBand fabric test utilities
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=772288 --- Comment #12 from Fedora Update System 2012-01-11 14:07:56 EST --- infiniband-diags-1.5.12-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/infiniband-diags-1.5.12-1.fc16 -- 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 772288] Review Request: infiniband-diags - various InfiniBand fabric test utilities
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=772288 --- Comment #13 from Fedora Update System 2012-01-11 14:08:04 EST --- infiniband-diags-1.5.12-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/infiniband-diags-1.5.12-1.fc15 -- 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 772288] Review Request: infiniband-diags - various InfiniBand fabric test utilities
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=772288 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 772260] Review Request: srptools - utilities for mounting SRP shared resources
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=772260 --- Comment #6 from Fedora Update System 2012-01-11 13:50:53 EST --- srptools-0.0.4-11.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/srptools-0.0.4-11.fc15 -- 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 772260] Review Request: srptools - utilities for mounting SRP shared resources
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=772260 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 772260] Review Request: srptools - utilities for mounting SRP shared resources
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=772260 --- Comment #5 from Fedora Update System 2012-01-11 13:50:44 EST --- srptools-0.0.4-11.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/srptools-0.0.4-11.fc16 -- 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 772234] Review Request: qperf - Performance testing utility for RDMA networks
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=772234 --- Comment #4 from Fedora Update System 2012-01-11 13:48:13 EST --- qperf-0.4.6-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/qperf-0.4.6-4.fc16 -- 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 772234] Review Request: qperf - Performance testing utility for RDMA networks
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=772234 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 772234] Review Request: qperf - Performance testing utility for RDMA networks
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=772234 --- Comment #5 from Fedora Update System 2012-01-11 13:48:20 EST --- qperf-0.4.6-4.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/qperf-0.4.6-4.fc15 -- 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 772233] Review Request: mstflint - Mellanox firmware burning utility
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=772233 --- Comment #5 from Fedora Update System 2012-01-11 13:40:59 EST --- mstflint-1.4-6.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/mstflint-1.4-6.fc16 -- 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 772233] Review Request: mstflint - Mellanox firmware burning utility
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=772233 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 772233] Review Request: mstflint - Mellanox firmware burning utility
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=772233 --- Comment #6 from Fedora Update System 2012-01-11 13:41:07 EST --- mstflint-1.4-6.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/mstflint-1.4-6.fc15 -- 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 773021] Review Request: piglit - Collection of automated tests for OpenGL implementations
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=773021 Scott Tsai changed: What|Removed |Added Status|NEW |ASSIGNED CC||scottt...@gmail.com AssignedTo|nob...@fedoraproject.org|scottt...@gmail.com Flag||fedora-review? -- 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 772229] Review Request: perftest - Performance testing programs for InfiniBand networks
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=772229 --- Comment #6 from Fedora Update System 2012-01-11 13:23:21 EST --- perftest-1.3.0-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perftest-1.3.0-2.fc16 -- 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 772229] Review Request: perftest - Performance testing programs for InfiniBand networks
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=772229 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 772229] Review Request: perftest - Performance testing programs for InfiniBand networks
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=772229 --- Comment #7 from Fedora Update System 2012-01-11 13:23:29 EST --- perftest-1.3.0-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perftest-1.3.0-2.fc15 -- 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 773357] Review Request: felix-gogo-shell - Community OSGi R4 Service Platform Implementation - Basic Commands
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=773357 Roland Grunberg changed: What|Removed |Added Flag|fedora-review? |fedora-review+ -- 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 773357] Review Request: felix-gogo-shell - Community OSGi R4 Service Platform Implementation - Basic Commands
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=773357
--- Comment #1 from Roland Grunberg 2012-01-11 12:55:37
EST ---
Package Review
==
=== REQUIRED ITEMS ===
[X] Rpmlint output:
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[X] Package is named according to the Package Naming Guidelines[1].
[X] Spec file name must match the base package name, in the format
%{name}.spec.
[X] Package meets the Packaging Guidelines[2].
[X] Package successfully compiles and builds into binary rpms.
[X] Buildroot definition is not present
[X] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[X] License field in the package spec file matches the actual license.
License type:
[X] 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] All independent sub-packages have license of their own
[X] Spec file is legible and written in American English.
[X] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package: 9f530f0009df4948b25c49cde14cb2a5
MD5SUM upstream package: 9f530f0009df4948b25c49cde14cb2a5
[X] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[X] Package must own all directories that it creates or must require other
packages for directories it uses.
[X] Package does not contain duplicates in %files.
[X] File sections do not contain %defattr(-,root,root,-) unless changed with
good reason
[X] Permissions on files are set properly.
[X] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[X] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[X] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[X] Package does not own files or directories owned by other packages.
[X] Javadoc documentation files are generated and included in -javadoc
subpackage
[X] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[X] Packages have proper BuildRequires/Requires on jpackage-utils
[X] Javadoc subpackages have Require: jpackage-utils
[X] Package uses %global not %define
[-] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-] If source tarball includes bundled jar/class files these need to be
removed prior to building
[X] All filenames in rpm packages must be valid UTF-8.
[X] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[X] If package contains pom.xml files install it (including depmaps) even when
building with ant
[X] pom files has correct add_maven_depmap
=== Maven ===
[X] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-] If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[-] If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why
it's needed in a comment
[X] Package DOES NOT use %update_maven_depmap in %post/%postun
[X] Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro
=== Other suggestions ===
[X] If possible use upstream build method (maven/ant/javac)
[X] Avoid having BuildRequires on exact NVR unless necessary
[X] Package has BuildArch: noarch (if possible)
[X] Latest version is packaged.
[X] Reviewer should test that the package builds in mock.
Tested on:
fedora-rawhide-i386 mock environment
builds with mvn -fae clean install
Looks good to me.
--
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 773357] Review Request: felix-gogo-shell - Community OSGi R4 Service Platform Implementation - Basic Commands
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=773357 Roland Grunberg changed: What|Removed |Added Flag||fedora-review? -- 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 767622] Review Request: unixODBC-gui-qt - Qt tools for unixODBC
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=767622 Honza Horak changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2012-01-11 12:07:10 -- 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 574545] Review Request: python26-mysqldb : Interface to MySQL for python26 on EPEL5
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=574545 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 574545] Review Request: python26-mysqldb : Interface to MySQL for python26 on EPEL5
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=574545 --- Comment #15 from Fedora Update System 2012-01-11 11:55:39 EST --- python26-mysqldb-1.2.3-2.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python26-mysqldb-1.2.3-2.el5 -- 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 242311] Review Request: perl-Time-Duration - rounded or exact English expression of durations
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=242311 --- Comment #23 from Jon Ciesla 2012-01-11 11:36:26 EST --- Fixed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 588428] Review Request: rubygem-addressable - Improved URI/URL handling
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=588428 --- Comment #22 from Shawn Starr 2012-01-11 11:31:44 EST --- Can I get an update from the SCM team on this approval? -- 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 721179] Review Request: rubygem-extlib - Support library for DataMapper and Merb
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=721179 --- Comment #31 from Shawn Starr 2012-01-11 11:32:18 EST --- Can someone please review? -- 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
