[Bug 1018393] Review Request: lv2-artyfx-plugins - a collection of LV2 audio plugins

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018393

Brendan Jones  changed:

   What|Removed |Added

Summary|Review  Request:|Review  Request:
   |lv2-artyfx-plugins  |lv2-artyfx-plugins - a
   ||collection of LV2 audio
   ||plugins



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876317] Review Request: ec2-metadata - EC2 instance metadata query tool

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=876317

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|ec2-metadata-0.1.1-1.fc19   |ec2-metadata-0.1.1-1.fc20



--- Comment #15 from Fedora Update System  ---
ec2-metadata-0.1.1-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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1008059] Review Request: xfce4-whiskermenu-plugin - An alternate application launcher for Xfce

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1008059



--- Comment #7 from Fedora Update System  ---
xfce4-whiskermenu-plugin-1.1.1-1.fc18 has been submitted as an update for
Fedora 18.
https://admin.fedoraproject.org/updates/xfce4-whiskermenu-plugin-1.1.1-1.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1008059] Review Request: xfce4-whiskermenu-plugin - An alternate application launcher for Xfce

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1008059



--- Comment #6 from Fedora Update System  ---
xfce4-whiskermenu-plugin-1.1.1-1.fc19 has been submitted as an update for
Fedora 19.
https://admin.fedoraproject.org/updates/xfce4-whiskermenu-plugin-1.1.1-1.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1008059] Review Request: xfce4-whiskermenu-plugin - An alternate application launcher for Xfce

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1008059

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

[Bug 1008059] Review Request: xfce4-whiskermenu-plugin - An alternate application launcher for Xfce

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1008059



--- Comment #5 from Fedora Update System  ---
xfce4-whiskermenu-plugin-1.1.1-1.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/xfce4-whiskermenu-plugin-1.1.1-1.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016943] Review Request: crystal - KDE WM theme (KDE4-compatible version)

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016943

Rex Dieter  changed:

   What|Removed |Added

 CC||chitl...@gmail.com,
   ||rdie...@math.unl.edu
  Component|Package Review  |crystal
   Assignee|nob...@fedoraproject.org|chitl...@gmail.com



--- Comment #4 from Rex Dieter  ---
I think the crystal maintainer isn't very active these days, I assume you'd be
interested in helping co-maintain it?  If so, I can help with sponsoring...

re-assigning to crystal component.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1017766] Review Request: google-crosextra-caladea-fonts - Sans-serif font metric-compatible with Cambria font

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017766



--- Comment #11 from Parag  ---
Thanks for your quick reply.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018057] Review Request: golang-googlecode-goprotobuf - Go support for Google protocol buffers

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018057

Lokesh Mandvekar  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |golang-googlecode-protobuf  |golang-googlecode-goprotobu
   |- Go support for Google |f - Go support for Google
   |protocol buffers|protocol buffers



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018384] Review Request: tinyca2 - TinyCA graphical openssl based CA

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018384



--- Comment #4 from Paul Wouters  ---
Spec URL: ftp://ftp.nohats.ca/tinyca2/tinyca2.spec
SRPM URL:ftp://ftp.nohats.ca/tinyca2/tinyca2-0.7.6-0.9.20070611.fc19.src.rpm

* Sat Oct 12 2013 Paul Wouters  - 0.7.6-0.9.20070611
- cleanup macro style, remove el5 compat items, use global instead of define
- Clean vendor
- Fixup perl requires
- Fixup locale handling
- Preserve timestamps with install
- Fixup mixed tab/spaces

This addresses 1) 2) 4) 8) and 9)
For 5) 6) and 7) well, upstream has been dead since 2007 :) These patches are
mostly to make things work on newer versions of perl.

3) Why should it be a separate file? (also this way if we change bindir again
it remains working :)

10) I agree it is a bit silly. But its automatically generated based on where
it is installed. Is it worth changing this? That would be a big change and
upstream would never merge it in.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 995283] python-flask-restless - Provides simple generation of ReSTful APIs

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=995283

Fedora Update System  changed:

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|ERRATA  |---
   Keywords||Reopened



--- Comment #13 from Fedora Update System  ---
Package python-flask-restless-0.12.0-1.fc19:
* should fix your issue,
* was pushed to the Fedora 19 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing
python-flask-restless-0.12.0-1.fc19'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-18920/python-flask-restless-0.12.0-1.fc19
then log in and leave karma (feedback).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
nodejs-node-static-0.7.1-2.fc19 has been pushed to the Fedora 19 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016441] Review Request: obapps - Graphical tool for configuring per-application settings in Openbox

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016441

Patrick Uiterwijk  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jwrigl...@gmail.com,
   ||puiterw...@redhat.com
 Depends On||1015761
   Assignee|puiterw...@gmail.com|puiterw...@redhat.com
  Flags||needinfo?(jwrigley7@gmail.c
   ||om)



--- Comment #1 from Patrick Uiterwijk  ---
Please apply a fix for #1015761 (or an update fixing it)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1015761
[Bug 1015761] [abrt] obapps-0.1.7-5.fc19:
_controls.py:4560:GetItem:TypeError: in method 'ListCtrl_GetItem', expected
argument 2 of type 'long'
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018384] Review Request: tinyca2 - TinyCA graphical openssl based CA

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018384

Patrick Uiterwijk  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



--- Comment #3 from Patrick Uiterwijk  ---
Package Review
==

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


Issues:
===
1. Buildroot, rm -rf $BUILDROOT and %clean are present: is it your intention to
also package for EL5?
If not, please remove them
2. Please use just one style of variables: either $BUILDROOT style or
%{_man_dir} style
3. Please move the desktop file to a source file instead of writing it from the
spec
4. Needs the explicit perl requires:
https://fedoraproject.org/wiki/Packaging:Perl#Versioned_MODULE_COMPAT_Requires
5. You should ask upstream to add license text
6. Please add justification to patches, and try to send them upstream
7. Please add info or url on how to get Source tarball
8. Please use %global instead of %define
9. Please fix the locales:
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Handling_Locale_Files
10. Final provides seem insane: why is it providing perl(CA), perl(GUI),
amongst others?


= MUST items =

Generic:
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[-]: 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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "GPL (v2 or later) (with incorrect FSF address)", "Unknown or generated".
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[!]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[!]: The spec file handles locales properly.
[!]: 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.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
[x]: Package complies to the Packaging Guidelines
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[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]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: %config files are marked noreplace or the reason is justified.
[x]: Macros in Summary, %description expandable at SRPM build time.
[!]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install or desktop-
 file-validate if there is such a file.
[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]: No %config files under /usr.
[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

Perl:
[!]: Package contains the mandatory BuildRequires and Requires:.
 Note: Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo
 $version)) missing?

= SHOULD items =

Generic:
[x]: Reviewer should test that the package builds in mo

[Bug 1015765] Review Request: islamic-menus - Islamic menus for desktops conforming with xdg standards

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015765



--- Comment #6 from Mosaab Alzoubi  ---
- Fix spec name.

Spec : http://ojuba.org/oji/SPECS/islamic-menus.spec
SRPM : http://ojuba.org/oji/SRPMS/islamic-menus-1.0.5-4.oji.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1015775] Review Request: TuxCut - TuxCut is a netcut "in windows os " like software written in PyQt

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015775



--- Comment #11 from Mosaab Alzoubi  ---
- Add mark of source URL.

Spec : http://ojuba.org/oji/SPECS/tuxcut.spec
SRPM : http://ojuba.org/oji/SRPMS/tuxcut-5.0-11.oji.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1015776] Review Request: winetricks - Winetricks is a package manager for win32 dlls and applications on posix.

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015776



--- Comment #2 from Mosaab Alzoubi  ---
After many revisions :)

Spec : http://ojuba.org/oji/SPECS/winetricks.spec
SRPM : http://ojuba.org/oji/SRPMS/winetricks-20130919-2.oji.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018393] Review Request: lv2-artyfx-plugins

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018393

Brendan Jones  changed:

   What|Removed |Added

 Blocks||805236 (FedoraAudio)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=805236
[Bug 805236] Tracker: Fedora Audio Spin
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018393] New: Review Request: lv2-artyfx-plugins

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018393

Bug ID: 1018393
   Summary: Review  Request: lv2-artyfx-plugins
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: brendan.jones...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Arty FX is a plugin bundle of artistic “RT” effects. The aim of this plugin
collection is to allow the designing of your sound just as you desired using a
fast, effiecient workflow.

http://openavproductions.com/artyfx/

SRPM:
http://bsjones.fedorapeople.org/reviews/lv2-artyfx-plugins-0-0.1.20131011gita4d52ec.fc20.src.rpm
SPEC: http://bsjones.fedorapeople.org/reviews/lv2-artyfx-plugins.spec

rpmlint
/home/bsjones/rpmbuild/SRPMS/lv2-artyfx-plugins-0-0.1.20131011gita4d52ec.fc20.src.rpm
/home/bsjones/rpmbuild/RPMS/x86_64/lv2-artyfx-plugins-0-0.1.20131011gita4d52ec.fc20.x86_64.rpm
~/rpmbuild/SPECS/lv2-artyfx-plugins.spec 
lv2-artyfx-plugins.src: W: spelling-error %description -l en_US workflow ->
work flow, work-flow, workforce
lv2-artyfx-plugins.x86_64: W: spelling-error %description -l en_US workflow ->
work flow, work-flow, workforce
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018384] Review Request: tinyca2 - TinyCA graphical openssl based CA

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018384

Patrick Uiterwijk  changed:

   What|Removed |Added

 CC||puiterw...@redhat.com
   Assignee|nob...@fedoraproject.org|puiterw...@redhat.com
  Flags||fedora-review?



--- Comment #2 from Patrick Uiterwijk  ---
I will review this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018384] Review Request: tinyca2 - TinyCA graphical openssl based CA

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018384



--- Comment #1 from Paul Wouters  ---
This is a package revival. According to dead.package:

This package was retired due to no active owner on 2011-02-23

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018384] New: Review Request: tinyca2 - TinyCA graphical openssl based CA

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018384

Bug ID: 1018384
   Summary: Review Request: tinyca2 - TinyCA graphical openssl
based CA
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pwout...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL: ftp://ftp.nohats.ca/tinyca2/tinyca2.spec
SRPM URL:ftp://ftp.nohats.ca/tinyca2/tinyca2-0.7.6-0.8.20070611.fc19.src.rpm
Description: TinyCA2 is a graphical tool written in Perl/Gtk to manage a small
Certification Authority (CA) using openssl.

TinyCA supports
- creation and revocation of x509 - S/MIME certificates.
- PKCS#10 requests.
- exporting certificates as PEM, DER, TXT, and PKCS#12.
- server certificates for use in web servers, email servers, IPsec,
  and more.
- client certificates for use in web browsers, email clients, IPsec,
  and more.
- creation and management of SubCAs

Fedora Account System Username: pwouters

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016803] Review Request: rodent-icon-theme - SVG scalable icon theme by Rodent

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016803



--- Comment #10 from Mario Blättermann  ---
(In reply to Antonio Trande from comment #9)
> What about index.theme file from rodent-icon-theme ?
> It's already installed by xfce4-icon-theme in /usr/share/icons/Rodent and we
> need to update it with new directories from rodent-icon-theme.

Your package must not touch a file from another package. But I don't know what
to do in this case. Just an idea: Rodent icons are an addition to
xfce4-icon-theme, as far as I can evaluate. What about to ask the
xfce4-icon-theme maintainers to add the Rodent icons as a second source,
omitting the duplicates? The index file could be patched to recognize the extra
folders.

And another idea: Rename the package, the index file and the folders in a way
that nothing can cause an installation conflict. I propose "Rodent-new" or
anything similar.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 904640] Review Request: rubygem-domain_name - Domain Name manipulation library for Ruby

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=904640

Ken Dreyer  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #9 from Ken Dreyer  ---
Looks good to me! Approved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016753] Review Request: nodejs-bson - bson parser for node.js

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016753

Troy Dawson  changed:

   What|Removed |Added

 Depends On||1016770



--- Comment #5 from Troy Dawson  ---
Spec URL: http://tdawson.fedorapeople.org/nodejs/nodejs-bson.spec
SRPM URL:
http://tdawson.fedorapeople.org/nodejs/nodejs-bson-0.2.3-1.fc20.src.rpm

- Updated to latest version - 0.2.3
- Updated BuildRequires to require nan
- Included test, though it is not enabled due to none of the packages being in
Fedora.
-- Created Source1 (the tests) and put how it was created in comments
-- Added BuildRequires for npm(gleak), npm(nodeunit), npm(one)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1016770
[Bug 1016770] Review Request: nodejs-nan - Native Abstractions for Node.js
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016770] Review Request: nodejs-nan - Native Abstractions for Node.js

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016770

Troy Dawson  changed:

   What|Removed |Added

 Blocks||1016753




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1016753
[Bug 1016753] Review Request: nodejs-bson - bson parser for node.js
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016803] Review Request: rodent-icon-theme - SVG scalable icon theme by Rodent

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016803



--- Comment #9 from Antonio Trande  ---
(In reply to Mario Blättermann from comment #8)
> (In reply to Mario Blättermann from comment #7)
> > Currently xfce4-icon-theme doesn't contain scalable icons, only the 48x48
> > ones.
> Sorry for the blurb. Indeed, there are scalable icons in xfce4-icon-theme.
> To don't cause package conflicts, you *could* let co-own some folders by
> both packages, but in case of file conflicts, you have to remove the
> appropriate icons from your package.
> 

What about index.theme file from rodent-icon-theme ?
It's already installed by xfce4-icon-theme in /usr/share/icons/Rodent and we
need to update it with new directories from rodent-icon-theme.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016943] Review Request: crystal - KDE WM theme (KDE4-compatible version)

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016943

Mario Blättermann  changed:

   What|Removed |Added

 CC||mario.blaetterm...@gmail.co
   ||m



--- Comment #3 from Mario Blättermann  ---
(In reply to Ben Nemec from comment #2)
> I guess I thought that was what I was doing - the wiki instructions on
> adding new packages led me to open this bug.  Did you have something else in
> mind?

You should open a bug for the existing crystal package, not a review request.
The package is outdated and needs some effort to make it work with KDE 4.x.
Review requests are only for completely new packages or that ones which have
been orphaned or retired for some reasons.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1015775] Review Request: TuxCut - TuxCut is a netcut "in windows os " like software written in PyQt

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015775



--- Comment #10 from Mosaab Alzoubi  ---
- Full exit from opt dir.
- Add rule to polkit.

Spec : http://ojuba.org/oji/SPECS/tuxcut.spec
SRPM : http://ojuba.org/oji/SRPMS/tuxcut-5.0-10.oji.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 908116] Review Request: openshift-origin-console - The OpenShift Management Console

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=908116



--- Comment #11 from Michael Scherer  ---
Doesn't build, it is missing a requires on systemd :

+ /usr/lib/rpm/redhat/brp-java-repack-jars
Traitement des fichiers : openshift-origin-console-1.5.18-2.fc19.noarch
erreur : Le fichier doit commencer par « / » :
%{_unitdir}/openshift-console.service
Erreur de construction de RPM :
Le fichier doit commencer par « / » : %{_unitdir}/openshift-console.service
Child return code was: 1

( ie, it doesn't expand %{_unitdir}

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 908116] Review Request: openshift-origin-console - The OpenShift Management Console

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=908116

Michael Scherer  changed:

   What|Removed |Added

  Flags|needinfo?(m...@zarb.org)|



--- Comment #10 from Michael Scherer  ---
Oups, damn, I forgot about it :/

Will try to do it during the weekend.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1017766] Review Request: google-crosextra-caladea-fonts - Sans-serif font metric-compatible with Cambria font

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017766

Yohan Graterol  changed:

   What|Removed |Added

  Flags|needinfo?(yohangraterol92@g |
   |mail.com)   |



--- Comment #10 from Yohan Graterol  ---
Of course, now I'm at the work. Please, wait tomorrow when I will be at my
house. Excuse me Parag.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1017766] Review Request: google-crosextra-caladea-fonts - Sans-serif font metric-compatible with Cambria font

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017766

Parag  changed:

   What|Removed |Added

  Flags||needinfo?(yohangraterol92@g
   ||mail.com)



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018057] Review Request: golang-googlecode-protobuf - Go support for Google protocol buffers

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018057



--- Comment #3 from Matthew Miller  ---
It looks like this package has two parts: goprotobuf/proto, the library, and
goprotobuf/protoc-gen-go, a protocol compiler plugin. I _think_ etcd just needs
the former, and already has its code generated.

I think we probably should include both, possibly as different subpackages.

Or, maybe we keep the goprotobuf/proto part in the devel package and build
protoc-gen-go as part of the base package (which would then be arch-specific
because protoc-gen-go is a binary).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1015775] Review Request: TuxCut - TuxCut is a netcut "in windows os " like software written in PyQt

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015775

Mosaab Alzoubi  changed:

   What|Removed |Added

 Depends On||458379




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=458379
[Bug 458379] Review Request: wondershaper - Simple network shaper
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1017766] Review Request: google-crosextra-caladea-fonts - Sans-serif font metric-compatible with Cambria font

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017766



--- Comment #9 from Parag  ---
Yohan,
   Can you check recent update above for the review of this package?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1015775] Review Request: TuxCut - TuxCut is a netcut "in windows os " like software written in PyQt

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015775



--- Comment #9 from Mosaab Alzoubi  ---
OK Rewritten ::

Spec : http://ojuba.org/oji/SPECS/tuxcut.spec
SRPM : http://ojuba.org/oji/SRPMS/tuxcut-5.0-9.oji.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016370] Review Request: rubygem-capillary - Generate a JSON payload from Git log output

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016370

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
rubygem-capillary-1.0.3-3.fc20 has been pushed to the Fedora 20 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895



--- Comment #6 from Fedora Update System  ---
nodejs-node-static-0.7.1-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/nodejs-node-static-0.7.1-2.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895

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

[Bug 1000154] Review Request: rubygem-openshift-origin-dns-nsupdate - Provides a DNS service update plugin using nsupdate

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1000154



--- Comment #2 from Troy Dawson  ---
Spec URL:
http://tdawson.fedorapeople.org/openshift-origin/rubygem-openshift-origin-dns-nsupdate.spec
SRPM URL:
http://tdawson.fedorapeople.org/openshift-origin/rubygem-openshift-origin-dns-nsupdate-1.10.2.1-2.fc20.src.rpm

- gem_cache exlcuded
- gem_docdir moved to ubygem-openshift-origin-dns-nsupdate-doc

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018057] Review Request: golang-googlecode-protobuf - Go support for Google protocol buffers

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018057

Matthew Miller  changed:

   What|Removed |Added

 Blocks||1018007
  Flags||fedora-review?



--- Comment #2 from Matthew Miller  ---
(In reply to Lokesh Mandvekar from comment #0)
> Note: This package uses code.google.com/p/goprotobuf . There's also
> code.google.com/p/protobuf, but I'm guessing the distinction is clear
> because of the _golang_ in the package name

I think it's better to leave the extra "go" in there -- it makes it more clear
even if it's a bit unwieldy. Also matches what Debian is doing, where they use
golang-godebiancontrol as an example. (On the other hand, they leave out the
full path unless there's a conflict. I think what we're doing with always
including it (the "googlecode" part) is better because it's basically just
coincidence which one we package first if there are overlaps.

> Also, should this block Bug 1005504 (already closed) ?

No, that ship has sailed. :) I filed bug #1018007 for these.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1018007
[Bug 1018007] etcd third-party libs need to be broken out into individual
(upstream-based packages)
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 995283] python-flask-restless - Provides simple generation of ReSTful APIs

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=995283



--- Comment #12 from Fedora Update System  ---
python-flask-restless-0.12.0-1.fc19 has been submitted as an update for Fedora
19.
https://admin.fedoraproject.org/updates/python-flask-restless-0.12.0-1.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 995283] python-flask-restless - Provides simple generation of ReSTful APIs

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=995283



--- Comment #11 from Fedora Update System  ---
python-flask-restless-0.12.0-1.fc18 has been submitted as an update for Fedora
18.
https://admin.fedoraproject.org/updates/python-flask-restless-0.12.0-1.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018057] Review Request: golang-googlecode-protobuf - Go support for Google protocol buffers

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018057



--- Comment #1 from Vincent Batts  ---
I do not suppose there is any harm in having the Provides: include the
/testdata directories. Often these directories only have *_test.go source (if
any source at all), so they would not be import'able.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895



--- Comment #5 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895

Troy Dawson  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Troy Dawson  ---
New Package SCM Request
===
Package Name: nodejs-node-static
Short Description: Simple, compliant file streaming module for node
Owners: tdawson
Branches: f18 f19 f20 el6
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 965895] Review Request: nodejs-node-static - Simple, compliant file streaming module for node

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=965895



--- Comment #3 from Troy Dawson  ---
Thanks.
I have changed the *'s to " and a few other minor description changes.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973069] Review Request:lpf-spotify-client - build and install spotify-client rpm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973069



--- Comment #11 from Tom "spot" Callaway  ---
I need to think about this some more. This is an entirely new case. We'd be
telling consumers of Fedora "this file must only be executed/run for personal
use", which is generally incompatible with every other file in Fedora.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1014601] Review Request: python-cssmin - A Python port of the YUI CSS compression algorithm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1014601



--- Comment #17 from Zbigniew Jędrzejewski-Szmek  ---
Can you also submit f20 package?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016294] Review Request: kryo-serializers - Additional kryo for standard JDK types

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016294



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

Don't request rawhide/devel, it's automatic.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016294] Review Request: kryo-serializers - Additional kryo for standard JDK types

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016294

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973069] Review Request:lpf-spotify-client - build and install spotify-client rpm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973069



--- Comment #10 from Alec Leamas  ---
Hm...I take your message as this can't be hosted on fedora with this limitation
("maybe" is worst possible outcome here). 

Since this is not what they intend: Can you think of any wording which resolves
this while still enforcing a non-redistribution condition on the created rpms?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973069] Review Request:lpf-spotify-client - build and install spotify-client rpm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973069



--- Comment #9 from Tom "spot" Callaway  ---
Yes, but they have only granted permission for lpf-spotify to perform
"automatic downloading and repackaging of the spotify software for personal
use"

They're effectively imposing a use restriction on the act of
downloading/repackaging, and I don't think we can do that.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973069] Review Request:lpf-spotify-client - build and install spotify-client rpm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973069



--- Comment #8 from Alec Leamas  ---
Note that the limitation is about what's produced by lpf-spotify, not
lpf-spotify  itself. We distribute an open-source tool which produces non-free
software. Isn't this somewhat similar to an open-sourced client to a service
which is not free?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973069] Review Request:lpf-spotify-client - build and install spotify-client rpm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973069



--- Comment #7 from Tom "spot" Callaway  ---
The only concern I have is "for personal use". I'm not sure it is possible for
us to comply with that license restriction.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 973069] Review Request:lpf-spotify-client - build and install spotify-client rpm

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=973069



--- Comment #6 from Alec Leamas  ---
Three months... but at last a reply [1]. In my eyes, this looks good. And in
yours?


[1]
http://community.spotify.com/t5/Help-Desktop-Linux-Mac-and/What-license-does-the-linux-spotify-client-use/m-p/557782?nobounce#M63523

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1014738] Review Request: python-astropy - A Community Python Library for Astronomy

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1014738



--- Comment #2 from Sergio Pascual  ---
Spec URL: http://guaix.fis.ucm.es/~spr/python-astropy.spec
SRPM URL: http://guaix.fis.ucm.es/~spr/python-astropy-0.2.4-2.fc19.src.rpm

The bundled cfistio library in f20+ has been removed. It uses now system
libraries. When python3-configobject reaches stable I will enable de python3
package

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018133] New: Review Request: php-pecl-memprof - Memory usage profiler

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018133

Bug ID: 1018133
   Summary: Review Request: php-pecl-memprof - Memory usage
profiler
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



Spec URL:
https://raw.github.com/remicollet/remirepo/9d45fad9fdc86b786846679f5078aa74c4d4fa86/php/pecl/php-pecl-memprof/php-pecl-memprof.spec
SRPM URL:
http://rpms.famillecollet.com/SRPMS/php-pecl-memprof-1.0.0-2.remi.src.rpm
Description: 
Memory usage profiler for PHP scripts.

Fedora Account System Username: remi

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 904640] Review Request: rubygem-domain_name - Domain Name manipulation library for Ruby

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=904640



--- Comment #8 from Mamoru TASAKA  ---
Thank you for comments!

http://mtasaka.fedorapeople.org/Review_request/mechanize-related/rubygem-domain_name-0.5.13-2.fc.src.rpm
http://mtasaka.fedorapeople.org/Review_request/mechanize-related/rubygem-domain_name.spec

* Fri Oct 11 2013 Mamoru TASAKA  - 0.5.13-2
- Remove redundant BR

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 908116] Review Request: openshift-origin-console - The OpenShift Management Console

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=908116

Marek Mahut  changed:

   What|Removed |Added

 CC||mma...@redhat.com
  Flags||needinfo?(m...@zarb.org)



--- Comment #9 from Marek Mahut  ---
What's the latest on this review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894482] Review Request: rubygem-openshift-origin-auth-kerberos - OpenShift plugin for kerberos auth service

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894482

Marek Mahut  changed:

   What|Removed |Added

 CC||mma...@redhat.com
  Flags||needinfo?(m...@zarb.org)



--- Comment #20 from Marek Mahut  ---
What's the latest on this review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1000154] Review Request: rubygem-openshift-origin-dns-nsupdate - Provides a DNS service update plugin using nsupdate

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1000154



--- Comment #1 from Marek Mahut  ---
Thank you for the review.

Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=6046995
Scratch build (< f19):
http://koji.fedoraproject.org/koji/taskinfo?taskID=6049796


* Please exclude %{gem_cache}, as we do not need it in the RPM
* I propose to move %{gem_docdir} into its separate -doc package due to its
size (400KB+ vs 80K of code)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1000154] Review Request: rubygem-openshift-origin-dns-nsupdate - Provides a DNS service update plugin using nsupdate

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1000154

Marek Mahut  changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1017767] Review Request: google-crosextra-carlito-fonts - Sans-serif font metric-compatible with Calibri font

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017767



--- Comment #11 from Parag  ---
$ fontaine Carlito-Regular.ttf | grep characterCount
"characterCount":"2116",
$ fontaine calibri.ttf | grep characterCount
"characterCount":"2161",

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1017766] Review Request: google-crosextra-caladea-fonts - Sans-serif font metric-compatible with Cambria font

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017766



--- Comment #8 from Parag  ---
1) right. I just copied the inside font information as summary initially which
is now fixed above.
2) Camibra font coverage
$ fontaine Cambria.ttf | grep characterCount
"characterCount":"3231"
$ fontaine Caladea-Regular.ttf | grep characterCount
"characterCount":"417"

How should we add in summary about this? ms Camibra font is having large
character/unicode coverage.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018092] Review Request: barman - Backup and Recovery Manager for PostgreSQL

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018092

Peter Robinson  changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018092] Review Request: barman - Backup and Recovery Manager for PostgreSQL

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018092

Dale Macartney  changed:

   What|Removed |Added

 CC||pbrobin...@gmail.com
 Depends On||1018090




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1018090
[Bug 1018090] Review Request: python-argcomplete - Bash tab completion for
argparse
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018090] Review Request: python-argcomplete - Bash tab completion for argparse

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018090

Dale Macartney  changed:

   What|Removed |Added

 Blocks||1018092




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1018092
[Bug 1018092] Review Request: barman - Backup and Recovery Manager for
PostgreSQL
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018092] New: Review Request: barman - Backup and Recovery Manager for PostgreSQL

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018092

Bug ID: 1018092
   Summary: Review Request: barman - Backup and Recovery Manager
for PostgreSQL
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: dbmacart...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



SPEC: http://dbmacartney.fedorapeople.org/barman/barman.spec
SRPM: http://dbmacartney.fedorapeople.org/barman/barman-1.2.3-1.fc19.src.rpm

Description: 
Barman (backup and recovery manager) is an administration
tool for disaster recovery of PostgreSQL servers written in Python.
It allows to perform remote backups of multiple servers
in business critical environments and help DBAs during the recovery phase.
Barman's most wanted features include backup catalogs, retention policies,
remote recovery, archiving and compression of WAL files and backups.
Barman is written and maintained by PostgreSQL professionals 2ndQuadrant.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018090] Review Request: python-argcomplete - Bash tab completion for argparse

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018090

Peter Robinson  changed:

   What|Removed |Added

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



--- Comment #2 from Peter Robinson  ---
As Dale's sponsor I'll review this

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018090] Review Request: python-argcomplete - Bash tab completion for argparse

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018090



--- Comment #1 from Dale Macartney  ---
koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6049657

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1018090] New: Review Request: python-argcomplete - Bash tab completion for argparse

2013-10-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1018090

Bug ID: 1018090
   Summary: Review Request: python-argcomplete - Bash tab
completion for argparse
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: dbmacart...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org



SPEC:
http://dbmacartney.fedorapeople.org/python-argcomplete/python-argcomplete.spec
SRPM:
http://dbmacartney.fedorapeople.org/python-argcomplete/python-argcomplete-0.6.3-1.fc19.src.rpm


Description:
Argcomplete provides easy, extensible command line tab completion of
arguments for your Python script.

It makes two assumptions:

 * You're using bash as your shell
 * You're using argparse to manage your command line arguments/options

Argcomplete is particularly useful if your program has lots of
options or subparsers, and if your program can dynamically suggest
completions for your argument/option values (for example, if the user
is browsing resources over the network).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review