[Bug 722713] Review Request: ace - Optimal route search in a complete graph

2011-08-17 Thread bugzilla
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=722713

--- Comment #4 from Mauricio Cleveland mauricio.clevel...@gmail.com 
2011-08-17 01:59:24 EDT ---
(In reply to comment #3)
 %defattr line is no longer needed unless you are building for the RPEL 
 
 Remove Requires:
 http://fedoraproject.org/wiki/Packaging:Guidelines#Requires
 Since RPM will find the dependency needed automaticly.
 
 BuildRoot line is not required unless building for the EPEL
 Also each time you update the SPEC file, bump up the release and note the
 change in the Changelog

OK.

 
 According to the homepage, the package is GPLv3+

OK.

 
 Also the %clean section isn't needed except for EPEL
 Prefix: tag isn't needed either
 

OK.

 Running rpmlint on the RPM package(not SRPM) gives that you need a man page 
 for
 it as well

bash-4.2$ rpmlint ace-1.5-4.fc15.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

 
 Description is too long
 http://fedoraproject.org/wiki/Common_Rpmlint_issues#description-line-too-long
 

OK.

 This is how mine would look if I did it, except with my name and email of
 course:
 http://github.com/ndowens/Fedora-Rpms/raw/master/SPECS/ace.spec
 
 The only issue that rpmlint gives me is that a man page is needed

OK.

http://code.google.com/p/curvatura/source/browse/ace.spec
http://code.google.com/p/curvatura/source/browse/RPMS/ace-1.5-4.fc15.noarch.rpm
http://code.google.com/p/curvatura/source/browse/RPMS/ace-1.5-4.fc15.src.rpm

I hope this all right.

-- 
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 728555] Review Request: screenie-composer - Fancy screenshot composer

2011-08-17 Thread bugzilla
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=728555

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-08-17 
02:41:46 EDT ---
screenie-composer-1.0.0-0.2.20110805git49c2630.fc16 has been submitted as an
update for Fedora 16.
https://admin.fedoraproject.org/updates/screenie-composer-1.0.0-0.2.20110805git49c2630.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 728555] Review Request: screenie-composer - Fancy screenshot composer

2011-08-17 Thread bugzilla
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=728555

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-08-17 
02:41:54 EDT ---
screenie-composer-1.0.0-0.2.20110805git49c2630.fc15 has been submitted as an
update for Fedora 15.
https://admin.fedoraproject.org/updates/screenie-composer-1.0.0-0.2.20110805git49c2630.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 728555] Review Request: screenie-composer - Fancy screenshot composer

2011-08-17 Thread bugzilla
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=728555

Fedora Update System upda...@fedoraproject.org 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 225645] Merge Review: cleanfeed

2011-08-17 Thread bugzilla
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=225645

Lukáš Nykrýn lnyk...@redhat.com changed:

   What|Removed |Added

 CC||lnyk...@redhat.com
 AssignedTo|nob...@fedoraproject.org|lnyk...@redhat.com

-- 
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 712272] Review Request: ghc-unix-compat - A portable POSIX-compatibility layer for Haskell

2011-08-17 Thread bugzilla
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=712272

Jens Petersen peter...@redhat.com 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 712272] Review Request: ghc-unix-compat - A portable POSIX-compatibility layer for Haskell

2011-08-17 Thread bugzilla
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=712272

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|peter...@redhat.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 662259] Review Request: git-annex - Manage files with git, without checking their contents in

2011-08-17 Thread bugzilla
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=662259

Bug 662259 depends on bug 713361, which changed state.

Bug 713361 Summary: Review Request: ghc-pcre-light - Haskell binding to pcre
https://bugzilla.redhat.com/show_bug.cgi?id=713361

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|MODIFIED
 Status|MODIFIED|ON_QA
 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #6 from Jens Petersen peter...@redhat.com 2011-08-17 03:23:26 EDT 
---
I assume Andrew is not intending to review this?

-- 
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 731066] Review Request: django-annoying - Eliminate annoying things in the Django framework

2011-08-17 Thread bugzilla
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=731066

Rahul Sundaram methe...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review+

--- Comment #1 from Rahul Sundaram methe...@gmail.com 2011-08-17 04:19:01 EDT 
---
You don't need to override CFLAGS explicitly in this case.  Other than that, 
this follows the standard template.  

==  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 731110] Review Request: maven-filtering - Shared component providing resource filtering

2011-08-17 Thread bugzilla
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=731110

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2011-08-17 
04:26:30 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Rpmlint output:
0 errors, 0 warnings.
Impressive :)
[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: ASL 2.0
[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:0fb6fb5d407bd7c8d752762859fe530c
MD5SUM upstream package:0fb6fb5d407bd7c8d752762859fe530c
[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.
[x]  Package requires 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.
[x]  Fully versioned dependency in subpackages, if present.
[x]  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
[x]  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.


*** 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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Tomas Mraz tm...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||tm...@redhat.com
 AssignedTo|nob...@fedoraproject.org|tm...@redhat.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 608574] Review Request: tinymce - Web based Javascript HTML WYSIWYG editor control

2011-08-17 Thread bugzilla
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=608574

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #16 from Jens Petersen peter...@redhat.com 2011-08-17 05:36:27 
EDT ---
Could you please push the updates for releases into Bodhi?

-- 
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 690728] Review Request: Nitrate - A test case management system written in Django

2011-08-17 Thread bugzilla
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=690728

--- Comment #53 from Jens Petersen peter...@redhat.com 2011-08-17 05:34:20 
EDT ---
Do you have an update to this package then?
so we can continue with the review or
do you prefer we quickly review the new packages first?

-- 
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 630206] Review Request: ghc-hledger-lib - Core types and utilities for working with hledger data

2011-08-17 Thread bugzilla
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=630206

--- Comment #11 from Jens Petersen peter...@redhat.com 2011-08-17 05:40:03 
EDT ---
Ping-a-ling?!

-- 
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 690728] Review Request: Nitrate - A test case management system written in Django

2011-08-17 Thread bugzilla
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=690728

--- Comment #54 from Yuguang Wang yuw...@redhat.com 2011-08-17 05:46:48 EDT 
---
I'd prefer a quick review on the new packages first.
for nitrate, I think I could provide a new package early next week.
Thanks. :)
(In reply to comment #53)
 Do you have an update to this package then?
 so we can continue with the review or
 do you prefer we quickly review the new packages first?

-- 
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 225645] Merge Review: cleanfeed

2011-08-17 Thread bugzilla
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=225645

Lukáš Nykrýn lnyk...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?

--- Comment #4 from Lukáš Nykrýn lnyk...@redhat.com 2011-08-17 05:53:22 EDT 
---
Checked git commit:
a79564bb2bc60af43dde6748eafb23d853f80bde

YES source files match upstream
YES package meets naming and versioning guidelines. 
YES specfile is properly named, is cleanly written and uses macros consistently
YES dist tag is present
YES clean section and buildroot present
YES license field matches the actual license
YES license is open source-compatible 
YES License text included in package
YES latest version is being packaged
N/A BuildRequires are proper
N/A compiler flags are appropriate
YES package builds in mock (Rawhide/x86_64)
N/A debuginfo package looks complete
YES rpmlint is silent
YES final provides and requires look sane
 -- dependence on perl is added automatically, but maybe it would by good to
define it explicitly
N/A %check is present and all tests pass
N/A no shared libraries are added to the regular linker search paths
YES owns the directories it creates
YES doesn't own any directories it shouldn't
YES no duplicates in %files
N/A scriptlets must be sane
YES code, not content
N/A large documentation files must go in a -doc subpackage
YES %docs are not necessary for the proper functioning of the package
YES no headers
YES no pkgconfig files
YES no libtool .la droppings.
YES not a GUI app.

Additional comments:
User news is not by default in system, so it use root, I don't know if it is
desired behavior.

-- 
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 717864] Review Request: ghc-case-insensitive - Haskell case-insensitive string comparison

2011-08-17 Thread bugzilla
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=717864

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Jens Petersen peter...@redhat.com 2011-08-17 05:53:22 EDT 
---
Thank you for reviewing :)

New Package SCM Request
===
Package Name: ghc-case-insensitive
Short Description: Haskell case-insensitive string comparison
Owners: petersen
Branches: f14 f15 f16
InitialCC: haskell-sig

-- 
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 726210] Review Request: freewrl - X3D / VRML visualization program

2011-08-17 Thread bugzilla
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=726210

--- Comment #19 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 06:00:18 EDT ---
freewrl-1.22.12-0.3.pre2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/freewrl-1.22.12-0.3.pre2.el6

-- 
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 675557] Review Request: matreshka - set of Ada libraries to help to develop information systems

2011-08-17 Thread bugzilla
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=675557

--- Comment #22 from Pavel Zhukov pa...@zhukoff.net 2011-08-17 06:11:11 EDT 
---
New build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3279184

SRPMS: 
http://landgraf.fedorapeople.org/packages/requested/matreshka/matreshka-0.1.1-5.fc15.src.rpm
SPEC:
http://landgraf.fedorapeople.org/packages/requested/matreshka/matreshka.spec

rpmlint: http://fpaste.org/S5OQ/

-- 
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 708115] Review Request: python-atfork - Provides an API to register pre and post callbacks when forking

2011-08-17 Thread bugzilla
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=708115

--- Comment #3 from Martin Krizek mkri...@redhat.com 2011-08-17 06:38:34 EDT 
---
(In reply to comment #2)
 (In reply to comment #1)
  Hi James, my *informal* review:
 
 Hi Martin, many thanks for your informal review feedback!
 
  snip
 
  [ TODO ] MUST: A package must own all directories that it creates. If it 
  does
  not create a directory that it uses, then it should require a package which
  does create that directory.
 
 This should be [ OK ] as far as I know, since the package only includes a
 python library at this time.
 

TODO was for me, I didn't notice that I forgot to change it to OK. You are
right, sorry about that.

  [ BAD ] SHOULD: If the source package does not include license text(s) as a
  separate file from upstream, the packager SHOULD query upstream to include 
  it.
  === License is not included.
 
 Thank you for finding this, I have raised this issue with upstream and will
 monitor for updates.  
 
 https://code.google.com/p/python-atfork/issues/detail?id=2
 
  [ BAD ] SHOULD: your package should contain man pages for binaries/scripts. 
  If
  it doesn't, work with upstream to add them where they make sense. === No 
  man
  pages.
 
 There are no scripts or binaries included with python-atfork.  My 
 understanding
 is that this should be NA for this particular package.  Feel free to correct
 me, though.

Right.

-- 
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 573929] Review Request: perl-Compress-Raw-Zlib - Low-Level Interface to zlib compression library

2011-08-17 Thread bugzilla
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=573929

--- Comment #31 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 06:54:09 EDT ---
perl-threads-shared-1.37-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/perl-threads-shared-1.37-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 708115] Review Request: python-atfork - Provides an API to register pre and post callbacks when forking

2011-08-17 Thread bugzilla
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=708115

--- Comment #4 from Neal Becker ndbeck...@gmail.com 2011-08-17 07:07:56 EDT 
---
I use python-atfork along with python-signalfd.  Are you interested in
packaging python-signalfd?

http://launchpad.net/python-signalfd

-- 
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 573929] Review Request: perl-Compress-Raw-Zlib - Low-Level Interface to zlib compression library

2011-08-17 Thread bugzilla
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=573929

--- Comment #32 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 07:09:51 EDT ---
perl-parent-0.225-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/perl-parent-0.225-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 663181] Review Request: ompc - MATLAB to Python syntax adapting compiler

2011-08-17 Thread bugzilla
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=663181

--- Comment #5 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-17 07:19:31 
EDT ---
Ping, what is the status 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 710904] Review Request: octave-communications - Communications for Octave

2011-08-17 Thread bugzilla
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=710904

--- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-17 07:18:44 
EDT ---
Please add comments about the functionality of Patch0 and Patch1.

**

Why aren't you using %{octave_pkg_build}?

**

You need to add the -v flag to the mkoctfile commend so that build flags can be
seen.

**

The package fails to build:

+ '[' -e
/builddir/build/BUILDROOT/octave-communications-1.0.10-1.fc15.x86_64/usr/share/octave/packages/communications-1.0.10/packinfo/on_uninstall.m
']'
+ echo 'function on_uninstall (desc)'
/var/tmp/rpm-tmp.7FEKFz: line 44:
/builddir/build/BUILDROOT/octave-communications-1.0.10-1.fc15.x86_64/usr/share/octave/packages/communications-1.0.10/packinfo/on_uninstall.m:
No such file or directory
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.7FEKFz (%install)
Bad exit status from /var/tmp/rpm-tmp.7FEKFz (%install)
Child returncode was: 1

-- 
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 658420] Review Request: zorba - General purpose XQuery processor

2011-08-17 Thread bugzilla
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=658420

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #13 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-17 07:30:43 
EDT ---
Whoa, seems we all had forgotten this. As per comment #6, all the issues in the
review were attended to, and the only thing blocking the review was the
bundling exception that has been granted.

However, there has been a change of guidelines, and you MUST make the Requires
architecture specific:
 Requires: %{name} = %{version}-%{release}%{?_isa}

Otherwise I believe there are no other issues, and thus this package must be

APPROVED


Please add the %{?_isa} stuff before import to git.

-- 
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 717864] Review Request: ghc-case-insensitive - Haskell case-insensitive string comparison

2011-08-17 Thread bugzilla
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=717864

--- Comment #5 from Jon Ciesla l...@jcomserv.net 2011-08-17 08:02:11 EDT ---
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 226505] Merge Review: ttcp

2011-08-17 Thread bugzilla
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=226505

Michal Sekletar msekl...@redhat.com changed:

   What|Removed |Added

 CC||msekl...@redhat.com
 AssignedTo|nob...@fedoraproject.org|msekl...@redhat.com

-- 
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 727780] Review Request: qt-ruby - Ruby bindings for Qt 4

2011-08-17 Thread bugzilla
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=727780

Jaroslav Reznik jrez...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #13 from Jaroslav Reznik jrez...@redhat.com 2011-08-17 08:22:09 
EDT ---
Ok,
thanks Than and Vit! I agree with Than regarding rails_support but of course if
someone asks, it can be packaged separately.

rpmlint ruby-qt-4.7.0-1.fc15.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

package builds correctly
http://koji.fedoraproject.org/koji/taskinfo?taskID=3279572

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 729512] Review Request: graphite2 - Font rendering capabilities for complex non-Roman writing systems

2011-08-17 Thread bugzilla
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=729512

--- Comment #4 from Martin Krizek mkri...@redhat.com 2011-08-17 08:28:03 EDT 
---
$ rpmlint rpmbuild/SRPMS/graphite2-1.0.1-2.fc15.src.rpm
rpmbuild/SPECS/graphite2.spec rpmbuild/RPMS/x86_64/graphite2-*
graphite2.x86_64: W: no-manual-page-for-binary gr2fonttest
graphite2.x86_64: W: no-manual-page-for-binary comparerenderer
graphite2.x86_64: W: no-manual-page-for-binary gr2fonttest
graphite2.x86_64: W: no-manual-page-for-binary comparerenderer
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/gr_features.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/direct_machine.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/TtfTypes.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/opcodes.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Bidi.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Face.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Face.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/include/graphite2/XmlLog.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/include/graphite2/Types.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/gr_font.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/SegCacheEntry.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Sparse.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/NameTable.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/opcode_table.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/XmlTraceLog.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/GlyphFaceCache.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/NameTable.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/tests/comparerenderer/CompareRenderer.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/SegCache.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/include/graphite2/Font.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/SegCache.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/SegCacheStore.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/CachedFace.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Slot.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/TtfUtil.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Font.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/gr_segment.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Main.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/tests/comparerenderer/RenderedLine.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Silf.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/FeatureMap.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/tests/comparerenderer/FeatureParser.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Segment.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/Segment.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/tests/comparerenderer/Renderer.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/CmapCache.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/SegCacheStore.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/XmlTraceLogTags.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/tests/comparerenderer/GrRenderer.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/GlyphFaceCache.cpp
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/FeatureVal.h
graphite2-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/graphite2-1.0.1/src/GlyphFace.cpp

[Bug 589867] Review Request: logcheck - analyzes logfiles and sends email

2011-08-17 Thread bugzilla
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=589867

--- Comment #36 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 08:23:47 EDT ---
logcheck-1.3.13-4.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/logcheck-1.3.13-4.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 729512] Review Request: graphite2 - Font rendering capabilities for complex non-Roman writing systems

2011-08-17 Thread bugzilla
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=729512

--- Comment #5 from Martin Krizek mkri...@redhat.com 2011-08-17 08:30:49 EDT 
---
During informal review I didn't find any issues, except those stated in the
rpmlint output. I do however have some comments on the spec file:

I think following lines
cp LICENSE $RPM_BUILD_ROOT/usr/share/graphite2

%files
%doc /usr/share/graphite2/LICENSE

should be replaced with just

%files
%doc LICENSE

Also this line:
%doc /usr/share/graphite2/*.cmake

seems incorrect to me. Assuming *.cmake belongs to %doc (which I don't know)
and therefore package can run without those files - is it neccessary to use the
whole path? At least I would replace /usr/share with the macro (and graphite2
for that matter).

-- 
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 708115] Review Request: python-atfork - Provides an API to register pre and post callbacks when forking

2011-08-17 Thread bugzilla
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=708115

--- Comment #5 from James Laska jla...@redhat.com 2011-08-17 08:34:17 EDT ---
(In reply to comment #4)
 I use python-atfork along with python-signalfd.  Are you interested in
 packaging python-signalfd?
 
 http://launchpad.net/python-signalfd

Sure!  Contact me directly and we can figure out how to proceed.  I have it
packaged locally now ... but I'd need some clarification from upstream on the
appropriate LICENSE.

-- 
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 225622] Merge Review: boost

2011-08-17 Thread bugzilla
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=225622

--- Comment #40 from Denis Arnaud denis.arnaud_fed...@m4x.org 2011-08-17 
08:41:00 EDT ---
The mainstream Boost package is now tracked by #711845. So, this bug could be
closed.

-- 
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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|rdie...@math.unl.edu
   Flag||fedora-review?

--- Comment #2 from Rex Dieter rdie...@math.unl.edu 2011-08-17 08:46:25 EDT 
---
I can help review this.

-- 
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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

--- Comment #3 from Richard Shaw hobbes1...@gmail.com 2011-08-17 08:53:20 EDT 
---
Great! I've also got spacenavd and spnavcfg packaged but haven't submitted
review requests just 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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Tomas Mraz tm...@redhat.com 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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

--- Comment #4 from Rex Dieter rdie...@math.unl.edu 2011-08-17 09:00:46 EDT 
---
naming: ok

license: BSD, ok

sources: ok
b85a0f4ab711e2d4f73a40e2e371f5ae  libspnav-0.2.2.tar.gz

scriptlets: ok


1.  MUST:  build doesn't use $RPM_OPT_FLAGS.  maybe use something like:
--- configure.opt   2011-08-17 07:47:12.275486930 -0500
+++ configure   2011-08-17 07:52:24.599256027 -0500
@@ -74,7 +74,7 @@
 fi

 if [ $OPT = 'yes' ]; then
-   echo 'opt = -O3' Makefile
+   echo 'opt = -O3 $(RPM_OPT_FLAGS)' Makefile
 fi

 if [ $X11 = 'yes' ]; then


2.  MUST.  static library build/packaged.   Please provide
justification/rationale for doing so, or remove it.

3.  SHOULD.  In %files, be explicit about what soname to package, so future abi
bumps don't come as a surprise, use something like
%files
%{_libdir}/libspnav.so.0*
instead?

4.  SHOULD.   Given all the configure/makefile hacks (optflags, DESTDIR, lib64)
in the .spec, I'm wondering if it may be more worthwhile to make an
upstreamable patch instead?  I can help do that, if that's agreeable with you.

-- 
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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

Miroslav Lichvar mlich...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Miroslav Lichvar mlich...@redhat.com 2011-08-17 09:12:46 
EDT ---
It looks good. 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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

--- Comment #5 from Richard Shaw hobbes1...@gmail.com 2011-08-17 09:12:15 EDT 
---
(In reply to comment #4)
 1.  MUST:  build doesn't use $RPM_OPT_FLAGS.  maybe use something like:
 --- configure.opt   2011-08-17 07:47:12.275486930 -0500
 +++ configure   2011-08-17 07:52:24.599256027 -0500
 @@ -74,7 +74,7 @@
  fi
 
  if [ $OPT = 'yes' ]; then
 -   echo 'opt = -O3' Makefile
 +   echo 'opt = -O3 $(RPM_OPT_FLAGS)' Makefile
  fi
 
  if [ $X11 = 'yes' ]; then

I'll get this updated and see if a patch wouldn't be better then a bunch of sed
hacks. I usually just sed update CFLAGS = to CFLAGS += 


 2.  MUST.  static library build/packaged.   Please provide
 justification/rationale for doing so, or remove it.

Yeah, I was wondering about that. I only packaged it because it built it. I
guess I just need to rm -f it so I don't get an installed but unpackaged
error


 3.  SHOULD.  In %files, be explicit about what soname to package, so future 
 abi
 bumps don't come as a surprise, use something like
 %files
 %{_libdir}/libspnav.so.0*
 instead?

OK


 4.  SHOULD.   Given all the configure/makefile hacks (optflags, DESTDIR, 
 lib64)
 in the .spec, I'm wondering if it may be more worthwhile to make an
 upstreamable patch instead?  I can help do that, if that's agreeable with you.

I'll ask but these makefiles are VERY simple and the packages have not been
updated recently so I wonder how active upstream is.

I'll post a new spec and SRPM shortly.

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Tomas Mraz tm...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(kalevlember@gmail
   ||.com)

--- Comment #11 from Tomas Mraz tm...@redhat.com 2011-08-17 09:19:12 EDT ---
There is a problem that the -devel subpackage puts doc files in
/usr/share/gtk-doc/html but this directory is not owned by anything that is
required by p11-kit-devel.

Also parts of the documentation document the configuration file format and that
should be perhaps documented somehow in the main package, or even some kind of
example config files included in the main 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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

--- Comment #6 from Richard Shaw hobbes1...@gmail.com 2011-08-17 09:33:22 EDT 
---
Spec URL: http://hobbes1069.fedorapeople.org/libspnav/libspnav.spec
SRPM URL:
http://hobbes1069.fedorapeople.org/libspnav/libspnav-0.2.2-2.fc15.src.rpm

* Wed Aug 17 2011 Richard Shaw hobbes1...@gmail.com - 0.2.2-2
- Patched make file to honor Fedora CFLAGS defaults.
- Removed static library package.
- Other minor updates to the spec file.

This addresses 1-3. If you want to help with the makefile that's great as I'm
not an expert at it :)

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(kalevlember@gmail |
   |.com)   |

--- Comment #12 from Kalev Lember kalevlem...@gmail.com 2011-08-17 09:47:52 
EDT ---
(In reply to comment #11)
 There is a problem that the -devel subpackage puts doc files in
 /usr/share/gtk-doc/html but this directory is not owned by anything that is
 required by p11-kit-devel.

That directory is owned by the p11-kit-devel package itself to avoid a runtime
dependency on the gtk-doc package:
$ rpm -qlp p11-kit-devel-0.3-1.fc15.x86_64.rpm | grep gtk-doc
/usr/share/gtk-doc
/usr/share/gtk-doc/html
/usr/share/gtk-doc/html/p11-kit
/usr/share/gtk-doc/html/p11-kit/api-index-full.html
/usr/share/gtk-doc/html/p11-kit/config-format.html
...

See the section in the packaging guidelines on File and Directory Ownership [1]
that specifically explains how to deal with gtk-doc directory ownership:
https://fedoraproject.org/wiki/Packaging/Guidelines#The_directory_is_owned_by_a_package_which_is_not_required_for_your_package_to_function


 Also parts of the documentation document the configuration file format and
 that should be perhaps documented somehow in the main package, or even
 some kind of example config files included in the main package.

I don't really want to put large html documentation in the main package.
p11-kit is probably going to be included in the live CD and it's nice to keep
the size down. However, instead of having the docs in the -devel package, it
would be easy to split them out to a separate -doc package. In the future, when
upstream has some example config files, we could also put these in the -doc
subpackage. Would that be better than shipping all the docs in -devel?

-- 
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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Ankur Sinha sanjay.an...@gmail.com 2011-08-17 10:40:32 
EDT ---
Thank you for the review Miroslav :)

New Package SCM Request
===
Package Name: gnumed
Short Description: The gnumed client
Owners: ankursinha
Branches: f14 f15 f16 
InitialCC: susmit mrceresa

-- 
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 662259] Review Request: git-annex - Manage files with git, without checking their contents in

2011-08-17 Thread bugzilla
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=662259

--- Comment #7 from Andrew McNabb amcn...@mcnabbs.org 2011-08-17 10:47:22 EDT 
---
I originally started to, but as I got into it, all of the Haskell-specific
stuff creeped me out.

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Tomas Mraz tm...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #13 from Tomas Mraz tm...@redhat.com 2011-08-17 10:48:56 EDT ---
You're right, I somehow overlooked that in the file list.

As for the documentation - I don't think splitting it to a separate -doc
subpackage is a solution. The configuration file format is really simple and
the documentation or example would be pretty minimal and doing a separate
subpackage for it would be an overkill.

I'd really like to see the global and example module config file directly in
the main package. And/or manual page for the p11-kit binary. However I will not
block the package on this.

rpmlint output:
rpmlint -v SRPMS/p11-kit-0.3-1.fc14.src.rpm RPMS/x86_64/p11-kit*
p11-kit.src: I: checking
p11-kit.src: I: checking-url http://p11-glue.freedesktop.org/p11-kit.html
(timeout 10 seconds)
p11-kit.src: I: checking-url
http://p11-glue.freedesktop.org/releases/p11-kit-0.3.tar.gz (timeout 10
seconds)
p11-kit-debuginfo.x86_64: I: checking
p11-kit-debuginfo.x86_64: I: checking-url
http://p11-glue.freedesktop.org/p11-kit.html (timeout 10 seconds)
p11-kit-devel.x86_64: I: checking
p11-kit-devel.x86_64: I: checking-url
http://p11-glue.freedesktop.org/p11-kit.html (timeout 10 seconds)
p11-kit.x86_64: I: checking
p11-kit.x86_64: I: checking-url http://p11-glue.freedesktop.org/p11-kit.html
(timeout 10 seconds)
p11-kit.x86_64: W: no-manual-page-for-binary p11-kit
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

A manual page for the p11-kit would be nice however this is not a blocker.

The package is named according to the naming guidelines.
The package meets licensing and packaging guidelines.
The source tarball is the same as the upstream tarball.

Note for future - there will have to be probably added gettext if the
localization of p11-kit is completed and configure also searches for the
gtk-doc binaries. But they are both not needed to build the package now.

The requires on the base package should be changed to
 Requires: %{name}%{?_isa} = %{version}-%{release}
But this is again a nitpick, but please change it before the import.

The package is ACCEPTed.

-- 
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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

--- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-08-17 10:48:55 EDT ---
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #15 from Kalev Lember kalevlem...@gmail.com 2011-08-17 11:01:56 
EDT ---
New Package SCM Request
===
Package Name: p11-kit
Short Description: Library for loading and sharing PKCS#11 modules
Owners: kalev mclasen
Branches:
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 730764] Review Request: wxpdfdoc - A library for creating PDF documents in C++ with wxWidgets

2011-08-17 Thread bugzilla
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=730764

--- Comment #4 from Domingo Becker domingobec...@gmail.com 2011-08-17 
11:00:22 EDT ---
New spec with Dan's proposal for every 64 bit architectures.

Spec URL: http://beckerde.fedorapeople.org/wxpdfdoc/wxpdfdoc.spec

SRPM URL:
http://beckerde.fedorapeople.org/wxpdfdoc/wxpdfdoc-0.9.1-3.fc14.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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

--- Comment #14 from Kalev Lember kalevlem...@gmail.com 2011-08-17 11:00:15 
EDT ---
(In reply to comment #13)
 The requires on the base package should be changed to
  Requires: %{name}%{?_isa} = %{version}-%{release}
 But this is again a nitpick, but please change it before the import.

Will do. Thanks for the review, Tomáš!

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

--- Comment #16 from Tomas Mraz tm...@redhat.com 2011-08-17 11:26:30 EDT ---
Can you please add me as comaintainer?

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

--- Comment #17 from Jon Ciesla l...@jcomserv.net 2011-08-17 11:28:49 EDT ---
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 662259] Review Request: git-annex - Manage files with git, without checking their contents in

2011-08-17 Thread bugzilla
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=662259

--- Comment #8 from Ben Boeckel maths...@gmail.com 2011-08-17 11:29:37 EDT ---
git-annex is on Hackage now with a cabal build system, so I'll update to use it
rather than the Makefile stuff sometime this week hopefully.

-- 
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 728691] Review Request: django-registration - A user-registration application for Django

2011-08-17 Thread bugzilla
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=728691

--- Comment #10 from Praveen Kumar kumarpraveen.nit...@gmail.com 2011-08-17 
11:35:39 EDT ---
New Package SCM Request
===
Package Name: django-registration
Short Description: A user-registration application for Django
Owners: sundaram pjp kumarpraveen
Branches: f14 f15 f16 el6

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

--- Comment #18 from Kalev Lember kalevlem...@gmail.com 2011-08-17 11:38:17 
EDT ---
Grr, I forgot to request the f16 branch. Tomas, I think you'll need to request
the ACLs for the master branch in pkgdb, but the SCM admin request should cover
the ACLs in f16.

-- 
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 731066] Review Request: django-annoying - Eliminate annoying things in the Django framework

2011-08-17 Thread bugzilla
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=731066

--- Comment #2 from Praveen Kumar kumarpraveen.nit...@gmail.com 2011-08-17 
11:34:14 EDT ---
New Package SCM Request
===
Package Name: django-annoying
Short Description: Eliminate annoying things in the Django framework
Owners: sundaram pjp kumarpraveen
Branches: f14 f15 f16 el6

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #19 from Kalev Lember kalevlem...@gmail.com 2011-08-17 11:38:41 
EDT ---
Package Change Request
==
Package Name: p11-kit
New Branches: f16
Owners: kalev mclasen tmraz

-- 
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

--- Comment #20 from Jon Ciesla l...@jcomserv.net 2011-08-17 11:40:36 EDT ---
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 725905] Review Request: p11-kit - Library for loading and sharing PKCS#11 modules

2011-08-17 Thread bugzilla
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=725905

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||p11-kit-0.3-2.fc16
 Resolution||NEXTRELEASE
Last Closed||2011-08-17 11:59:48

--- Comment #21 from Kalev Lember kalevlem...@gmail.com 2011-08-17 11:59:48 
EDT ---
Package imported and built; closing the ticket.

-- 
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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

Magnus Tuominen magnus.tuomi...@gmail.com changed:

   What|Removed |Added

 CC||magnus.tuomi...@gmail.com

--- Comment #6 from Magnus Tuominen magnus.tuomi...@gmail.com 2011-08-17 
12:09:21 EDT ---
There seems to be missing some build requires: qca2-devel and maybe boost-devel
too

-- 
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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 12:16:23 EDT ---
gnumed-0.9.9-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/gnumed-0.9.9-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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 12:16:31 EDT ---
gnumed-0.9.9-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/gnumed-0.9.9-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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

Fedora Update System upda...@fedoraproject.org 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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 12:16:40 EDT ---
gnumed-0.9.9-4.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/gnumed-0.9.9-4.fc14

-- 
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 727780] Review Request: qt-ruby - Ruby bindings for Qt 4

2011-08-17 Thread bugzilla
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=727780

Ngo Than t...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs+

--- Comment #14 from Ngo Than t...@redhat.com 2011-08-17 12:19:09 EDT ---
Thanks!

New Package SCM Request
===
Package Name: ruby-qt
Short Description: Ruby bindings for Qt 4
Owners: than rdieter jreznik kkofler ltinkl rnovacek
Branches: 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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-17 Thread bugzilla
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=731455

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

 CC||methe...@gmail.com
  Alias||django-authopenid

-- 
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 731455] New: Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: django-authopenid - Django application to integrate 
Django authentication system with OpenID

https://bugzilla.redhat.com/show_bug.cgi?id=731455

   Summary: Review Request: django-authopenid - Django application
to integrate Django authentication system with OpenID
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sanjay.an...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://ankursinha.fedorapeople.org/django-authopenid/django-authopenid.spec
SRPM URL:
http://ankursinha.fedorapeople.org/django-authopenid/django-authopenid-1.0-1.20110817hg.fc15.src.rpm
Description: 
A Django application that allows you to authenticate your users 
while using django auth contrib.

This application allows a user to connect to you website with :

-legacy account : username/password
-openid url 

The idea is to have a workflow integrating openid and legacy authentification.

If the user connects with an openid he could associate it with its legaccy
account or just create a new django account. When the user is connected you
could manage him like you usually do with `auth contrib or django-registration
which is the default.

This application also provide views to:
 * change password
 * change account email
 * change associated openid
 * delete account
 * send a new password 

Source produced By Benoît Chesneau for Enki Multimedia for the Friendsnippets
project.


=

Only spelling errors in rpmlint :)

-- 
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 663181] Review Request: ompc - MATLAB to Python syntax adapting compiler

2011-08-17 Thread bugzilla
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=663181

--- Comment #6 from Alon Levy al...@redhat.com 2011-08-17 13:52:11 EDT ---
My second attempt at a rpm package at the time, the real blocker was a problem
with setup.py, I don't remember what it was unfortunately, so I'll try to give
it another try, I still want to get it in.

Alon

-- 
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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

--- Comment #8 from Rex Dieter rdie...@math.unl.edu 2011-08-17 13:52:21 EDT 
---
See bug #731456 for jreen 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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Depends on||731456(jreen)

-- 
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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2011-08-17 13:50:15 EDT 
---
Spec URL: http://rdieter.fedorapeople.org/rpms/tomahawk/tomahawk.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/tomahawk/tomahawk-0.2.1-2.fc15.src.rpm

%changelog
* Wed Aug 17 2011 Rex Dieter rdie...@fedoraproject.org 0.2.1-2
- unbundle jreen, BR: jreen-devel
- BR: boost-devel qca2-devel
- Requires: qca-ossl

-- 
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 731456] New: Review Request: jreen - Qt XMPP Library

2011-08-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: jreen - Qt XMPP Library

https://bugzilla.redhat.com/show_bug.cgi?id=731456

   Summary: Review Request: jreen - Qt XMPP Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rdie...@math.unl.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://rdieter.fedorapeople.org/rpms/tomahawk/jreen.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/tomahawk/jreen-0.1.0-0.2.20110816.fc15.src.rpm
Description: Qt XMPP Library

-- 
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 731456] Review Request: jreen - Qt XMPP Library

2011-08-17 Thread bugzilla
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=731456

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Blocks||713461, 656997(kde-reviews)
  Alias||jreen

-- 
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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-17 Thread bugzilla
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=731455

--- Comment #1 from Rahul Sundaram methe...@gmail.com 2011-08-17 14:10:01 EDT 
---
Use the description and tarball from
http://pypi.python.org/pypi/django-authopenid/ instead of generating it
yourself.  Snapshots can change API and break existing software that depends on
it.  Unless upstream does a new stable release, we shouldn't be typically using
it.

-- 
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 715180] Review Request: FreeMat - A free environment for rapid engineering, scientific prototyping and data processing

2011-08-17 Thread bugzilla
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=715180

Richard Shaw hobbes1...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|hobbes1...@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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

--- Comment #9 from Rex Dieter rdie...@math.unl.edu 2011-08-17 14:42:13 EDT 
---
f16 scratch build (using bundled jreen for now, to allow non-bootstrapped
building):
http://koji.fedoraproject.org/koji/taskinfo?taskID=3280479

-- 
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 730851] Review Request: chocolate-doom - Conservative Doom source port

2011-08-17 Thread bugzilla
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=730851

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

 CC||sanjay.an...@gmail.com
 AssignedTo|nob...@fedoraproject.org|sanjay.an...@gmail.com
   Flag||fedora-review?

--- Comment #1 from Ankur Sinha sanjay.an...@gmail.com 2011-08-17 14:39:02 
EDT ---
Hello,

I shall take up this ticket. 

Thanks,
Ankur

-- 
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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-17 Thread bugzilla
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=731455

--- Comment #2 from Ankur Sinha sanjay.an...@gmail.com 2011-08-17 14:46:23 
EDT ---
Hello,

Updated

http://ankursinha.fedorapeople.org/django-authopenid/django-authopenid.spec

http://ankursinha.fedorapeople.org/django-authopenid/django-authopenid-1.0.1-2.fc15.src.rpm

Thanks,
Ankur

-- 
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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-17 Thread bugzilla
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=731455

--- Comment #4 from Rahul Sundaram methe...@gmail.com 2011-08-17 14:52:03 EDT 
---
Note that you can remove the first line that defines python_sitelib as that is
redundant now. You should also use name and version macros more instead of
hardcoding those but these are not blockers.

-- 
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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-17 Thread bugzilla
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=731455

Rahul Sundaram methe...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #3 from Rahul Sundaram methe...@gmail.com 2011-08-17 14:49:29 EDT 
---


==  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 731456] Review Request: jreen - Qt XMPP Library

2011-08-17 Thread bugzilla
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=731456

--- Comment #1 from Rex Dieter rdie...@math.unl.edu 2011-08-17 14:49:37 EDT 
---
f16 scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3280425

-- 
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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2011-08-17 14:51:39 EDT 
---
good enough methinks, 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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

Richard Shaw hobbes1...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Flag||fedora-cvs?

--- Comment #8 from Richard Shaw hobbes1...@gmail.com 2011-08-17 15:06:41 EDT 
---
New Package SCM Request
===
Package Name: libspnav
Short Description: Open source alternative to 3DConnextion drivers
Owners: hobbes1069
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 731515] New: Review Request: qtweetlib - Qt Twitter library

2011-08-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: qtweetlib - Qt Twitter library

https://bugzilla.redhat.com/show_bug.cgi?id=731515

   Summary: Review Request: qtweetlib - Qt Twitter library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rdie...@math.unl.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://rdieter.fedorapeople.org/rpms/tomahawk/qtweetlib.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/tomahawk/qtweetlib-0.0-0.1.20110602.fc15.src.rpm
Description: Qt Twitter library

-- 
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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Alias||tomahawk

-- 
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 713461] Review Request: tomahawk - The Social Music Player

2011-08-17 Thread bugzilla
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=713461

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Depends on||731515(qtweetlib)

-- 
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 731515] Review Request: qtweetlib - Qt Twitter library

2011-08-17 Thread bugzilla
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=731515

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Blocks||713461(tomahawk),
   ||656997(kde-reviews)
  Alias||qtweetlib

-- 
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 731519] New: Review Request: spacenavd - A free, compatible alternative for 3Dconnecxion's input drivers

2011-08-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: spacenavd - A free, compatible alternative for 
3Dconnecxion's input drivers

https://bugzilla.redhat.com/show_bug.cgi?id=731519

   Summary: Review Request: spacenavd - A free, compatible
alternative for 3Dconnecxion's input drivers
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: hobbes1...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://hobbes1069.fedorapeople.org/spacenavd/spacenavd.spec
SRPM URL:
http://hobbes1069.fedorapeople.org/spacenavd/spacenavd-0.5-1.fc15.src.rpm
Description:
Spacenavd, is a free software replacement user-space driver (daemon), for
3Dconnexion's space-something 6dof input devices. It's compatible with the
original 3dxsrv proprietary daemon provided by 3Dconnexion, and works
perfectly with any program that was written for the 3Dconnexion driver.

-- 
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 731519] Review Request: spacenavd - A free, compatible alternative for 3Dconnecxion's input drivers

2011-08-17 Thread bugzilla
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=731519

Richard Shaw hobbes1...@gmail.com changed:

   What|Removed |Added

  Alias||spacenavd

--- Comment #1 from Richard Shaw hobbes1...@gmail.com 2011-08-17 15:17:51 EDT 
---
rpmlint output:

# ls *.rpm
spacenavd-0.5-1.fc15.src.rpm spacenavd-debuginfo-0.5-1.fc15.x86_64.rpm
spacenavd-0.5-1.fc15.x86_64.rpm

# rpmlint *.rpm
spacenavd.src:32: W: rpm-buildroot-usage %build %configure
--prefix=%{buildroot}%{_prefix}

Necessary because make file does not support DESTDIR


spacenavd.x86_64: W: no-manual-page-for-binary spacenavd

Is a daemon, not an end user binary.


spacenavd.x86_64: W: no-manual-page-for-binary spnavd_ctl

Not sure what to do here, upstream doesn't provide anything beyond a README.


spacenavd.x86_64: W: service-default-enabled /etc/rc.d/init.d/spacenavd

You'll only install this if you need to so should be on by default.


spacenavd.x86_64: W: incoherent-subsys /etc/rc.d/init.d/spacenavd $prog

Rpmlint has issues with $ variables in init scripts which is why this is a
warning and not an error. Not really a problem.


3 packages and 0 specfiles checked; 0 errors, 5 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 731524] New: Review Request: gnatcoll - Suite of reusable software components and utilities

2011-08-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gnatcoll - Suite of reusable software components and 
utilities

https://bugzilla.redhat.com/show_bug.cgi?id=731524

   Summary: Review Request: gnatcoll - Suite of reusable software
components and utilities
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pa...@zhukoff.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://landgraf.fedorapeople.org/packages/requested/gnatcoll/gnatcoll.spec
SRPM URL:
http://landgraf.fedorapeople.org/packages/requested/gnatcoll/gnatcoll-2011-3.fc15.src.rpm
Description: The GNAT Component Collection is a suite of reusable software
components and utilities. It has been used by AdaCore in developing the GNAT
tool set, the GPS Integrated Development Environment, and GNAT Tracker, its
web-based customer support interface

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3280577

-- 
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 731525] New: Review Request: spnavcfg - Spacenav daemon interactive configuration program

2011-08-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: spnavcfg - Spacenav daemon interactive configuration 
program

https://bugzilla.redhat.com/show_bug.cgi?id=731525

   Summary: Review Request: spnavcfg - Spacenav daemon interactive
configuration program
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: hobbes1...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://hobbes1069.fedorapeople.org/spnavcfg/spnavcfg.spec
SRPM URL:
http://hobbes1069.fedorapeople.org/spnavcfg/spnavcfg-0.2.1-1.fc15.src.rpm
Description:
Spacenav daemon interactive configuration program.

-- 
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 731525] Review Request: spnavcfg - Spacenav daemon interactive configuration program

2011-08-17 Thread bugzilla
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=731525

Richard Shaw hobbes1...@gmail.com changed:

   What|Removed |Added

  Alias||spnavcfg

--- Comment #1 from Richard Shaw hobbes1...@gmail.com 2011-08-17 15:28:23 EDT 
---
rpmlint output:

$ ls *.rpm
spnavcfg-0.2.1-1.fc15.src.rpm spnavcfg-debuginfo-0.2.1-1.fc15.x86_64.rpm
spnavcfg-0.2.1-1.fc15.x86_64.rpm

$ rpmlint *.rpm
spnavcfg.src: W: spelling-error Summary(en_US) Spacenav - Spacemen
spnavcfg.src: W: spelling-error %description -l en_US Spacenav - Spacemen
spnavcfg.src:29: W: rpm-buildroot-usage %build %configure
--prefix=%{buildroot}%{_prefix}
spnavcfg.x86_64: W: spelling-error Summary(en_US) Spacenav - Spacemen
spnavcfg.x86_64: W: spelling-error %description -l en_US Spacenav - Spacemen

spnavcfg.x86_64: E: setuid-binary /usr/bin/spnavcfg root 04775L
spnavcfg.x86_64: E: non-standard-executable-perm /usr/bin/spnavcfg 04775L
spnavcfg.x86_64: E: non-standard-executable-perm /usr/bin/spnavcfg 04775L
- They uses setuid in order to allow normal users to write to the config file
in /etc. Supposedly it's secure because they have a frontend (gui) and backend
part of the program so only the backend runs as root.


spnavcfg.x86_64: W: no-manual-page-for-binary spnavcfg
- This is a gui only app.

3 packages and 0 specfiles checked; 3 errors, 6 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 731524] Review Request: gnatcoll - Suite of reusable software components and utilities

2011-08-17 Thread bugzilla
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=731524

--- Comment #1 from Pavel Zhukov pa...@zhukoff.net 2011-08-17 15:38:13 EDT ---
new build (Add ldconfig and ExcludeArch)
SRPM URL:
http://landgraf.fedorapeople.org/packages/requested/gnatcoll/gnatcoll-2011-4.fc15.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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

--- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-08-17 15:58:20 EDT ---
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 708554] Review Request: umph - Command line tool for parsing video links from Youtube feeds

2011-08-17 Thread bugzilla
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=708554

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-08-17 
16:20:48 EDT ---
umph-0.1.8-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/umph-0.1.8-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 708554] Review Request: umph - Command line tool for parsing video links from Youtube feeds

2011-08-17 Thread bugzilla
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=708554

Fedora Update System upda...@fedoraproject.org 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 708554] Review Request: umph - Command line tool for parsing video links from Youtube feeds

2011-08-17 Thread bugzilla
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=708554

--- Comment #4 from Fedora Update System upda...@fedoraproject.org 2011-08-17 
16:19:11 EDT ---
umph-0.1.8-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/umph-0.1.8-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 728757] Review Request: gnumed - The gnumed client

2011-08-17 Thread bugzilla
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=728757

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 16:24:52 EDT ---
gnumed-0.9.9-4.fc16 has been pushed to the Fedora 16 testing repository.

-- 
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 728555] Review Request: screenie-composer - Fancy screenshot composer

2011-08-17 Thread bugzilla
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=728555

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-08-17 
16:24:21 EDT ---
screenie-composer-1.0.0-0.2.20110805git49c2630.fc16 has been pushed to the
Fedora 16 testing repository.

-- 
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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-17 Thread bugzilla
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=731191

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-08-17 17:02:56 EDT ---
libspnav-0.2.2-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libspnav-0.2.2-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


  1   2   >