[Bug 1168511] Review Request: radamsa - test case generator for robustness testing, aka a fuzzer

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168511



--- Comment #16 from Niranjan MR  ---
Updated spec file: https://mrniranjan.fedorapeople.org/radamsa.spec
UPdated SRPM: https://mrniranjan.fedorapeople.org/radamsa-0.4-2.fc21.src.rpm

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

[Bug 1174288] Review Request: owl-lisp - functional dialect of Scheme

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174288



--- Comment #3 from Niranjan MR  ---
Updated Spec file: https://mrniranjan.fedorapeople.org/owl-lisp.spec
Updated SRPM: https://mrniranjan.fedorapeople.org/owl-lisp-0.1.7-2.fc21.src.rpm

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

[Bug 1176658] Review Request: adwaita-qt - Adwaita Qt theme

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176658

Niranjan MR  changed:

   What|Removed |Added

 CC||niran...@ashoo.in



--- Comment #1 from Niranjan MR  ---
This is un-official package review. 

From the spec file contains: "0.4.copr%{?dist}", I believe  this dist tag is
okay for copr , but for fedora package you would like to modify the dist tag as
per the guidelines specified here:

https://fedoraproject.org/wiki/Packaging:NamingGuidelines?rd=Packaging/NamingGuidelines#Version_Tag
https://fedoraproject.org/wiki/Packaging:NamingGuidelines?rd=Packaging/NamingGuidelines#The_.25.7B.3Fdist.7D_Tag

Package Review
==

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



= MUST items =
[ ]: Provides: bundled(gnulib) in place as required.
 Note: Sources not installed
[ ]: Package does not contain kernel modules.
[ ]: Package contains no static executables.
[ ]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[ ]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[ ]: License field in the package spec file matches the actual license.
 Note: There is no build directory. Running licensecheck on vanilla
 upstream sources. Licenses found: "LGPL (v2.1 or later)", "GPL (v2 or
 later)". Detailed output of licensecheck in /home/orion/review/review-
 adwaita-qt/licensecheck.txt
[ ]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/themes/Adwaita/qt4
[ ]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/themes/Adwaita/qt4,
 /usr/share/themes/Adwaita
[ ]: %build honors applicable compiler flags or justifies otherwise.
[ ]: Package contains no bundled libraries without FPC exception.
[ ]: Changelog in prescribed format.
[ ]: Sources contain only permissible code or content.
[ ]: Package contains desktop file if it is a GUI application.
[ ]: Development files must be in a -devel package
[ ]: Package uses nothing in %doc for runtime.
[ ]: Package consistently uses macros (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.
[ ]: Requires correct, justified where necessary.
[ ]: Spec file is legible and written in American English.
[ ]: Package contains systemd file(s) if in need.
[ ]: Useful -debuginfo package or justification otherwise.
[ ]: Package is not known to require an ExcludeArch tag.
[ ]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 30720 bytes in 2 files.
[ ]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[ ]: If the source package does not include l

[Bug 1060924] Review Request: rubygem-unicode - Unicode normalization library for Ruby

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060924



--- Comment #12 from Fedora Update System  ---
rubygem-unicode-0.4.4.1-2.el7 has been submitted as an update for Fedora EPEL
7.
https://admin.fedoraproject.org/updates/rubygem-unicode-0.4.4.1-2.el7

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

[Bug 1060924] Review Request: rubygem-unicode - Unicode normalization library for Ruby

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060924



--- Comment #11 from Fedora Update System  ---
rubygem-unicode-0.4.4.1-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/rubygem-unicode-0.4.4.1-2.fc20

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

[Bug 1060924] Review Request: rubygem-unicode - Unicode normalization library for Ruby

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060924



--- Comment #10 from Fedora Update System  ---
rubygem-unicode-0.4.4.1-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/rubygem-unicode-0.4.4.1-2.fc21

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

[Bug 1060924] Review Request: rubygem-unicode - Unicode normalization library for Ruby

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060924

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

[Bug 1174933] Review Request: python-rq - Library for creating background jobs, and processing them.

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174933

Niranjan Mallapadi Raghavender  changed:

   What|Removed |Added

 CC||mnira...@redhat.com



--- Comment #1 from Niranjan Mallapadi Raghavender  ---
This is un-official review of the package. 

The source cannot be downloaded.

INFO: Getting .spec and .srpm Urls from : 1174933
INFO:   --> SRPM url:
https://dbruno.fedorapeople.org/python-rq/python-rq-0.4.6-1.fc21.src.rpm
INFO:   --> Spec url: https://dbruno.fedorapeople.org/python-rq/python-rq.spec
INFO: Using review directory: /home/makerpm/rpmbuild/SPECS/1174933-python-rq
INFO: Downloading .spec and .srpm files
INFO: No upstream for (Source2): README-pythonrq.md
INFO: Downloading (Source0):
http://pypi.python.org/packages/source/a/apache-libcloud/rq-0.4.6.tar.gz
WARNING: Cannot download url:
http://pypi.python.org/packages/source/a/apache-libcloud/rq-0.4.6.tar.gz
INFO: No upstream for (Source0): rq-0.4.6.tar.gz
INFO: No upstream for (Source1): LICENSE-pythonrq

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

[Bug 663181] Review Request: ompc - MATLAB to Python syntax adapting compiler

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=663181



--- Comment #17 from Susi Lehtola  ---
Looks like this passed below my radar.

(In reply to Alon Levy from comment #15)
> Regarding the license: I haven't contacted upstream yet about it, I am
> waiting a response for the patches anyway, when that gets back I'll ask
> about the LGPL files in there (lex.py, yacc.py, ply, gplot) and GPL
> (examples/appengine). I'm not packaging examples/appengine right now, so I
> could just change the license to BSD & LGPL. (would that just be License:
> BSD & LGPL ?)

Yes, but you need to include the LGPL version as in the guidelines.

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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445

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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445



--- Comment #5 from Fedora Update System  ---
nodejs-grunt-angular-templates-0.5.7-1.fc21 has been submitted as an update for
Fedora 21.
https://admin.fedoraproject.org/updates/nodejs-grunt-angular-templates-0.5.7-1.fc21

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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445



--- Comment #4 from Fedora Update System  ---
nodejs-grunt-angular-templates-0.5.7-1.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-grunt-angular-templates-0.5.7-1.fc20

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

[Bug 1176595] Review Request: hypre - High performance matrix preconditioners

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176595



--- Comment #2 from Dave Love  ---
I don't know where they went, but they're back now.

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303



--- Comment #10 from Fedora Update System  ---
kde-plasma-redshift-1.0-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/kde-plasma-redshift-1.0-2.fc20

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303



--- Comment #9 from Fedora Update System  ---
kde-plasma-redshift-1.0-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/kde-plasma-redshift-1.0-2.fc21

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303

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

[Bug 1060254] Review Request: nodejs-deep-eql - Improved deep equality testing for Node.js and the browser

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060254

Piotr Popieluch  changed:

   What|Removed |Added

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



--- Comment #8 from Piotr Popieluch  ---
Package Change Request
==
Package Name: nodejs-deep-eql
New Branches: el6 epel7
Owners: piotrp tomh jamielinux

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

[Bug 1060245] Review Request: nodejs-simple-assert - Vanilla assertions for Node.js

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060245

Piotr Popieluch  changed:

   What|Removed |Added

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



--- Comment #9 from Piotr Popieluch  ---
Package Change Request
==
Package Name: nodejs-simple-assert
New Branches: el6 epel7
Owners: piotrp tomh jamielinux

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

[Bug 1060250] Review Request: nodejs-type-detect - Improved typeof detection for Node.js and the browser

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060250

Piotr Popieluch  changed:

   What|Removed |Added

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



--- Comment #8 from Piotr Popieluch  ---
Package Change Request
==
Package Name: nodejs-type-detect
New Branches: el6 epel7
Owners: piotrp tomh jamielinux

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

[Bug 1161834] Review Request: girl - GNOME Internet Radio Locator

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1161834

Michael Schwendt  changed:

   What|Removed |Added

 CC||bugs.mich...@gmx.net



--- Comment #10 from Michael Schwendt  ---
A brief look:

> Requires: gtk2 >= 2.24.24
> Requires: libgnome >= 2.32.1
> Requires: libxml2 >= 2.9.1
> Requires: gnome-vfs2 >= 2.24.4
> Requires: libgnomeui >= 2.24.5
> Requires: totem >= 3.10.1
> Requires: streamripper >= 1.64.6

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

That section is not specific to library packages.


> %install
> 
> %make_install
> make DESTDIR=%{buildroot} install

Double-check this %install section, please. Run "rpm -E %make_install" as a
hint.


> %lang(cs)
> %lang(pt_BR)

No-op and hence should be deleted.


> %{_datadir}/girl/girl-0.1.dtd

https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership
https://fedoraproject.org/wiki/Packaging:UnownedDirectories


%{_datadir}/appdata/girl.appdata.xml

https://fedoraproject.org/wiki/Packaging:Guidelines#AppData_files
https://fedoraproject.org/wiki/Packaging:AppData

Note the bottom about validating the file.


%{_datadir}/applications/girl.desktop

https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage


> %{_datadir}/icons/hicolor/16x16/apps/girl.png
> %{_datadir}/icons/hicolor/22x22/apps/girl.png
> %{_datadir}/icons/hicolor/24x24/apps/girl.png
> %{_datadir}/icons/hicolor/32x32/apps/girl.png
> %{_datadir}/icons/hicolor/48x48/apps/girl.png
> %{_datadir}/icons/hicolor/256x256/apps/girl.png

https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache


> %{_mandir}/man1/girl.1.gz

A growing number of reviewers suggest replacing the '.gz' with '*', because
that makes it possible to disable/change/customise the compression that is
applied by rpmbuild on-the-fly.

  %{_mandir}/man1/girl.1*

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

[Bug 1174976] Review Request: openstack-heat-gbp - Group Based Policy add-on for OpenStack Orchestration

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174976

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

[Bug 1174976] Review Request: openstack-heat-gbp - Group Based Policy add-on for OpenStack Orchestration

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174976



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

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303



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

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

[Bug 443249] Review Request: rubygem-hoe - Hoe is a simple rake/rubygems helper for project Rakefiles

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=443249

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 443249] Review Request: rubygem-hoe - Hoe is a simple rake/rubygems helper for project Rakefiles

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=443249



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

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

[Bug 214730] Review Request: pungi - Distribution compose tool

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=214730

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 214730] Review Request: pungi - Distribution compose tool

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=214730



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

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303

Christian Dersch  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #7 from Christian Dersch  ---
New Package SCM Request
===
Package Name: kde-plasma-redshift
Short Description: Plasmoid for redshift
Upstream URL: http://simgunz.org/projects/redshift-plasmoid
Owners: lupinix
Branches: f20 f21
InitialCC:

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303



--- Comment #6 from Christian Dersch  ---
Thank you for reviewing the package :) Will update the URL.

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

[Bug 1176658] Review Request: adwaita-qt - Adwaita Qt theme

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176658

Martin Bříza  changed:

   What|Removed |Added

Summary|Review Request:  -|



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

[Bug 1176658] New: Review Request: -

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176658

Bug ID: 1176658
   Summary: Review Request:  - 
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mbr...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://fpaste.org/162175/41927214/
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/mbriza/qt-gtk/fedora-21-x86_64/adwaita-qt-0-0.4.copr.fc21/adwaita-qt-0-0.4.copr.fc21.src.rpm
Description: Adwaita theme for Qt applications
Fedora Account System Username: mbriza

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

[Bug 1159660] Review Request: WALinuxAgent - The Windows Azure Linux Agent

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159660



--- Comment #7 from Rich Mattes  ---
Yeah that original spec was a big crufty mess...  The package is simple enough
that an all-new spec file is probably less work than sorting out the original. 
It looks pretty good now.

As far as your notes go:
- I think noreplace was the right way to go, otherwise local changes get blown
away on every package update.  The fedora-review warning was probably trying to
make sure you weren't trying to do the wrong thing.
- According to [1], it's OK to own dirs from other packages in the case where
the other package doesn't provide any core functionality for your package.  If
you don't need logrotate, you should own the directory, otherwise you need a
Requires: logrotate.
- I think systemd is the right call for epel7. It's the supported init system
now, so providing native service files over sysv init files is preferable.  The
only argument I can see for shipping the init script is el6->el7 upgrades, but
el7 has been out long enough that it's probably not a huge issue.

As for The Big Checklist goes, everything is now fixed except for:
- The logrotate.d directory ownership
- According to [2], you need to BR python2-devel or python3-devel.

[1]
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#The_directory_is_owned_by_a_package_which_is_not_required_for_your_package_to_function
[2] https://fedoraproject.org/wiki/Packaging:Python#BuildRequires

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303

Rex Dieter  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Rex Dieter  ---
naming: ok

sources: ok
32b142c966d496e50bebb599e374b1c0  v1.0.tar.gz

license: ok

scriptlets: ok

macros: ok


You may want to consider updating project URL, sources to point to
http://simgunz.org/projects/redshift-plasmoid


Otherwise, simple and clean, APPROVED.

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

[Bug 1162303] Review Request: kde-plasma-redshift - Plasmoid for redshift

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162303

Rex Dieter  changed:

   What|Removed |Added

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



--- Comment #4 from Rex Dieter  ---
I can review this today.

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

[Bug 1174976] Review Request: openstack-heat-gbp - Group Based Policy add-on for OpenStack Orchestration

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174976

Robert Kukura  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Robert Kukura  ---
New Package SCM Request
===
Package Name: openstack-heat-gbp
Short Description: Group Based Policy add-on for OpenStack Orchestration
Upstream URL: https://launchpad.net/group-based-policy-automation
Owners: rkukura
Branches: f20 f21
InitialCC:

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

[Bug 1174976] Review Request: openstack-heat-gbp - Group Based Policy add-on for OpenStack Orchestration

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174976

Haïkel Guémar  changed:

   What|Removed |Added

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



--- Comment #2 from Haïkel Guémar  ---
I herebry approve this package into Fedora Packages Collection since it
complies with our guidelines.
Please submit a SCM request

Package Review
==

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



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "Apache (v2.0)", "Unknown or generated", "*No copyright* Apache (v2.0)".
 3 files have unknown license. Detailed output of licensecheck in
 /home/haikel/1174976-openstack-heat-gbp/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[ ]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: 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.
[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]: SourceX tarball generation or download is documented.
 Note: Package contains tarball without URL, check comments
[ ]: Descr

[Bug 443249] Review Request: rubygem-hoe - Hoe is a simple rake/rubygems helper for project Rakefiles

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=443249

Troy Dawson  changed:

   What|Removed |Added

 CC||tdaw...@redhat.com
  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #7 from Troy Dawson  ---
Package Change Request
==
Package Name: rubygem-hoe
New Branches: epel7
Owners: tdawson

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

[Bug 214730] Review Request: pungi - Distribution compose tool

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=214730

Troy Dawson  changed:

   What|Removed |Added

 CC||tdaw...@redhat.com
  Flags||fedora-cvs?



--- Comment #10 from Troy Dawson  ---
Package Change Request
==
Package Name: pungi
New Branches: epel7
Owners: tdawson

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

[Bug 1176595] Review Request: hypre - High performance matrix preconditioners

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176595

Antonio Trande  changed:

   What|Removed |Added

 CC||anto.tra...@gmail.com



--- Comment #1 from Antonio Trande  ---
URLs broken.

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

[Bug 505154] Tracker: Review Requests for Science and Technology related packages

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=505154

Dave Love  changed:

   What|Removed |Added

 Depends On||1176595




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1176595
[Bug 1176595] Review Request: hypre - High performance matrix
preconditioners
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176595] Review Request: hypre - High performance matrix preconditioners

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176595

Dave Love  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR),
   ||505154 (FE-SCITECH)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=505154
[Bug 505154] Tracker: Review Requests for Science and Technology related
packages
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176595] New: Review Request: hypre - High performance matrix preconditioners

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176595

Bug ID: 1176595
   Summary: Review Request: hypre - High performance matrix
preconditioners
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: d.l...@liverpool.ac.uk
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://loveshack.fedorapeople.org/review/hypre.spec
SRPM URL: https://loveshack.fedorapeople.org/review/hypre-2.8.0b-14.el6.src.rpm
Description:
Hypre is a set of matrix preconditioning libraries to aid in the
solution of large systems of linear equations.
Fedora Account System Username: loveshack

This is required for future packages, like petsc.
Copr builds: http://copr.fedoraproject.org/coprs/loveshack/livhpc/build/63979/

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

[Bug 1132671] Review Request: rubygem-compass-import-once - Ruby Module to speed up Sass compilation

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1132671

Vít Ondruch  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Vít Ondruch  ---
* The spec file differs from SRPM
  - There is BR: rubygem(sass-globbing) enabled in SRPM while disabled in the
.spec you provided. The latter seems to be what you probably wanted.

* Test suite
  - It seems that 

sed -i '/sass-globbing/ s/^/#/' test/test_helper.rb
mv test/fixtures/with_globbing.scss{,.disable}

Should be enough to execute the test suite without sass-globbing.

* Patching using %patch0 macro
  - I'd suggest to apply the patch using "%patch0 -p1" in %prep section instead
of the %build section.

Otherwise, the package looks good => APPROVED

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

[Bug 505154] Tracker: Review Requests for Science and Technology related packages

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=505154

Dave Love  changed:

   What|Removed |Added

 Depends On||1176593




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1176593
[Bug 1176593] Review Request: autodock_vina - Docking of small molecules to
proteins
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176593] Review Request: autodock_vina - Docking of small molecules to proteins

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176593

Dave Love  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR),
   ||505154 (FE-SCITECH)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=505154
[Bug 505154] Tracker: Review Requests for Science and Technology related
packages
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176593] New: Review Request: autodock_vina - Docking of small molecules to proteins

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176593

Bug ID: 1176593
   Summary: Review Request: autodock_vina - Docking of small
molecules to proteins
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: d.l...@liverpool.ac.uk
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://loveshack.fedorapeople.org/review/autodock_vina.spec
SRPM URL:
https://loveshack.fedorapeople.org/review/autodock_vina-1.1.2-1.el6.src.rpm
Description:
AutoDock Vina is a program to support drug discovery, molecular
docking and virtual screening of compound libraries. It offers
multi-core capability, high performance and enhanced accuracy
and ease of use.

The same institute also developed autodock, which is widely used.

O. Trott, A. J. Olson, AutoDock Vina: improving the speed and accuracy
of docking with a new scoring function, efficient optimization and
multithreading, Journal of Computational Chemistry 31 (2010) 455-461

Fedora Account System Username: loveshack

Requested: https://fedoraproject.org/wiki/SIGs/SciTech/BioinformaticsWishList

Copr builds: http://copr.fedoraproject.org/coprs/loveshack/livhpc/build/20492/

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

[Bug 1171238] Review Request: nodejs-package-info - Get the information of a npm package

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171238
Bug 1171238 depends on bug 1171236, which changed state.

Bug 1171236 Summary: Review Request: nodejs-got - Simplified HTTP/HTTPS requests
https://bugzilla.redhat.com/show_bug.cgi?id=1171236

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |NEXTRELEASE



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

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236

Parag  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-12-22 08:08:07



--- Comment #10 from Parag  ---
This is now built for all requested branches.

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

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236



--- Comment #9 from Fedora Update System  ---
nodejs-got-2.2.0-1.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-got-2.2.0-1.el7

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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445



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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445

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

[Bug 530684] Review Request: nordugrid-arc - Advanced Resource Connector Grid Middleware

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=530684

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |



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

[Bug 530684] Review Request: nordugrid-arc - Advanced Resource Connector Grid Middleware

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=530684



--- Comment #24 from Jon Ciesla  ---
Complete.

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

[Bug 1150566] Review Request: wcm - Wal Commander

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150566

Sergey Kosarevsky  changed:

   What|Removed |Added

 CC||s...@linderdaum.com



--- Comment #5 from Sergey Kosarevsky  ---
FYI, 0.18.1 released today.

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

[Bug 1176277] Review Request: Shinobi - Meta build system for Ninja

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176277



--- Comment #7 from Ivan Romanov  ---
# disable case-sensitive search
Provides:   %{binary}

and

Shinobi.src:24: W: unversioned-explicit-provides %{binary}

Confuse me. Can it be fixed? Or maybe just remove Provides?

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

[Bug 1176277] Review Request: Shinobi - Meta build system for Ninja

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176277



--- Comment #6 from Ivan Romanov  ---
Package Review
==

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


Issues:
===
- All build dependencies are listed in BuildRequires, except for any that are
  listed in the exceptions section of Packaging Guidelines.
  Note: These BR are not needed: diffutils
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "Unknown or generated". 6 files have unknown license. Detailed output of
 licensecheck in /home/taurus/1176277-Shinobi/licensecheck.txt
[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.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 3 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[?]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: %check is present and all tests pass.
[-]: Packages should try to preserve timestamps of original installed files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: Dist tag is present (not strictly required in

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236



--- Comment #8 from Fedora Update System  ---
nodejs-got-2.2.0-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-got-2.2.0-1.fc20

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

[Bug 1171230] Review Request: nodejs-registry-url - Get the set npm registry URL

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171230



--- Comment #11 from Fedora Update System  ---
nodejs-registry-url-2.0.0-2.el6 has been submitted as an update for Fedora EPEL
6.
https://admin.fedoraproject.org/updates/nodejs-registry-url-2.0.0-2.el6

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

[Bug 1176277] Review Request: Shinobi - Meta build system for Ninja

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176277



--- Comment #5 from Ivan Romanov  ---
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Exceptions_2
BR diffutils SHOULD be removed.

https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Manpages
Package SHOULD have manpage. Furthemore it console tool. Maybe you should ask
upstream to convert references.md to manpage.

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

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236



--- Comment #7 from Fedora Update System  ---
nodejs-got-2.2.0-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-got-2.2.0-1.el6

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

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236

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

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236



--- Comment #6 from Fedora Update System  ---
nodejs-got-2.2.0-1.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/nodejs-got-2.2.0-1.fc21

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

[Bug 1020456] Review Request: vagrant - an automation tool used to manage development environments

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1020456

Arun S A G  changed:

   What|Removed |Added

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



--- Comment #20 from Arun S A G  ---
i will take a look at this before christmas. Possibly tonight or tomorrow!

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

[Bug 1132670] Review Request: rubygem-compass-core - The Compass core stylesheet library

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1132670

Vít Ondruch  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Vít Ondruch  ---
* Test suite
  - I'd suggest to execute the test suite using following command:

ruby -Ilib:test/units \
  -e 'Dir.glob "./test/units/*_test.rb", &method(:require)'

  - Please expand the test suite in %check section. This way you keep in RPM
just the content which was originally shipped by upstream. This also avoids
several rpmlint complains.
  - It seems that the timecop and true are needed just for integration
tests. Since there is circular dependency between true and compass, I'd
suggest to use boostrap macro [1] to disable the integration tests during
bootstrap.

Otherwise the package is legit => APPROVED


[1] https://fedoraproject.org/wiki/Packaging:Guidelines#Bootstrapping

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

[Bug 1172525] Review Request: zbackup - A versatile deduplicating backup tool

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172525



--- Comment #17 from Fedora Update System  ---
zbackup-1.3-4.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/zbackup-1.3-4.fc20

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

[Bug 1171230] Review Request: nodejs-registry-url - Get the set npm registry URL

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171230



--- Comment #10 from Fedora Update System  ---
nodejs-registry-url-2.0.0-2.el7 has been submitted as an update for Fedora EPEL
7.
https://admin.fedoraproject.org/updates/nodejs-registry-url-2.0.0-2.el7

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

[Bug 1171236] Review Request: nodejs-got - Simplified HTTP/HTTPS requests

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171236



--- Comment #5 from Parag  ---
I will build this with new version (disabled tests) but looks like enabling
tests require few more packages to be added to fedora which I will do in next
few days.

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

[Bug 1142508] Review Request: rubygem-true - Testing framework for Sass libraries

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1142508



--- Comment #3 from Vít Ondruch  ---
> * Test suite
>   - Since there seems to be circular dependency, I suggest you to follow the
> bootstrapping guidelines [1].
>   - When the test suite is shipped in separate source package, I prefer to
> unpack it in the %check section. It does not pollute the original
> installation then.

Actually, looking into compass-core, it might make more sense to boostrap
compass-core instead of rubygem-true.

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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445

Piotr Popieluch  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Piotr Popieluch  ---
Thank you for review

New Package SCM Request
===
Package Name: nodejs-grunt-angular-templates
Short Description: Grunt build task to concatenate & register your AngularJS
templates
Upstream URL: https://github.com/ericclemmons/grunt-angular-templates
Owners: piotrp
Branches: f20 f21 el6 epel7

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

[Bug 1172525] Review Request: zbackup - A versatile deduplicating backup tool

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172525



--- Comment #16 from Fedora Update System  ---
zbackup-1.3-4.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/zbackup-1.3-4.fc21

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

[Bug 1172525] Review Request: zbackup - A versatile deduplicating backup tool

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172525

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

[Bug 851680] Review Request: mingw-libidl - MinGW Windows IDL Parsing Library.

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851680

Erik van Pienbroek  changed:

   What|Removed |Added

  Flags||needinfo?(greg.hellings@gma
   ||il.com)



--- Comment #6 from Erik van Pienbroek  ---
Hi Greg,

I just noticed that you haven't imported this package in the Fedora git yet.
Could you please do this so we can close this ticket?

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

[Bug 1176445] Review Request: nodejs-grunt-angular-templates - Grunt build task to concatenate & register your AngularJS templates

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176445

Parag AN(पराग)  changed:

   What|Removed |Added

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



--- Comment #1 from Parag AN(पराग)  ---
Review:

+ Package built successful in mock (f22 x86_64)

+ rpmlint on generated rpms gave output
nodejs-grunt-angular-templates.noarch: W: spelling-error %description -l en_US
minifying -> magnifying, indemnifying, signifying
nodejs-grunt-angular-templates.noarch: W: spelling-error %description -l en_US
templateCache -> template Cache, template-cache, contemplate
nodejs-grunt-angular-templates.noarch: W: only-non-binary-in-usr-lib
nodejs-grunt-angular-templates.noarch: W: dangling-symlink
/usr/lib/node_modules/grunt-angular-templates/node_modules/html-minifier
/usr/lib/node_modules/html-minifier
nodejs-grunt-angular-templates.src: W: spelling-error %description -l en_US
minifying -> magnifying, indemnifying, signifying
nodejs-grunt-angular-templates.src: W: spelling-error %description -l en_US
templateCache -> template Cache, template-cache, contemplate
2 packages and 0 specfiles checked; 0 errors, 6 warnings.

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

+ License is "MIT" and its text is given in LICENSE file.

+ follows nodejs packaging guidelines.

Looks good.
APPROVED.

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

[Bug 1142508] Review Request: rubygem-true - Testing framework for Sass libraries

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1142508



--- Comment #2 from Vít Ondruch  ---
* The .spec file you are linking differs from the .spec from SRPM
  - But the difference is just subtle, so not a blocker.

* Update to the latest upstream version.
  - There is already available true 1.0.1. Please consider update to the latest
upstream version.

* Test suite
  - Could you please execute the gem's test suite?
  - Since there seems to be circular dependency, I suggest you to follow the
bootstrapping guidelines [1].
  - When the test suite is shipped in separate source package, I prefer to
unpack it in the %check section. It does not pollute the original
installation then.

* License file should be in main package
  - Please move the LICENSE.txt file into main package.

* Move the VERSION file into -doc sub-package
  - It does not look like the VERSION file is used for runtime. Please move it
into the -doc subpackage.


[1] https://fedoraproject.org/wiki/Packaging:Guidelines#Bootstrapping

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

[Bug 1082401] Review Request: nodejs-lazystream - Create streams lazily when they are read from or written to

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082401

Parag AN(पराग)  changed:

   What|Removed |Added

 CC||panem...@gmail.com



--- Comment #8 from Parag AN(पराग)  ---
No idea why this package has not been built even after having 2 maintainers.

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

[Bug 1082401] Review Request: nodejs-lazystream - Create streams lazily when they are read from or written to

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082401

Parag AN(पराग)  changed:

   What|Removed |Added

 CC||piotr1...@gmail.com



--- Comment #7 from Parag AN(पराग)  ---
*** Bug 1176452 has been marked as a duplicate of this bug. ***

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

[Bug 1176452] Review Request: nodejs-lazystream - Open Node Streams on demand

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176452

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||panem...@gmail.com
 Resolution|--- |DUPLICATE
Last Closed||2014-12-22 06:33:18



--- Comment #1 from Parag AN(पराग)  ---
It looks like this package is already approved in Fedora but never built on
koji.

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

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

[Bug 1142508] Review Request: rubygem-true - Testing framework for Sass libraries

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1142508

Vít Ondruch  changed:

   What|Removed |Added

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



--- Comment #1 from Vít Ondruch  ---
I'm taking this for a review.

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

[Bug 1162234] Request review: nss-securepass - NSS library for SecurePass.

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162234



--- Comment #4 from Giuseppe Paterno'  ---
Location for SPEC and SRPM have changed:

SPEC: http://pubs.gpaterno.com/external/nss_securepass.spec
SRPM: http://pubs.gpaterno.com/external/nss-securepass-0.2-1.el6.src.rpm

Thanks.

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

[Bug 1176307] Review Request: torrent-file-editor - Qt based GUI tool designed to create and edit .torrent files

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176307



--- Comment #7 from Ivan Romanov  ---
* Sun Dec 21 2014 Ivan Romanov  - 0.1.0-3
- corrected updating icon cache
- added updating MIME type database

Spec URL:
https://raw.githubusercontent.com/drizt/torrent-file-editor-package/4a02a5079cac69cbd32f2b9de743e6d5f3d71b46/torrent-file-editor.spec
SRPM URL:
https://kojipkgs.fedoraproject.org//work/tasks/5518/8455518/torrent-file-editor-0.1.0-3.fc22.src.rpm

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

[Bug 1172974] Review Request: nodejs-split - Split a text stream into a line stream

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172974



--- Comment #10 from Fedora Update System  ---
nodejs-split-0.3.2-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-split-0.3.2-1.el6

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

[Bug 1173204] Review Request: nodejs-as-number - Tiny function that returns the given number, or a default value

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1173204



--- Comment #8 from Fedora Update System  ---
nodejs-as-number-1.0.0-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-as-number-1.0.0-1.el6

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

[Bug 1172975] Review Request: nodejs-stream-reduce - Reduce stream data to a single value

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172975



--- Comment #8 from Fedora Update System  ---
nodejs-stream-reduce-1.0.3-1.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-stream-reduce-1.0.3-1.el6

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

[Bug 1171749] Review Request: nodejs-ascii-tree - A node module for generating a text tree in ASCII

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171749



--- Comment #8 from Fedora Update System  ---
nodejs-ascii-tree-0.1.1-1.el6 has been submitted as an update for Fedora EPEL
6.
https://admin.fedoraproject.org/updates/nodejs-ascii-tree-0.1.1-1.el6

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

[Bug 1171231] Review Request: nodejs-object-assign - ES6 Object.assign() ponyfill

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171231



--- Comment #9 from Fedora Update System  ---
nodejs-object-assign-2.0.0-2.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-object-assign-2.0.0-2.el6

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

[Bug 1172975] Review Request: nodejs-stream-reduce - Reduce stream data to a single value

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172975



--- Comment #7 from Fedora Update System  ---
nodejs-stream-reduce-1.0.3-1.el7 has been submitted as an update for Fedora
EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-stream-reduce-1.0.3-1.el7

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

[Bug 1173204] Review Request: nodejs-as-number - Tiny function that returns the given number, or a default value

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1173204



--- Comment #7 from Fedora Update System  ---
nodejs-as-number-1.0.0-1.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-as-number-1.0.0-1.el7

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

[Bug 1172973] Review Request: nodejs-from - Easy way to make a Readable Stream

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172973



--- Comment #9 from Fedora Update System  ---
nodejs-from-0.1.3-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-from-0.1.3-1.el6

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

[Bug 1171749] Review Request: nodejs-ascii-tree - A node module for generating a text tree in ASCII

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171749



--- Comment #7 from Fedora Update System  ---
nodejs-ascii-tree-0.1.1-1.el7 has been submitted as an update for Fedora EPEL
7.
https://admin.fedoraproject.org/updates/nodejs-ascii-tree-0.1.1-1.el7

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

[Bug 1172974] Review Request: nodejs-split - Split a text stream into a line stream

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172974



--- Comment #9 from Fedora Update System  ---
nodejs-split-0.3.2-1.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-split-0.3.2-1.el7

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

[Bug 1171231] Review Request: nodejs-object-assign - ES6 Object.assign() ponyfill

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171231



--- Comment #8 from Fedora Update System  ---
nodejs-object-assign-2.0.0-2.el7 has been submitted as an update for Fedora
EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-object-assign-2.0.0-2.el7

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

[Bug 1172974] Review Request: nodejs-split - Split a text stream into a line stream

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172974



--- Comment #8 from Fedora Update System  ---
nodejs-split-0.3.2-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-split-0.3.2-1.fc20

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

[Bug 1172973] Review Request: nodejs-from - Easy way to make a Readable Stream

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172973



--- Comment #8 from Fedora Update System  ---
nodejs-from-0.1.3-1.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-from-0.1.3-1.el7

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

[Bug 1173204] Review Request: nodejs-as-number - Tiny function that returns the given number, or a default value

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1173204



--- Comment #6 from Fedora Update System  ---
nodejs-as-number-1.0.0-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-as-number-1.0.0-1.fc20

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

[Bug 1171231] Review Request: nodejs-object-assign - ES6 Object.assign() ponyfill

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171231



--- Comment #7 from Fedora Update System  ---
nodejs-object-assign-2.0.0-2.fc20 has been submitted as an update for Fedora
20.
https://admin.fedoraproject.org/updates/nodejs-object-assign-2.0.0-2.fc20

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

[Bug 1171749] Review Request: nodejs-ascii-tree - A node module for generating a text tree in ASCII

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171749



--- Comment #6 from Fedora Update System  ---
nodejs-ascii-tree-0.1.1-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-ascii-tree-0.1.1-1.fc20

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

[Bug 1172975] Review Request: nodejs-stream-reduce - Reduce stream data to a single value

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172975



--- Comment #6 from Fedora Update System  ---
nodejs-stream-reduce-1.0.3-1.fc20 has been submitted as an update for Fedora
20.
https://admin.fedoraproject.org/updates/nodejs-stream-reduce-1.0.3-1.fc20

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

[Bug 1172973] Review Request: nodejs-from - Easy way to make a Readable Stream

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1172973



--- Comment #7 from Fedora Update System  ---
nodejs-from-0.1.3-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-from-0.1.3-1.fc20

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

[Bug 1171231] Review Request: nodejs-object-assign - ES6 Object.assign() ponyfill

2014-12-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1171231



--- Comment #6 from Fedora Update System  ---
nodejs-object-assign-2.0.0-2.fc21 has been submitted as an update for Fedora
21.
https://admin.fedoraproject.org/updates/nodejs-object-assign-2.0.0-2.fc21

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

  1   2   >