[Bug 923163] Review request: gnome-weather - A Weather application for GNOME

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923163

Mathieu Bridon  changed:

   What|Removed |Added

 CC||boche...@fedoraproject.org

--- Comment #1 from Mathieu Bridon  ---
I just tried reviewing this package, and it doesn't build:

+ ./configure --build=x86_64-redhat-linux-gnu --host=x86_64-redhat-linux-gnu
--program-prefix= --disable-dependency-tracking --prefix=/usr
--exec-prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin --sysconfdir=/etc
--datadir=/usr/share --includedir=/usr/include --libdir=/usr/lib64
--libexecdir=/usr/libexec --localstatedir=/var --sharedstatedir=/var/lib
--mandir=/usr/share/man --infodir=/usr/share/info --disable-static
/var/tmp/rpm-tmp.4dZ8zy: line 34: ./configure: No such file or directory

The problem is your invocation of the %setup macro:
%setup -q -c gnome-weather -n gnome-weather

This means that rpmbuild will first create a "gnome-weather" folder, then
extract the tarball inside.

So you end up with the path:
gnome-weather/gnome-weather-3.7.92/

The contents of the archive (including the configure file) are in that last
subfolder.

But rpmbuild will only enter the gnome-weather folder, and as such not find the
contents.

You could just use the simplest form of the setup macro:
%setup -q

And that fixes the issue.

-- 
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=UvPku9OMfx&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922165] Review Request: openstack-ceilometer - OpenStack measurement collection service

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922165

--- Comment #5 from Kashyap Chamarthy  ---
Ok, re-ran with the new SRPM, no installation errors this time, posting the
review tool result for reference. Still need to do manual investigation:

#--#

Package Review
==

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



= MUST items =

Generic:
[ ]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[ ]: Package contains no bundled libraries without FPC exception.
[ ]: Changelog in prescribed format.
[ ]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf %{buildroot} present but not required
[ ]: Sources contain only permissible code or content.
[ ]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[ ]: Package contains desktop file if it is a GUI application.
[ ]: 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 python-
 ceilometer , openstack-ceilometer-common , openstack-ceilometer-compute ,
 openstack-ceilometer-central , openstack-ceilometer-collector ,
 openstack-ceilometer-api
[ ]: Package complies to the Packaging Guidelines
[ ]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "Apache (v2.0)", "Unknown or generated", "*No copyright* Apache (v2.0)".
 3 files have unknown license. Detailed output of licensecheck in
 /home/kashyap/rpmbuild/SRPMS/openstack-ceilometer/licensecheck.txt
[ ]: License file installed when any subpackage combination is installed.
[ ]: Package consistently uses macro is (instead of hard-coded directory
 names).
[ ]: Package is named according to the Package Naming Guidelines.
[ ]: 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.
[ ]: Package must own all directories that it creates.
[ ]: Package does not own files or directories owned by other packages.
[ ]: Requires correct, justified where necessary.
[ ]: Spec file is legible and written in American English.
[ ]: Package contains systemd file(s) if in need.
[ ]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 30720 bytes in 3 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: %config files are marked noreplace or the reason is justified.
[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]: Spec file lacks Packager, Vendor, PreReq tags.
[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]: No %config files under /usr.
[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).

Python:
[ ]: Python eggs must not download any dependencies during the build process.
[ ]: A package which is used by another package via an egg interface should
 provide egg info.
[ ]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep

= 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.
[ ]: Final provides and requires are sane (see attachments).
[ ]: Package functions as described.
[ ]: Latest version is packaged.
[ ]: Package does not i

[Bug 896310] Review Request: odeint - A C++ library for numerically solving Ordinary Differential Equations

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=896310

Rich Mattes  changed:

   What|Removed |Added

 CC||richmat...@gmail.com
   Assignee|nob...@fedoraproject.org|richmat...@gmail.com
  Flags||fedora-review?

--- Comment #4 from Rich Mattes  ---
I'll go ahead and take this, and try to get it done this weekend.

-- 
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=eXIyX3ooh7&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 825409] Review Request: gazebo - 3D multi-robot simulator with dynamics

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=825409

--- Comment #6 from Rich Mattes  ---
Rebased to release 1.5.0, bundled libs still not fixed

Spec URL: http://rmattes.fedorapeople.org/RPMS/gazebo/gazebo.spec
SRPM URL:
http://rmattes.fedorapeople.org/RPMS/gazebo/gazebo-1.5.0-1.fc18.src.rpm

-- 
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=JCgLEbPzv9&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 871197] Review Request: python-catkin - Official build system of ROS

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=871197

Ankur Sinha (FranciscoD)  changed:

   What|Removed |Added

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

--- Comment #3 from Ankur Sinha (FranciscoD)  ---
Hi Rich,

I'll try and review this by the weekend. 

Thanks,
Ankur

-- 
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=0XXmiv3kr4&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 871205] Review Request: urdfdom - U-Robot Description Format Document Object Model

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=871205

--- Comment #1 from Rich Mattes  ---
Updated to latest upstream release:

Spec URL: http://rmattes.fedorapeople.org/RPMS/urdfdom/urdfdom.spec
SRPM URL:
http://rmattes.fedorapeople.org/RPMS/urdfdom/urdfdom-0.2.7-1.fc18.src.rpm

rpmlint output:
$ rpmlint urdfdom.spec ../RPMS/x86_64/urdfdom-*
urdfdom.spec: W: invalid-url Source0: urdfdom-0.2.7.tar.bz2
urdfdom.x86_64: W: spelling-error %description -l en_US parsers -> parser,
parses, parers
urdfdom.x86_64: E: invalid-soname /usr/lib64/liburdfdom_sensor.so
liburdfdom_sensor.so
urdfdom.x86_64: E: invalid-soname /usr/lib64/liburdfdom_world.so
liburdfdom_world.so
urdfdom.x86_64: E: invalid-soname /usr/lib64/liburdfdom_model_state.so
liburdfdom_model_state.so
urdfdom.x86_64: E: invalid-soname /usr/lib64/liburdfdom_model.so
liburdfdom_model.so
urdfdom.x86_64: W: no-manual-page-for-binary check_urdf
urdfdom.x86_64: W: no-manual-page-for-binary urdf_mem_test
urdfdom.x86_64: W: no-manual-page-for-binary urdf_to_graphiz
urdfdom-devel.x86_64: W: no-documentation
3 packages and 1 specfiles checked; 4 errors, 6 warnings.


I think this invalid soname error is new.  It's currently creating whatever
CMake defaults to, and I'm not sure it's against the packaging guidelines.

-- 
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=cYw7IxOz3F&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 871204] Review Request: urdfdom-headers - The URDF (U-Robot Description Format) headers

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=871204

--- Comment #3 from Rich Mattes  ---
Updated to latest upstream version:

Spec URL:
http://rmattes.fedorapeople.org/RPMS/urdfdom-headers/urdfdom-headers.spec
SRPM URL:
http://rmattes.fedorapeople.org/RPMS/urdfdom-headers/urdfdom-headers-0.2.2-1.fc18.src.rpm


rpmlint output:
$ rpmlint urdfdom-headers.spec
../RPMS/noarch/urdfdom-headers-devel-0.2.2-1.fc18.noarch.rpm 
urdfdom-headers.spec: W: invalid-url Source0: urdfdom-headers-0.2.2.tar.bz2
1 packages and 1 specfiles checked; 0 errors, 1 warnings.

The LICENSE file was changed to BSD after 0.2.2 was tagged.  I removed the
LICENSE file from %doc until it is corrected in the next release

-- 
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=q3uMG7lAA2&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 871203] Review Request: console-bridge - Lightweight set of macros used for reporting information in libraries

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=871203

--- Comment #4 from Rich Mattes  ---
Updated to latest upstream version:
Spec URL:
http://rmattes.fedorapeople.org/RPMS/console-bridge/console-bridge.spec
SRPM URL:
http://rmattes.fedorapeople.org/RPMS/console-bridge/console-bridge-0.1.4-1.fc18.src.rpm

rpmlint:
$ rpmlint ./console-bridge.spec ../RPMS/x86_64/console-bridge-*
./console-bridge.spec: W: invalid-url Source0:
ros-console_bridge-0.1.4-0-g96c8951.tar.gz
console-bridge.x86_64: E: invalid-soname /usr/lib64/libconsole_bridge.so
libconsole_bridge.so
console-bridge.x86_64: W: no-documentation
console-bridge-devel.x86_64: W: no-documentation
3 packages and 1 specfiles checked; 1 errors, 3 warnings.

-- 
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=fYEfTv4LdU&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 871197] Review Request: python-catkin - Official build system of ROS

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=871197

--- Comment #2 from Rich Mattes  ---
Alright I think the package is in good enough shape to be reviewed now.  After
a day or two that turned into a month or two...

Spec URL: http://rmattes.fedorapeople.org/rospackages/catkin/python-catkin.spec
SRPM URL:
http://rmattes.fedorapeople.org/rospackages/catkin/python-catkin-0.4.5-4.gitd4f1f24.fc18.src.rpm

rpmlint output:
$ rpmlint python-catkin.spec
../RPMS/noarch/python-catkin-0.4.5-4.gitd4f1f24.fc18.noarch.rpm 
python-catkin.spec: W: invalid-url Source0: ros-catkin-0.4.5-0-gd4f1f24.tar.gz
python-catkin.noarch: W: no-manual-page-for-binary catkin_install_parse
python-catkin.noarch: W: no-manual-page-for-binary git-catkin
python-catkin.noarch: W: no-manual-page-for-binary git-catkin-track-all
python-catkin.noarch: W: no-manual-page-for-binary catkin-version
python-catkin.noarch: W: no-manual-page-for-binary catkin-parse-stack
python-catkin.noarch: W: no-manual-page-for-binary catkin-bump-version
python-catkin.noarch: W: no-manual-page-for-binary
catkin-build-debs-of-workspace
python-catkin.noarch: W: no-manual-page-for-binary catkin-topological-order
1 packages and 1 specfiles checked; 0 errors, 9 warnings.

The package is a checkout from github, and the checkout command is included in
the spec comments.  The package also doesn't have manpages, so the warnings are
all OK.

-- 
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=YGJSyyux9Z&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919265] Review Request: Bijiben - Note taking app

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919265

--- Comment #15 from Mathieu Bridon  ---
(In reply to comment #14)
> License is correct. Lifting FE-Legal.

Thanks Tom!

(In reply to comment #13)
> I can sponsor you

Like I said in comment 11, the package was in a state where I'd approve it if
Pierre-Yves didn't need a sponsor (or if I was a sponsor myself).

So there's no need for me any more here, he's all yours Matthias. :)

-- 
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=OkjSm1yMkV&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 875308] Review Request: mate-menu-editor - MATE Desktop menu editor

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875308

--- Comment #17 from Fedora Update System  ---
mate-menu-editor-1.5.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/mate-menu-editor-1.5.0-2.fc17

-- 
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=YTE0U2HbiV&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 875308] Review Request: mate-menu-editor - MATE Desktop menu editor

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875308

--- Comment #16 from Fedora Update System  ---
mate-menu-editor-1.5.0-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/mate-menu-editor-1.5.0-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=2RzQuHcjb4&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 875308] Review Request: mate-menu-editor - MATE Desktop menu editor

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875308

Fedora Update System  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=uk1etl5ey4&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882559] Review Request: mate-file-manager-image-converter - MATE Desktop file manager image converter plugin

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882559

--- Comment #20 from Fedora Update System  ---
mate-file-manager-image-converter-1.5.0-1.fc17 has been submitted as an update
for Fedora 17.
https://admin.fedoraproject.org/updates/mate-file-manager-image-converter-1.5.0-1.fc17

-- 
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=tKYCfvT0K8&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882559] Review Request: mate-file-manager-image-converter - MATE Desktop file manager image converter plugin

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882559

--- Comment #21 from Fedora Update System  ---
mate-file-manager-image-converter-1.5.0-1.fc18 has been submitted as an update
for Fedora 18.
https://admin.fedoraproject.org/updates/mate-file-manager-image-converter-1.5.0-1.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=vtvrRvcneP&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882559] Review Request: mate-file-manager-image-converter - MATE Desktop file manager image converter plugin

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882559

Fedora Update System  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=4ECJxfg7v9&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923564] Review Request: NetworkManager-ssh - NetworkManager VPN plugin for SSH

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923564

--- Comment #6 from Dan Fruehauf  ---
Hey again,

 - All versions were greater, so they were removed. Not depending on these
versions anymore.

 - Fixed release tag according to:
http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Release_Tag
http://fedoraproject.org/wiki/Packaging:SourceURL#Github

 - Added COPYING file

 - Removed ChangeLog file from RPM package as a full changelog is in the spec

 - Decided it'll be a pre release as things are still in development over there

Files are here for your reinspection:
Spec URL: http://nm-ssh.cloudapp.net/NetworkManager-ssh.spec
SRPM URL:
http://nm-ssh.cloudapp.net/NetworkManager-ssh-0.0.3-pre1.20130321gitcf6c00f.fc18.src.rpm

Appreciated.

-- 
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=Fmdslybs55&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 902017] Review Request: openshift-java-client - OpenShift Java Client

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=902017

--- Comment #13 from Gerard Ryan  ---
I think it's ready now Tomas, please have a look when you get a chance, 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=c6MtFJMifw&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 902017] Review Request: openshift-java-client - OpenShift Java Client

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=902017

Gerard Ryan  changed:

   What|Removed |Added

  Flags||needinfo?(tra...@redhat.com
   ||)

--- Comment #12 from Gerard Ryan  ---
Spec URL:
http://galileo.fedorapeople.org/openshift-java-client/2.0.3-2/openshift-java-client.spec
SRPM URL:
http://galileo.fedorapeople.org/openshift-java-client/2.0.3-2/openshift-java-client-2.0.3-2.20130320git7073708.fc18.src.rpm

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5150244
(license tag in spec is wrong for this build, but is correct for the spec/srpm
above)

- Updated to include all the changes that have happened upstream
-- Licensing fixed.
-- André has removed the bundled Base64Coder class

- Updated to use new maven macros.

-- 
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=sQHxZxKmA9&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 911186] Review Request: mocha - A simple, flexible, fun test framework for Node.js

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=911186

--- Comment #14 from Fedora Update System  ---
mocha-1.8.2-3.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/mocha-1.8.2-3.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=MbvTYJvWv3&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 911186] Review Request: mocha - A simple, flexible, fun test framework for Node.js

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=911186

Fedora Update System  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=DN0q7co7ch&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 910136] Review Request: nodejs-node-expat - Fast libexpat XML SAX parser binding for Node.js

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=910136

--- Comment #9 from Fedora Update System  ---
nodejs-node-expat-2.0.0-4.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/nodejs-node-expat-2.0.0-4.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=OAfnXAiEOa&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915337] Review Request: nmon - Nigel's performance MONitor for Linux

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915337

--- Comment #13 from Fedora Update System  ---
nmon-14g-3.fc18 has been pushed to the Fedora 18 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=9HAPWuidU5&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915337] Review Request: nmon - Nigel's performance MONitor for Linux

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915337

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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=rYFF5VDQA0&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 907513] Review Request: clearlooks-phenix-theme - Clearlooks-Phenix gtk theme

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=907513

--- Comment #12 from Fedora Update System  ---
clearlooks-phenix-3.0.15-1.fc18 has been pushed to the Fedora 18 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=mJUiRKiyFf&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 907513] Review Request: clearlooks-phenix-theme - Clearlooks-Phenix gtk theme

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=907513

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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=wPPGx1qWkK&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 914695] Review Request: php-Smarty2 - Template/Presentation Framework for PHP

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=914695

--- Comment #9 from Fedora Update System  ---
php-Smarty2-2.6.27-1.fc18, gallery2-2.3.2-7.fc18 has been pushed to the Fedora
18 stable repository.  If problems still persist, please make note of it in
this bug report.

-- 
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=r44V1eDDXW&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 914695] Review Request: php-Smarty2 - Template/Presentation Framework for PHP

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=914695

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2013-03-20 17:40:57

-- 
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=qKB0Fyy60D&a=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

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878188

--- Comment #54 from Fedora Update System  ---
qt5-qtbase-5.0.1-6.fc18 has been pushed to the Fedora 18 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=SGMF8NpQ8d&a=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

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878188

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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=lxLmNuGSMd&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 911932] Review Request: bccc - The buddycloud console client

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=911932

Bug 911932 depends on bug 911940, which changed state.

Bug 911940 Summary: python-sleekxmpp is out of date
https://bugzilla.redhat.com/show_bug.cgi?id=911940

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA

-- 
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=eMJnsBBFrS&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919429] Review Request: gimp-dbp - David's Batch Processor for the Gimp

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919429

--- Comment #9 from Fedora Update System  ---
gimp-dbp-1.1.9-2.fc17 has been pushed to the Fedora 17 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=dKxqy7DekX&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919429] Review Request: gimp-dbp - David's Batch Processor for the Gimp

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919429

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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=fYwFPXh3xk&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

Antonio Trande  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #24 from Antonio Trande  ---
New Package SCM Request
===
Package Name: metis
Short Description: Serial Graph Partitioning and Fill-reducing Matrix Ordering
Owners: sagitter
Branches: f17 f18 f19 el6

-- 
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=itemEEVWXb&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922485] Review Request: nodejs-mbtiles - Utilities and tilelive integration for the MBTiles format

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922485

Tom Hughes  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #4 from Tom Hughes  ---
New Package SCM Request
===
Package Name: nodejs-mbtiles
Short Description: Utilities and tilelive integration for the MBTiles format
Owners: tomh
Branches: f18 f19
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=QfdWamBJhc&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #23 from Antonio Trande  ---
(In reply to comment #22)
> You should write it as
> 
> License: ASL 2.0 and BSD and LGPLv2+
> 

Fixed. 
Thank you.

Spec URL: http://sagitter.fedorapeople.org/metis/metis.spec
SRPM URL: http://sagitter.fedorapeople.org/metis/metis-5.0.3-7.fc18.src.rpm

-- 
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=d4SAOoGCPv&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 910125] Review Request: nodejs-formidable - A Node.js module for parsing form data, especially file uploads

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=910125

--- Comment #11 from Fedora Update System  ---
nodejs-formidable-1.0.13-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/nodejs-formidable-1.0.13-1.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=kPbzpQ10Lp&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 909907] Review Request: php-horde-Horde-Crypt - Horde Cryptography API

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=909907

Nick Bebout  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
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=ezGncAZVWE&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 909706] Review Request: php-horde-Horde-Vfs- Virtual File System API

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=909706

Nick Bebout  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
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=hCWfcdO8JX&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 908389] Review Request: php-horde-Horde-Text-Filter - Horde Text Filter API

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=908389

Nick Bebout  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
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=jSoBXj1CeG&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 890839] Review Request: slic3r - G-code generator for 3D printers (RepRap, Makerbot, Ultimaker etc.)

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=890839

T.C. Hollingsworth  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #15 from T.C. Hollingsworth  ---
Fine by me.  LC_NUMERIC issues with the decimal seperator have been popping up
in other packages lately anyway, so that would be sufficient for those in the
know anyway.  ;-)

This package is 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=1ZoEQl5rC6&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785494] Review Request: php-horde-Horde-Compress - Horde Compression API

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785494

--- Comment #11 from Nick Bebout  ---
Updated

Spec URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Compress.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Compress-2.0.3-3.fc17.src.rpm

-- 
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=kSGJJaomsv&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922485] Review Request: nodejs-mbtiles - Utilities and tilelive integration for the MBTiles format

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922485

T.C. Hollingsworth  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #3 from T.C. Hollingsworth  ---
Well, you could have simply moved the file in %prep instead of deleting it and
it would have been fine too.  Sorry, I should have been a little more specific.

Anyway, this package is 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=9hWuHKriax&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 891232] nodejs-ronn - markdown to roff/html converter

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891232

--- Comment #6 from T.C. Hollingsworth  ---
Spec: http://patches.fedorapeople.org/npm/nodejs-ronn.spec
SRPM: http://patches.fedorapeople.org/npm/nodejs-ronn-0.3.8-6.fc18.src.rpm

* Wed Mar 20 2013 T.C. Hollingsworth  - 0.3.8-6
- fix require on markdown

-- 
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=CWOYlhTg3C&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785453] Review Request: php-horde-Horde-Mail - Horde Mail Library

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785453

--- Comment #6 from Nick Bebout  ---
Updated

Spec URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Mail.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Mail-2.0.4-1.fc17.src.rpm

-- 
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=ukmm3dBxMJ&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785492] Review Request: php-horde-Horde-Controller - Horde Controller libraries

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785492

--- Comment #10 from Nick Bebout  ---
Updated

Spec URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Controller.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Controller-2.0.1-3.fc17.src.rpm

-- 
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=wPPAbcIxxa&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922485] Review Request: nodejs-mbtiles - Utilities and tilelive integration for the MBTiles format

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922485

--- Comment #2 from Tom Hughes  ---
The deletion was just to make install easier - it can't be deleted in %setup
because we do need to install it.

Fixed now anyway, and manual pages added:

Spec URL: http://download.compton.nu/nodejs/nodejs-mbtiles.spec
SRPM URL: http://download.compton.nu/nodejs/nodejs-mbtiles-0.2.8-2.fc18.src.rpm

-- 
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=F2kDYTPVRQ&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785484] Review Request: php-horde-Horde-Template - Horde Template System

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785484

--- Comment #5 from Nick Bebout  ---
Updated

Spec URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Template.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Template-2.0.1-3.fc17.src.rpm

-- 
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=pVVFZHZ9Xz&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #22 from Paulo Andrade  
---
You should write it as

License: ASL 2.0 and BSD and LGPLv2+

I did look at gkregex GKlib headers and sources
earlier in the review, and they are not built;
it uses system regex (regcomp/regexec), that
in Linux just happens to be updated versions
of those GKlib files.

It does not use random.c by default either.

But, it "compiles" all the files, just that
does not generate any code, unless some
preprocessor symbol is defined, for example
USE_GKREGEX to use LGPL regex.

But it actually builds a few, like getopt.c,
it just renames the symbols adding a gk_ prefix.
It also uses glibc qsort macros to implement
several specialized quick sort routines.

-- 
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=EAldqwNMYp&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785451] Review Request: php-horde-Horde-Token - Horde Token API

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785451

--- Comment #7 from Nick Bebout  ---
Updated

Spec URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Token.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Token-2.0.3-3.fc17.src.rpm

-- 
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=EfzaisBzd2&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785469] Review Request: php-horde-Horde-LoginTasks - Horde Login Tasks System

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785469

Nick Bebout  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
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=aauA9MxAKz&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785469] Review Request: php-horde-Horde-LoginTasks - Horde Login Tasks System

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785469

--- Comment #12 from Nick Bebout  ---
Updated

Spec URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-LoginTasks.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-LoginTasks-2.0.2-4.fc17.src.rpm

-- 
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=E7cd0Oqdj5&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785465] Review Request: php-horde-Horde-Group - Horde User Groups System

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785465

--- Comment #9 from Nick Bebout  ---
Updated

Spec URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Group.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Group-2.0.2-1.fc17.src.rpm

-- 
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=smll8Z8Png&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785472] Review Request: php-horde-Horde-Db - Horde Database Libraries

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785472

--- Comment #9 from Nick Bebout  ---
Updated

Spec URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Db.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Db-2.0.2-1.fc17.src.rpm

-- 
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=BHFX2NEbjp&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920751] Review Request: satyr - A set of tools to create anonymous, machine-friendly problem reports

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920751

Michael Schwendt  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #4 from Michael Schwendt  ---
The %changelog could have mentioned the update from 0.1 to 0.2 more explicitly.
It is good habit to explicitly mention that the source tarball has been
replaced and to sum up what sort of update it is (in case it's possible to
classify the update in a concise way), e.g. whether it's just a maintenance
release, whether it includes just a few bug-fixes or also new features, major
rewrites or minor changes.

https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs

[...]

Other than that, you've fixed the only issues with the packaging, and hereby it
is

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=1IsZ52JcQT&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785447] Review Request: php-horde-Horde-Auth - Horde Authentication API

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785447

--- Comment #15 from Nick Bebout  ---
Updated

Spec URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Auth.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Auth-2.0.4-1.fc17.src.rpm

-- 
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=t4BYgbysP2&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #21 from Antonio Trande  ---
(In reply to comment #20)
> It is up to you if wish to make a new package for review with
> the man page correction, otherwise I see no other problems,
> just make sure to add the correction to the final package.

I still have some doubts about license (see comment#0).

> 
> I consider the package APPROVED.

Okay.

-- 
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=wvcPV4n8nI&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785452] Review Request: php-horde-Horde-Stream-Filter - Horde Stream filters

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=785452

--- Comment #10 from Nick Bebout  ---
Updated

Spec URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Stream-Filter.spec
SRPM URL:
http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-Stream-Filter-2.0.1-4.fc17.src.rpm

-- 
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=TQ0Z6NynhW&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 877651] Review Request: sagemath - A free open-source mathematics software system

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=877651

--- Comment #57 from Paulo Andrade  
---
Update:

- Update to sagemath 5.8.
- Do full cleanup of notebook package on uninstall.
- Remove with_sage_cython build conditional.
- Remove with_sage_networkx build conditional.
- Add nopari2.6 patch to not rely on not yet available interfaces.
- Add cryptominisat patch to build package in f18.

Spec URL: http://pcpa.fedorapeople.org/sagemath.spec
SRPM URL:
http://pcpa.fedorapeople.org/sagemath/SRPMS/sagemath-5.8-1.fc20.src.rpm


The nopari2.6 is a simple patch due to using features from
the upcoming pari 2.6 in the patched pari.spkg.

The cryptominisat patch is because it now requires
cryptominisat 2.9.6 and does not build with f18
cryptominisat 2.9.5.

Remaining changes, not worth to add to changelog, or that
I forgot to :-), are patch rediffs, minor corrections
to documentation build and adding ldconfig to post and
postun.

-- 
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=ymjfiVdzte&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

Paulo Andrade  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #20 from Paulo Andrade  
---
It is up to you if wish to make a new package for review with
the man page correction, otherwise I see no other problems,
just make sure to add the correction to the final package.

I consider the package APPROVED.


BTW, I see you did not yet populate the MUMPS git tree:
http://pkgs.fedoraproject.org/cgit/MUMPS.git/
Do you need some help with 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=auRev1vcDb&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

Paulo Andrade  changed:

   What|Removed |Added

  Flags||fedora-review?

--- Comment #19 from Paulo Andrade  
---
The permissions warning appears bogus, and should
be caused by not installing files owned by root:

$ rpm2cpio metis-5.0.3-6.fc19.x86_64.rpm |cpio -id
2077 blocks
$ ll usr/bin/ndmetis
-rwxr-xr-x 1 pcpa pcpa 40648 Mar 20 16:25 usr/bin/ndmetis

The missing manual page warning is due to explicitly
compressing them. Should not do it, just copy the
manual pages to the proper directory and let rpmbuild
compress them. Also, change this:

mkdir %{buildroot}%{_datadir}
mkdir %{buildroot}%{_mandir} ; mkdir %{buildroot}%{_mandir}/man1

into

mkdir -p %{buildroot}%{_mandir}/man1

-- 
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=uyMv65tMfY&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923960] New: Review Request: JMapViewer - JMapViewer is a java component to integrate an OSM map view into your Java application.

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923960

Bug ID: 923960
   Summary: Review Request: JMapViewer - JMapViewer is a java
component to integrate an OSM map view into your Java
application.
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: johannes.l...@googlemail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://hannes.fedorapeople.org/JMapViewer.spec
SRPM URL: http://hannes.fedorapeople.org/JMapViewer-2013.02.10-1.fc18.src.rpm
Description: JMapViewer is a Java Swing component for integrating OSM maps 
into your Java application. 
JMapViewer allows you to set markers on the map or zoom 
to a specific location on the map.
Fedora Account System Username: hannes

I need to remove the probably trademarked logo from the upstream release
package, that's why I need to create a new tar ball without the logo and the
relevant code.
https://lists.fedoraproject.org/pipermail/legal/2012-May/001905.html

-- 
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=ZVThxzBSq9&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #18 from Paulo Andrade  
---

Package Review
==

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


Issues:
===
- Permissions on files are set properly.
  Note: See rpmlint output
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions
- Spec file lacks Packager, Vendor, PreReq tags.
  Note: Found : Packager: pcpa 
  See: (this test has no URL)


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: 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.
[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.
[x]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[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. Licenses found:
 "BSD", "BSD (2 clause)", "LGPL (v2.1 or later) (with incorrect FSF
 address)", "Unknown or generated". 4 files have unknown license. Detailed
 output of licensecheck in /home/pcpa/920518-metis/licensecheck.txt
[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.
[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]: 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.
[x]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 419840 bytes in 4 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]: Fully versioned dependency in subpackages, if present.
[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:
[x]: 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.
[x]: Scriptlets must be sane, if used.
[x]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if avail

[Bug 923959] New: Review Request: JOrtho - A spell checker for Java

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923959

Bug ID: 923959
   Summary: Review Request: JOrtho - A spell checker for Java
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: johannes.l...@googlemail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://hannes.fedorapeople.org/JOrtho.spec
SRPM URL: http://hannes.fedorapeople.org/JOrtho-0.5-1.fc18.src.rpm
Description: 
JOrtho is a spell checker for Java. The library works with any
JTextComponent from the Swing framework and checks as you type.
The dictionary is based on the free Wiktionary.org, and is 
applicable for multiple languages. You can select the 
spell checking language via a context menu. The features 
of JOrtho are the highlighting of potentially wrongly spelled words, 
a context menu with suggestions for correct forms of the word
and a context menu with option to change the checking language. 
At the moment there are eight languages for spell checking available: 
English, German, French, Spanish, Italian, Russian, Polish, and Arabic.

Fedora Account System Username: hannes

-- 
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=AKQapCcdQE&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922165] Review Request: openstack-ceilometer - OpenStack measurement collection service

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922165

--- Comment #4 from Pádraig Brady  ---
Spec URL: http://fedorapeople.org/~pbrady/ceilometer/openstack-ceilometer.spec
SRPM URL:
http://fedorapeople.org/~pbrady/ceilometer/openstack-ceilometer-2013.1-0.3.g3.fc18.src.rpm
Description: OpenStack ceilometer provides services to measure and
collect metrics from OpenStack components.
Fedora Account System Username: pbrady

-- 
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=uRaI2GXl3Z&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #17 from Antonio Trande  ---
Spec URL: http://sagitter.fedorapeople.org/metis/metis.spec
SRPM URL: http://sagitter.fedorapeople.org/metis/metis-5.0.3-6.fc18.src.rpm

It should be functioning.

-- 
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=loqazwU5X0&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 891232] nodejs-ronn - markdown to roff/html converter

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891232

Tom Hughes  changed:

   What|Removed |Added

 CC||t...@compton.nu

--- Comment #5 from Tom Hughes  ---
There's an error in the unbundling patch - when it creates the markdown parser
it should do:

  var md = require('markdown').markdown;

not:

  var md = require('markdown');

as the top level parse function exported by the unbundled markdown returns HTML
rather than the parse tree that ronn wants.

-- 
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=QUIxqNOX3q&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919265] Review Request: Bijiben - Note taking app

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919265

Tom "spot" Callaway  changed:

   What|Removed |Added

 Blocks|182235 (FE-Legal)   |
 CC||tcall...@redhat.com

--- Comment #14 from Tom "spot" Callaway  ---
License is correct. Lifting FE-Legal.

-- 
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=AEX6YTqw3g&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #16 from Paulo Andrade  
---
Probably you have it installed in the mock chroot.
Try "mock --clean" and run fedora-review 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=UNkaSPs33J&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #15 from Antonio Trande  ---
Created attachment 713341
  --> https://bugzilla.redhat.com/attachment.cgi?id=713341&action=edit
build.log from fedora-reviw results directory

-- 
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=92QZPLltiG&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 920518] Review Request: metis - Serial Graph Partitioning and Fill-reducing Matrix Ordering

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=920518

--- Comment #14 from Antonio Trande  ---
(In reply to comment #13)
> Appears good looking in the spec, but when trying
> fedora-review, it fails in %check because the binaries
> are not in $PATH:
> 
> + cd /builddir/build/BUILD
> ~/build/BUILD/metis-5.0.3/graphs ~/build/BUILD/metis-5.0.3
> + cd metis-5.0.3
> + pushd graphs
> + ndmetis mdual.graph
> /var/tmp/rpm-tmp.lerKCw: line 30: ndmetis: command not found

Not for me.(In reply to comment #13)
> Appears good looking in the spec, but when trying
> fedora-review, it fails in %check because the binaries
> are not in $PATH:
> 
> + cd /builddir/build/BUILD
> ~/build/BUILD/metis-5.0.3/graphs ~/build/BUILD/metis-5.0.3
> + cd metis-5.0.3
> + pushd graphs
> + ndmetis mdual.graph
> /var/tmp/rpm-tmp.lerKCw: line 30: ndmetis: command not found

Not for me.
I show you the build.log from my fedora-review output.

-- 
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=cmhC6YLTK2&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919094] Review Request: perl-App-p - Steroids for your perl one-liners

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919094

--- Comment #3 from Jon Ciesla  ---
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=Z8G5dxuoNa&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919094] Review Request: perl-App-p - Steroids for your perl one-liners

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919094

Petr Pisar  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #2 from Petr Pisar  ---
New Package SCM Request
===
Package Name: perl-App-p
Short Description: Steroids for your perl one-liners
Owners: ppisar jplesnik psabata
Branches: 
InitialCC: perl-sig

-- 
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=EiLAKPaSjX&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919094] Review Request: perl-App-p - Steroids for your perl one-liners

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919094

Petr Šabata  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #1 from Petr Šabata  ---
No issues.  Nice app.  Approving.

-- 
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=h9LblXkvtZ&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915791] Review Request: perl-MogileFS-Server - Server part of the MogileFS distributed file system

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915791

--- Comment #6 from Petr Pisar  ---
SRPM URL:
http://ppisar.fedorapeople.org/perl-MogileFS-Server/perl-MogileFS-Server-2.67-1.fc20.src.rpm

-- 
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=1DEscH7kMe&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915791] Review Request: perl-MogileFS-Server - Server part of the MogileFS distributed file system

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915791

--- Comment #5 from Petr Pisar  ---
Updated package is available:
Spec URL:
http://ppisar.fedorapeople.org/perl-MogileFS-Server/perl-MogileFS-Server.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-MogileFS-Server/perl-MogileFS-Server-2.67-0.fc20.src.rpm

-- 
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=vuZDBtyO8d&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915791] Review Request: perl-MogileFS-Server - Server part of the MogileFS distributed file system

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915791

--- Comment #4 from Petr Pisar  ---
These are some time-outs or locking problems in SQLite:

# Creating 100 files...
# created 10/100
[Wed Mar 20 16:51:06 2013] crash log: Database error from
MogileFS::Store/lib/MogileFS/Store.pm/1219: database is locked at
lib/MogileFS/Worker/Monitor.pm line 378.
# created 20/100
[Wed Mar 20 16:51:07 2013] Child 2640 (monitor) died: 256 (UNEXPECTED)
[Wed Mar 20 16:51:07 2013] Job monitor has only 0, wants 1, making 1.
# created 30/100
MogileFS::Backend: timed out after 3s against 127.0.0.1:7001 when sending
command: [create_close
domain=testdom&fid=43&devid=2&path=http://127.0.1.1:7500/dev2/0/000/000/43.fid&size=8192&key=manyhundred_38
] at /usr/share/perl5/vendor_perl/MogileFS/NewHTTPFile.pm line 335.
# Tests were run but no plan was declared and done_testing() was not seen.
[Wed Mar 20 16:51:16 2013] ending run due to SIGTERM

that occur only in mock environment. There are some massive parallel tests and
SQLite is not capable to handle them probably. Other option is to use MySQL or
Postgres back-end, but AFAIK both are fuzzy in mock.

I'm going to disable the tests.

-- 
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=XQTbI8jCCN&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919094] Review Request: perl-App-p - Steroids for your perl one-liners

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919094

Petr Šabata  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.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=wVKlI3DRKX&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 852211] Review Request: dunst - dmenu-ish lightweight notification-daemon

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=852211

Lukas Zapletal  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2013-03-20 11:35:43

--- Comment #36 from Lukas Zapletal  ---
rawhide and f19

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

-- 
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=3DYG3iHq1k&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923831] New: Review Request: python-ipaddress - Port of the python 3.3+ ipaddress module to 2.6 and 2.7

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923831

Bug ID: 923831
   Summary: Review Request: python-ipaddress -  Port of the python
3.3+ ipaddress module to 2.6 and 2.7
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: matt_dom...@dell.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://domsch.com/linux/fedora/python-ipaddress/python-ipaddress.spec
SRPM URL:
http://domsch.com/linux/fedora/python-ipaddress/python-ipaddress-1.0.3-1.fc18.src.rpm
Description: ipaddress provides the capabilities to create, manipulate and
operate
on IPv4 and IPv6 addresses and networks.

The functions and classes in this module make it straightforward to
handle various tasks related to IP addresses, including checking
whether or not two hosts are on the same subnet, iterating over all
hosts in a particular subnet, checking whether or not a string
represents a valid IP address or network definition, and so on.

Fedora Account System Username: mdomsch

rpmlint output (all ignorable; there's no documentation or license file
included):
python-ipaddress.src: W: spelling-error %description -l en_US subnet -> subset,
sublet, sub net
python-ipaddress.noarch: W: spelling-error %description -l en_US subnet ->
subset, sublet, sub net
python-ipaddress.noarch: W: no-documentation
2 packages and 1 specfiles checked; 0 errors, 3 warnings.

-- 
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=zEDyeTJEY3&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915791] Review Request: perl-MogileFS-Server - Server part of the MogileFS distributed file system

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915791

--- Comment #3 from Petr Šabata  ---
Hmmm, many of the tests fail in mock.
It also seems your will need SCTP support which is optional in Fedora.  This
will require thorough inspection...

-- 
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=CBHHR0nd8f&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 726910] Review Request: brandy - A BBC BASIC interpreter for Linux (was dead.package)

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=726910

--- Comment #2 from Chris Tyler  ---
Missed a BR, let's try this again.

Spec URL:
http://england.proximity.on.ca/chris/package-review/brandy/brandy.spec
SRPM URL:
http://england.proximity.on.ca/chris/package-review/brandy/brandy-1.20-2.pre5.fc18.src.rpm

-- 
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=WUteFjTJsS&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 907261] Review Request: poly2tri - A 2D constrained Delaunay triangulation library

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=907261

--- Comment #14 from Miro Hrončok  ---
- Using soname version 1.0
- Corrected Makefile to actually produce the library with soname version 1.0
- Added comment about the Makefile and WAF

Spec URL: https://raw.github.com/hroncok/SPECS/master/poly2tri.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/poly2tri-0.0-4.20120407hgacf81f1f1764.fc18.src.rpm

-- 
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=6yIdPevYla&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 890839] Review Request: slic3r - G-code generator for 3D printers (RepRap, Makerbot, Ultimaker etc.)

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=890839

--- Comment #14 from Miro Hrončok  ---
Oh, OK.

Spec URL: https://raw.github.com/hroncok/SPECS/master/slic3r.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/slic3r-0.9.8-4.fc18.src.rpm

As the reasons for slic3r-english-locale.patch are complicated, I wrote:

# Use English decimal separator for numbers
# Reasons are a bit complicated and are described in the patch
Patch1: %{name}-english-locale.patch

Hope, that's enough (I don't want the long justification in spec as a comment).

-- 
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=JYuEQCOPMy&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 726910] Review Request: brandy - A BBC BASIC interpreter for Linux (was dead.package)

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=726910

--- Comment #1 from Chris Tyler  ---
Updated to 1.20pre5.

Spec URL:
http://england.proximity.on.ca/chris/package-review/brandy/brandy.spec
SRPM URL:
http://england.proximity.on.ca/chris/package-review/brandy/brandy-1.20-1.pre5.fc18.src.rpm

-- 
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=C6XmD8owHS&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922745] Review Request: perl-Module-Load - Run-time require of both modules and files

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922745

--- Comment #5 from Petr Pisar  ---
Waiting on unblocking .

-- 
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=XE1N6inZj2&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922479] Review Request: nodejs-uri-js - URI parsing/validating/resolving library for Javascript

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922479

--- Comment #8 from Fedora Update System  ---
nodejs-uri-js-1.4.2-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/nodejs-uri-js-1.4.2-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=zbih0Oh5o0&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922479] Review Request: nodejs-uri-js - URI parsing/validating/resolving library for Javascript

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922479

Fedora Update System  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=nNMVDG4FrW&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 915920] Review Request: qt5-qtsvg - Qt5 - QtSvg component

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915920

Dan Horák  changed:

   What|Removed |Added

 Blocks||907398

-- 
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=nAqL7EdsaY&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922745] Review Request: perl-Module-Load - Run-time require of both modules and files

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922745

--- Comment #4 from Jon Ciesla  ---
Unretired.

-- 
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=gxWhRvMhCn&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923460] Review Request: libqmi - glib helper library for the Qualcomm MSM Interface (QMI) protocol

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923460

--- Comment #2 from Dan Williams  ---
Yanko: oops :)  Didn't realize there was another request, sorry about that.

-- 
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=A4WMoBoWBX&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882559] Review Request: mate-file-manager-image-converter - MATE Desktop file manager image converter plugin

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882559

--- Comment #19 from Jon Ciesla  ---
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=cwakoYZeQB&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 875308] Review Request: mate-menu-editor - MATE Desktop menu editor

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875308

--- Comment #15 from Jon Ciesla  ---
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=jyZg0kZI0p&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923136] Review Request: perl-Digest-MD5 - Perl interface to the MD5 algorithm

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923136

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Digest-MD5-2.52-1.fc20
 Resolution|--- |RAWHIDE
Last Closed||2013-03-20 08:01:19

--- Comment #7 from Petr Pisar  ---
Thank you for the review and the 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=0KI1lf9D9u&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922479] Review Request: nodejs-uri-js - URI parsing/validating/resolving library for Javascript

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922479

--- Comment #7 from Pavol Babinčák  ---
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=ClPIAYRJJw&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922479] Review Request: nodejs-uri-js - URI parsing/validating/resolving library for Javascript

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922479

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |
  Flags||fedora-cvs?

--- Comment #6 from Tom Hughes  ---
Package Change Request
==
Package Name: nodejs-uri-js
New Branches: f19
Owners: tomh

-- 
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=INQpTsDjhK&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 919265] Review Request: Bijiben - Note taking app

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=919265

--- Comment #13 from Matthias Clasen  ---
I can sponsor you

-- 
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=KLqPi3owKO&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923564] Review Request: NetworkManager-ssh - NetworkManager VPN plugin for SSH

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923564

--- Comment #5 from Eduardo Echeverria  ---
Oh, i'm sorry, i had a copy n' paste error (in the case of
NetworkManager-devel) , this not applied to the dependent packages with epoch,
which must be declared anywhere as  %{epoch}:%{version}-%{release}

-- 
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=uVctHrPGDS&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923470] Review Request: python-q - Quick and dirty python debugging output

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=923470

--- Comment #3 from Pavol Babinčák  ---
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=7cui6gL9oR&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 922841] Review Request: perl-Text-ParseWords - Parse text into an array of tokens or array of arrays

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=922841

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Text-ParseWords-3.29-1
   ||.fc20
 Resolution|--- |RAWHIDE
Last Closed||2013-03-20 06:46:00

--- Comment #4 from Petr Pisar  ---
Thank you for the review and the 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=da7sQ8GZBW&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review