[Bug 1246761] Review Request: libqtelegram-ae - Qt Telegram API wrapper library

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246761

Helio Chissini de Castro  changed:

   What|Removed |Added

  Flags|needinfo?(he...@kde.org)|



--- Comment #12 from Helio Chissini de Castro  ---
Nope, none of the 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 1240122] Review Request: mysema-apt-maven-plugin - Maven APT plugin

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1240122



--- Comment #28 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/mysema-apt-maven-plugin

-- 
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 1283327] New: Review Request: python-sphinxcontrib-pecanwsme - Extension to Sphinx for documenting APIs built with Pecan and WSME

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283327

Bug ID: 1283327
   Summary: Review Request: python-sphinxcontrib-pecanwsme -
Extension to Sphinx for documenting APIs built with
Pecan and WSME
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jp...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jpena.fedorapeople.org/python-sphinxcontrib-pecanwsme/python-sphinxcontrib-pecanwsme.spec
SRPM URL:
https://jpena.fedorapeople.org/python-sphinxcontrib-pecanwsme/python-sphinxcontrib-pecanwsme-0.8.0-1.fc24.src.rpm
Description: Extension to Sphinx for documenting APIs built with Pecan and WSME
Fedora Account System Username: jpena

Koji scratch build available at
http://koji.fedoraproject.org/koji/taskinfo?taskID=11893963

Please note the Python 3 subpackage is currently disabled, due to lack of
upstream support (see
https://github.com/dreamhost/sphinxcontrib-pecanwsme/pull/14).

-- 
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 1281876] Review Request: enki - Extensible text editor for programmers

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281876



--- Comment #13 from Christian Dersch  ---
Package itself is fine now :) I'll approve when upstream clarified GPLv2 vs
GPLv3 stuff.


Package Review
==

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



= MUST items =

Generic:
[?]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[?]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "GPL (v3 or later)", "Unknown or generated". 343 files have
 unknown license. Detailed output of licensecheck in
 /home/review/1281876-enki/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[?]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: 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]: update-desktop-database is invoked in %post and %postun if package
 contains desktop file(s) with a MimeType: entry.
 Note: desktop file(s) with MimeType entry in enki
[x]: gtk-update-icon-cache is invoked in %postun and %posttrans if package
 contains icons.
 Note: icons in enki
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 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 %license.
[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 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]: Dist tag is present.
[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 does not use a name that already exists.
[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:
[x]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.

> Done by Raphael, https://github.com/hlamer/enki/issues/344

[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = 

[Bug 1277499] Review Request: php-apigen-theme-default - Default Theme for Apigen

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277499



--- Comment #11 from Fedora Update System  ---
php-apigen-theme-default-1.0.2-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update php-apigen-theme-default'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-8d50d9a410

-- 
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 1277397] Review Request: php-nette-di - Nette Dependency Injection Component

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277397
Bug 1277397 depends on bug 1277388, which changed state.

Bug 1277388 Summary: Review Request: php-nette-php-generator - Nette PHP 
Generator
https://bugzilla.redhat.com/show_bug.cgi?id=1277388

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---



-- 
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 1281532] Review Request: Pagila - Example database for PostgreSQL

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281532



--- Comment #19 from Fedora Update System  ---
pagila-0.10.1-3.fc23 has been pushed to the Fedora 23 testing repository. If
problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update pagila'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-bc9e521874

-- 
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 1277497] Review Request: php-apigen-theme-bootstrap - Twitter Bootstrap Theme for Apigen

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277497



--- Comment #12 from Fedora Update System  ---
php-apigen-theme-bootstrap-1.1.3-1.fc23 has been pushed to the Fedora 23
testing repository. If problems still persist, please make note of it in this
bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update php-apigen-theme-bootstrap'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-31340a213d

-- 
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 1270382] Review Request: nodejs-on-headers - Execute a listener when a response is about to write headers

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1270382



--- Comment #10 from Fedora Update System  ---
nodejs-on-headers-1.0.1-2.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update nodejs-on-headers'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-4654cbe245

-- 
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 1277384] Review Request: php-nette-finder - Nette Finder: Files Searching

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277384



--- Comment #12 from Fedora Update System  ---
php-nette-finder-2.3.1-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update php-nette-finder'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b5cc56ae7a

-- 
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 1277395] Review Request: php-nette-component-model - Nette Component Model

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277395



--- Comment #12 from Fedora Update System  ---
php-nette-component-model-2.2.4-2.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update php-nette-component-model'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b6513b801d

-- 
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 1277397] Review Request: php-nette-di - Nette Dependency Injection Component

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277397
Bug 1277397 depends on bug 1277388, which changed state.

Bug 1277388 Summary: Review Request: php-nette-php-generator - Nette PHP 
Generator
https://bugzilla.redhat.com/show_bug.cgi?id=1277388

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE



-- 
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 1277484] Review Request: php-nette - Nette Framework

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277484
Bug 1277484 depends on bug 1277388, which changed state.

Bug 1277388 Summary: Review Request: php-nette-php-generator - Nette PHP 
Generator
https://bugzilla.redhat.com/show_bug.cgi?id=1277388

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE



-- 
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 1277388] Review Request: php-nette-php-generator - Nette PHP Generator

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277388

Remi Collet  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE
Last Closed|2015-11-18 04:24:50 |2015-11-18 11:57:10



-- 
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 1277484] Review Request: php-nette - Nette Framework

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277484
Bug 1277484 depends on bug 1277388, which changed state.

Bug 1277388 Summary: Review Request: php-nette-php-generator - Nette PHP 
Generator
https://bugzilla.redhat.com/show_bug.cgi?id=1277388

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---



-- 
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 1277388] Review Request: php-nette-php-generator - Nette PHP Generator

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277388

Fedora Update System  changed:

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---



--- Comment #13 from Fedora Update System  ---
php-nette-php-generator-2.3.4-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update php-nette-php-generator'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-171d31d51d

-- 
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 1268372] Review Request: openstack-app-catalog-ui - openstack horizon plugin for the openstack app-catalog

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1268372



--- Comment #20 from Kevin Fox  ---
PING

-- 
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 1269609] Review Request: ari-backup - A wrapper around rdiff-backup

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1269609



--- Comment #34 from Fedora Update System  ---
ari-backup-1.0.10-3.fc23 has been pushed to the Fedora 23 testing repository.
If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update ari-backup'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-24ef0be480

-- 
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 791263] Review Request: mockito - A Java mocking framework

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=791263

Roman Mohr  changed:

   What|Removed |Added

 CC||ro...@fenkhuber.at
  Flags|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 1283296] New: Review Request: pam-u2f - PAM authentication over U2F

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283296

Bug ID: 1283296
   Summary: Review Request: pam-u2f - PAM authentication over U2F
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: spartacu...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://www.variantweb.net/pub/review/pam-u2f.spec
SRPM URL: https://www.variantweb.net/pub/review/pam-u2f-1.0.3-1.fc23.src.rpm
Description:
The PAM U2F module provides an easy way to integrate the Yubikey (or
other U2F-compliant authenticators) into your existing user
authentication infrastructure. PAM is used by GNU/Linux, Solaris and Mac
OS X for user authentication.
Fedora Account System Username: spartacus06

-- 
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 1281779] Review Request: perl-Filesys-Statvfs - Perl extension for statvfs() and fstatvfs()

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281779



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/perl-Filesys-Statvfs

-- 
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 1281722] Review Request: perl-Unix-Mknod - Perl extension for mknod, major, minor, and makedev

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281722



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/perl-Unix-Mknod

-- 
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 1283296] Review Request: pam-u2f - PAM authentication over U2F

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283296

Seth Jennings  changed:

   What|Removed |Added

 Attachment|0   |1
#1096146 is||
   obsolete||



--- Comment #2 from Seth Jennings  ---
Created attachment 1096147
  --> https://bugzilla.redhat.com/attachment.cgi?id=1096147=edit
fedora-review

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

[Bug 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914



--- Comment #8 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/perl-Lingua-Translit

-- 
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 1283296] Review Request: pam-u2f - PAM authentication over U2F

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283296



--- Comment #1 from Seth Jennings  ---
Created attachment 1096146
  --> https://bugzilla.redhat.com/attachment.cgi?id=1096146=edit
fedora-review

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

[Bug 791263] Review Request: mockito - A Java mocking framework

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=791263

Roman Mohr  changed:

   What|Removed |Added

 CC||rm...@redhat.com



--- Comment #32 from Roman Mohr  ---
Package Change Request
==
Package Name: mockito
New Branches: epel7
Owners: rfenkhuber
InitialCC: java-sig

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

[Bug 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282917



--- Comment #6 from Denis Fateyev  ---
- Added perl(File::Spec) and perl(File::Basename) to Requires
- Removed perl(Config) and perl(Fnctl) from BuildRequires

Spec path: http://www.fateyev.com/RPMS/Fedora22/perl-Test-mysqld.spec

Rawhide scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=11894029

-- 
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911



--- Comment #7 from Denis Fateyev  ---
On the contrary, I'm planning to maintain it in EPEL too. Thanks for the
detailed clarifications, anyway.

- Added perl(integer) to BR (thought that all concerned Perl versions had it so
omitted it initially);
- Dropped META.json.

EL6 doesn't support %license tag so I didn't used it. The Group tag is also
kept for more EPEL adaptivity.

Spec path: http://www.fateyev.com/RPMS/Fedora22/perl-Crypt-Salsa20.spec

Rawhide scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=11894191

-- 
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 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282917



--- Comment #5 from Upstream Release Monitoring 
 ---
dfateyev's scratch build of perl-Test-mysqld-0.17-3.fc20.denf.src.rpm for epel7
completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11894024

-- 
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911



--- Comment #6 from Upstream Release Monitoring 
 ---
dfateyev's scratch build of perl-Crypt-Salsa20-0.03-2.fc20.denf.src.rpm for
epel7 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11894175

-- 
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 1240122] Review Request: mysema-apt-maven-plugin - Maven APT plugin

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1240122



--- Comment #29 from Fedora Update System  ---
mysema-apt-maven-plugin-1.1.3-1.fc23 has been submitted as an update to Fedora
23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-9e34b0eaf6

-- 
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 1281739] Review Request: python-libpagure - A Python library for Pagure APIs

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281739

Adam Miller  changed:

   What|Removed |Added

  Flags||fedora-review+



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

[Bug 1240122] Review Request: mysema-apt-maven-plugin - Maven APT plugin

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1240122



--- Comment #30 from Raphael Groner  ---
My suggestion was more like the following but it's up to you. You may not want
to use a snapshot tarball if official version is propagated with Version and
Release tags.

> %global artifactId apt-maven-plugin
> %global srcversion %(echo %{version} |tr . _)
…
> URL:   https://github.com/querydsl/%{artifactId}
> Source0:   %{url}/archive/%{srcversion}.tar.gz#/%{name}-%{version}.tar.gz
> # Add license file fix https://github.com/querydsl/apt-maven-plugin/issues/37
> Patch0:%{url}/commit/7d890eb13a971300a2229e7c0912a1f4bd4d4df8.patch
…
> %prep
> %setup -q -n %{artifactId}-%{srcversion}
> %patch0 -p1
…

-- 
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 1281739] Review Request: python-libpagure - A Python library for Pagure APIs

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281739



--- Comment #13 from Adam Miller  ---
Package Review
==

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



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 7 files have unknown license. Detailed
 output of licensecheck in /home/admiller/reviews/1281739-python-
 libpagure/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/lib/python3.5/site-packages,
 /usr/lib/python3.5
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/lib/python3.5/site-
 packages, /usr/lib/python3.5
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: 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 %license.
[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]: Dist tag is present.
[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 does not use a name that already exists.
[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.
[ ]: 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).
[-]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python3-libpagure
[x]: 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.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps 

[Bug 1240122] Review Request: mysema-apt-maven-plugin - Maven APT plugin

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1240122

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|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 1283376] New: Review Request: nodejs-aws-sign2 - Node.js module for AWS signing

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283376

Bug ID: 1283376
   Summary: Review Request: nodejs-aws-sign2 - Node.js module for
AWS signing
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-aws-sign2-0.6.0-1.fc23.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-aws-sign2-0.6.0-1.fc23.noarch.rpm
Description: Node.js module for AWS signing
Fedora Account System Username: piotrp

-- 
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 1283376] Review Request: nodejs-aws-sign2 - Node.js module for AWS signing

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283376

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||994934




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=994934
[Bug 994934] nodejs-request-2.66.0 is available
-- 
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 1246904] Review Request: gnome-shell-extension-panel-osd - An extension to configure the place where notifications are shown

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246904



--- Comment #11 from Jens Lody  ---
Latest upstream, mainly ttranslation and compatiblity with gnome-shell 3.19.1 .
https://rpm.jenslody.de/review/gnome-shell-extension-panel-osd-1-0.3.20151118git61c1bd4.el7.centos.src.rpm

Successful copr-builds:
https://copr.fedoraproject.org/coprs/jenslody/gnome-shell-extensions/monitor/

-- 
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 1269649] Review Request: aeskulap - Full open source replacement for commercially available DICOM viewer

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1269649



--- Comment #4 from Jens Lody  ---
Actual copr-builds:
https://copr.fedoraproject.org/coprs/jenslody/Tests/monitor/

-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178



--- Comment #22 from Fedora Update System  ---
python-yaql-0.2.7-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-c22d266f42

-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

Marcos  changed:

   What|Removed |Added

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



--- Comment #23 from Marcos  ---
Hi Alan,

The new branch f23 for python-yaql was requested, the build was successful and
the package was pushed to bodhi for testing
https://bodhi.fedoraproject.org/updates/FEDORA-2015-c22d266f42.

Is there any other step that I need to accomplish in order to finish with this
first version of python-yaql?

Thank you.

-- 
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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914



--- Comment #5 from Upstream Release Monitoring 
 ---
dfateyev's scratch build of perl-Lingua-Translit-0.22-2.fc20.denf.src.rpm for
f23 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11891896

-- 
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 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282917



--- Comment #3 from Denis Fateyev  ---
Fixed Requires according RHBZ#1282956 along other small changes.

Spec path: http://www.fateyev.com/RPMS/Fedora22/perl-Test-mysqld.spec

Rawhide scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=11891926

-- 
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 1283190] New: Review Request: nodejs-is-my-json-valid - A JSONSchema validator that uses code generation to be extremely fast

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283190

Bug ID: 1283190
   Summary: Review Request: nodejs-is-my-json-valid -  A
JSONSchema validator that uses code generation to be
extremely fast
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-is-my-json-valid.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-is-my-json-valid-2.12.2-1.fc23.src.rpm
Description: A JSONSchema validator that uses code generation to be extremely
fast
Fedora Account System Username: piotrp

-- 
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 1282903] Review Request: golang-github-sallyom-Register - Golang binary for registering OCI containers with systemd-machined

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282903

Jan Chaloupka  changed:

   What|Removed |Added

  Flags||needinfo?(somalley@redhat.c
   ||om)



--- Comment #1 from Jan Chaloupka  ---
Hi Sally,

can you regenerate the spec file with the latest build for f23 [1]?. I have
built it today. It contains some updates of generated spec file, making the
spec file more clean. Karmas on it would be appreciated [2].

'gofed repo2spec --detect https://github.com/sallyom/Register --commit
79c22398b30da927f4fd58aa45b5155bd859d34e --with-build' will do the magic.

In general:
- it is always good to provide devel subpackage so other can import your source
codes. Even if the current devel subpackage would be empty, it is scanned by
other tools.
- any [Build]Required golang package should be a virtual provide, not physical
one, i.e. use golang(github.com/godbus/dbus) instead of
golang-github-godbus-dbus-devel. In future the name of the
golang-github-godbus-dbus package can change. Virtual provides does not change
unless the imported package is removed.

Can you provide LICENSE file for https://github.com/sallyom/Register? Without
that the package can not be packaged in Fedora.

[1] http://koji.fedoraproject.org/koji/buildinfo?buildID=700023
[2] https://bodhi.fedoraproject.org/updates/FEDORA-2015-bd2922d501

-- 
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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914



--- Comment #3 from Denis Fateyev  ---
Thanks for the review, Jitka.

> - make - spec file - line 38, 44, 52
> - coreutils - spec file - line 46, 47, 56
> - findutils - spec file - line 46, 47

Aren't they populated in build environment automatically? I believe I've seen
once a list of packages which aren't required in BR, but surprisingly I can't
find it right 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 1283167] Review Request: nodejs-jsonpointer - Simple JSON Addressing

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283167

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

Alan Pevec  changed:

   What|Removed |Added

  Flags|needinfo?(ape...@redhat.com |needinfo?(marcos.fermin.lob
   |)   |o...@cern.ch)
   |needinfo?(karlthered@gmail. |
   |com)|



--- Comment #25 from Alan Pevec  ---
Why push obsolete 0.2.7 version?
Re. 1.0.2 no need for re-review, just push it to distgit and build in Koji!

-- 
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 1283150] New: Review Request: nodejs-require-directory - Recursively iterates over specified directory, require()'ing each file

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283150

Bug ID: 1283150
   Summary: Review Request: nodejs-require-directory - Recursively
iterates over specified directory, require()'ing each
file
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-require-directory.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-require-directory-2.1.1-1.fc23.src.rpm
Description: Recursively iterates over specified directory, require()'ing each
file
Fedora Account System Username: piotrp

-- 
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911

Petr Šabata  changed:

   What|Removed |Added

  Flags|fedora_requires_release_not |fedora-review?
   |e?  |



--- Comment #2 from Petr Šabata  ---
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911



--- Comment #4 from Denis Fateyev  ---
Adjusted BRs fully as requested in a previous review, small cleanup.

Spec path: http://www.fateyev.com/RPMS/Fedora22/perl-Crypt-Salsa20.spec

Rawhide scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=11892279

-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

Alan Pevec  changed:

   What|Removed |Added

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



--- Comment #21 from Alan Pevec  ---
ok, so epel7 is no go, since it requires latest OpenStack deps which do not
plan to push there, we'll keep EL7 builds in RDO.

For f23, please request branch in pkgdb:
https://fedoraproject.org/wiki/PackageDB_admin_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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

Marcos  changed:

   What|Removed |Added

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



--- Comment #24 from Marcos  ---
Hi,

New version in the horizon: 1.0.2

Spec URL:
http://mferminl.web.cern.ch/mferminl/fedorapkg/python-yaql/1.0.2/python-yaql.spec
SRPM URL:
http://mferminl.web.cern.ch/mferminl/fedorapkg/python-yaql/1.0.2/python-yaql-1.0.2-1.fc23.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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914



--- Comment #4 from Jitka Plesnikova  ---
All dependencies should be listed due to the latest version of the packaging
guidelines. The list which you mention was removed.


For more info:
https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRequires_2
https://fedorahosted.org/fpc/ticket/497

-- 
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 1283150] Review Request: nodejs-require-directory - Recursively iterates over specified directory, require()'ing each file

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283150

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
-- 
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 1283166] New: Review Request: eclipse-xtext-antlr-generator - Xtext adapter to ANTLR's generator

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283166

Bug ID: 1283166
   Summary: Review Request: eclipse-xtext-antlr-generator - Xtext
adapter to ANTLR's generator
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mat.bo...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://fedorapeople.org/~mbooth/eclipse-xtext-antlr-generator.spec
SRPM URL:
https://fedorapeople.org/~mbooth/eclipse-xtext-antlr-generator-2.1.1-1.git18383dd.fc24.src.rpm
Description: Xtext adapter to ANTLR's generator.
Fedora Account System Username: mbooth

-- 
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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914



--- Comment #6 from Denis Fateyev  ---
Thanks, didn't know about these recent changes in guidelines. 
Fixed BuildRequires, improved source path, fixed another small details
mentioned.

Spec path: http://www.fateyev.com/RPMS/Fedora22/perl-Lingua-Translit.spec

Rawhide scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=11891891

-- 
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 1283167] New: Review Request: nodejs-jsonpointer - Simple JSON Addressing

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283167

Bug ID: 1283167
   Summary: Review Request: nodejs-jsonpointer - Simple JSON
Addressing
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-jsonpointer.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-jsonpointer-3.0.0-1.fc23.src.rpm
Description: Simple JSON Addressing
Fedora Account System Username: piotrp

-- 
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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #7 from Jitka Plesnikova  ---
It looks good now.

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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911



--- Comment #3 from Upstream Release Monitoring 
 ---
dfateyev's scratch build of perl-Crypt-Salsa20-0.03-2.fc20.denf.src.rpm for
rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11892278

-- 
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 1189269] Review Request: qparted - Visual partition editor based on Qt framework [a fork of qtparted]

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1189269
Bug 1189269 depends on bug 1188780, which changed state.

Bug 1188780 Summary: RFE: No menu icon
https://bugzilla.redhat.com/show_bug.cgi?id=1188780

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX



-- 
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 1204614] Review Request: gofed - Tool for development of golang devel packages

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1204614

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |MODIFIED



--- Comment #26 from Fedora Update System  ---
gofed-0.0.10-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-ae3431a7c2

-- 
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 1204614] Review Request: gofed - Tool for development of golang devel packages

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1204614



--- Comment #26 from Fedora Update System  ---
gofed-0.0.10-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-ae3431a7c2

-- 
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 1283190] Review Request: nodejs-is-my-json-valid - A JSONSchema validator that uses code generation to be extremely fast

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283190

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)
 Depends On||1283167




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1283167
[Bug 1283167] Review Request: nodejs-jsonpointer - Simple JSON Addressing
-- 
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 1283167] Review Request: nodejs-jsonpointer - Simple JSON Addressing

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283167

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||1283190




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1283190
[Bug 1283190] Review Request: nodejs-is-my-json-valid -  A JSONSchema
validator that uses code generation to be extremely fast
-- 
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911

Petr Pisar  changed:

   What|Removed |Added

 CC||ppi...@redhat.com



--- Comment #1 from Petr Pisar  ---
Petr, wrong flag (fedora_requires_release_note)?

-- 
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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914

Jitka Plesnikova  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|jples...@redhat.com



--- Comment #2 from Jitka Plesnikova  ---
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

$ rpm -qp --requires perl-Lingua-Translit-0.22-1.fc24.noarch.rpm | sort | uniq
-c
  1 /usr/bin/perl
  1 perl(:MODULE_COMPAT_5.22.0)
  1 perl(:VERSION) >= 5.8.0
  1 perl(Carp)
  1 perl(Encode)
  1 perl(Exporter)
  1 perl(Getopt::Long)
  1 perl(Lingua::Translit)
  1 perl(Lingua::Translit::Tables)
  1 perl(strict)
  1 perl(utf8)
  1 perl(warnings)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1

Binary requires are Ok.

$ rpm -qp --provides perl-Lingua-Translit-0.22-1.fc24.noarch.rpm | sort | uniq
-c
  1 perl(Lingua::Translit) = 0.22
  1 perl(Lingua::Translit::Tables) = 0.10
  1 perl-Lingua-Translit = 0.22-1.fc24

Binary provides are Ok.

$ rpmlint ./perl-Lingua-Translit*
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Rpmlint is ok

BuildRequires
FIX: Please add BRs:
- perl(Exporter) - Lingua-Translit-0.22/lib/Lingua/Translit/Tables.pm - line 74
- perl(utf8) - Lingua-Translit-0.22/lib/Lingua/Translit/Tables.pm - line 12
  and tests
- make - spec file - line 38, 44, 52
- coreutils - spec file - line 46, 47, 56
- findutils - spec file - line 46, 47

FIX: Replace variable PERL_INSTALL_ROOT obsoleted with DESTDIR name in
  %install.
TODO: Remove the deleting empty directories in %install section. This is
  default behavior.


Please correct all 'FIX' issues and consider fixing 'TODO' items and provide
new spec file.

-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

Alan Pevec  changed:

   What|Removed |Added

 CC||ape...@redhat.com
  Flags||needinfo?(marcos.fermin.lob
   ||o...@cern.ch)



--- Comment #19 from Alan Pevec  ---
No Bodhi updates are needed for f24/Rawhide
you need them to push to the f23, f22 or epel7 i.e. current releases.

BTW please update to the latest release!
1.0.0 was release on Aug 20 and 1.0.2 on Sep 16 so it should've been included
in the review already i.e. this was not correct!
[x]: Latest version is packaged.

-- 
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 1277397] Review Request: php-nette-di - Nette Dependency Injection Component

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277397
Bug 1277397 depends on bug 1277388, which changed state.

Bug 1277388 Summary: Review Request: php-nette-php-generator - Nette PHP 
Generator
https://bugzilla.redhat.com/show_bug.cgi?id=1277388

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE



-- 
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 1277484] Review Request: php-nette - Nette Framework

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277484
Bug 1277484 depends on bug 1277388, which changed state.

Bug 1277388 Summary: Review Request: php-nette-php-generator - Nette PHP 
Generator
https://bugzilla.redhat.com/show_bug.cgi?id=1277388

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE



-- 
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 1277388] Review Request: php-nette-php-generator - Nette PHP Generator

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277388

Remi Collet  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE
Last Closed|2015-11-16 11:52:23 |2015-11-18 04:24:50



-- 
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911

Petr Šabata  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.com
  Flags||fedora_requires_release_not
   ||e?



-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1257178

Marcos  changed:

   What|Removed |Added

  Flags|needinfo?(karlthered@gmail. |needinfo?(ape...@redhat.com
   |com)|)
   |needinfo?(marcos.fermin.lob |
   |o...@cern.ch)  |



--- Comment #20 from Marcos  ---
$ fedpkg switch-branch epel7
Switched to branch 'epel7'
$ git merge master
... OK
$ git push
Total 0 (delta 0), reused 0 (delta 0)
remote: Emitting a message to the fedmsg bus.
remote: * Publishing information for 1 commits
To ssh://mfl...@pkgs.fedoraproject.org/python-yaql
   fcb303f..2207ae7  epel7 -> epel7
$ fedpkg build
FAILED http://koji.fedoraproject.org/koji/taskinfo?taskID=11891084 

The build.log:

...
writing manifest file 'yaql.egg-info/SOURCES.txt'
+ sphinx-build doc/source html
Making output directory...
Running Sphinx v1.1.3
Extension error:
Could not import extension oslosphinx (exception: No module named oslosphinx)
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.4Yx7eC (%build)
Bad exit status from /var/tmp/rpm-tmp.4Yx7eC (%build)
Child return code was: 1
EXCEPTION: Command failed. See logs for output.


I tried:
$ fedpkg switch-branch f23
Could not execute switch_branch: Unknown remote branch origin/f23

And other fedora versions like f23 and I can not change the branch in order to
push to other branch

-- 
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 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282917

Jitka Plesnikova  changed:

   What|Removed |Added

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



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

[Bug 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914



--- Comment #1 from Upstream Release Monitoring 
 ---
jplesnik's scratch build of perl-Lingua-Translit-0.22-1.fc24.src.rpm for
rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11891179

-- 
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 1282914] Review Request: perl-Lingua-Translit - Transliterates text between writing systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282914

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jples...@redhat.com
  Flags||fedora-review?



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

[Bug 1283218] Review Request: nodejs-should-promised - Some sort of promise assertion helpers

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283218

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||1283228




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1283228
[Bug 1283228] Review Request nodejs-har-validator - Extremely fast HTTP
Archive (HAR) validator using JSON Schema
-- 
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 1239148] Review Request: thefuck - App that corrects your previous console command

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1239148

Matías Kreder  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-11-18 09:41:10



-- 
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 1246762] Review Request: telegramqml - Qt Qml qtelegram wrapper library

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246762

Rex Dieter  changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu
  Alias||telegramqml



-- 
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 1283228] New: Review Request nodejs-har-validator - Extremely fast HTTP Archive (HAR) validator using JSON Schema

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283228

Bug ID: 1283228
   Summary: Review Request nodejs-har-validator - Extremely fast
HTTP Archive (HAR) validator using JSON Schema
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-har-validator.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-har-validator-2.0.2-1.fc23.src.rpm
Description: Extremely fast HTTP Archive (HAR) validator using JSON Schema
Fedora Account System Username: piotrp

-- 
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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282911



--- Comment #5 from Petr Šabata  ---
I'll assume this is for Fedora only.

* Missing a BR, as already pointed out in the other current reviews:
  - perl(integer), used in lib/Crypt/Salsa20.pm (lots)

* strict and warnings are used in the Makefile.PL, too.
  You may want to list them in the top block, for consistence.
  Well, this depends on your style preference...

* Drop META.json from %doc, it's of no use to your end users.

* Package the license text using the %license macro.

* If you really don't need EPEL compatibility, you can:
  1) ignore the abovementioned coreutils & findutils point.
  2) Add a 6.76+ version dependency constraint to EU::MM.
  3) Generate the Makefile with `NO_PACKLIST=1'.
  This will allow you to drop line 46.

* The Group tag is optional, too.

-- 
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 1283190] Review Request: nodejs-is-my-json-valid - A JSONSchema validator that uses code generation to be extremely fast

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283190

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||1283228




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1283228
[Bug 1283228] Review Request nodejs-har-validator - Extremely fast HTTP
Archive (HAR) validator using JSON Schema
-- 
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 1283150] Review Request: nodejs-require-directory - Recursively iterates over specified directory, require()'ing each file

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283150

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||1283228




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1283228
[Bug 1283228] Review Request nodejs-har-validator - Extremely fast HTTP
Archive (HAR) validator using JSON Schema
-- 
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 1283228] Review Request nodejs-har-validator - Extremely fast HTTP Archive (HAR) validator using JSON Schema

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283228

Piotr Popieluch  changed:

   What|Removed |Added

 Depends On||1283150, 1283190




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1283150
[Bug 1283150] Review Request: nodejs-require-directory - Recursively
iterates over specified directory, require()'ing each file
https://bugzilla.redhat.com/show_bug.cgi?id=1283190
[Bug 1283190] Review Request: nodejs-is-my-json-valid -  A JSONSchema
validator that uses code generation to be extremely fast
-- 
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 1276114] Review Request: nodejs-generate-object-property - Generate safe JS code that can used to reference a object property

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1276114

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||1283190




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1283190
[Bug 1283190] Review Request: nodejs-is-my-json-valid -  A JSONSchema
validator that uses code generation to be extremely fast
-- 
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 1283190] Review Request: nodejs-is-my-json-valid - A JSONSchema validator that uses code generation to be extremely fast

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283190

Piotr Popieluch  changed:

   What|Removed |Added

 Depends On||1276114




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1276114
[Bug 1276114] Review Request: nodejs-generate-object-property - Generate
safe JS code that can used to reference a object property
-- 
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 1243550] Review Request: openstack-aodh - OpenStack Telemetry Alarming

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1243550

Haïkel Guémar  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2015-11-18 08:34:49



-- 
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 1283218] Review Request: nodejs-should-promised - Some sort of promise assertion helpers

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283218

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
-- 
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 1283218] New: Review Request: nodejs-should-promised - Some sort of promise assertion helpers

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283218

Bug ID: 1283218
   Summary: Review Request: nodejs-should-promised - Some sort of
promise assertion helpers
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-should-promised.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-should-promised-0.3.1-1.fc23.src.rpm
Description: Some sort of promise assertion helpers
Fedora Account System Username: piotrp

-- 
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 1246761] Review Request: libqtelegram-ae - Qt Telegram API wrapper library

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246761

Rex Dieter  changed:

   What|Removed |Added

  Alias||libqtelegram-ae



-- 
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 1246763] Review Request: cutegram - A Qt Qml based Telegram client

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246763

Rex Dieter  changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu
  Alias||cutegram



-- 
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 1283228] Review Request nodejs-har-validator - Extremely fast HTTP Archive (HAR) validator using JSON Schema

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1283228

Piotr Popieluch  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||994934
 Depends On||1283218




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=994934
[Bug 994934] nodejs-request-2.66.0 is available
https://bugzilla.redhat.com/show_bug.cgi?id=1283218
[Bug 1283218] Review Request: nodejs-should-promised - Some sort of promise
assertion helpers
-- 
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 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1282917



--- Comment #4 from Jitka Plesnikova  ---
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

$ rpm -qp --requires perl-Test-mysqld-0.17-1.fc24.noarch.rpm | sort | uniq -c
  1 mysql-server
  1 perl(:MODULE_COMPAT_5.22.0)
  1 perl(:VERSION) >= 5.8.0
  1 perl(Class::Accessor::Lite)
  1 perl(Cwd)
  1 perl(DBD::mysql)
  1 perl(DBI)
  1 perl(File::Copy::Recursive)
  1 perl(File::Temp)
  1 perl(POSIX)
  1 perl(Time::HiRes)
  1 perl(strict)
  1 perl(warnings)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1

FIX: Add run Requires
- perl(File::Spec) - Test-mysqld-0.17/lib/Test/mysqld.pm:183
- perl(File::Basename) - Test-mysqld-0.17/lib/Test/mysqld.pm:184

$ rpm -qp --provides perl-Test-mysqld-0.17-1.fc24.noarch.rpm | sort | uniq -c
  1 perl(Test::mysqld) = 0.17
  1 perl-Test-mysqld = 0.17-1.fc24

Binary provides are Ok.

$ rpmlint ./perl-Test-mysqld*
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
Rpmlint is ok

BuildRequires:
FIX: Please remove from BRs
- perl(Config) - Module::Install::Can not used for build
- perl(Fnctl) - Module::Install::Makefile not used for build

Please correct all 'FIX' issues and provide new spec file.

Otherwise the package looks good.

-- 
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 1212909] New package request: clufter - Tool for transforming/analyzing cluster configuration formats

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1212909

errata-xmlrpc  changed:

   What|Removed |Added

 Status|VERIFIED|RELEASE_PENDING



-- 
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 1240122] Review Request: mysema-apt-maven-plugin - Maven APT plugin

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1240122



--- Comment #31 from gil cattaneo  ---
(In reply to Raphael Groner from comment #30)
> My suggestion was more like the following but it's up to you. You may not
> want to use a snapshot tarball if official version is propagated with
> Version and Release tags.

No thanks, for me is useless

-- 
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 1269649] Review Request: aeskulap - Full open source replacement for commercially available DICOM viewer

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1269649



--- Comment #6 from Jens Lody  ---
(In reply to Antonio Trande from comment #5)
> # Note: upstream is dead, so patches can not be included in the original
> sources.
> # The source does not exist any longer on the old server, but can be
> downloaded via archive.org.
> # Is it allowed to do it this way ?
> 
> Why you don't upload your patched source code as new maintained project?

To be honest:
It just did not come in my mind, even if it would obviously be the simplest
solution.
This would obsolete all the patches, that clutter the spec-file now.

One question is, what to do with the naming in such a case, where upstream is
dead ?

-- 
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 1208911] Review Request: doublecmd - Twin-panel (commander-style) file manager

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1208911



--- Comment #24 from Upstream Release Monitoring 
 ---
raphgro's scratch build of doublecmd-0.6.6-2.fc23.src.rpm for rawhide completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11895688

-- 
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 1277497] Review Request: php-apigen-theme-bootstrap - Twitter Bootstrap Theme for Apigen

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1277497



--- Comment #14 from Fedora Update System  ---
php-apigen-theme-bootstrap-1.1.3-1.fc22 has been pushed to the Fedora 22
testing repository. If problems still persist, please make note of it in this
bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update php-apigen-theme-bootstrap'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b69f61c452

-- 
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 1281532] Review Request: Pagila - Example database for PostgreSQL

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1281532



--- Comment #21 from Fedora Update System  ---
pagila-0.10.1-3.fc22 has been pushed to the Fedora 22 testing repository. If
problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update pagila'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-db5080529c

-- 
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