[Bug 994152] Review Request: libprocess - Library that provides an actor style message-passing

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=994152

Ralf Corsepius rc040...@freenet.de changed:

   What|Removed |Added

 CC||rc040...@freenet.de



--- Comment #10 from Ralf Corsepius rc040...@freenet.de ---
Package fails to build on f20:
...
src/process.cpp: In function 'void process::send_file(ev_loop*, ev_io*, int)':
src/process.cpp:1096:22: error: 'sendfile' is not a member of 'os'
 ssize_t length = os::sendfile(s, fd, offset, size);
...

Further issues:

* The compiler is invoked with broken arguments:
... -I.//include -I -I/src -I -I   ...

The -I.//include doesn't look cosher, but real issue here is the subsequent -I
without arguments:
# touch foo.cc
# gcc -o foo.o -c foo.cc -I
gcc: error: missing path after ‘-I’

You are only not seeing this error with this package, because these -I's
swallow their next argument (which happen to be further -I's):
# gcc -o foo.o -c foo.cc -I -I


* consider to append --disable-static to %configure
This avoids building the static libs and thus cuts down compilation times by
ca. factor 2.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=n0lBjXOgaYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1001728] Review Request: rubygem-rkerberos - A Ruby interface for the the Kerberos library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1001728

Björn besser82 Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

 CC||bjoern.es...@gmail.com



--- Comment #2 from Björn besser82 Esser bjoern.es...@gmail.com ---
(In reply to Simon A. Erat from comment #1)
 Hello Miroslav
 
 Informal Review
 
 * Valid License named in specfile:
   Artistic 2.0
 * Missing License:
   Either as file or link in readme/manual of the package

There MUST be a copy of LICENSE within, in this case.  Link within README would
not be sufficient according to Artistic 2.0 Permissions for Redistribution of
the Standard Version No. 2 ... provided that you duplicate all of the
original copyright notices and associated disclaimers ...

You should add this as another SOURCE to spec-file:

  Source1: http://www.perlfoundation.org/attachment/legal/artistic-2_0.txt

copy it into src-tree `cp -a %{SOURCE1} COPYING` during %prep and include it
for %doc.


 * Package fails to buld as noarch
   Is an 'interface' really required to be the same arch as the
 host-application

In this case yes, because it builds and provides a C-compiled interface.


 ##!!
 Processing files: rubygem-rkerberos-0.1.2-3.fc19.noarch
 + popd
 + exit 0
 Provides: rkerberos.so rubygem(rkerberos) = 0.1.2 rubygem-rkerberos =
 0.1.2-3.fc19
 Requires(rpmlib): rpmlib(CompressedFileNames) = 3.0.4-1 rpmlib(FileDigests)
 = 4.6.0-1 rpmlib(PayloadFilesHavePrefix) = 4.0-1
 Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3)
 libc.so.6(GLIBC_2.4) libcom_err.so.2 libcrypt.so.1 libdl.so.2
 libkadm5clnt_mit.so.8 libkadm5clnt_mit.so.8(kadm5clnt_mit_8_MIT)
 libkrb5.so.3 libkrb5.so.3(krb5_3_MIT) libm.so.6 libpthread.so.0 librt.so.1
 libruby.so.2.0 rtld(GNU_HASH)
 error: Arch dependent binaries in noarch package

There you get it.  C-compiled stuff :)


 Issues:
 ===
 - gems should require rubygems package
   Note: Requires: rubygems missing in rubygem-rkerberos-doc
   See: http://fedoraproject.org/wiki/Packaging:Ruby#RubyGems

False positve from f-r.  It is present, but with SCL-macro prefixed:
  Requires: %{?scl_prefix}rubygems


 - Pure Ruby package must be built as noarch
 - Package contains Requires: ruby(release).

False positive here, too.  C-compiled interface.


 [-]: Development (unversioned) .so files in -devel subpackage, if present.
  Note: Unversioned so-files in private %_libdir subdirectory (see
  attachment). Verify they are not in ld path.
   ---Its in: /usr/lib64/gems/ruby/rkerberos-0.1.2/lib
  What does that mean?   

This can actually be marked as PASS, because:

  * It is supposed for C-compiled interfaces to have no SO-Version.

  * Applies to *.so-files which are directly placed inside %{_libdir}, mostly.


 [ ]: License field in the package spec file matches the actual license.
  Note: Checking patched sources after %prep for licenses. Licenses found:
  Unknown or generated. 20 files have unknown license. Detailed output
 of
  licensecheck in /home/simon/1001728-rubygem-rkerberos/licensecheck.txt

This can be marked PASS.  Having no explicit license commented inside the
file usualy means: Same license as in distributed LICENSE/COPYING.


 [!]: License file installed when any subpackage combination is installed.

Yes, obviously there is no license file


 [x]: Package must own all directories that it creates.
  Note: Directories without known owners:
  /usr/share/gems/gems/rkerberos-0.1.2, /usr/share/gems,
  /usr/share/gems/doc, /usr/share/gems/gems

This would need some more manual inspection...


 [x]: %build honors applicable compiler flags or justifies otherwise.

LDFLAGS are not applied when linking obj to so.  I'd usually recommend to have
%configure ||: on top of %build, so all FLAGS get exported properly.


 [ ]: Package contains no bundled libraries without FPC exception.

Can be marked PASS.  There seem no bundled files / libs or subsets of them.


 [ ]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
  beginning of %install.
  Note: rm -rf %{buildroot} present but not required

False positive.  This is PASS.


 [!]: Development files must be in a -devel package
--they are in a -debug package

What???  The debug-pkg looks sane to me.  It is the regular fashion that -debug
contains a copy of all sources and the DWARF-part of the linked-binaries.


 [ ]: Package uses nothing in %doc for runtime.

PASS


 [ ]: Useful -debuginfo package or justification otherwise.
-- cant tell, idk what ruby coders need to debug usefull

Having a look inside it should tell ;)  There must be the sources and the
DWARF-parts of every build binary inside.  To me it looks good.  ;)


 [ ]: Package is not known to require an ExcludeArch tag.
  Note: Test run failed
 [x]: Large documentation must go in a 

[Bug 1001728] Review Request: rubygem-rkerberos - A Ruby interface for the the Kerberos library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1001728

Björn besser82 Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=60MTBzVXara=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004678] Review Request: python-cov-core - Plugin core for use by pytest-cov, nose-cov and nose2-cov

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004678

Bohuslav Slavek Kabrda bkab...@redhat.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
  Flags||fedora-cvs?



--- Comment #3 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
(In reply to Christopher Meng from comment #2)
 Ah... bugzilla just caught errors. 
 
 Please note that python-devel should be python2-devel.

Oh snap, I always forget that :)

Thanks for the review, I'll replace python-devel with python2-devel before
committing.


New Package SCM Request
===
Package Name: python-cov-core
Short Description: Plugin core for use by pytest-cov, nose-cov and nose2-cov
Owners: bkabrda
Branches: f20
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=wzfaVqpBTEa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004963] Review Request: openstack-heat-templates - Heat example templates and DIB elements

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004963

Jamie Nguyen jamieli...@fedoraproject.org changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
 Status|NEW |ASSIGNED
 CC||jamieli...@fedoraproject.or
   ||g
   Assignee|nob...@fedoraproject.org|jamieli...@fedoraproject.or
   ||g



--- Comment #1 from Jamie Nguyen jamieli...@fedoraproject.org ---
Issues:
===

1) Please add a URL tag.

2) Please use the GitHub Source URL guidelines:
https://fedoraproject.org/wiki/Packaging:SourceURL#Github

3) This is a pre-release package (as upstream have not made any releases yet),
so please use the snapshot package guidelines and pre-release package naming
guidelines:
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages

4) There is no version indicated upstream, and I can't find any indication that
'0.0.1' will be the version they will start with. Start versioning from '0'
instead, so something like openstack-heat-templates-0-0.1.20130813git. Upstream
might start from version '0.0.0', in which case '0.0.1' will be newer than
upstream.

5) The files are not purely documentation files and include shell scripts.
%{_datadir}/%{name} is probably a better location.

6) There are no %build or %install sections. These aren't compulsory, but it's
common practice to include them anyway even if you're not running any commands
inside them. It'll also silence the rpmlint warnings. This is not a MUST
though, so I won't block the review on this.

7) I'd say openshift-origin/ directory should be put into a separate subpackage
possibly called openshift-origin-templates.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=x5CoxMlZsTa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1001351] Review Request: glite-lbjp-common-db - Database engine abstraction wrapper

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1001351



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
glite-lbjp-common-db-3.2.8-2.fc19 has been submitted as an update for Fedora
19.
https://admin.fedoraproject.org/updates/glite-lbjp-common-db-3.2.8-2.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=mbHx1OzZtOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1001351] Review Request: glite-lbjp-common-db - Database engine abstraction wrapper

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1001351



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
glite-lbjp-common-db-3.2.8-2.fc18 has been submitted as an update for Fedora
18.
https://admin.fedoraproject.org/updates/glite-lbjp-common-db-3.2.8-2.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=MnTL2KJTCta=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005172] New: Review Request: python-llvmpy - Python bindings for LLVM

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005172

Bug ID: 1005172
   Summary: Review Request: python-llvmpy - Python bindings for
LLVM
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sergio.pa...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: http://guaix.fis.ucm.es/~spr/python-llvmpy.spec
SRPM URL: http://guaix.fis.ucm.es/~spr/python-llvmpy-0.12.0-1.fc19.src.rpm
Description: llvmpy is a Python wrapper around the llvm C++ library which 
allows simple access to compiler tools.
Fedora Account System Username: sergiopr

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=a1vN6SqXxXa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] New: Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173

Bug ID: 1005173
   Summary: Review Request: SDL2_image - Image loading library for
SDL
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: i.gnatenko.br...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: http://ignatenkobrain.fedorapeople.org/for-review/SDL2_image.spec
SRPM URL:
http://ignatenkobrain.fedorapeople.org/for-review/SDL2_image-2.0.0-1.fc20.src.rpm
Description: 
Simple DirectMedia Layer (SDL) is a cross-platform multimedia library
designed to provide fast access to the graphics frame buffer and audio
device.  This package contains a simple library for loading images of
various formats (BMP, PPM, PCX, GIF, JPEG, PNG) as SDL surfaces.
Fedora Account System Username: ignatenkobrain

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=BBQr19xIbza=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005172] Review Request: python-llvmpy - Python bindings for LLVM

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005172

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|cicku...@gmail.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=iS2DQTzx21a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821404] Review Request: gimp-dds-plugin - A plugin for GIMP allows to load/save in the DDS format

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821404

Vasiliy Glazov vasc...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2013-09-06 07:50:27



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Rq6YlXib2Ja=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004337] Review Request: python3-docs - Documentation for the Python 3 programming language

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004337

Simon A. Erat erat.si...@gmail.com changed:

   What|Removed |Added

 CC||erat.si...@gmail.com



--- Comment #1 from Simon A. Erat erat.si...@gmail.com ---
Informal Review - failed
-

Spec  not 'clear'


Use the %{name} makro instead of self created ${python}.
Within the spec, and the source, Python is written in small and capital
letters, both ways.
 Fix naming issues.

%build - looks quite empty, is that enough?
 %install - no files to move/cp?


You want to make a documention package of python 3 only.
Why dont you download the docs-tarball from python?
http://docs.python.org/3/download.html
Further its 3.2.2, not the latest package 3.4.0.


Fix this and try again.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=gFxVDUmZaea=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004951] Review Request: qtbrowserplugin - Qt Solutions Component: Browser Plugin

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004951

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
 Status|NEW |ASSIGNED
 CC||limburg...@gmail.com
   Assignee|nob...@fedoraproject.org|limburg...@gmail.com
  Flags||fedora-review?



--- Comment #3 from Jon Ciesla limburg...@gmail.com ---
Looking. . .

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=inY7NykpSRa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004337] Review Request: python3-docs - Documentation for the Python 3 programming language

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004337

Björn besser82 Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bjoern.es...@gmail.com
   Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com
  Flags||fedora-review?



--- Comment #2 from Björn besser82 Esser bjoern.es...@gmail.com ---
FTBFS on rawhide!  Excerpt from build.log:

  (%build): /bin/sh -e /var/tmp/rpm-tmp.Dn6X33
  + umask 022
  + cd /builddir/build/BUILD
  + cd Python-3.3.2
  + make -C Doc html
  make: Entering directory `/builddir/build/BUILD/Python-3.3.2/Doc'
  Checking out Sphinx...
  /bin/sh: line 2: svn: command not found
  make: Leaving directory `/builddir/build/BUILD/Python-3.3.2/Doc'
  make: *** [checkout] Error 127

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=HbUZGl6q5Fa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004337] Review Request: python3-docs - Documentation for the Python 3 programming language

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004337



--- Comment #3 from Björn besser82 Esser bjoern.es...@gmail.com ---
Thoughts on spec-file:

  * You can strip a lot of the macro-madness, which has 0-impact on the
actual result.

  * There is an easy way of getting the needed python-version:
  %global pyver %(v=`python3 --version 21 | sed -es/.ython[ \t]*//`;echo
$v)
which is better than hardcoding it ;)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=4C04LY6XaUa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||limburg...@gmail.com
   Assignee|nob...@fedoraproject.org|limburg...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jon Ciesla limburg...@gmail.com ---
Good:

- rpmlint checks return:

SDL2_image.x86_64: W: no-manual-page-for-binary showimage
Each executable in standard binary directories should have a man page.

None exists.

SDL2_image-devel.x86_64: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files

Move README.txt into -devel

- package meets naming guidelines
- package meets packaging guidelines
- license ( LGPLv2+ ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file
- devel package ok
- no .la files
- post/postun ldconfig ok
- devel requires base package n-v-r 

Otherwise very clean.  Move the README prior to import.

APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=DGQQhAFmKoa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004337] Review Request: python3-docs - Documentation for the Python 3 programming language

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004337



--- Comment #4 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
Thanks for your comments, guys:
- As for the %{python} macro, where in the specfile do I use it in an
inappropriate place?
- The macro madness is there because when the package goes through the
review, I want it to have minimal diff with python-docs package (which this
package is based on, BTW). So each spec will hardcode the python version, but
each will be different.
- %build - yes, I just create documentation and remove the .buildinfo dir,
nothing else needed
- If I'm referring to the same docs as you, they are already generated and
can't be regeneraged. I consider that bad, everything should be built from
source, IMO, even just documentation.
- As for the Koji failure, that's interesting... I didn't experience it in my
local mock, but I admit I didn't scrub it before building. I'll investigate and
repost the spec and srpm once I find the problem.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=NMCE2lcdAGa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173

Igor Gnatenko i.gnatenko.br...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
OK. Will move README at import.

New Package SCM Request
===
Package Name: SDL2_image
Short Description: Image loading library for SDL
Owners: ignatenkobrain
Branches: f19 f20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=xLJtscXPEJa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004337] Review Request: python3-docs - Documentation for the Python 3 programming language

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004337



--- Comment #5 from Björn besser82 Esser bjoern.es...@gmail.com ---
(In reply to Bohuslav Slavek Kabrda from comment #4)
 Thanks for your comments, guys:
 - As for the %{python} macro, where in the specfile do I use it in an
 inappropriate place?

Actually nowhere, it was just about that macro-medness, I think. :)


 - The macro madness is there because when the package goes through the
 review, I want it to have minimal diff with python-docs package (which
 this package is based on, BTW). So each spec will hardcode the python
 version, but each will be different.

Why do you want to keep the diff as small as possible?  Is there any not
foreseeable reason in it?

btw. Why don't you just generate  pkg the docs during build of regular
python3-pkg?  Afaik, you are the maintainer of it, too.

 - If I'm referring to the same docs as you, they are already generated and
 can't be regeneraged. I consider that bad, everything should be built from
 source, IMO, even just documentation.

Just explained that to Simon in 1:1 irc.  ;)

 - As for the Koji failure, that's interesting... I didn't experience it in
 my local mock, but I admit I didn't scrub it before building. I'll
 investigate and repost the spec and srpm once I find the problem.

It seems there is something on the make file, which tries to do some
SVN-checkout...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=cN4CQJQkQua=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004845] Review Request: python-libarchive - Fully functional Python wrapper for libarchive.

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004845

Paul Wouters pwout...@redhat.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |



--- Comment #10 from Paul Wouters pwout...@redhat.com ---
On f19 it builds, bug segfaults

tar zcvf example.tar.gz /tmp/.Test-unix/
tar: Removing leading `/' from member names
/tmp/.Test-unix/

 import libarchive.
 a  = libarchive.Archive('example.tar.gz')
 for entry in a:
... print entry.pathname
... a.close()
  File stdin, line 3
a.close()
^
SyntaxError: invalid syntax
 a
libarchive.Archive object at 0x7f14629bdc50
 a.close()
 
 for entry in a:
... print entry.pathname
... 
Segmentation fault

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=qL561TCONPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173



--- Comment #3 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=N0vyJkLLYga=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=qAF6XA6NI5a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173



--- Comment #4 from Fedora Update System upda...@fedoraproject.org ---
SDL2_image-2.0.0-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/SDL2_image-2.0.0-2.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=zT8TDCm2oPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=NdXtm0mmXba=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173



--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
SDL2_image-2.0.0-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/SDL2_image-2.0.0-2.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=E7Qd92SVg7a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915907] Review Request: qt5-qtquick1 - Qt5 - QtQuick1 component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915907

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

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |



--- Comment #2 from Rex Dieter rdie...@math.unl.edu ---

Spec URL: http://rdieter.fedorapeople.org/rpms/qt5/qt5-qtquick1.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/qt5/qt5-qtquick1-5.1.1-1.fc19.src.rpm

%changelog
* Tue Sep 03 2013 Rex Dieter rdie...@fedoraproject.org 5.1.1-1
- 5.1.1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=KHT2fVB4SJa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 813832] Review Request: linuxdcpp - linux port of dc++

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=813832



--- Comment #9 from Jon Ciesla limburg...@gmail.com ---
Do you intend to import and build this at any point?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=JOmuzhnVo9a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 961223] Review Request: ocrfeeder - A document layout analysis and optical character recognition system

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=961223



--- Comment #10 from Jon Ciesla limburg...@gmail.com ---
Any updated on this?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=xGuUwXFSpBa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 961223] Review Request: ocrfeeder - A document layout analysis and optical character recognition system

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=961223

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC||jro...@redhat.com
  Flags||needinfo?(jro...@redhat.com
   ||)



--- Comment #11 from Christopher Meng cicku...@gmail.com ---
I can patch it for pillow but I still want upstream to do this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=qN4r0Uc3Dqa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005172] Review Request: python-llvmpy - Python bindings for LLVM

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005172

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Christopher Meng cicku...@gmail.com ---
Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Large documentation must go in a -doc subpackage. Large could be size (~1MB)
  or number of files.
  Note: Documentation size is 2416640 bytes in 171 files.
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#PackageDocumentation


Rpmlint
---
Checking: python-llvmpy-0.12.0-1.fc20.i686.rpm
  python-llvmpy-0.12.0-1.fc20.src.rpm
python-llvmpy.src: W: spelling-error %description -l en_US llvm - llama
2 packages and 0 specfiles checked; 0 errors, 1 warnings.

-- Ignorable


Provides

python-llvmpy:
python-llvmpy
python-llvmpy(x86-32)



Unversioned so-files

python-llvmpy: /usr/lib/python2.7/site-packages/llvmpy/_api.so
python-llvmpy: /usr/lib/python2.7/site-packages/llvmpy/_capsule.so

-- Ignorable


Generated by fedora-review 0.5.0 (920221d) last change: 2013-08-30
Command line :/usr/bin/fedora-review -rvn python-llvmpy-0.12.0-1.fc19.src.rpm
Buildroot used: fedora-rawhide-i386
Active plugins: Generic, Python, Shell-api, C/C++
Disabled plugins: Java, SugarActivity, Perl, R, PHP, Ruby
Disabled flags: EPEL5, EXARCH, DISTTAG

***
APPROVED.

Suggestions:

1. %{buildroot}/%{python_sitearch} can be %{buildroot}%{python_sitearch}

2. Source can be Source0

3. Is it possible to use llvmpy as its %{name}? I think upstream won't release
2 version for py2 and py3 in parallel.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=cwLJ439IDXa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 994152] Review Request: libprocess - Library that provides an actor style message-passing

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=994152

Timothy St. Clair tstcl...@redhat.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |



--- Comment #11 from Timothy St. Clair tstcl...@redhat.com ---
re comment #10: requires stout-0.1.2-2.c784a5e.fc20 which is in koji and has
been recently been submitted to testing.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=6r9jOPDL7Ma=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005172] Review Request: python-llvmpy - Python bindings for LLVM

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005172



--- Comment #2 from Christopher Meng cicku...@gmail.com ---
Also if you think -doc is needed, please add -doc subpackage.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=sAd5V4gVy5a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005172] Review Request: python-llvmpy - Python bindings for LLVM

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005172



--- Comment #3 from Christopher Meng cicku...@gmail.com ---
Also again(sorry),

I just checked documentation of llvmpy and found:

python -c import llvm; llvm.test() is used for simple test, is it possible
for you to add it into %check?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=i2aR6ojXrOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 927461] Review Request: ros - The Robot Operating System

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927461

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC||cicku...@gmail.com
  Alias||ros



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=fXWnskjmBPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 972346] Review Request: ros-common_msgs - Common ROS Messages

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972346

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

  Alias||ros-common_msgs



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=wVfDlDHCRIa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915903] Review Request: qt5-qtjsbackend - Qt5 - QtJSBackend component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915903

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Bl7iIG6sMna=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915909] Review Request: qt5-qtdeclarative - Qt5 - QtDeclarative component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915909



--- Comment #17 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=14SKrIbYYXa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915909] Review Request: qt5-qtdeclarative - Qt5 - QtDeclarative component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915909

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=gFeRoRfzj9a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826520] Review Request: hiera - A simple hierarchical database supporting plugin data sources

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826520

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

   What|Removed |Added

   Fixed In Version|hiera-1.0.0-3.fc18  |hiera-1.0.0-3.el6



--- Comment #25 from Fedora Update System upda...@fedoraproject.org ---
hiera-1.0.0-3.el6 has been pushed to the Fedora EPEL 6 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=FOtys6o9eZa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 822328] Review Request: libmediainfo - Supplies technical and tag information about a video or audio file

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=822328

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|cicku...@gmail.com  |nob...@fedoraproject.org



--- Comment #15 from Christopher Meng cicku...@gmail.com ---
Drop the review, please ask others, sorry.

Only copy things generated before.

License check:


*No copyright* zlib/libpng
--
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/MediaInfoLib/Project/MSCS2008/Example/HowToUse_Dll.cs
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/MediaInfoLib/Project/MSCS2010/Example/HowToUse_Dll.cs
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/MediaInfoLib/Source/ThirdParty/tinyxml2/tinyxml2.cpp
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/MediaInfoLib/Source/ThirdParty/tinyxml2/tinyxml2.h

and

Public domain
-
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/MediaInfoLib/Source/ThirdParty/md5/md5.c

Seems bundled libs/incorrect license tag.



==
Rpmlint
---
Checking: libmediainfo-0.7.64-3.fc20.i686.rpm
  libmediainfo-devel-0.7.64-3.fc20.i686.rpm
  libmediainfo-0.7.64-3.fc20.src.rpm
libmediainfo-devel.i686: W: file-not-utf8
/usr/share/doc/libmediainfo-devel/Example/HowToUse_Dll.cs


Also, all files should be installed with -p option.

%post -n %{name} -p /sbin/ldconfig can just be

%post -p /sbin/ldconfig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Gn54vifkHYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915903] Review Request: qt5-qtjsbackend - Qt5 - QtJSBackend component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915903



--- Comment #16 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=1wS2SWzvy4a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 912960] Review Request: rubygem-gdk3 - Ruby binding of GDK-3.x

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=912960

Toshio Ernie Kuratomi a.bad...@gmail.com changed:

   What|Removed |Added

 CC||a.bad...@gmail.com



--- Comment #30 from Toshio Ernie Kuratomi a.bad...@gmail.com ---
Thread start:
https://lists.fedoraproject.org/pipermail/packaging/2013-September/009486.html

Email from me with patches that probably fix the code itself here:
https://lists.fedoraproject.org/pipermail/packaging/2013-September/009493.html

Let me know if you need a direct copy of the patches since the mailman web
archives have the patches inline.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=GTuZPeFqMea=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029

Juan Hernández juan.hernan...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Juan Hernández juan.hernan...@redhat.com ---
New Package SCM Request
===
Package Name: ovirt-engine-sdk-python
Short Description: oVirt Engine Software Development Kit (Python)
Owners: jhernand  oschreib
Branches: f18 f19 f20 el6
InitialCC: mpastern

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=PwLnGMw8TGa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=878188



--- Comment #58 from Rex Dieter rdie...@math.unl.edu ---
whoa, slow down. :)

I've been meaning to work on epel-6 builds, but only *after* all the reviews
were done.

So, now that the deed is done, be aware I made some el6-related commits to
master branch recently, if it all possible, please do make a best effort to
keep master/el6 branches merged or mergeable.

If branch-specific commits are required, please try to discuss it first. 
Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=sSaUz68fbGa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029



--- Comment #10 from Jon Ciesla limburg...@gmail.com ---
SCM request and review summary name don't match, please correct.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=irjQ7wBOA5a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=QBDRc7tESHa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=878188



--- Comment #59 from Richard Shaw hobbes1...@gmail.com ---
No problem :)

I noticed the spec updates! I only need the four components at this time.

Build is failing on ppc64 though :(

In file included from tools/qcryptographichash.cpp:74:
tools/../../3rdparty/sha3/KeccakF-1600-opt64.c: In function 'void
KeccakAbsorb576bits(unsigned char*, const unsigned char*)':
tools/../../3rdparty/sha3/KeccakF-1600-opt64.c:348: error: 'fromBytesToWord'
was not declared in this scope

http://koji.fedoraproject.org/koji/getfile?taskID=5905860name=build.log

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=VYsz4wHVKya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #8 from Christopher Meng cicku...@gmail.com ---
APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=WOj82rkhd5a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005406] New: Review Request: python-happybase - Python library to interact with Apache HBase

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005406

Bug ID: 1005406
   Summary: Review Request: python-happybase - Python library to
interact with Apache HBase
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: dpri...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: http://fedorapeople.org/~dprince/python-happybase.spec
SRPM URL: http://fedorapeople.org/~dprince/python-happybase-0.6-1.src.rpm
Description: Python library to interact with Apache HBase
Fedora Account System Username: dprince

This is an upcoming requirement of OpenStack Ceilometer.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Pv63z4pFRPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005405] New: Review Request: python-thrift - Thrift Python Software Library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005405

Bug ID: 1005405
   Summary: Review Request: python-thrift - Thrift Python Software
Library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: dpri...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: http://fedorapeople.org/~dprince/python-thrift.spec
SRPM URL: http://fedorapeople.org/~dprince/python-thrift-0.9.1-1.src.rpm

Description: Thrift is provided as a set of Python packages. The top level
package is thrift, and there are subpackages for the protocol, transport, and
server code. Each package contains modules using standard Thrift naming
conventions i.e. TProtocol, TTransport) and implementations in corresponding
module (i.e. TSocket).  There is also a subpackage reflection, which contains
the generated code for the reflection structures.

Fedora Account System Username: dprince

-

This is an upcoming requirement of OpenStack Ceilometer which requires
python-happybase.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=pYW88OfQuta=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915903] Review Request: qt5-qtjsbackend - Qt5 - QtJSBackend component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915903

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

   What|Removed |Added

 CC||hobbes1...@gmail.com
  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #15 from Richard Shaw hobbes1...@gmail.com ---
Package Change Request
==
Package Name: qt5-qtjsbackend
New Branches: el6
Owners: hobbes1069
InitialCC: 

Needed for future review request.
Co-maintainers welcomed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=GNnXG6AOpJa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 972348] Review Request: actionlib - Interface for pre-emptible tasks

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972348

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC||cicku...@gmail.com
  Alias||actionlib



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=EuECOMItC6a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=878188



--- Comment #57 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=esdvHlx8eYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=878188

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=hwvUaqAWL6a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 928584] Review Request: ros-std_msgs - Standard ROS Messages

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=928584

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC||cicku...@gmail.com
  Flags||needinfo?(sanjay.ankur@gmai
   ||l.com)



--- Comment #9 from Christopher Meng cicku...@gmail.com ---
This ticket has blocked another review for a long time.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Fc0UOqKJuca=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk-python - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=iSpa6tvLnpa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk-python - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029



--- Comment #12 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=2SGzKn5qB7a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005372] New: Review Request: python-oslo-messaging - OpenStack common Messaging library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005372

Bug ID: 1005372
   Summary: Review Request: python-oslo-messaging - OpenStack
common Messaging library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: dpri...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: http://fedorapeople.org/~dprince/python-oslo-messaging.spec
SRPM URL:
http://fedorapeople.org/~dprince/python-oslo-messaging-1.2.0.a4.gb6d8087-0.1.230.b6d8087_d17ff27.src.rpm
Description: OpenStack common Messaging library
Fedora Account System Username: dprince

This is based off of a master tarball on Sept 6th 2013.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=iVrJM3Kohka=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005173] Review Request: SDL2_image - Image loading library for SDL

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005173

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

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
SDL2_image-2.0.0-4.fc20 has been pushed to the Fedora 20 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=SZKElJbdJna=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 927374] Review Request: rubygem-rugged - Ruby binding to the libgit2 linkable library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927374

Troy Dawson tdaw...@redhat.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |



--- Comment #17 from Troy Dawson tdaw...@redhat.com ---
Spec URL: http://tdawson.fedorapeople.org/review/rubygem-rugged.spec
SRPM URL:
http://tdawson.fedorapeople.org/review/rubygem-rugged-0.19.0-3.fc20.src.rpm

- Added version to ruby(abi)
- Added %{?rubygems_default_filter} before description
- Added BR: git
-- didn't use /usr/bin/git because that would break on older Fedora versions
- Removed line where I was removing hidden files
- Added full test directory from github (Source1)
-- In comments say how I get and package Source1
-- Let developers know of gemspec bug, they have already thanked me and said
they will fix it.
- Remove BR: rubygem(test-unit)
- Fix up tests so they all pass
-- Added LANG variable
-- run git config --global user.name John Doe

Thanks for the help with the tests.  I would have never found those missing
directories, and switching to minitest fixed the exit code problem.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ecUMWRjY2ca=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987884] Review Request: swizzle - Stream-based parsing code

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987884

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2013-09-06 13:32:24



--- Comment #1 from gil cattaneo punto...@libero.it ---


*** This bug has been marked as a duplicate of bug 1004997 ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Ubv3XXtZ33a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004997] Review Request: swizzle - Stream-based parsing code

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004997



--- Comment #1 from gil cattaneo punto...@libero.it ---
*** Bug 987884 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=SvzSAontjIa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 824218] Review Request: polly - Linux Twitter client designed for multiple columns of multiple accounts

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=824218

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

  Flags||needinfo?(aal...@gmail.com)



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=8j2crCmZlWa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1004029] Rename Request: ovirt-engine-sdk-python - oVirt Engine Software Development Kit (Python)

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1004029

Juan Hernández juan.hernan...@redhat.com changed:

   What|Removed |Added

Summary|Rename Request: |Rename Request:
   |ovirt-engine-sdk - oVirt|ovirt-engine-sdk-python -
   |Engine Software Development |oVirt Engine Software
   |Kit (Python)|Development Kit (Python)
  Flags||fedora-cvs?



--- Comment #11 from Juan Hernández juan.hernan...@redhat.com ---
New Package SCM Request
===
Package Name: ovirt-engine-sdk-python
Short Description: oVirt Engine Software Development Kit (Python)
Owners: jhernand  oschreib
Branches: f18 f19 f20 el6
InitialCC: mpastern

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=rjcIycWscBa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 928584] Review Request: ros-std_msgs - Standard ROS Messages

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=928584

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

  Flags|needinfo?(sanjay.ankur@gmai |
   |l.com)  |



--- Comment #10 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
You're welcome to take over and continue the review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=U3z6amsr8Ca=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=878188

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

   What|Removed |Added

 CC||hobbes1...@gmail.com
  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #56 from Richard Shaw hobbes1...@gmail.com ---
Package Change Request
==
Package Name: qt5-qtbase
New Branches: el6
Owners: hobbes1069
InitialCC: 

Needed for future review request.
Co-maintainers welcomed!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=jfreIdQQzqa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 635256] Review Request: qtop - tool for monitoring PBS systems

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=635256

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
  Flags||needinfo?(fo...@cern.ch)



--- Comment #30 from Christopher Meng cicku...@gmail.com ---
What's going on here? Have you found a sponsor?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=nAMUwyLFEma=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005320] New: Review Request: openstack-puppet - Puppet modules used to install OpenStack

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005320

Bug ID: 1005320
   Summary: Review Request: openstack-puppet - Puppet modules used
to install OpenStack
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: roh...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: http://rohara.fedorapeople.org/puppet/openstack-puppet.spec
SRPM URL:
http://rohara.fedorapeople.org/puppet/openstack-puppet-2013.2-1.fc18.noarch.rpm
Description: A collection of Puppet modules used to install OpenStack.
Fedora Account System Username: rohara

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Yp0DOWY4Nva=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915909] Review Request: qt5-qtdeclarative - Qt5 - QtDeclarative component

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=915909

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

   What|Removed |Added

 CC||hobbes1...@gmail.com
  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #16 from Richard Shaw hobbes1...@gmail.com ---
Package Change Request
==
Package Name: qt5-qtdeclarative
New Branches: el6
Owners: hobbes1069
InitialCC: 

Needed for future review request.
Co-maintainers welcomed!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=paeyUPkMPxa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973868] Re-Review Request: lifeograph - A diary program

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973868

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

   What|Removed |Added

   Fixed In Version|lifeograph-0.11.1-2.fc19|lifeograph-0.11.1-2.fc18



--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
lifeograph-0.11.1-2.fc18 has been pushed to the Fedora 18 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=diIFh3QAG9a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 980071] Review Request: python-xpyb - X Python Binding, based on the X C Binding (XCB) library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=980071

Tomas Dabašinskas tdaba...@redhat.com changed:

   What|Removed |Added

 Blocks||1005447



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=DQTZ2jCtGYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=878188

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

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |



--- Comment #60 from Rex Dieter rdie...@math.unl.edu ---
I guess there's also:

tools/qregularexpression.cpp: In member function
'QRegularExpressionMatchPrivate* QRegularExpressionPrivate::doMatch(const
QString, int, QRegularExpression::MatchType, QRegularExpression::MatchOptions,
bool, const QRegularExpressionMatchPrivate*) const':
tools/qregularexpression.cpp:1241: error: passing 'const
QAtomicPointerpcre16_extra' as 'this' argument of 'T*
QBasicAtomicPointerT::loadAcquire() [with T = pcre16_extra]' discards
qualifiers

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=QYF56uzcE2a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 980168] Review Request: python-tornadIO2 - Socket.io server on top of Tornado framework

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=980168

Yohan Graterol yohangratero...@gmail.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
  Flags|fedora-review?  |fedora-review+



--- Comment #10 from Yohan Graterol yohangratero...@gmail.com ---
Hello Richard!

The spec and srpm is fine!



PACKAGE APPROVED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=i7RSgopP26a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 989325] Review Request: libgme - Video game music file emulation/playback library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=989325

Yohan Graterol yohangratero...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Yohan Graterol yohangratero...@gmail.com ---
Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[-]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: %build honors applicable compiler flags or justifies otherwise.
[-]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[-]: Package requires other packages for directories it uses.
[-]: Package uses nothing in %doc for runtime.
[-]: Package is not known to require ExcludeArch.
[-]: Fully versioned dependency in subpackages, if present.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in libgme-
 devel
[x]: Package complies to the Packaging Guidelines
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. 
[x]: License file installed when any subpackage combination is installed.
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[-]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[-]: Useful -debuginfo package or justification otherwise.
[-]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 92160 bytes in 5 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Each %files section contains %defattr if rpm  4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[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]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[-]: Scriptlets must be sane, if used.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is 

[Bug 959942] Review Request: irker - IRC Message Relay

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=959942

Yohan Graterol yohangratero...@gmail.com changed:

   What|Removed |Added

  Flags||needinfo+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=i3DIa4vy4va=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 989325] Review Request: libgme - Video game music file emulation/playback library

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=989325

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Christopher Meng cicku...@gmail.com ---
Thank you.

I will fix wrong-file-end-of-line-encoding issues before import.




New Package SCM Request
===
Package Name: libgme
Short Description: Video game music file emulation/playback library
Owners: cicku
Branches: f20 f19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=iV34OgUrOfa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 994859] Review Request: python-pygit2 - Python bindings for libgit2

2013-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=994859

Yohan Graterol yohangratero...@gmail.com changed:

   What|Removed |Added

  Flags||needinfo+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=slYpXLj5zwa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review