[Bug 1309782] Review Request: bugyou_plugins - Plugins and Services for Bugyou

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1309782



--- Comment #6 from Fedora Update System  ---
bugyou_plugins-0.1-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-fccd1b9dd2

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

[Bug 1309782] Review Request: bugyou_plugins - Plugins and Services for Bugyou

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1309782



--- Comment #7 from Fedora Update System  ---
bugyou_plugins-0.1-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-2780aa93c2

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

[Bug 1309782] Review Request: bugyou_plugins - Plugins and Services for Bugyou

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1309782



--- Comment #8 from Fedora Update System  ---
bugyou_plugins-0.1-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-18eaad36f3

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

[Bug 1312609] Review Request: nodejs-archiver-utils - Utility functions for nodejs-archiver

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312609

Piotr Popieluch  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2016-02-29 03:57:32



--- Comment #6 from Piotr Popieluch  ---
built in  rawhide+f24

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

[Bug 1127967] Review Request: python-releases - A Sphinx extension for changelog manipulation

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1127967



--- Comment #9 from Pierre-YvesChibon  ---
I am sorry for the time-out :(

Eduardo, would you mind updating the spec file to the latest 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 1312561] Review Request: gkrellm-sun - Sun clock plugin for GKrellM

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312561



--- Comment #5 from Fedora Update System  ---
gkrellm-2.3.6-0.1.rc1.git20160226.fc23 gkrellm-sun-1.0.0-20.fc23 has been
submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-e10fd8290e

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

[Bug 1312561] Review Request: gkrellm-sun - Sun clock plugin for GKrellM

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312561

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 1312561] Review Request: gkrellm-sun - Sun clock plugin for GKrellM

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312561



--- Comment #6 from Fedora Update System  ---
gkrellm-2.3.6-0.1.rc1.git20160226.fc22 gkrellm-sun-1.0.0-20.fc22 has been
submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-9be2619671

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

[Bug 1312577] Review Request: python-pyopengl - Python bindings for OpenGL

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312577



--- Comment #9 from Jonathan Underwood  ---
Great, thanks for the review, much appreciated.

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

[Bug 1107127] Review Request: procenv - Utility to show process environment

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1107127



--- Comment #51 from Fedora Update System  ---
procenv-0.44-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 1293100] Review Request: tarantool - an in-memory database and Lua application server

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1293100

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #30 from Fedora Update System  ---
tarantool-1.6.8.530-2.el7 has been pushed to the Fedora EPEL 7 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-220abf23ed

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

[Bug 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145



--- Comment #10 from Jonathan Underwood  ---

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.
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "LGPL (v3 or later)", "Unknown or generated", "*No copyright*
 Public domain". 9 files have unknown license. Detailed output of
 licensecheck in /home/jgu/Fedora/1310145-python-igor/licensecheck.txt

Public Domain needs adding to the License field in the spec.

[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: 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.
[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.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python2-igor , python3-igor
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[!]: Patches link to upstream bugs/comments/lists or are otherwise
 justified.
This needs addressing by adding comments to spec file. Have you
submitted these patches upstream?



[x]: 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
 architectur

[Bug 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145



--- Comment #11 from Jonathan Underwood  ---
So, please address the following:

1) Add Public Domain to license field

2) Add comments about the patches, and submit upstream where appropriate,
giving a link in a comment for each patch.

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

[Bug 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145

Jonathan Underwood  changed:

   What|Removed |Added

  Flags|fedora-review?  |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 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145



--- Comment #12 from Jonathan Underwood  ---
APPROVED

I'll trust you to make those minor changes. Thanks for the contribution to
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 1305382] Review Request: tristripper - Triangle stripification (algorithm by Tanguy Fautre)

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305382

Raphael Groner  changed:

   What|Removed |Added

 Status|POST|RELEASE_PENDING



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

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305496

Tomas Repik  changed:

   What|Removed |Added

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



--- Comment #4 from Tomas Repik  ---
(In reply to jiri vanek from comment #3)
> [!]: %check is present and all tests pass.
> 
> Unless there is some reason I cant see, i probably insists execution of
> tests during build. there is even  junit in In original sources.

Tests are executed during build, using maven-surefire-plugin. Only
HistogramPerfTest is not run, but I don't think it's necessary. As [1] tells:
"Optional %check section
Runs projects integration tests. Unit test are usually run in %build section,
so if there are no integration tests available, this section is omitted"

> [!]: Package functions as described.
> 
> There is quite a lot interesting things in the source tarball. One of them
> is launcher. Others are web based histrograms. I would expect them to work
> in installed application. Rightnow they are just missing.

I can't see any more files needed to be installed. Can you be more specific?

[1] https://fedorahosted.org/released/javapackages/doc/#_example_rpm_project

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

[Bug 1293100] Review Request: tarantool - an in-memory database and Lua application server

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1293100



--- Comment #31 from Fedora Update System  ---
tarantool-1.6.8.530-2.fc22 has been pushed to the Fedora 22 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-c7394b6681

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

[Bug 1293100] Review Request: tarantool - an in-memory database and Lua application server

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1293100



--- Comment #32 from Fedora Update System  ---
tarantool-1.6.8.530-2.fc23 has been pushed to the Fedora 23 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-98716494c3

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

[Bug 1310674] Review Request: rubygem-github-linguist - GitHub Language detection

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310674



--- Comment #6 from Jiri Popelka  ---
How should I handle the missing dependencies (for tests) ? Do I need to package
them or is there other option ?

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

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305496

jiri vanek  changed:

   What|Removed |Added

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



--- Comment #5 from jiri vanek  ---
(In reply to Tomas Repik from comment #4)
> (In reply to jiri vanek from comment #3)
> > [!]: %check is present and all tests pass.
> > 
> > Unless there is some reason I cant see, i probably insists execution of
> > tests during build. there is even  junit in In original sources.
> 
> Tests are executed during build, using maven-surefire-plugin. Only
> HistogramPerfTest is not run, but I don't think it's necessary. As [1]
> tells: "Optional %check section
> Runs projects integration tests. Unit test are usually run in %build
> section, so if there are no integration tests available, this section is
> omitted"

I always forget that forcing maven run build and tests one by one is a bit
hackisch.  So yes, I donto wont to foorce you to split build and test run. And
integration tests seems really not necessary.
>  
> > [!]: Package functions as described.
> > 
> > There is quite a lot interesting things in the source tarball. One of them
> > is launcher. Others are web based histrograms. I would expect them to work
> > in installed application. Rightnow they are just missing.
> 
> I can't see any more files needed to be installed. Can you be more specific?
> 
> [1] https://fedorahosted.org/released/javapackages/doc/#_example_rpm_project

You are saying yoursef:  48: %jpackage_script
org.fedoraproject.helloworld.HelloWorld "" "" %{name} helloworld true

:)

More serious - there is launcher HistogramLogProcessor. I watched it more
closely, and although it probably deserves to be rewritten and packed inside 
as launcher, it do not seems to be doing main functionality of this package.

Originally I thougth that this program will take an fil/input of numbers and
create histogram..image...

But looking to list of mainclasses:
src/perf/java/org/HdrHistogram/HistogramPerfTest.java:public static void
main(String[] args) {
src/main/java/org/HdrHistogram/HistogramLogProcessor.java:public static
void main(final String[] args)  {
src/examples/java/SimpleHistogramExample.java:public static void main(final
String[] args) {


My assumption was probably wrong. Thoughts?

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

[Bug 1312360] Review Request: python-rcssmin - CSS Minifier

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312360



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-rcssmin

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

[Bug 1311932] Review Request: mu - A simple Python editor for micro:bit

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1311932



--- Comment #5 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/mu

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

[Bug 1312577] Review Request: python-pyopengl - Python bindings for OpenGL

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312577



--- Comment #10 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-pyopengl

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

[Bug 1169966] Review Request: rkt - CLI for running app containers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169966



--- Comment #54 from Joe Brockmeier  ---
(In reply to Haïkel Guémar from comment #53)
> Built and imported in F24 and rawhide for over a week now.

Fantastic! This is great news.

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

[Bug 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145



--- Comment #13 from Zbigniew Jędrzejewski-Szmek  ---
Thanks!

I updated License and opened a PR upstream:
https://github.com/wking/igor/pull/1

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

[Bug 1083962] Review Request: maxscale - A database-centric proxy that works with MariaDB and MySQL

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083962



--- Comment #20 from Oden Eriksson  ---
Heya to you too :-)

Years later, based on http://hhorak.fedorapeople.org/maxscale and the Mageia
package maxscale-1.3.0-3.mga6.src.rpm I have a new candidate up on my site
here:

http://nux.se/maxscale/
http://nux.se/maxscale/maxscale-1.3.0-1.el7.centos.src.rpm
http://nux.se/maxscale/maxscale.spec

Please test this one.

Cheers.

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

[Bug 1312577] Review Request: python-pyopengl - Python bindings for OpenGL

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312577

Jonathan Underwood  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2016-02-29 09:35:28



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

[Bug 1169966] Review Request: rkt - CLI for running app containers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169966



--- Comment #55 from Lokesh Mandvekar  ---
(In reply to Joe Brockmeier from comment #54)
> (In reply to Haïkel Guémar from comment #53)
> > Built and imported in F24 and rawhide for over a week now.
> 
> Fantastic! This is great news.

Umm, don't celebrate just yet, this only works with setenforce 0 for now. Also,
I hope this doesn't get pushed into f24 stable/release until that's sorted.
This got into f24 when it was cut from 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 1270322] Review Request: chromium - A WebKit (Blink) powered web browser

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270322



--- Comment #24 from Tom "spot" Callaway  ---
New SRPM: https://spot.fedorapeople.org/chromium-48.0.2564.116-3.fc24.src.rpm
New SPEC: https://spot.fedorapeople.org/chromium.spec

Adds chrome-remote-desktop subpackage, Requires: u2f-hidraw-policy.

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

[Bug 1119075] Review Request: itop-itsm - Simple, web based IT Service Management tool

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1119075

Jon Ciesla  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NOTABUG
Last Closed||2016-02-29 10:00:21



--- Comment #8 from Jon Ciesla  ---
Please reopen if still of interest.

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

[Bug 961223] Review Request: ocrfeeder - A document layout analysis and optical character recognition system

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=961223

Jon Ciesla  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NOTABUG
Last Closed||2016-02-29 10:00:57



--- Comment #22 from Jon Ciesla  ---
Submitted non-responsive.

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

[Bug 1260254] Review Request: mongo-c-driver - High-performance MongoDB driver for C

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1260254

Remi Collet  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2016-02-29 10:06:59



--- Comment #6 from Remi Collet  ---
Taking review.

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

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

[Bug 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939



--- Comment #2 from Upstream Release Monitoring 
 ---
remi's scratch build of mongo-c-driver-1.3.3-2.remi.src.rpm for rawhide failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=13177835

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

[Bug 1312939] New: Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939

Bug ID: 1312939
   Summary: Review Request: mongo-c-driver - Client library
written in C for MongoDB
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/5326a6d805d8935bc49a42149c0f3b476647727e/mongo-c-driver/mongo-c-driver.spec
SRPM URL:
http://rpms.famillecollet.com/SRPMS/mongo-c-driver-1.3.3-2.remi.src.rpm
Description: 
mongo-c-driver is a client library written in C for MongoDB.


Fedora Account System Username: remi

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

[Bug 1269056] Review Request: php-pecl-mongodb - MongoDB driver for PHP

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1269056

Remi Collet  changed:

   What|Removed |Added

 Depends On||1312939




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1312939
[Bug 1312939] Review Request: mongo-c-driver - Client library written in C
for MongoDB
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939

Remi Collet  changed:

   What|Removed |Added

 Blocks||1269056 (pecl/mongodb)
 CC||i...@cicku.me



--- Comment #1 from Remi Collet  ---
*** Bug 1260254 has been marked as a duplicate of this bug. ***


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269056
[Bug 1269056] Review Request: php-pecl-mongodb - MongoDB driver for PHP
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1269056] Review Request: php-pecl-mongodb - MongoDB driver for PHP

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1269056
Bug 1269056 depends on bug 1260254, which changed state.

Bug 1260254 Summary: Review Request: mongo-c-driver - High-performance MongoDB 
driver for C
https://bugzilla.redhat.com/show_bug.cgi?id=1260254

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE



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

[Bug 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939



--- Comment #3 from Upstream Release Monitoring 
 ---
remi's scratch build of mongo-c-driver-1.3.3-2.remi.src.rpm for f24 failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=13177894

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

[Bug 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939



--- Comment #4 from Upstream Release Monitoring 
 ---
remi's scratch build of mongo-c-driver-1.3.3-2.remi.src.rpm for f23 failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=13177898

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

[Bug 1312296] Review Request: 3dprinter-udev-rules - Rules for udev to give regular users access to operate 3D printers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312296



--- Comment #5 from Fedora Update System  ---
3dprinter-udev-rules-0.1-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-739a9100da

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

[Bug 1312296] Review Request: 3dprinter-udev-rules - Rules for udev to give regular users access to operate 3D printers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312296

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 1298665] Review Request: libvma - Dramatically improves performance of socket based applications

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1298665



--- Comment #7 from Michal Schmidt  ---
> ExcludeArch: %{arm} %{ix86} s390x ppc64 ppc64le

Every use of ExcludeArch should have a comment explaining the reason.

> %description
> libvma is a LD_PRELOAD-able library that boosts performance
> of TCP and UDP traffic

Missing a dot to end the sentence.

> Part of Mellanox's enhanced services

Does it mean libvma works only with Mellanox hardware?
If yes, please rephrase it in a clearer way.
If it does not mean that, then does it mean anything at all?

> Allows application written over standard socket API
> to run over Infiniband/Ethernet from user space with full network stack bypass
> and get better throughput, latency and packets/sec rate.

It allows applications [...]

> %pre
> if [ `grep "^[^#]" /etc/security/limits.conf /etc/security/limits.d/* 2> 
> /dev/null |grep memlock|grep unlimited | wc -l` -le 0 ]; then

Looks like it could be simplified to:
if ! grep -qs '^[^#].*memlock.*unlimited' /etc/security/limits.conf
/etc/security/limits.d/*; then

Anyway, do you really need to emit the message?
It is unusual for a scriptlet in Fedora to emit any output whatsoever.

> %files
> %{_libdir}/%{name}*.so.*
> %{_libdir}/%{name}.so

You may want to add a comment here saying you need libvma.so in the main
package
(as opposed to -devel) so that 'LD_PRELOAD=libvma.so ' works.

> %doc README.txt journal.txt VMA_VERSION

I see your upstream tarball contains a COPYING file, that you're not including
in the package.
Ideally it would contain the full text of the GPLv2
(Would you please make that change upstream?
 Currently it has only a copy of the header as it appears in the source files.)
and be included in the package using the %license tag (not %doc).

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

[Bug 1312296] Review Request: 3dprinter-udev-rules - Rules for udev to give regular users access to operate 3D printers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312296



--- Comment #6 from Fedora Update System  ---
3dprinter-udev-rules-0.1-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-86837fdc16

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

[Bug 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145



--- Comment #14 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-igor

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

[Bug 1270375] Review Request: nacl-arm-newlib - C library intended for use on embedded systems

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270375



--- Comment #7 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nacl-arm-newlib

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

[Bug 1312963] New: Review Request: glibc-arm-linux-gnu - Cross Compiled GNU C Library targeted at arm-linux-gnu

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312963

Bug ID: 1312963
   Summary: Review Request: glibc-arm-linux-gnu - Cross Compiled
GNU C Library targeted at arm-linux-gnu
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: tcall...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://spot.fedorapeople.org/glibc-arm-linux-gnu.spec
SRPM URL: https://spot.fedorapeople.org/glibc-arm-linux-gnu-2.23-1.fc24.src.rpm
Description: 
This is a Cross Compiled version of the GNU C Library, which can be used to
compile and link binaries for the arm-linux-gnu platform, instead of for the
native platform.

Fedora Account System Username: spot

A few notes: As this is a cross-compiled version of glibc, it will not run on
the target system (unless your target system is ARM, and then why are you
installing a cross-compiler for ARM?). This is why the package is noarch, and
why rpmlint throws warnings (they can be ignored). We could make the package
arch specific, but the bits _should_ be identical regardless of architecture of
the host 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

[Bug 1312963] Review Request: glibc-arm-linux-gnu - Cross Compiled GNU C Library targeted at arm-linux-gnu

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312963

Tom "spot" Callaway  changed:

   What|Removed |Added

 Blocks||1270405




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1270405
[Bug 1270405] Review Request: native_client - Google Native Client
Toolchain
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1270405] Review Request: native_client - Google Native Client Toolchain

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270405

Tom "spot" Callaway  changed:

   What|Removed |Added

 Depends On||1312963




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1312963
[Bug 1312963] Review Request: glibc-arm-linux-gnu - Cross Compiled GNU C
Library targeted at arm-linux-gnu
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1270375] Review Request: nacl-arm-newlib - C library intended for use on embedded systems

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270375



--- Comment #8 from Fedora Update System  ---
nacl-arm-newlib-2.1.0-3.git373135e.fc22 has been submitted as an update to
Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-9a083ecdf1

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

[Bug 1270375] Review Request: nacl-arm-newlib - C library intended for use on embedded systems

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270375

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



--- Comment #9 from Fedora Update System  ---
nacl-arm-newlib-2.1.0-3.git373135e.fc23 has been submitted as an update to
Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-86e9d3f14a

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

[Bug 1270375] Review Request: nacl-arm-newlib - C library intended for use on embedded systems

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270375



--- Comment #8 from Fedora Update System  ---
nacl-arm-newlib-2.1.0-3.git373135e.fc22 has been submitted as an update to
Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-9a083ecdf1

--- Comment #9 from Fedora Update System  ---
nacl-arm-newlib-2.1.0-3.git373135e.fc23 has been submitted as an update to
Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-86e9d3f14a

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

[Bug 1310869] Review Request: python-zanata2fedmsg - A web app bridging zanata webhooks to fedmsg

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310869

Ralph Bean  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2016-02-29 11:37:03



--- Comment #5 from Ralph Bean  ---
Bodhi updates submitted: 
http://bodhi.fedoraproject.org/updates/python-zanata2fedmsg

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

[Bug 1270405] Review Request: native_client - Google Native Client Toolchain

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270405



--- Comment #3 from Tom "spot" Callaway  ---
New SRPM:
https://spot.fedorapeople.org/native_client-48.0.2564.82-2.fc23.src.rpm
New SPEC: https://spot.fedorapeople.org/native_client.spec

No, it doesn't currently work for i686. Nacl/PNacl is disabled for ia32. It
might be possible to enable at some point, but one awful torture pit at a time.
:)

As far as /usr vs %{_prefix}, if the macro is longer than the actual directory,
and extremely unlikely to change in my lifetime, I'm not likely to use it
religiously. That said, if it's a review blocker, I'll fix it.

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

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305496



--- Comment #6 from Tomas Repik  ---
Well, first of all thanks for the review, I appreciate it. 

> Thoughts?
For me it's more like a library than a real runable application although it
would make sense as well. I could try and create something like subpackage with
the main class from HistogramLogProcessor.java, but it's not the core function
and not important to include.

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

[Bug 1169966] Review Request: rkt - CLI for running app containers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169966



--- Comment #56 from Daniel Walsh  ---
Working with coreos on the SELinux issues, a couple can be fixed simply, the
others might involve code changes in RKT.

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

[Bug 1298665] Review Request: libvma - Dramatically improves performance of socket based applications

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1298665



--- Comment #8 from al...@mellanox.com ---
(In reply to Michal Schmidt from comment #7)
> > ExcludeArch: %{arm} %{ix86} s390x ppc64 ppc64le
> 
> Every use of ExcludeArch should have a comment explaining the reason.

I will fix it.

> > %description
> > libvma is a LD_PRELOAD-able library that boosts performance
> > of TCP and UDP traffic
> 
> Missing a dot to end the sentence.

I will fix it.

> > Part of Mellanox's enhanced services
> 
> Does it mean libvma works only with Mellanox hardware?
> If yes, please rephrase it in a clearer way.
> If it does not mean that, then does it mean anything at all?

libvma can be supported by RDMA capable devices that support
IBV_QPT_RAW_PACKET QPs.
I will rephrase the description.

> > Allows application written over standard socket API
> > to run over Infiniband/Ethernet from user space with full network stack 
> > bypass
> > and get better throughput, latency and packets/sec rate.
> 
> It allows applications [...]

I will fix it.

> > %pre
> > if [ `grep "^[^#]" /etc/security/limits.conf /etc/security/limits.d/* 2> 
> > /dev/null |grep memlock|grep unlimited | wc -l` -le 0 ]; then

> Looks like it could be simplified to:
> if ! grep -qs '^[^#].*memlock.*unlimited' /etc/security/limits.conf
> /etc/security/limits.d/*; then
> 
> Anyway, do you really need to emit the message?
> It is unusual for a scriptlet in Fedora to emit any output whatsoever.

We print this message in order to notify about system configuration change
(memlock limit). I agree that this is not crucial and can be removed.

> > %files
> > %{_libdir}/%{name}*.so.*
> > %{_libdir}/%{name}.so
>
> You may want to add a comment here saying you need libvma.so in the main
> package
> (as opposed to -devel) so that 'LD_PRELOAD=libvma.so ' works.

Good idea, I will add this comment.

> > %doc README.txt journal.txt VMA_VERSION
> 
> I see your upstream tarball contains a COPYING file, that you're not
> including in the package.
> Ideally it would contain the full text of the GPLv2
> (Would you please make that change upstream?
>  Currently it has only a copy of the header as it appears in the source
> files.)

I am checking if we can make this change.

> and be included in the package using the %license tag (not %doc).

I will add the COPYING file to the package under the %license tag.

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

[Bug 1312604] Review Request: nodejs-code-point-at - ES2015 String#codePointAt() ponyfill

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312604



--- Comment #2 from Jared Smith  ---
Oops, sorry about that.  Updated packages below:

Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-code-point-at/nodejs-code-point-at.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-code-point-at/nodejs-code-point-at-1.0.0-2.fc25.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 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939



--- Comment #5 from Upstream Release Monitoring 
 ---
remi's scratch build of mongo-c-driver-1.3.3-2.remi.src.rpm for f23 failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=13179468

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

[Bug 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939



--- Comment #6 from Upstream Release Monitoring 
 ---
remi's scratch build of mongo-c-driver-1.3.3-2.remi.src.rpm for rawhide
completed http://koji.fedoraproject.org/koji/taskinfo?taskID=13179467

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

[Bug 1312939] Review Request: mongo-c-driver - Client library written in C for MongoDB

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312939



--- Comment #7 from Remi Collet  ---
- add patch to skip online tests open
https://github.com/mongodb/mongo-c-driver/pull/314
- temporarily disable test suite on F24+ (#1313018)

https://github.com/remicollet/remirepo/commit/6f00a8bc2afd4197d5adf6d31e4eafdf4c15b64c

- temporarily disable test suite on arm  (#1303864)

https://github.com/remicollet/remirepo/commit/fd955b84d82f18615dafc80c83cae2724af6311b


Spec:
https://raw.githubusercontent.com/remicollet/remirepo/fd955b84d82f18615dafc80c83cae2724af6311b/mongo-c-driver/mongo-c-driver.spec
Srpm: http://rpms.famillecollet.com/SRPMS/mongo-c-driver-1.3.3-2.remi.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 1310145] Review Request: python-igor - Parser for Igor Binary Waves (.ibw) and Packed Experiment (.pxp) files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310145

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||python-igor-0.2-4.20150408g
   ||it2c2a79d.fc24
 Resolution|--- |CURRENTRELEASE
Last Closed||2016-02-29 13:55:33



--- Comment #15 from Zbigniew Jędrzejewski-Szmek  ---
Built in rawhide, F24, F23, F22. Updates submitted.

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

[Bug 1312604] Review Request: nodejs-code-point-at - ES2015 String#codePointAt() ponyfill

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312604



--- Comment #3 from Tom Hughes  ---
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: "MIT/X11 (BSD like)", "Unknown or generated". 3 files have
 unknown license. Detailed output of licensecheck in /home/tom/1312604
 -nodejs-code-point-at/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 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]: 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

= SHOULD items =

Generic:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[-]: 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).
[?]: 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]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a working URL.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: Spec use %global instead of %define unless justified.

= EXTRA items =

Generic:
[x]: Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: Spec file according to URL is the same as in SRPM.


Rpmlint
---
Checking: nodejs-code-point-at-1.0.0-2.fc25.noarch.rpm
  

[Bug 1312604] Review Request: nodejs-code-point-at - ES2015 String#codePointAt() ponyfill

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312604

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Tom Hughes  ---
Needs a BR on npm(number-is-nan) when tests are enabled, but other than that it
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 1312654] Review Request: nodejs-is-fullwidth-code-point - Check if given Unicode code point is fullwidth

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312654



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-is-fullwidth-code-point

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

[Bug 1311152] Review Request: nodejs-mz - Modernize node.js to current ECMAScript standards

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1311152



--- Comment #2 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-mz

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

[Bug 1312604] Review Request: nodejs-code-point-at - ES2015 String#codePointAt() ponyfill

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312604



--- Comment #5 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-code-point-at

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

[Bug 1310796] Review Request: python-etcd - a python client for etcd

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1310796

Colin Walters  changed:

   What|Removed |Added

 CC||walt...@redhat.com



--- Comment #3 from Colin Walters  ---
Can you add a link in the spec to the upstream source?  It is linked from the
very bottom of the pypy page, but often when looking at a package I want to
know "where do I find the git".  See:

https://fedoraproject.org/wiki/User:Walters/Packaging_VCS_key_proposal

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

[Bug 1305737] Review Request: reprepro - Tool to handle local repositories of Debian packages

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305737

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-02-29 17:20:22



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

[Bug 1305737] Review Request: reprepro - Tool to handle local repositories of Debian packages

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305737



--- Comment #17 from Fedora Update System  ---
reprepro-4.17.0-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 1305737] Review Request: reprepro - Tool to handle local repositories of Debian packages

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305737



--- Comment #18 from Fedora Update System  ---
reprepro-4.17.0-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 1312296] Review Request: 3dprinter-udev-rules - Rules for udev to give regular users access to operate 3D printers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312296

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
3dprinter-udev-rules-0.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.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-86837fdc16

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

[Bug 1270375] Review Request: nacl-arm-newlib - C library intended for use on embedded systems

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270375

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
nacl-arm-newlib-2.1.0-3.git373135e.fc22 has been pushed to the Fedora 22
testing repository. If problems still persist, please make note of it in this
bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-9a083ecdf1

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

[Bug 1312561] Review Request: gkrellm-sun - Sun clock plugin for GKrellM

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312561

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
gkrellm-2.3.6-0.1.rc1.git20160226.fc22, gkrellm-sun-1.0.0-20.fc22 has been
pushed to the Fedora 22 testing repository. If problems still persist, please
make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-9be2619671

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

[Bug 1309782] Review Request: bugyou_plugins - Plugins and Services for Bugyou

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1309782

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
bugyou_plugins-0.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.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-2780aa93c2

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

[Bug 1312296] Review Request: 3dprinter-udev-rules - Rules for udev to give regular users access to operate 3D printers

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312296



--- Comment #8 from Fedora Update System  ---
3dprinter-udev-rules-0.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.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-739a9100da

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

[Bug 1270375] Review Request: nacl-arm-newlib - C library intended for use on embedded systems

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270375



--- Comment #11 from Fedora Update System  ---
nacl-arm-newlib-2.1.0-3.git373135e.fc23 has been pushed to the Fedora 23
testing repository. If problems still persist, please make note of it in this
bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-86e9d3f14a

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

[Bug 1312561] Review Request: gkrellm-sun - Sun clock plugin for GKrellM

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1312561



--- Comment #8 from Fedora Update System  ---
gkrellm-2.3.6-0.1.rc1.git20160226.fc23, gkrellm-sun-1.0.0-20.fc23 has been
pushed to the Fedora 23 testing repository. If problems still persist, please
make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-e10fd8290e

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

[Bug 1309782] Review Request: bugyou_plugins - Plugins and Services for Bugyou

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1309782



--- Comment #10 from Fedora Update System  ---
bugyou_plugins-0.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.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-18eaad36f3

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

[Bug 1309782] Review Request: bugyou_plugins - Plugins and Services for Bugyou

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1309782



--- Comment #11 from Fedora Update System  ---
bugyou_plugins-0.1-1.el7 has been pushed to the Fedora EPEL 7 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-fccd1b9dd2

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

[Bug 1299146] Review Request: python-epub - Python library for reading EPUB files

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1299146

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #18 from Fedora Update System  ---
python-epub-0.5.2-4.el7 has been pushed to the Fedora EPEL 7 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-11455d1187

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

[Bug 1202064] Review Request: knock - A port-knocking server/client

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1202064

Mike Goodwin  changed:

   What|Removed |Added

 CC||m...@mgoodwin.net



--- Comment #7 from Mike Goodwin  ---
Any news on this?

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

[Bug 1268716] Review Request: cjdns - IP6 VPN with crypto address allocation

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1268716



--- Comment #38 from Stuart D Gathman  ---
Submitted release 6.  I moved all the tools to /usr/libexec/cjdns.  This make
rpmlint much happier.  Now it just complains about lack of documentation and
the /var/empty/cjdns dir.

I also added a few man pages.  I'll add more man pages as I get time.

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

[Bug 1268716] Review Request: cjdns - IP6 VPN with crypto address allocation

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1268716



--- Comment #39 from Upstream Release Monitoring 
 ---
sdgathman's scratch build of cjdns-17.3-6.fc22.src.rpm for rawhide completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=13184124

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

[Bug 1313155] New: Review Request: gluster-nagios-addons - Plugins to monitor gluster services on hosts

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313155

Bug ID: 1313155
   Summary: Review Request: gluster-nagios-addons - Plugins to
monitor gluster services on hosts
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rnach...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://rnachimu.fedorapeople.org/gluster-nagios-addons.spec

SRPM URL:
https://rnachimu.fedorapeople.org/gluster-nagios-addons-0.2.5-1.el6.src.rpm 

Description: Gluster-nagios-addons package contains a set of Nagios plugins,
scripts and configurations files which helps to monitor Gluster services
running on a Glusterfs server. These plugins will be used by the nagios server
extensions available in 'nagios-server-addons' packages.

Fedora Account System Username: rnachimu

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

[Bug 1313157] New: Review Request: nagios-server-addons - Nagios server plugins and helpers for Gluster monitoring

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313157

Bug ID: 1313157
   Summary: Review Request: nagios-server-addons - Nagios server
plugins and helpers for  Gluster monitoring
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rnach...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://rnachimu.fedorapeople.org/nagios-server-addons.spec

SRPM URL:
https://rnachimu.fedorapeople.org/nagios-server-addons-0.2.3-1.el6.src.rpm

Description: Nagios-server-addons package contains a set of Nagios server
plugins and helper scripts for Gluster monitoring. This package should be
installed on the Nagios server to enable Nagios monitoring for gluster
deployments. These plugins will communicate with gluster nodes using NRPE (
Nagios Remote Plug-in Executor) plugins installed by 'gluster-nagios-addons'.


Fedora Account System Username: rnachimu

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

[Bug 1313157] Review Request: nagios-server-addons - Nagios server plugins and helpers for Gluster monitoring

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313157

Ramesh N  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1313155



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

[Bug 1313155] Review Request: gluster-nagios-addons - Plugins to monitor gluster services on hosts

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313155

Ramesh N  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1313157



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

[Bug 1313157] Review Request: nagios-server-addons - Nagios server plugins and helpers for Gluster monitoring

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313157

Ramesh N  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1313159



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

[Bug 1313159] Review Request: gluster-nagios-common - Common utilities required for gluster-nagios-common and nagios-server-addons

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313159

Ramesh N  changed:

   What|Removed |Added

 CC||nde...@redhat.com
   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1313155,
   ||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1313157



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

[Bug 1313159] New: Review Request: gluster-nagios-common - Common utilities required for gluster-nagios-common and nagios-server-addons

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313159

Bug ID: 1313159
   Summary: Review Request: gluster-nagios-common - Common
utilities required for gluster-nagios-common and
nagios-server-addons
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rnach...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://rnachimu.fedorapeople.org/gluster-nagios-common.spec

SRPM URL:
https://rnachimu.fedorapeople.org/gluster-nagios-common-0.2.3-1.fc25.src.rpm

Description: gluster-nagios-common package contains the common utilities
required for packages gluster-nagios-addons and nagios-server-addons.


Fedora Account System Username: rnachimu

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

[Bug 1313155] Review Request: gluster-nagios-addons - Plugins to monitor gluster services on hosts

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1313155

Ramesh N  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1313159



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

[Bug 1298665] Review Request: libvma - Dramatically improves performance of socket based applications

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1298665



--- Comment #9 from Upstream Release Monitoring 
 ---
alexva's scratch build of libvma-7.0.14-1.fc24.src.rpm for rawhide failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=13185983

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

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

2016-02-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1305496



--- Comment #7 from jiri vanek  ---
I can agree on it. But Still I would say  - upstream have an launcher =>
include launcher.
No subpackage needed.

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