[Bug 1150393] Review Request: tengine - A high performance web server and reverse proxy server

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150393



--- Comment #11 from Jan Kaluža jkal...@redhat.com ---
(In reply to Christopher Meng from comment #10)
 You may need to revise the package since this package shares the same source
 code from nginx.

As I already stated in description of this review, Tengine is fork of nginx
keeping the binary compatibility with nginx, but bringing more features. It's
therefore clear the source code will be shared. It's the same situation as with
mysql vs. mariadb.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144014] Review Request: gphotofs - A FUSE filesystem module to mount your camera as a filesystem

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144014

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

   What|Removed |Added

   Fixed In Version||gphotofs-0.5-1.fc20
 Resolution|NEXTRELEASE |ERRATA



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
gphotofs-0.5-1.fc20 has been pushed to the Fedora 20 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1129220] Review Request: bash-argsparse - An high level argument parsing library for bash

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129220



--- Comment #12 from Haïkel Guémar karlthe...@gmail.com ---
Since this package complies with Fedora Packaging guidelines, I hereby approve
it into Fedora Packages Collection. Please submit a scm request.

Then, you're sponsored into the packager group, welcome back :)


Package Review
==

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



= MUST items =

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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 WTFPL. Detailed output of licensecheck in /home/haikel/1129220-bash-
 argsparse/licensecheck.txt
[x]: 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.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 245760 bytes in 51 files.
[x]: Package complies to the Packaging Guidelines
[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).
[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 requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[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]: 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

= 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.
[-]: 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.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: Dist tag is present (not strictly required in GL).
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a working URL.
[x]: Spec use %global instead of %define unless justified.

= EXTRA 

[Bug 1109717] Review Request: python-evtx - Windows Event Log parsing library and tools

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109717



--- Comment #8 from Richard W.M. Jones rjo...@redhat.com ---
http://oirase.annexia.org/reviews/python-evtx/
http://oirase.annexia.org/reviews/python-evtx/python-evtx.spec
http://oirase.annexia.org/reviews/python-evtx/python-evtx-0.3.2-0.1.20140326git8838756.src.rpm

Updated with the suggestions from comment 7.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1145100] Review Request: zeromq2 - Software library for fast, message-based applications

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1145100



--- Comment #2 from Thomas Spura toms...@fedoraproject.org ---
I think both approaches are possible in principle. Yet, in my opinion adding
compat packages to the same spec is not as legible as two different packages,
and you always rebuild all packages, if you change anything in your spec file
as you need to bump your globrelease...

I cannot find a guideline, that explicitly forbids your approach, but I would
have thought so...

So in summary: Yes I need this :)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1155778] Review Request: python-regex - Alternative regular expression module, to replace re

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1155778



--- Comment #3 from Thomas Moschny thomas.mosc...@gmx.de ---
Asked upstream to include a LICENSE file:
https://code.google.com/p/mrab-regex-hg/issues/detail?id=124

The permission problem is not present in koji builds, see the scratch build.
Could be a umask issue when building locally.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1060502] Review Request: golang-github-mitchellh-cli - A library for implementing powerful command-line interfaces in Go

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060502

Jan Chaloupka jchal...@redhat.com changed:

   What|Removed |Added

 CC||jchal...@redhat.com



--- Comment #9 from Jan Chaloupka jchal...@redhat.com ---
Hello. I am checking all golang packages from github. Just noticed this package
has git done but there is no spec file committed so far.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1155778] Review Request: python-regex - Alternative regular expression module, to replace re

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1155778

Christian Dersch chrisder...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Christian Dersch chrisder...@gmail.com ---
Approved! Koji builds show no permission problems and you asked upstream =
package is fine now :)

Greetings
Christian

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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



--- Comment #1 from Parag AN(पराग) panem...@gmail.com ---
Review:

+ mock build is successful for f22.

+ rpmlint output on rpms gave
nodejs-object-inspect.noarch: W: only-non-binary-in-usr-lib
2 packages and 0 specfiles checked; 0 errors, 1 warnings.

+ Source verified with upstream as (sha256sum)
srpm tarball
:fe38e8bfdf2d183c2a50b8c0e315d7a87ac41a0db292318bb11fdf5ed3def324
upstream tarball
:fe38e8bfdf2d183c2a50b8c0e315d7a87ac41a0db292318bb11fdf5ed3def324

+ License MIT is valid and is included in its own file as %doc

+ rest looks following packaging guidelines.

APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123069] Review Request: nodejs-resumer - A stream that starts paused and resumes on the next tick

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123069

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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



--- Comment #2 from Parag AN(पराग) panem...@gmail.com ---
Review:

+ mock build is successful for f22.

+ rpmlint output on rpms gave
nodejs-resumer.noarch: W: only-non-binary-in-usr-lib
nodejs-resumer.noarch: W: dangling-symlink
/usr/lib/node_modules/resumer/node_modules/through
/usr/lib/node_modules/through
2 packages and 0 specfiles checked; 0 errors, 2 warnings.

+ Source verified with upstream as (sha256sum)
srpm tarball
:9830d5e074876e38d9652bf385d97d9fc469dbf6f86a3275e551f648ad538bc2
upstream tarball
:9830d5e074876e38d9652bf385d97d9fc469dbf6f86a3275e551f648ad538bc2

+ License MIT is valid and is included in its own file as %doc

+ rest looks following packaging guidelines.

APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

  Flags||fedora-cvs-



--- Comment #2 from Tom Hughes t...@compton.nu ---
New Package SCM Request
===
Package Name: nodejs-object-inspect
Short Description: String representations of objects in node and the browser
Upstream URL: https://github.com/substack/object-inspect
Owners: tomh
Branches: f20 f21
InitialCC: jamielinux

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123069] Review Request: nodejs-resumer - A stream that starts paused and resumes on the next tick

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123069

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Tom Hughes t...@compton.nu ---
New Package SCM Request
===
Package Name: nodejs-resumer
Short Description: A stream that starts paused and resumes on the next tick
Upstream URL: https://github.com/substack/resumer
Owners: tomh
Branches: f20 f21
InitialCC: jamielinux

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331



--- Comment #16 from Mikolaj Izdebski mizde...@redhat.com ---
Works for me with the following changes:
1. I had to add BR on ant-junit, without that it didn't build for me
2. I added alias needed by Gradle:
%mvn_alias ':tesla-polyglot-{*}' org.sonatype.pmaven:pmaven-@1

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1129220] Review Request: bash-argsparse - An high level argument parsing library for bash

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129220



--- Comment #13 from Dams an...@livna.org ---
Thank you very much !

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331



--- Comment #17 from gil cattaneo punto...@libero.it ---
(In reply to Mikolaj Izdebski from comment #16)
 Works for me with the following changes:
 1. I had to add BR on ant-junit, without that it didn't build for me
Done
 2. I added alias needed by Gradle:
 %mvn_alias ':tesla-polyglot-{*}' org.sonatype.pmaven:pmaven-@1
Done

Spec URL: http://gil.fedorapeople.org/tesla-polyglot.spec
SRPM URL: http://gil.fedorapeople.org/tesla-polyglot-0.1.0-2.fc19.src.rpm

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1129220] Review Request: bash-argsparse - An high level argument parsing library for bash

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129220

Dams an...@livna.org changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #14 from Dams an...@livna.org ---
New Package SCM Request
===
Package Name: bash-argsparse
Short Description: An high level argument parsing library for bash
Upstream URL: https://github.com/Anvil/bash-argsparse
Owners: anvil
Branches: f20 f21 el6 epel7
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331

Michael Simacek msima...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #18 from Michael Simacek msima...@redhat.com ---
Package Review
==

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



= MUST items =

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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 MIT/X11 (BSD like), Unknown or generated. 122 files have unknown
 license. Detailed output of licensecheck in /home/msimacek/855331-tesla-
 polyglot/licensecheck.txt
 The files under MIT don't end up in binary RPM
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/maven-poms/tesla-polyglot, /usr/share
 /maven-metadata
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/maven-poms/tesla-
 polyglot, /usr/share/maven-metadata, /usr/share/java/tesla-polyglot
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 30720 bytes in 8 files.
[x]: Package complies to the Packaging Guidelines
[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).
[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 does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[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]: 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

Java:
[x]: Bundled jar/class files should be removed before build
[x]: Packages have proper BuildRequires/Requires on jpackage-utils
 Note: Maven packages do not need to (Build)Require jpackage-utils. It is
 pulled in by maven-local
[x]: Javadoc documentation files are generated and included in -javadoc
 subpackage
[x]: Javadoc subpackages should not have Requires: jpackage-utils
[x]: Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlink)

Maven:
[x]: If package contains pom.xml files install it (including depmaps) even
 when building with ant
[x]: POM files have correct Maven mapping
[x]: If package uses '-Dmaven.local.depmap' explain why it was needed in a
 comment
[x]: Maven packages should use new style packaging
[x]: Old add_to_maven_depmap macro is not being used
[x]: Packages DO NOT have Requires(post) and Requires(postun) on 

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331

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

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #19 from gil cattaneo punto...@libero.it ---
Thanks for the review!

New Package SCM Request
===
Package Name: tesla-polyglot
Short Description: Modules to enable Maven usage in other JVM languages
Owners: gil
Branches: f21
InitialCC: java-sig

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331
Bug 855331 depends on bug 859711, which changed state.

Bug 859711 Summary: Review Request: jruby-maven-plugins - JRuby Maven Mojos
https://bugzilla.redhat.com/show_bug.cgi?id=859711

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 859711] Review Request: jruby-maven-plugins - JRuby Maven Mojos

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=859711

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

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2014-10-23 07:10:46



--- Comment #4 from gil cattaneo punto...@libero.it ---
No more required

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1155210] Review Request: compat-drumstick - C++/Qt4 wrapper around the ALSA library sequencer interface

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1155210



--- Comment #5 from Robin Lee robinlee.s...@gmail.com ---
(In reply to Kevin Kofler from comment #4)
 Huh? compat-drumstick is NOT the upstream name! The upstream name is just
 drumstick.
 
 The compat- prefix was added to signal that it's an old version of
 drumstick, to install in parallel with the current drumstick 1.x.x. And
 there what I'm saying is that the drumstick0 naming scheme is more commonly
 used for that kind of compatibility package in Fedora. The '0' being not the
 soname soversion there, but the human-readable version number (drumstick
 0.x.x, the new Qt 5 one is 1.x.x).

Actually naming as drumstick-qt4 or drumstick0 is OK for me. I would change the
package name if you take the reviewer.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 476483] Review Request: ghc-paths - library for information about ghc paths (renamed to ghc-ghc-paths)

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=476483



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 476483] Review Request: ghc-paths - library for information about ghc paths (renamed to ghc-ghc-paths)

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=476483

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 630198] Review Request: ghc-safe - Library for safe (pattern match free) functions

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=630198



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 664221] Review Request: ghc-cautious-file - Provides ways to write a file cautiously

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=664221

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 630198] Review Request: ghc-safe - Library for safe (pattern match free) functions

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=630198

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 664221] Review Request: ghc-cautious-file - Provides ways to write a file cautiously

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=664221



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 703503] Review Request: ghc-xdg-basedir - Haskell library implementing XDG base directory spec

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=703503



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 703503] Review Request: ghc-xdg-basedir - Haskell library implementing XDG base directory spec

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=703503

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 712270] Review Request: ghc-derive - A program and library to derive instances for data types

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=712270

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 712280] Review Request: ghc-ghc-mtl - A mtl compatible with GHC-API monads and monad-transformers

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=712280

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 712280] Review Request: ghc-ghc-mtl - A mtl compatible with GHC-API monads and monad-transformers

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=712280



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 712270] Review Request: ghc-derive - A program and library to derive instances for data types

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=712270



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331



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

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855588] Review Request: ghc-concrete-typerep - Provides Binary and Hashable instances for TypeRep.

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855588



--- Comment #25 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855588] Review Request: ghc-concrete-typerep - Provides Binary and Hashable instances for TypeRep.

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855588

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123069] Review Request: nodejs-resumer - A stream that starts paused and resumes on the next tick

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123069



--- Comment #4 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123069] Review Request: nodejs-resumer - A stream that starts paused and resumes on the next tick

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123069

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1129220] Review Request: bash-argsparse - An high level argument parsing library for bash

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129220

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1129220] Review Request: bash-argsparse - An high level argument parsing library for bash

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129220



--- Comment #15 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1120865] Review Request: golang-github-BurntSushi-toml

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1120865



--- Comment #15 from Fedora Update System upda...@fedoraproject.org ---
golang-github-BurntSushi-toml-0-0.3.git2ceedfe.el6 has been submitted as an
update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/golang-github-BurntSushi-toml-0-0.3.git2ceedfe.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1120865] Review Request: golang-github-BurntSushi-toml

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1120865



--- Comment #16 from Fedora Update System upda...@fedoraproject.org ---
golang-github-BurntSushi-toml-0-0.3.git2ceedfe.fc20 has been submitted as an
update for Fedora 20.
https://admin.fedoraproject.org/updates/golang-github-BurntSushi-toml-0-0.3.git2ceedfe.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1120865] Review Request: golang-github-BurntSushi-toml

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1120865



--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
golang-github-BurntSushi-toml-0-0.3.git2ceedfe.fc21 has been submitted as an
update for Fedora 21.
https://admin.fedoraproject.org/updates/golang-github-BurntSushi-toml-0-0.3.git2ceedfe.fc21

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1141807] Review Request: golang-github-coreos-go-etcd - The official etcd v0.2 client library for Go

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1141807



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
golang-github-coreos-go-etcd-0.2.0-0.4.rc1.git6fe04d5.el6 has been submitted as
an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/golang-github-coreos-go-etcd-0.2.0-0.4.rc1.git6fe04d5.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1141807] Review Request: golang-github-coreos-go-etcd - The official etcd v0.2 client library for Go

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1141807



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
golang-github-coreos-go-etcd-0.2.0-0.4.rc1.git6fe04d5.fc20 has been submitted
as an update for Fedora 20.
https://admin.fedoraproject.org/updates/golang-github-coreos-go-etcd-0.2.0-0.4.rc1.git6fe04d5.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1141807] Review Request: golang-github-coreos-go-etcd - The official etcd v0.2 client library for Go

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1141807



--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
golang-github-coreos-go-etcd-0.2.0-0.4.rc1.git6fe04d5.fc21 has been submitted
as an update for Fedora 21.
https://admin.fedoraproject.org/updates/golang-github-coreos-go-etcd-0.2.0-0.4.rc1.git6fe04d5.fc21

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

  Flags|fedora-cvs- |fedora-cvs?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964

Michael Simacek msima...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||msima...@redhat.com
   Assignee|nob...@fedoraproject.org|msima...@redhat.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1051840] Review Request: python-libqutrub - Arabic verb conjuagtion library

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1051840



--- Comment #4 from Mosaab Alzoubi moc...@hotmail.com ---
Done:

Spec: http://ojuba.org/test/python-libqutrub.spec
Srpm: http://ojuba.org/test/python-libqutrub-1.0-2.oj35.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144033] Review Request: perl-ExtUtils-CBuilder - Compile and link C code for Perl modules

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144033

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Petr Pisar ppi...@redhat.com ---
New Package SCM Request
===
Package Name: perl-ExtUtils-CBuilder
Short Description: Compile and link C code for Perl modules
Upstream URL: http://search.cpan.org/dist/ExtUtils-CBuilder/
Owners: ppisar jplesnik psabata
Branches: 
InitialCC: perl-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1033413] Review Request: smuxi - An user-friendly IRC client based on GNOME/GTK+

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1033413

Jeff Peeler jpee...@redhat.com changed:

   What|Removed |Added

 CC||jpee...@redhat.com



--- Comment #16 from Jeff Peeler jpee...@redhat.com ---
I was looking for this in Fedora and came across this review. Could the spec
files be re-uploaded once more? (Not fully committed here yet, but I am
interested in helping this along.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1156086] New: Review Request: openmx - Open source package for Material eXplorer

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1156086

Bug ID: 1156086
   Summary: Review Request: openmx - Open source package for
Material eXplorer
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: marcin.du...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://marcindulak.fedorapeople.org/packages/openmx/openmx.spec
SRPM URL:
https://marcindulak.fedorapeople.org/packages/openmx/openmx-3.7.8-1.fc21.src.rpm
Description: OpenMX (Open source package for Material eXplorer) is a software
package for nano-scale material simulations based on density functional
theories (DFT), norm-conserving pseudopotentials, and pseudo-atomic localized
basis functions.
Fedora Account System Username: marcindulak

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964



--- Comment #9 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/staxmate.spec
SRPM URL: http://gil.fedorapeople.org/staxmate-2.2.1-1.fc19.src.rpm

- update to 2.2.1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144068] Review Request: perl-ExtUtils-Install - Install Perl files from here to there

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144068

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Petr Pisar ppi...@redhat.com ---
New Package SCM Request
===
Package Name: perl-ExtUtils-Install
Short Description: Install Perl files from here to there
Upstream URL: http://search.cpan.org/dist/ExtUtils-Install/
Owners: ppisar jplesnik psabata
Branches: 
InitialCC: perl-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964



--- Comment #10 from gil cattaneo punto...@libero.it ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=7933903

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in other JVM languages

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331



--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
tesla-polyglot-0.1.0-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/tesla-polyglot-0.1.0-2.fc21

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071



--- 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144033] Review Request: perl-ExtUtils-CBuilder - Compile and link C code for Perl modules

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144033



--- 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144033] Review Request: perl-ExtUtils-CBuilder - Compile and link C code for Perl modules

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144033

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144068] Review Request: perl-ExtUtils-Install - Install Perl files from here to there

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144068



--- 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144068] Review Request: perl-ExtUtils-Install - Install Perl files from here to there

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144068

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964

Michael Simacek msima...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #11 from Michael Simacek msima...@redhat.com ---
Package Review
==

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



= MUST items =

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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Unknown or generated. 68 files have unknown license. Detailed output of
 licensecheck in /home/msimacek/818964-staxmate/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/maven-metadata
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/maven-metadata
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 6 files.
[x]: Package complies to the Packaging Guidelines
[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: No rpmlint messages.
[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 does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[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]: 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

Java:
[x]: Bundled jar/class files should be removed before build
[x]: Packages have proper BuildRequires/Requires on jpackage-utils
 Note: Maven packages do not need to (Build)Require jpackage-utils. It is
 pulled in by maven-local
[x]: Javadoc documentation files are generated and included in -javadoc
 subpackage
[x]: Javadoc subpackages should not have Requires: jpackage-utils
[x]: Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlink)

Maven:
[x]: If package contains pom.xml files install it (including depmaps) even
 when building with ant
[x]: POM files have correct Maven mapping
[x]: Maven packages should use new style packaging
[x]: Old add_to_maven_depmap macro is not being used
[x]: Packages DO NOT have Requires(post) and Requires(postun) on jpackage-
 utils for %update_maven_depmap macro
[x]: Package DOES NOT use %update_maven_depmap in %post/%postun
[x]: Packages use %{_mavenpomdir} instead of %{_datadir}/maven2/poms

= SHOULD items =

Generic:
[x]: If the source package does not include license text(s) as a separate file
 from 

[Bug 1047788] Review Request: python-naftawayh - Arabic word tagger

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047788



--- Comment #6 from Mosaab Alzoubi moc...@hotmail.com ---
Done:

Spec: http://ojuba.org/test/python-naftawayh.spec
Srpm: http://ojuba.org/test/python-naftawayh-0.2-3.oj35.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1146931] Review Request: python-freetype - Freetype python bindings

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1146931



--- Comment #3 from Mosaab Alzoubi moc...@hotmail.com ---
The requested URL /fedora-work/SRPMS/python-freetype-0.4.2-1.fc21.src.rpm was
not found on this server.


Re-Upload SRPM.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1056291] Review Request: python-kdcproxy - MS-KKDCP (kerberos proxy) WSGI module

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1056291

Nathaniel McCallum nathan...@natemccallum.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-10-23 11:31:37



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089493] Review Request: bashmount - A menu-driven bash script for mounting removable media

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089493

Till Maas opensou...@till.name changed:

   What|Removed |Added

 CC||opensou...@till.name



--- Comment #12 from Till Maas opensou...@till.name ---
(In reply to Jamie Nguyen from comment #8)
 New Package SCM Request
 ===
 Package Name: bashmount
 Short Description: A menu-driven bash script for mounting removable media
 Upstream URL: https://github.com/jamielinux/bashmount
 Owners: jamielinux
 Branches: f19 f20 f21 el6 epel7
 InitialCC:

There is only a master branch, therefore I guess this needs to be a Package
Change Request:
http://fedoraproject.org/wiki/Package_SCM_admin_requests#Package_Change_Requests_for_existing_packages

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1148365] Review Request: perl-URI-Encode - Simple percent Encoding/Decoding

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1148365

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

   What|Removed |Added

   Fixed In Version|perl-URI-Encode-0.09-1.fc20 |perl-URI-Encode-0.09-1.el6



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
perl-URI-Encode-0.09-1.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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1133198] Review Request: perl-Compress-LZ4 - Perl interface to the LZ4 compression library

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1133198

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

   What|Removed |Added

   Fixed In Version|perl-Compress-LZ4-0.20-1.fc |perl-Compress-LZ4-0.20-1.el
   |20  |6



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
perl-Compress-LZ4-0.20-1.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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1148365] Review Request: perl-URI-Encode - Simple percent Encoding/Decoding

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1148365

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

   What|Removed |Added

   Fixed In Version|perl-URI-Encode-0.09-1.el6  |perl-URI-Encode-0.09-1.el7



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
perl-URI-Encode-0.09-1.el7 has been pushed to the Fedora EPEL 7 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1133198] Review Request: perl-Compress-LZ4 - Perl interface to the LZ4 compression library

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1133198

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

   What|Removed |Added

   Fixed In Version|perl-Compress-LZ4-0.20-1.el |perl-Compress-LZ4-0.20-1.el
   |6   |7



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
perl-Compress-LZ4-0.20-1.el7 has been pushed to the Fedora EPEL 7 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123069] Review Request: nodejs-resumer - A stream that starts paused and resumes on the next tick

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123069



--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
nodejs-resumer-0.0.0-1.fc20,nodejs-object-inspect-1.0.0-1.fc20 has been
submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-resumer-0.0.0-1.fc20,nodejs-object-inspect-1.0.0-1.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123069] Review Request: nodejs-resumer - A stream that starts paused and resumes on the next tick

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123069

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071



--- Comment #4 from Fedora Update System upda...@fedoraproject.org ---
nodejs-resumer-0.0.0-1.fc20,nodejs-object-inspect-1.0.0-1.fc20 has been
submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-resumer-0.0.0-1.fc20,nodejs-object-inspect-1.0.0-1.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1123071] Review Request: nodejs-object-inspect - String representations of objects in node and the browser

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1123071

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1146931] Review Request: python-freetype - Freetype python bindings

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1146931



--- Comment #4 from Parag pnem...@redhat.com ---
re-uploaded the srpm at same location.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144659] Review Request: nodejs-text-table - Generate borderless text table strings suitable for printing to stdout

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144659

Parag pnem...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Parag pnem...@redhat.com ---
New Package SCM Request
===
Package Name: nodejs-text-table
Short Description: Generate borderless text table strings suitable for printing
to stdout
Upstream URL: https://github.com/substack/text-table
Owners: pnemade
Branches: f20 f21 epel7

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005499] Review Request: caja-terminal - Terminal embedded in Caja

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005499

Wolfgang Ulbrich chat-to...@raveit.de changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed|2013-10-02 21:12:41 |2014-10-23 13:50:53



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1156176] New: Review Request: golang-github-russross-blackfriday - Markdown processor implemented in Go

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1156176

Bug ID: 1156176
   Summary: Review Request: golang-github-russross-blackfriday -
Markdown processor implemented in Go
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: l...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://lsm5.fedorapeople.org/golang-github-russross-blackfriday/golang-github-russross-blackfriday.spec
SRPM URL:
https://lsm5.fedorapeople.org/golang-github-russross-blackfriday/SRPMS/golang-github-russross-blackfriday-1.2-1.fc22.src.rpm

Description: 
%{summary}

This package contains library source intended for building other packages
which use %{project}/%{repo}.

Fedora Account System Username: lsm5

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

$ rpmlint golang-github-russross-blackfriday.spec
SRPMS/golang-github-russross-blackfriday-1.2-1.fc22.src.rpm
RPMS/noarch/golang-github-russross-blackfriday-devel-1.2-1.fc22.noarch.rpm
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005498] Review Request: python-caja - Python bindings for Caja

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005498

Wolfgang Ulbrich chat-to...@raveit.de changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed|2013-09-29 20:45:24 |2014-10-23 13:47:51



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005499] Review Request: caja-terminal - Terminal embedded in Caja

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005499
Bug 1005499 depends on bug 1005498, which changed state.

Bug 1005498 Summary: Review Request: python-caja - Python bindings for Caja
https://bugzilla.redhat.com/show_bug.cgi?id=1005498

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 997829] Review Request: caja-actions - Caja extension for customizing the context menu

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=997829

Wolfgang Ulbrich chat-to...@raveit.de changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed|2014-07-26 20:28:51 |2014-10-23 13:47:17



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1156176] Review Request: golang-github-russross-blackfriday - Markdown processor implemented in Go

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1156176

Lokesh Mandvekar l...@fedoraproject.org changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1149649] Review Request: tuxfootball - funny soccer game

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1149649

Raphael Groner projects...@smart.ms changed:

   What|Removed |Added

   Assignee|musur...@gmail.com  |nob...@fedoraproject.org
 Whiteboard||NotReady



--- Comment #3 from Raphael Groner projects...@smart.ms ---
If you notice some issues that need to be solved before you want to start a
formal review, add these issues in a comment and set the Whiteboard of the bug
to contain NotReady. This helps other possible reviewers to notice that the
review request is not yet ready for further review action. 
https://fedoraproject.org/wiki/Package_Review_Process#Reviewer

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1130097] Review Request: python-cjson - Fast JSON encoder/decoder for Python

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1130097



--- Comment #17 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
---
Rel-eng ticket: https://fedorahosted.org/rel-eng/ticket/6021

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1139066] Review Request: jbrout - Photo manager, written in python/pygtk

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1139066

Matěj Cepl mc...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2014-10-23 14:27:09



--- Comment #11 from Matěj Cepl mc...@redhat.com ---
This should be closed a long time ago.

http://koji.fedoraproject.org/koji/packageinfo?packageID=6382

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964

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

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #12 from gil cattaneo punto...@libero.it ---
Thanks for the review

New Package SCM Request
===
Package Name: staxmate
Short Description: Light-weight Java framework for streaming XML processing
Owners: gil
Branches: f21
InitialCC: java-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089493] Review Request: bashmount - A menu-driven bash script for mounting removable media

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089493

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

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #13 from Jamie Nguyen jamieli...@fedoraproject.org ---
Package Change Request
==
Package Name: bashmount
New Branches: f19 f20 f21 el6 epel7
Owners: jamielinux

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1155778] Review Request: python-regex - Alternative regular expression module, to replace re

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1155778

Thomas Moschny thomas.mosc...@gmx.de changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Thomas Moschny thomas.mosc...@gmx.de ---
Many thanks for the super-fast review!

New Package SCM Request
===
Package Name: python-regex
Short Description: Alternative regular expression module, to replace re
Upstream URL: https://pypi.python.org/pypi/regex
Owners: thm
Branches: f20 f21 el6 epel7
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115670] Review Request: nodejs-grunt-contrib-connect - Start a connect web server

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115670



--- Comment #4 from Ralph Bean rb...@redhat.com ---
(In reply to Tom Hughes from comment #2)
 Again this should be noarch, and you may want to think about the tests.

Cool.  The next release will have the noarch declaration.

As for the tests, I'd rather leave them out to try and cut down on how large
this dep chain has become.

(In reply to Tom Hughes from comment #3)
 Oh, and there is no need to add requires for npm modules to the spec - they
 will be added automatically.

Would you mind if I left them in just to be explicit and consistent with other
packages?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115670] Review Request: nodejs-grunt-contrib-connect - Start a connect web server

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115670



--- Comment #6 from Ralph Bean rb...@redhat.com ---
Whoops, wrong URLs.

Spec URL: http://threebean.org/rpm/SPECS/nodejs-grunt-contrib-connect.spec
SRPM URL:
http://threebean.org/rpm/SRPMS/nodejs-grunt-contrib-connect-0.8.0-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115670] Review Request: nodejs-grunt-contrib-connect - Start a connect web server

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115670



--- Comment #5 from Ralph Bean rb...@redhat.com ---
Spec URL: http://threebean.org/rpm/nodejs-grunt-contrib-connect.spec
SRPM URL:
http://threebean.org/rpm/../SRPMS/nodejs-grunt-contrib-connect-0.8.0-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115669] Review Request: nodejs-grunt-banner - Adds a simple banner to files

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115669



--- Comment #4 from Ralph Bean rb...@redhat.com ---
Here's a release that adds the noarch declaration, but it doesn't yet try to
take on the tests and it leaves the npm module requirements in place.

Spec URL: http://threebean.org/rpm/SPECS/nodejs-grunt-banner.spec
SRPM URL:
http://threebean.org/rpm/SRPMS/nodejs-grunt-banner-0.2.3-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115673] Review Request: nodejs-grunt-contrib-cssmin - Compress CSS files

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115673



--- Comment #3 from Ralph Bean rb...@redhat.com ---
Here's a new release with the latest upstream tarball and the noarch
declaration.

As with the other reviews, I'd like to decline running the tests for now and
ask if I can keep the npm module requirements explicit:

Spec URL: http://threebean.org/rpm/SPECS/nodejs-grunt-contrib-cssmin.spec
SRPM URL:
http://threebean.org/rpm/SRPMS/nodejs-grunt-contrib-cssmin-0.10.0-1.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115674] Review Request: nodejs-grunt-contrib-less - Compile LESS files to CSS

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115674



--- Comment #3 from Ralph Bean rb...@redhat.com ---
Here's a new release with the latest upstream tarball and the noarch
declaration.

As with the other reviews, I'd like to decline running the tests for now and
ask if I can keep the npm module requirements explicit:

Spec URL: http://threebean.org/rpm/SPECS/nodejs-grunt-contrib-less.spec
SRPM URL:
http://threebean.org/rpm/SRPMS/nodejs-grunt-contrib-less-0.11.4-1.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1115676] Review Request: nodejs-grunt-html-validation - W3C html validation grunt plugin

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1115676



--- Comment #3 from Ralph Bean rb...@redhat.com ---
New release includes latest upstream, builds as noarch, fixes those
permissions, and removes unnecessary files (like Gruntfile.js).

Spec URL: http://threebean.org/rpm/SPECS/nodejs-grunt-html-validation.spec
SRPM URL:
http://threebean.org/rpm/SRPMS/nodejs-grunt-html-validation-0.1.18-1.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964

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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818964] Review Request: staxmate - Light-weight Java framework for streaming XML processing

2014-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818964



--- Comment #13 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

  1   2   >