[Bug 1268910] Review Request: h2o - HTTP server for HTTP/1.x and HTTP/2

[bugzilla]

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



--- Comment #5 from Michael Schwendt (Fedora Packager Sponsors Group) 
 ---
> Summary:  An optimized HTTP server with support for HTTP/1.x and 
> HTTP/2

> %description
> H2O is a very fast HTTP server written in C.

Why is %summary longer and more detailed than %description?

Typically, the latter expands on the former:

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

Suggestion:

  Summary: HTTP server

  %description
  H2O is a very fast HTTP server written in C. It supports HTTP/1.x and HTTP/2.


> %post
> /sbin/ldconfig
> %systemd_post %{name}.service

> %postun
> /sbin/ldconfig

Running ldconfig here doesn't achieve anything related to this package, because
this package doesn't store any shared lib in runtime linker's search path.


> %doc %{_datarootdir}/doc/%{name}

%_datarootdir is %_datadir, which is /usr/share, and files below /usr/share/doc
are implicitly marked as %doc. See "rpm -E %__docdir_path".

> %doc %{_mandir}/man1/%{name}.1.gz

Same for %_mandir.


> %config(noreplace) %{_sysconfdir}/%{name}/%{name}.conf
> 
> %{_libexecdir}/%{name}/setuidgid

The directories %_sysconfdir/%name and %_libexecdir/%{name} are not included
yet.

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


Consider pointing the fedora-review tool at this ticket:
fedora-review -b 1268910

-- 
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 1265636] Review Request: google-roboto-mono-fonts - Google Roboto Mono fonts

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
google-roboto-mono-fonts-2.000986-0.1.20150923git.fc23 has been pushed to the
Fedora 23 stable repository. If problems still persist, please make note of it
in this bug report.

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

[Bug 1262468] Review Request: php-punic - PHP-Unicode CLDR

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:15:01



-- 
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 1265484] Review Request: google-roboto-slab-fonts - Google Roboto Slab fonts

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:15:13



-- 
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 1262468] Review Request: php-punic - PHP-Unicode CLDR

[bugzilla]

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



--- Comment #15 from Fedora Update System  ---
php-punic-1.6.3-1.fc23 has been pushed to the Fedora 23 stable repository. If
problems still persist, please make note of it in this bug report.

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

[Bug 1250887] Review Request: preprocess - A portable multi-language file Python2 preprocessor

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:14:49



-- 
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 1265484] Review Request: google-roboto-slab-fonts - Google Roboto Slab fonts

[bugzilla]

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



--- Comment #18 from Fedora Update System  ---
google-roboto-slab-fonts-1.100263-0.3.20150923git.fc23 has been pushed to the
Fedora 23 stable repository. If problems still persist, please make note of it
in this bug report.

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

[Bug 1262469] Review Request: php-patchwork-utf8 - Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:14:56



-- 
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 1250887] Review Request: preprocess - A portable multi-language file Python2 preprocessor

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
preprocess-1.2.1-3.20150914gitb23422.fc23 has been pushed to the Fedora 23
stable repository. If problems still persist, please make note of it in this
bug report.

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

[Bug 1262469] Review Request: php-patchwork-utf8 - Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP

[bugzilla]

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



--- Comment #16 from Fedora Update System  ---
php-patchwork-utf8-1.2.3-3.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1265326] Review Request: nodejs-error-ex - Easy error subclassing and stack customization

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2015-09-27 02:43:43 |2015-10-05 14:14:42



-- 
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 1268910] Review Request: h2o - HTTP server for HTTP/1.x and HTTP/2

[bugzilla]

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



--- Comment #6 from marha...@gmail.com ---
Thanks for the comments!

I have updated the spec file as per your suggestions. I couldn't quite get the
fedora-review tool to work, but I'll give it another try with the updated spec
file path from this comment (forgot to use raw in the last one).

Spec URL: https://raw.githubusercontent.com/marhag87/h2o-rpm/master/h2o.spec
SRPM URL:
https://raw.githubusercontent.com/marhag87/h2o-rpm-src/master/h2o-1.5.0-4.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 1261134] Review Request: python-django-formtools - A set of high-level abstractions for Django forms

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:15:57



-- 
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 1260038] Review Request: python-XStatic-mdi - mdi (XStatic packaging standard)

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
python-XStatic-mdi-1.1.70.1-2.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1261134] Review Request: python-django-formtools - A set of high-level abstractions for Django forms

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
python-django-formtools-1.0-4.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1265636] Review Request: google-roboto-mono-fonts - Google Roboto Mono fonts

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:15:07



-- 
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 1262872] Review Request: perl-TeX-Encode - Encoding to LaTeX escapes

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:15:39



-- 
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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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

Bug 1265322 Summary: Review Request: nodejs-encoding - Convert encodings, uses 
iconv by default and fallbacks to iconv-lite if needed
https://bugzilla.redhat.com/show_bug.cgi?id=1265322

   What|Removed |Added

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



-- 
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 1265329] Review Request: nodejs-parse-json - Parse JSON with more helpful errors

[bugzilla]

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

Bug 1265326 Summary: Review Request: nodejs-error-ex - Easy error subclassing 
and stack customization
https://bugzilla.redhat.com/show_bug.cgi?id=1265326

   What|Removed |Added

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



-- 
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 1265326] Review Request: nodejs-error-ex - Easy error subclassing and stack customization

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
nodejs-error-ex-1.2.0-1.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1265322] Review Request: nodejs-encoding - Convert encodings, uses iconv by default and fallbacks to iconv-lite if needed

[bugzilla]

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



--- Comment #11 from Fedora Update System  ---
nodejs-encoding-0.1.11-2.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1265322] Review Request: nodejs-encoding - Convert encodings, uses iconv by default and fallbacks to iconv-lite if needed

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-05 14:14:04



-- 
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 1262872] Review Request: perl-TeX-Encode - Encoding to LaTeX escapes

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
perl-TeX-Encode-1.3-2.fc23 has been pushed to the Fedora 23 stable repository.
If problems still persist, please make note of it in this bug report.

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

[Bug 1260038] Review Request: python-XStatic-mdi - mdi (XStatic packaging standard)

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |ERRATA



-- 
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 1264179] Review Request: php-mediawiki-at-ease - Safe replacement to @ for suppressing warnings

[bugzilla]

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



--- Comment #15 from Fedora Update System  ---
php-mediawiki-at-ease-1.1.0-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-mediawiki-at-ease'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-f21ad62720

-- 
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 1231353] Review Request: php-cssjanus - Convert CSS stylesheets between left-to-right and right-to-left

[bugzilla]

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



--- Comment #15 from Fedora Update System  ---
php-cssjanus-1.1.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-cssjanus'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-37c1674e25

-- 
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 1268910] Review Request: h2o - HTTP server for HTTP/1.x and HTTP/2

[bugzilla]

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



--- Comment #7 from Upstream Release Monitoring 
 ---
marhag87's scratch build of h2o-1.5.0-4.fc22.src.rpm for f22 completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11339443

-- 
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 1197505] Review Request: libnfs - Client library for accessing NFS shares over a network

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
libnfs-1.9.8-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b520e9cc25

-- 
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 1197505] Review Request: libnfs - Client library for accessing NFS shares over a network

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
libnfs-1.9.8-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-d53d1a3547

-- 
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 1268742] New: Review Request: rubygem-bacon-colored_output - Colored output for Bacon test framework

[bugzilla]

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

Bug ID: 1268742
   Summary: Review Request: rubygem-bacon-colored_output - Colored
output for Bacon test framework
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-bacon-colored_output.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-bacon-colored_output-1.0.1-1.fc24.src.rpm
Description: Colored output for Bacon test framework.
Fedora Account System Username: ilgrad

-- 
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 1268744] New: Review Request: rubygem-ast - A library for working with Abstract Syntax Trees

[bugzilla]

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

Bug ID: 1268744
   Summary: Review Request: rubygem-ast - A library for working
with Abstract Syntax Trees
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-ast.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-ast-2.1.0-1.fc24.src.rpm
Description: A library for working with Abstract Syntax Trees.
Fedora Account System Username: ilgrad

-- 
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 1268742] Review Request: rubygem-bacon-colored_output - Colored output for Bacon test framework

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1268744




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268744
[Bug 1268744] Review Request: rubygem-ast - A library for working with
Abstract Syntax Trees
-- 
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 1268744] Review Request: rubygem-ast - A library for working with Abstract Syntax Trees

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Depends On||1268742




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268742
[Bug 1268742] Review Request: rubygem-bacon-colored_output - Colored output
for Bacon test framework
-- 
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 1268745] New: Review Request: rubygem-parser - A Ruby parser written in pure Ruby

[bugzilla]

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

Bug ID: 1268745
   Summary: Review Request: rubygem-parser - A Ruby parser written
in pure Ruby
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-parser.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-parser-2.2.2.6-1.fc24.src.rpm
Description: A Ruby parser written in pure Ruby.
Fedora Account System Username: ilgrad

-- 
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 1268745] Review Request: rubygem-parser - A Ruby parser written in pure Ruby

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Depends On||1268744




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268744
[Bug 1268744] Review Request: rubygem-ast - A library for working with
Abstract Syntax Trees
-- 
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 1268744] Review Request: rubygem-ast - A library for working with Abstract Syntax Trees

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1268745




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268745
[Bug 1268745] Review Request: rubygem-parser - A Ruby parser written in
pure Ruby
-- 
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 1197505] Review Request: libnfs - Client library for accessing NFS shares over a network

[bugzilla]

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

Ondrej Holy  changed:

   What|Removed |Added

 Blocks||1268670




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268670
[Bug 1268670] NFS support for gvfs is not available in Fedora
-- 
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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@redhat.com
   Assignee|nob...@fedoraproject.org|mru...@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 1268753] New: Review Request: rubygem-astrolabe - An object-oriented AST extension for Parser

[bugzilla]

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

Bug ID: 1268753
   Summary: Review Request: rubygem-astrolabe - An object-oriented
AST extension for Parser
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-astrolabe.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-astrolabe-1.3.1-1.fc24.src.rpm
Description: An object-oriented AST extension for Parser.
Fedora Account System Username: ilgrad

-- 
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 1268745] Review Request: rubygem-parser - A Ruby parser written in pure Ruby

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1268753




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268753
[Bug 1268753] Review Request: rubygem-astrolabe - An object-oriented AST
extension for Parser
-- 
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 1268753] Review Request: rubygem-astrolabe - An object-oriented AST extension for Parser

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Depends On||1268745




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268745
[Bug 1268745] Review Request: rubygem-parser - A Ruby parser written in
pure Ruby
-- 
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 1256058] Review Request: ghc-fdo-notify - Desktop Notifications client

[bugzilla]

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

Jens Petersen  changed:

   What|Removed |Added

 Blocks||1176888




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1176888
[Bug 1176888] git-annex-5.20150930 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 1268758] New: Review Request: rubygem-rubocop - Automatic Ruby code style checking tool

[bugzilla]

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

Bug ID: 1268758
   Summary: Review Request: rubygem-rubocop - Automatic Ruby code
style checking tool
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-rubocop.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-rubocop-0.34.2-1.fc24.src.rpm
Description: Automatic Ruby code style checking tool. Aims to enforce the
community-driven Ruby Style Guide.
Fedora Account System Username: ilgrad

-- 
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 1266186] Review Request: jeromq - Pure Java implementation of libzmq

[bugzilla]

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



--- Comment #4 from Mikolaj Izdebski  ---
Updated package:
- skipped running tests
- added comment clarify licensing (I kept license as LGPLv3 only)

Spec URL: http://mizdebsk.fedorapeople.org/review/jeromq/jeromq.spec
SRPM URL:
http://mizdebsk.fedorapeople.org/review/jeromq/jeromq-0.3.5-2.fc24.src.rpm
Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=11330087

-- 
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 1266186] Review Request: jeromq - Pure Java implementation of libzmq

[bugzilla]

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



--- Comment #5 from Upstream Release Monitoring 
 ---
mizdebsk's scratch build of jeromq-0.3.5-2.fc24.src.rpm for f24 completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11330087

-- 
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 1268699] Review Request: rubygem-rb-fsevent - FSEvents API with Signals catching (without RubyCocoa)

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1268765




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268765
[Bug 1268765] Review Request: rubygem-grape-entity - A simple facade for
managing the relationship between your model and API
-- 
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 1268765] Review Request: rubygem-grape-entity - A simple facade for managing the relationship between your model and API

[bugzilla]

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



--- Comment #1 from Ilya Gradina  ---
Depends: rubygem(rubocop) = 0.31.0
https://copr.fedoraproject.org/coprs/ilgrad/gitlab/build/123973/

-- 
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 1268771] New: Review Request: rubygem-grape - A simple Ruby framework for building REST-like APIs

[bugzilla]

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

Bug ID: 1268771
   Summary: Review Request: rubygem-grape - A simple Ruby
framework for building REST-like APIs
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-grape.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-grape-0.13.0-1.fc24.src.rpm
Description: A Ruby framework for rapid API development with great conventions.
Fedora Account System Username: ilgrad

-- 
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 1264661] Review Request: rubygem-generator_spec - Test Rails generators with RSpec

[bugzilla]

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



--- Comment #11 from Ilya Gradina  ---
Hi Vit,

I was refused.
https://admin.fedoraproject.org/pkgdb/packager/ilgrad/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 1268765] New: Review Request: rubygem-grape-entity - A simple facade for managing the relationship between your model and API

[bugzilla]

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

Bug ID: 1268765
   Summary: Review Request: rubygem-grape-entity - A simple facade
for managing the relationship between your model and
API
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-grape-entity.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-grape-entity-0.4.8-1.fc24.src.rpm
Description: Extracted from Grape, A Ruby framework for rapid API development
with great conventions.
Fedora Account System Username: ilgrad

-- 
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 1268765] Review Request: rubygem-grape-entity - A simple facade for managing the relationship between your model and API

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Depends On||1268699, 1268701, 1268758




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268699
[Bug 1268699] Review Request: rubygem-rb-fsevent -   FSEvents API with
Signals catching (without RubyCocoa)
https://bugzilla.redhat.com/show_bug.cgi?id=1268701
[Bug 1268701] Review Request: rubygem-growl - growlnotify bindings
https://bugzilla.redhat.com/show_bug.cgi?id=1268758
[Bug 1268758] Review Request: rubygem-rubocop - Automatic Ruby code style
checking tool
-- 
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 1268701] Review Request: rubygem-growl - growlnotify bindings

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1268765




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268765
[Bug 1268765] Review Request: rubygem-grape-entity - A simple facade for
managing the relationship between your model and API
-- 
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 1268758] Review Request: rubygem-rubocop - Automatic Ruby code style checking tool

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1268765




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268765
[Bug 1268765] Review Request: rubygem-grape-entity - A simple facade for
managing the relationship between your model and API
-- 
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 1264661] Review Request: rubygem-generator_spec - Test Rails generators with RSpec

[bugzilla]

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



--- Comment #10 from Vít Ondruch  ---
Ilya, this package was approved, you can continue with scm 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 1249743] Review Request: kaadbg - Remote debugger runner for kaa

[bugzilla]

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

René Ribaud  changed:

   What|Removed |Added

 CC||rene.rib...@gmail.com
   Assignee|nob...@fedoraproject.org|rene.rib...@gmail.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 1249786] Review Request: kaaedit - Console text editor

[bugzilla]

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



--- Comment #1 from René Ribaud  ---
Hello William,

First short review.
1- Forget what I said in my previous mail about naming. I did not notice it was
an application and not a module.
2- Currently the package can not be built due to missing dependencies.
2a- BuildRequires:  pyjf3 --> should be rename to python3-pyjf3
2b- BuildRequires:  kaadbg --> this one is missing in the rawhide repo.
I found your bug request (1249743) about it and I have assigned it to me.

So I will review 1249743 first and then this one.

-- 
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 1262645] Review Request: python-tortilla - A tiny library for creating wrappers around web APIs

[bugzilla]

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



--- Comment #3 from René Ribaud  ---
Colorclass and formats available in the repos.
Tortilla should build and be ready for the review 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 1231353] Review Request: php-cssjanus - Convert CSS stylesheets between left-to-right and right-to-left

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #14 from Fedora Update System  ---
php-cssjanus-1.1.1-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-cssjanus'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-d6f07abe80

-- 
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 1264179] Review Request: php-mediawiki-at-ease - Safe replacement to @ for suppressing warnings

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #14 from Fedora Update System  ---
php-mediawiki-at-ease-1.1.0-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-mediawiki-at-ease'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-af46e52467

-- 
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 1262470] Review Request: python-PyPDF2 - A pure Python library built as a PDF toolkit

[bugzilla]

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

Major Hayden  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #11 from Major Hayden  ---
New Package SCM Request
===
Package Name: python-PyPDF2
Short Description: Python PDF toolkit and library
Upstream URL: https://github.com/mstamy2/PyPDF2
Owners: mhayden
Branches: f21 f22 f23 el6 epel7
InitialCC:

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

[Bug 1268010] Review Request: libmooshika - helper library for RDMA

[bugzilla]

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

Orion Poplawski  changed:

   What|Removed |Added

 CC||or...@cora.nwra.com



--- Comment #4 from Orion Poplawski  ---
(In reply to Dave Love from comment #3)
> (In reply to Dominique Martinet from comment #2)
> > I think I do need to keep the devel package requirements on other -devel
> > packages, though, right?
> > As far as I can tell (rpm -qpi --requires ) it could not guess
> > these, and I couldn't find anything in the guidelines about this.
> 
> Yes, I looked without success, and it's quite inconsistent in packages
> generally.  I'd have thought the dependencies should be added, but
> it's difficult to verify them (and I don't know why rpmbuild doesn't
> try to follow #includes).

Yes, -devel packages should require the other devel packages needed to build
with it.  It's a commonly missed aspect.  I suspect it's just a bit too messy
to try to generate the proper provides/requires automatically.

-- 
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 1240116] Review Request: python-music21 - A toolkit for computational musicology

[bugzilla]

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



--- Comment #10 from Zbigniew Jędrzejewski-Szmek  ---
Pff, a lot of time passed.

Latest upstream version, with new style python2- and python3- subpackages:

Spec URL: http://in.waw.pl/~zbyszek/fedora/python-music21.spec
SRPM URL: http://in.waw.pl/~zbyszek/fedora/python-music21-2.1.2-1.fc24.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 1262469] Review Request: php-patchwork-utf8 - Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP

[bugzilla]

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



--- Comment #17 from Fedora Update System  ---
php-patchwork-utf8-1.2.3-3.fc22 has been pushed to the Fedora 22 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1261134] Review Request: python-django-formtools - A set of high-level abstractions for Django forms

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
python-django-formtools-1.0-4.fc22 has been pushed to the Fedora 22 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1265636] Review Request: google-roboto-mono-fonts - Google Roboto Mono fonts

[bugzilla]

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



--- Comment #15 from Fedora Update System  ---
google-roboto-mono-fonts-2.000986-0.1.20150923git.fc22 has been pushed to the
Fedora 22 stable repository. If problems still persist, please make note of it
in this bug report.

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

[Bug 1262468] Review Request: php-punic - PHP-Unicode CLDR

[bugzilla]

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



--- Comment #16 from Fedora Update System  ---
php-punic-1.6.3-1.fc22 has been pushed to the Fedora 22 stable repository. If
problems still persist, please make note of it in this bug report.

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

[Bug 1265484] Review Request: google-roboto-slab-fonts - Google Roboto Slab fonts

[bugzilla]

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



--- Comment #19 from Fedora Update System  ---
google-roboto-slab-fonts-1.100263-0.3.20150923git.fc22 has been pushed to the
Fedora 22 stable repository. If problems still persist, please make note of it
in this bug report.

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

[Bug 1269001] New: Review Request: tinycbor - Concise Binary Object Representation (CBOR) Library

[bugzilla]

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

Bug ID: 1269001
   Summary: Review Request: tinycbor - Concise Binary Object
Representation (CBOR) Library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://jsmith.fedorapeople.org/Packaging/tinycbor/tinycbor.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/tinycbor/tinycbor-0.2-1.fc23.src.rpm
Description: Concise Binary Object Representation (CBOR) Library
Fedora Account System Username: jsmith

-- 
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 1266200] Review Request: python-dulwich99 - A python implementation of the Git file formats and protocols--compat

[bugzilla]

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



--- Comment #7 from Upstream Release Monitoring 
 ---
jdulaney's scratch build of python-dulwich99-0.9.9-3.fc23.src.rpm for rawhide
completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11340929

-- 
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 1262470] Review Request: python-PyPDF2 - A pure Python library built as a PDF toolkit

[bugzilla]

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

Pete Travis  changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #10 from Pete Travis  ---
Package Review
==

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



= 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: "BSD (3 clause)", "Unknown or generated". 11 files have unknown
 license. Detailed output of licensecheck in /home/pete/fedpkg/1262470
 -python-PyPDF2/licensecheck.txt
[+]: License file installed when any subpackage combination is installed.
[+]: Package contains no bundled libraries without FPC exception.
[+]: Changelog in prescribed format.
[+]: Sources contain only permissible code or content.
[-]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[+]: Package 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.
[+]: Package is not known to require an ExcludeArch tag.
[+]: 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]: 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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[+]: Final provides and requires are sane (see attachments).
[-]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python3-PyPDF2 , python-PyPDF2-doc
[+]: Package functions as described.
[+]: Latest version is packaged.
[+]: 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, 

[Bug 1250887] Review Request: preprocess - A portable multi-language file Python2 preprocessor

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
preprocess-1.2.1-3.20150914gitb23422.fc22 has been pushed to the Fedora 22
stable repository. If problems still persist, please make note of it in this
bug report.

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

[Bug 1262872] Review Request: perl-TeX-Encode - Encoding to LaTeX escapes

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
perl-TeX-Encode-1.3-2.fc22 has been pushed to the Fedora 22 stable repository.
If problems still persist, please make note of it in this bug report.

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

[Bug 1197505] Review Request: libnfs - Client library for accessing NFS shares over a network

[bugzilla]

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



--- Comment #14 from Ross Lagerwall  ---
Yes, I've submitted updates for F22 and F23.

-- 
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 1234210] Review Request: pdf-stapler - tool for manipulating PDF documents from the command line

[bugzilla]

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



--- Comment #21 from Globe Trotter  ---
Just to clarify, new files have been uploaded here:

SRPM: http://maitra.public.iastate.edu/Fedora/pdf-stapler-0.3.3-1.fc22.src.rpm
SPEC: http://maitra.public.iastate.edu/Fedora/pdf-stapler.spec

Both use python-PyPDF2.

-- 
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 1268010] Review Request: libmooshika - helper library for RDMA

[bugzilla]

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



--- Comment #3 from Dave Love  ---
(In reply to Dominique Martinet from comment #2)

> Hmm I'll admit I didn't even try to build on EPEL5, will remove it.

Group does no harm, but some reviewers object to it being there.

> Right, will fix.
> I think I do need to keep the devel package requirements on other -devel
> packages, though, right?
> As far as I can tell (rpm -qpi --requires ) it could not guess
> these, and I couldn't find anything in the guidelines about this.

Yes, I looked without success, and it's quite inconsistent in packages
generally.  I'd have thought the dependencies should be added, but
it's difficult to verify them (and I don't know why rpmbuild doesn't
try to follow #includes).

> Hmm I wonder why I added the .la there, if anything it should have been in
> the -devel... But right rhel systems don't even have it, will remove.

That's something that is in the guidelines.  (I sympathize with "I
wonder why I ..."!)

> No problem, thanks for the review!

It's only an informal one.  I hope someone (Red Hat storage?) can test
it, as required for a proper review, and sponsor you.  They probably
expect to see more packaging and/or informal reviews of other things,
first.

If you have any more to contribute, even if it's not ready or
appropriate for review, I'd recommend setting up a copr repo.  Most of
what I install on our cluster goes into mine.

> I'll fix all of that and try
> fedora-review as soon as I get my laptop back home (might be a few days,
> will update the ticket with new URLs anyway)
> Our development servers are behind a kerberized proxy and the fedora tools
> can't seem to work with that, unfortunately :/

I don't know what it needs that would be blocked, but you can run
fedora-review on local package source, which is what I meant.  I've
only managed to do that in a rawhide vagrant box, though, which is
more painful than I'd like.

-- 
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 1249743] Review Request: kaadbg - Remote debugger runner for kaa

[bugzilla]

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



--- Comment #4 from William Moreno  ---
Spec URL: https://williamjmorenor.fedorapeople.org/rpmdev/kaadbg.spec
SRPM URL:
https://williamjmorenor.fedorapeople.org/rpmdev/kaadbg-0.3.0-1.fc22.src.rpm

Here is the updated links to the last release.

-- 
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 1262469] Review Request: php-patchwork-utf8 - Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP

[bugzilla]

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



--- Comment #18 from Fedora Update System  ---
php-patchwork-utf8-1.2.3-3.fc21 has been pushed to the Fedora 21 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1265484] Review Request: google-roboto-slab-fonts - Google Roboto Slab fonts

[bugzilla]

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



--- Comment #20 from Fedora Update System  ---
google-roboto-slab-fonts-1.100263-0.3.20150923git.fc21 has been pushed to the
Fedora 21 stable repository. If problems still persist, please make note of it
in this bug report.

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

[Bug 1265636] Review Request: google-roboto-mono-fonts - Google Roboto Mono fonts

[bugzilla]

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



--- Comment #16 from Fedora Update System  ---
google-roboto-mono-fonts-2.000986-0.1.20150923git.fc21 has been pushed to the
Fedora 21 stable repository. If problems still persist, please make note of it
in this bug report.

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

[Bug 1262872] Review Request: perl-TeX-Encode - Encoding to LaTeX escapes

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
perl-TeX-Encode-1.3-2.fc21 has been pushed to the Fedora 21 stable repository.
If problems still persist, please make note of it in this bug report.

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

[Bug 1262468] Review Request: php-punic - PHP-Unicode CLDR

[bugzilla]

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



--- Comment #17 from Fedora Update System  ---
php-punic-1.6.3-1.fc21 has been pushed to the Fedora 21 stable repository. If
problems still persist, please make note of it in this bug report.

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

[Bug 1249786] Review Request: kaaedit - Console text editor

[bugzilla]

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



--- Comment #3 from William Moreno  ---
Spec URL: https://williamjmorenor.fedorapeople.org/rpmdev/kaaedit.spec
SRPM URL:
https://williamjmorenor.fedorapeople.org/rpmdev/kaaedit-0.44.0-2.fc22.src.rpm

Fixed the requires

-- 
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 1266186] Review Request: jeromq - Pure Java implementation of libzmq

[bugzilla]

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

Jon Ciesla  changed:

   What|Removed |Added

  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 1264199] Review Request: php-wikimedia-avro - A library for using Avro with PHP

[bugzilla]

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



--- Comment #5 from Jon Ciesla  ---
This SCM request method has been deprecated. Please see
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 1266186] Review Request: jeromq - Pure Java implementation of libzmq

[bugzilla]

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



--- Comment #8 from Jon Ciesla  ---
This SCM request method has been deprecated. Please see
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 1264209] Review Request: php-wikimedia-ip-set - Library to match IP addresses against CIDR specifications

[bugzilla]

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

Jon Ciesla  changed:

   What|Removed |Added

  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 1264209] Review Request: php-wikimedia-ip-set - Library to match IP addresses against CIDR specifications

[bugzilla]

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



--- Comment #8 from Jon Ciesla  ---
This SCM request method has been deprecated. Please see
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 1268881] Review Request: perl-Coro-Multicore - Make Coro threads on multiple cores with specially supported modules

[bugzilla]

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

Jon Ciesla  changed:

   What|Removed |Added

  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 1264199] Review Request: php-wikimedia-avro - A library for using Avro with PHP

[bugzilla]

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

Jon Ciesla  changed:

   What|Removed |Added

  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 1234210] Review Request: pdf-stapler - tool for manipulating PDF documents from the command line

[bugzilla]

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



--- Comment #20 from Globe Trotter  ---
New files uploaded as per suggested lines. Please check and see if there is
anything else to be fixed.

-- 
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 1268881] Review Request: perl-Coro-Multicore - Make Coro threads on multiple cores with specially supported modules

[bugzilla]

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



--- Comment #5 from Jon Ciesla  ---
This SCM request method has been deprecated. Please see
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 1267003] Review Request: rows - A common, beautiful interface to tabular data, no matter the format

[bugzilla]

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

William Moreno  changed:

   What|Removed |Added

Summary|Review Request: python-rows |Review Request: rows - A
   |- A common, beautiful   |common, beautiful interface
   |interface to tabular data,  |to tabular data, no matter
   |no matter the format|the format



--- Comment #4 from William Moreno  ---
Spec URL: https://williamjmorenor.fedorapeople.org/rpmdev/rows.spec
SRPM URL:
https://williamjmorenor.fedorapeople.org/rpmdev/rows-0.1.1-2.fc22.src.rpm

=
Rename to rows
Include a python2-rows subpackage
Fix Requires following upstream recomends

-- 
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 1226719] Review Request: menulibre - FreeDesktop.org compliant menu editor

[bugzilla]

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

Orion Poplawski  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |
  Flags||fedora-review+



--- Comment #3 from Orion Poplawski  ---
Looks good.  Approved.  You may want to check out the latest python packaging
guidelines for some updates.  Sorry for the delay.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 1257410] Review Request: abcMIDI - ABC to/from MIDI conversion utilities

[bugzilla]

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



--- Comment #8 from Christopher Meng  ---
Please, read carefully:

http://fedoraproject.org/wiki/Package_Review_Process#Definitions_for_fedora-review_Flag_Settings

-- 
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 1234210] Review Request: pdf-stapler - tool for manipulating PDF documents from the command line

[bugzilla]

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



--- Comment #19 from Matthias Runge  ---

> > - there are tests in github repo. it would be great to execute them during
> > package build. It gives you a bit more feedback, if your build works (or 
> > not)
> 
> How do I do this? Sorry, I am not a python programmer and this is my first
> package anyway.

you could add something like:

%check
%{__python} setup.py test

to execute tests.

Be careful: build systems are not connected to the net, thus downloading from
pypi doesn't work.

you probably need to sed out requirements:

sed -i 's|"PyPDF2>=1.24"||' setup.py


That has to be done in %prep.

Requirements are handled in rpm anyways, there's no much value in keeping them
tied in python.

-- 
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 1249743] Review Request: kaadbg - Remote debugger runner for kaa

[bugzilla]

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



--- Comment #3 from René Ribaud  ---
Hello William,

I think you updated the spec but you didn't rebuild the srpms.
So LICENSE file from the patch is missing.

[root@d7c7b3161665 noarch]# rpm -qpl kaadbg-0.3.0-0.fc24.noarch.rpm 
/usr/lib/python3.4/site-packages/kaadbg
/usr/lib/python3.4/site-packages/kaadbg-0.3.0-py3.4.egg-info
/usr/lib/python3.4/site-packages/kaadbg-0.3.0-py3.4.egg-info/PKG-INFO
/usr/lib/python3.4/site-packages/kaadbg-0.3.0-py3.4.egg-info/SOURCES.txt
/usr/lib/python3.4/site-packages/kaadbg-0.3.0-py3.4.egg-info/dependency_links.txt
/usr/lib/python3.4/site-packages/kaadbg-0.3.0-py3.4.egg-info/top_level.txt
/usr/lib/python3.4/site-packages/kaadbg/__init__.py
/usr/lib/python3.4/site-packages/kaadbg/__pycache__
/usr/lib/python3.4/site-packages/kaadbg/__pycache__/__init__.cpython-34.pyc
/usr/lib/python3.4/site-packages/kaadbg/__pycache__/__init__.cpython-34.pyo
/usr/lib/python3.4/site-packages/kaadbg/__pycache__/debug.cpython-34.pyc
/usr/lib/python3.4/site-packages/kaadbg/__pycache__/debug.cpython-34.pyo
/usr/lib/python3.4/site-packages/kaadbg/__pycache__/run.cpython-34.pyc
/usr/lib/python3.4/site-packages/kaadbg/__pycache__/run.cpython-34.pyo
/usr/lib/python3.4/site-packages/kaadbg/debug.py
/usr/lib/python3.4/site-packages/kaadbg/run.py
/usr/share/doc/kaadbg
/usr/share/doc/kaadbg/README.rst


Otherwise that looks ok.

-- 
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 1266186] Review Request: jeromq - Pure Java implementation of libzmq

[bugzilla]

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

Mikolaj Izdebski  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #7 from Mikolaj Izdebski  ---
New Package SCM Request
===
Package Name: jeromq
Short Description: Pure Java implementation of libzmq
Owners: mizdebsk msrb msimacek
Branches: f23
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 759818] Review Request: buffer - General purpose buffer program

[bugzilla]

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



--- Comment #11 from Michael Schwendt (Fedora Packager Sponsors Group) 
 ---
> has been retired:
> but the reason doesn't seem obvious
> (I'm building it for each fedora release myself)

Build history is this:

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

After conversion to dist-git, package has been auto-rebuilt once but apparently
failed during the mass-rebuilds for F16 (and later) in 2011. Very likely due to
compiler changes. Typically, the package maintainers at that time have received
FTBFS notifications and could have looked into it.

It seems you've been the package maintainer for this before (via FAS account
"bcornec"), and later the package has been adapted by somebody else for some
time:

  http://pkgs.fedoraproject.org/cgit/buffer.git/log/

As pointed out before, some of the review tickets have not been visible on the
review queue tracker page for a long time because they been filed incorrectly.

mindi: bug 187317
mondo: bug 187318
mindi-busybox: bug 476234

-- 
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 1218758] Review Request: flmsg - Fast Light Message Amateur Radio Forms Manager

[bugzilla]

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



--- Comment #8 from Christopher Meng  ---
I may patch it when I'm back, but don't expect it will be handled soon, I'm
still on vacation.

Meanwhile I don't think it's a bundled library, only some code instead.

-- 
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 1268010] Review Request: libmooshika - helper library for RDMA

[bugzilla]

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

Dave Love  changed:

   What|Removed |Added

 CC||d.l...@liverpool.ac.uk



--- Comment #1 from Dave Love  ---
Unfortunately I can't review this properly as I can't run ganesha so
can't test it.  However, here are some quick comments on the spec
file.

* I don't know if you want to support it, but the Group tag is only
  needed for EPEL5;

* You normally shouldn't add explicit Requires for dynamic libraries
  except typically for devel packages depending on the non-devel
  version -- the packaging will usually do that automatically. 
  For the devel package the Requires should normally be like

  %packagedevel
  Requires:   %{name}-libs%{?_isa} = %{version}-%{release}
  ...

* make should have an argument %{?_smp_mflags} in the %build section,
  or a comment that it's not smp-safe;

* Don't package .la files (and normally not .a).  Under %files, You
  can use instead

  %exclude %{_libdir}/*a

  but you need .so in the devel package, not the main one;

* You need to add

  %post -p /sbin/ldconfig
  %postun -p /sbin/ldconfig

Sorry not to give references to the packaging guide for each item, but
I hope that helps, and it's easy to fix.  Don't be put off!  rpmlint
should warn about some of it, and it's worth running fedora-review on
the package (if you can -- I have trouble on an EPEL6 system).

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