date:20170221

[Bug 1410594] Review Request: python-beaker - Full-stack software and hardware integration testing system

2017-02-21 Thread bugzilla

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

Matthias Runge  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1410594] Review Request: python-beaker - Full-stack software and hardware integration testing system

2017-02-21 Thread bugzilla

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

Matthias Runge  changed:

   What|Removed |Added

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



--- Comment #1 from Matthias Runge  ---
I'll do the 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396790] Review Request: gnome-shell-extension-freon - GNOME Shell extension to display system temperature , voltage, and fan speed

2017-02-21 Thread bugzilla

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



--- Comment #7 from terrycloth  ---
I was thinking for a moment that maybe it's trying to say that I shouldn't run
`glib-compile-schemas` myself, because it's automatically part of the
%posttrans process?

But the spec does not actually compile *anything*, and there's no Makefile.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1411053] Review Request: nodejs-jschardet - Character encoding auto-detection in JavaScript ( port of python's chardet)

2017-02-21 Thread bugzilla

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

kaiwang  changed:

   What|Removed |Added

 CC||kaiw...@redhat.com



--- Comment #1 from kaiwang  ---
*This is un-official review of the package.*

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: "LGPL (v2.1 or later)", "Unknown or generated". 10 files have
 unknown license. Detailed output of licensecheck in
 /home/kermit/review/1411053-nodejs-jschardet/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.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 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:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: 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]: 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]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: SourceX is a working URL.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: Spec use

[Bug 1396790] Review Request: gnome-shell-extension-freon - GNOME Shell extension to display system temperature , voltage, and fan speed

2017-02-21 Thread bugzilla

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



--- Comment #6 from terrycloth  ---
Like one or a couple of my other pending review requests, one remaining issue
from fedora-review says something about gschemas, but I don't entirely
understand what it's trying to tell me to do:

Issues:
===
- glib-compile-schemas is run in %postun and %posttrans if package has
  *.gschema.xml files.
  Note: gschema file(s) in gnome-shell-extension-freon
  See:
 
http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#GSettings_Schema

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396790] Review Request: gnome-shell-extension-freon - GNOME Shell extension to display system temperature , voltage, and fan speed

2017-02-21 Thread bugzilla

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

terrycloth  changed:

   What|Removed |Added

Version|24  |25



--- Comment #5 from terrycloth  ---
Updated to upstream version 23.

Spec URL:
https://gitlab.com/terrycloth/packaging-gnome-shell-extension-freon/raw/master/gnome-shell-extension-freon.spec

SRPM URL:
https://andrew.tosk.in/tmp/gnome-shell-extension-freon-23-1.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1419226] Review Request: prelude-correlator

2017-02-21 Thread bugzilla

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



--- Comment #9 from Fedora Update System  ---
prelude-correlator-3.1.0-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-73969f8acd

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1343710] Review Request: chrome-gnome-shell - GNOME Shell integration for Chrome

2017-02-21 Thread bugzilla

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



--- Comment #20 from jeremy9...@gmail.com ---
Hello,

Firefox 52 come really soon (march 7) and maybe Jeremy should take the package,
at least for now, to be in time for Firefox 52 ?

Thanks !

https://wiki.mozilla.org/RapidRelease/Calendar

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423390] Review Request: prewikka

2017-02-21 Thread bugzilla

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |POST
  Flags||fedora-review+



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
Two small issues:
- COPYING is listed twice in %files
- the %description suggests that this is a GUI application. It'd be nice to
reword it somehow to say that this is a backend for a browser interface.

+ package name is OK
+ license is acceptable (GPLv2+)
+ license is specified correctly
+ latest version
+ builds and installs correctly
+ Provides/Requires/BuildRequires appear correct
+ no scriptlets

> I try another approch, it add a new warning in rpmlint but the SPEC is more 
> clean. What do you think ?

Nah, I think this is OK.

Package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1327979] Review Request: python-jupyter-notebook - A web-based notebook environment for interactive computing

2017-02-21 Thread bugzilla

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



--- Comment #16 from Zbigniew Jędrzejewski-Szmek  ---
%description -n python3-%{pypi_name}
should also use %_description.

I'm not quite convinced about the drop of python2 executables. But they can
always be added back, so let's continue on the presumption that python2 and
python3 executables are interchangeable.

Provides: bundled(jquery)
Provides: bundled(jquery-ui)
Provides: bundled(jquery-typeahead)
Provides: bundled(mathjax)
+ a bunch of other js stuff. I don't know if it's worthwhile to unbundle those.
Maybe mathjax?

Some fonts could be unbundled easily:
Requires: fontawesome-fonts
Requires: fontawesome-fonts-web
ln -vfs /usr/share/fonts/fontawesome/fontawesome-webfont.woff
%{python3_sitepackages}/notebook/static/components/font-awesome/fonts/fontawesome-webfont.woff
etc.

loading intersphinx inventory from
http://ipython.org/ipython-doc/dev/objects.inv...
loading intersphinx inventory from
https://nbconvert.readthedocs.io/en/latest/objects.inv...
loading intersphinx inventory from
https://nbformat.readthedocs.io/en/latest/objects.inv...
loading intersphinx inventory from
https://jupyter.readthedocs.io/en/latest/objects.inv...
→ those will fail in koji. It might be worth putting those objects.inv files in
sources and installing them into the cache for the build. But that's a minor
thing, can be fixed up once the package is in.

Looks good otherwise. I'll finish the review when the tornado dep 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1403600] Review Request: YafaRay - A free open-source raytracing render engine

2017-02-21 Thread bugzilla

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



--- Comment #21 from Luya Tshimbalanga  ---
Thank you Simone!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424861] Review Request: python-hupper - Integrated process monitor for developing servers

2017-02-21 Thread bugzilla

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

Kevin Fenzi  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2017-02-21 20:57:59



--- Comment #4 from Kevin Fenzi  ---
Built in 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1219662] Review Request: python-terminado - Terminals served to term.js using Tornado websockets

2017-02-21 Thread bugzilla

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl



--- Comment #16 from Zbigniew Jędrzejewski-Szmek  ---
nosetests → nosetests -v

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425658] New: Review Request: python-lit - Tool for running LLVM test suites

2017-02-21 Thread bugzilla

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

Bug ID: 1425658
   Summary: Review Request: python-lit - Tool for running LLVM
test suites
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: tstel...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://github.com/tstellar/python-lit/blob/master/python-lit.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/tstellar/llvm/fedora-25-x86_64/00515665-python-lit/python-lit-0.5.0-0.fc25.src.rpm
Description: This tools is part of the LLVM source tree, but is distributed as
a python module as well: https://pypi.python.org/pypi/lit  The LLVM community
recommends installing this tool using the python package, rather than
installing it from the LLVM tree: https://bugs.llvm.org//show_bug.cgi?id=8496
Fedora Account System Username: tstellar

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1123579] Review Request: rubygem-colorize - Add color methods to String class

2017-02-21 Thread bugzilla

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



--- Comment #28 from Fedora Update System  ---
rubygem-colorize-0.8.1-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-5673253c83

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1123579] Review Request: rubygem-colorize - Add color methods to String class

2017-02-21 Thread bugzilla

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



--- Comment #27 from Fedora Update System  ---
rubygem-colorize-0.8.1-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-c08a5b1590

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1123579] Review Request: rubygem-colorize - Add color methods to String class

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
python-iso3166-0.8-1.fc24 python-streamlink-0.3.2-1.fc24 has been submitted as
an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-1b050a4daf

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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



--- Comment #4 from Fedora Update System  ---
python-iso3166-0.8-1.fc25 python-streamlink-0.3.2-1.fc25 has been submitted as
an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-c32857ef03

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424634] Review Request: python-iso-639 - ISO 639 library for Python

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
python-iso-639-0.4.5-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-7f0c847df9

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1421858] Review Request: flannel-container - system container for flannel

2017-02-21 Thread bugzilla

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

container-rev...@lists.fedoraproject.org changed:

   What|Removed |Added

 CC||container-rev...@lists.fedo
   ||raproject.org



--- Comment #4 from container-rev...@lists.fedoraproject.org ---
Sponsored.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1421856] Review Request: etcd-container - system container for etcd

2017-02-21 Thread bugzilla

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

container-rev...@lists.fedoraproject.org changed:

   What|Removed |Added

 CC||container-rev...@lists.fedo
   ||raproject.org



--- Comment #5 from container-rev...@lists.fedoraproject.org ---
Sponsored.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423377] Review Request: prelude-lml-rules

2017-02-21 Thread bugzilla

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



--- Comment #4 from Fedora Update System  ---
prelude-lml-rules-3.1.0-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-b1e2ea3453

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423377] Review Request: prelude-lml-rules

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
prelude-lml-rules-3.1.0-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-6bd5b4f901

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423377] Review Request: prelude-lml-rules

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423377] Review Request: prelude-lml-rules

2017-02-21 Thread bugzilla

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



--- Comment #3 from Fedora Update System  ---
prelude-lml-rules-3.1.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-fbf6d92b64

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1419226] Review Request: prelude-correlator

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1419226] Review Request: prelude-correlator

2017-02-21 Thread bugzilla

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



--- Comment #8 from Fedora Update System  ---
prelude-correlator-3.1.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-79dc3a256d

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424859] Review Request: nodejs-encodeurl - Encode a URL to a percent-encoded form

2017-02-21 Thread bugzilla

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



--- Comment #4 from Piotr Popieluch  ---
I'm sorry I just found that
%{?nodejs_find_provides_and_requires}
is still needed on EL6.

I thought it was already automated with the last update of the macros.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423390] Review Request: prewikka

2017-02-21 Thread bugzilla

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



--- Comment #2 from Thomas Andrejak  ---
New files : 
SPEC : https://fedorapeople.org/~totol/prewikka.spec
SRPM : https://fedorapeople.org/~totol/prewikka-3.1.0-1.fc26.src.rpm

(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)
> fedora-review complains:
> [ ]: Package requires other packages for directories it uses.
>  Note: No known owner of /usr/lib/python2.7/site-
>  packages/prewikka/locale/de, /usr/lib/python2.7/site-
>  packages/prewikka/locale/pl/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/fr/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/it/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/pt_BR/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/de/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/fr, /usr/lib/python2.7/site-
>  packages/prewikka/locale/es/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/en, /usr/lib/python2.7/site-
>  packages/prewikka/locale, /usr/lib/python2.7/site-
>  packages/prewikka/locale/es, /usr/lib/python2.7/site-
>  packages/prewikka/locale/ru, /usr/lib/python2.7/site-
>  packages/prewikka/locale/pt_BR, /usr/lib/python2.7/site-
>  packages/prewikka/locale/ru/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka, /usr/lib/python2.7/site-
>  packages/prewikka/locale/it, /usr/lib/python2.7/site-
>  packages/prewikka/locale/en/LC_MESSAGES, /usr/lib/python2.7/site-
>  packages/prewikka/locale/pl
> This should be covered by -f %{name}.lang, looks like this doesn't work for
> some reason.

I try another approch, it add a new warning in rpmlint but the SPEC is more
clean. What do you think ?

> 
> I don't think you need to provide the versioned symlinks: they are only used
> when two versions of an application can be installed that behave
> *differently* for python2 and python3. Here, if python3 support is added in
> the future, there will be no difference in behaviour when running under
> python2 and python3, IIUC.

Done

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1333529] Review Request: opa-fm - OPA Fabric Manager

2017-02-21 Thread bugzilla

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



--- Comment #8 from Scott Breyer  ---
We've made changes to our spec file that should be in line with what you are
asking. The patch is presently making its way through our internal review and
test process.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1333531] Review Request: opa-ff - OPA Fast Fabric Tools

2017-02-21 Thread bugzilla

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



--- Comment #10 from Scott Breyer  ---
We've made changes to our spec file that should be in line with what you are
asking. The patch is presently making its way through our internal review and
test process.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424721] Review Request: php-maennchen-zipstream-php - A fast and simple streaming zip file downloader for PHP

2017-02-21 Thread bugzilla

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-maennchen-zipstream-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424851] Review Request: xplayer-plparser - Xplayer playlist parser

2017-02-21 Thread bugzilla

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



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/xplayer-plparser

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1423390] Review Request: prewikka

2017-02-21 Thread bugzilla

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



--- Comment #1 from Zbigniew Jędrzejewski-Szmek  ---
fedora-review complains:
[ ]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/lib/python2.7/site-
 packages/prewikka/locale/de, /usr/lib/python2.7/site-
 packages/prewikka/locale/pl/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/fr/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/it/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/pt_BR/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/de/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/fr, /usr/lib/python2.7/site-
 packages/prewikka/locale/es/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/en, /usr/lib/python2.7/site-
 packages/prewikka/locale, /usr/lib/python2.7/site-
 packages/prewikka/locale/es, /usr/lib/python2.7/site-
 packages/prewikka/locale/ru, /usr/lib/python2.7/site-
 packages/prewikka/locale/pt_BR, /usr/lib/python2.7/site-
 packages/prewikka/locale/ru/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka, /usr/lib/python2.7/site-
 packages/prewikka/locale/it, /usr/lib/python2.7/site-
 packages/prewikka/locale/en/LC_MESSAGES, /usr/lib/python2.7/site-
 packages/prewikka/locale/pl
This should be covered by -f %{name}.lang, looks like this doesn't work for
some reason.

I don't think you need to provide the versioned symlinks: they are only used
when two versions of an application can be installed that behave *differently*
for python2 and python3. Here, if python3 support is added in the future, there
will be no difference in behaviour when running under python2 and python3,
IIUC.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1415331] Review Request: enjarify - Translate Dalvik bytecode to equivalent Java bytecode

2017-02-21 Thread bugzilla

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



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
Raphael?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424859] Review Request: nodejs-encodeurl - Encode a URL to a percent-encoded form

2017-02-21 Thread bugzilla

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



--- Comment #3 from Piotr Popieluch  ---
Changes look fine. Only seems that the src.rpm is not uploaded. If you upload
it I can finish the fedora-review template and approve.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1333529] Review Request: opa-fm - OPA Fabric Manager

2017-02-21 Thread bugzilla

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



--- Comment #7 from Scott Breyer  ---
Hello Michal,

Regarding the guidelines for systemd, our subject matter expert is confident
after review that we are in compliance with the Fedora Packaging Guidelines.
Can you point to something specific that you believe is non-compliant?

Thanks,
Scott

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424788] Review Request: nodejs-safe-buffer - Node.js module for a safer buffer API

2017-02-21 Thread bugzilla

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

Piotr Popieluch  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2017-02-21 15:13:30



--- Comment #5 from Piotr Popieluch  ---
in 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424788] Review Request: nodejs-safe-buffer - Node.js module for a safer buffer API

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424788] Review Request: nodejs-safe-buffer - Node.js module for a safer buffer API

2017-02-21 Thread bugzilla

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



--- Comment #4 from Fedora Update System  ---
nodejs-safe-buffer-5.0.1-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-da67d96119

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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



--- Comment #39 from James Hogarth  ---
Oversight, I'll review on the next build.

I plan to add a migration document this weekend and I'll get it all sorted
before anything goes stable.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1219662] Review Request: python-terminado - Terminals served to term.js using Tornado websockets

2017-02-21 Thread bugzilla

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

Miro Hrončok  changed:

   What|Removed |Added

 Blocks||1327979




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1327979
[Bug 1327979] Review Request: python-jupyter-notebook - A web-based
notebook environment for interactive computing
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1327979] Review Request: python-jupyter-notebook - A web-based notebook environment for interactive computing

2017-02-21 Thread bugzilla

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

Miro Hrončok  changed:

   What|Removed |Added

 Depends On||1219662




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1219662
[Bug 1219662] Review Request: python-terminado - Terminals served to
term.js using Tornado websockets
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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

Ralf Corsepius  changed:

   What|Removed |Added

 CC||rc040...@freenet.de



--- Comment #38 from Ralf Corsepius  ---
James, 

/usr/share/doc/nextcloud/README.fedora contains many reference to ownCloud rsp.
owncloud. Is this intentional or an oversight?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1421245] Review Request: libcrush - C library to control placement in a hierarchy

2017-02-21 Thread bugzilla

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



--- Comment #10 from Loic Dachary  ---
Spec URL:
https://build.opensuse.org/package/view_file/home:ldachary/libcrush/libcrush.spec
SRPM URL:
http://download.opensuse.org/repositories/home:/ldachary/Fedora_25/src/libcrush-1.0.0-23.1.src.rpm

It has the changes mentionned above except two:

* %cmake is not used and the .spec explains why
* the libdir definition for 64-bit targets is unmodified because I believe it
is correct

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1411201] Review Request: js-jquery-file-upload - File Upload widget for jQuery

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
js-jquery-file-upload-9.17.0-1.fc25 has been submitted as an update to Fedora
25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-19f210f5aa

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1411201] Review Request: js-jquery-file-upload - File Upload widget for jQuery

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1411201] Review Request: js-jquery-file-upload - File Upload widget for jQuery

2017-02-21 Thread bugzilla

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



--- Comment #6 from Fedora Update System  ---
js-jquery-file-upload-9.17.0-1.fc24 has been submitted as an update to Fedora
24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-828ccca3cf

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1411201] Review Request: js-jquery-file-upload - File Upload widget for jQuery

2017-02-21 Thread bugzilla

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

Randy Barlow  changed:

   What|Removed |Added

   Fixed In Version||js-jquery-file-upload-9.17.
   ||0-1.fc26



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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



--- Comment #2 from Mohamed El Morabity  ---
Hello Athos,

many thanks for your review!

(In reply to Athos Ribeiro from comment #1)
> For python3 packages, it os not a good idea to have
> %{python3_sitelib}/*
> because you may end up owning directories that you should not own.
> This is not the case for your package though, but I suggest you be more
> specific there since the package may change.
I just followed the spec template on the Python packaging guidelines page. But
you're right anyway. I'll be careful for further updates, I promise ;)

> Is there any reason for not running the test suite for the package (upstream
> does provide one)? If not, it would be nice to run the test suite in %check.
Tests are not available in the Pypi tarball. They're available on the Github
upstream repo. Unfortunately, there's no release tag published on this repo.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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

Athos Ribeiro  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Athos Ribeiro  ---
Hello Mohamed,

For python3 packages, it os not a good idea to have
%{python3_sitelib}/*
because you may end up owning directories that you should not own.
This is not the case for your package though, but I suggest you be more
specific there since the package may change.

Is there any reason for not running the test suite for the package (upstream
does provide one)? If not, it would be nice to run the test suite in %check.

Package looks good. Approved.

Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
= 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.
[x]: License file installed when any subpackage combination is installed.
[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.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
[x]: Package complies to the Packaging Guidelines
[x]: Package 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: No rpmlint messages.
[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 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:
[-]: 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.
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[?]: Package should compile and build into binary rpms on all supported
 architectures.
[!]: %check is

[Bug 1421856] Review Request: etcd-container - system container for etcd

2017-02-21 Thread bugzilla

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



--- Comment #4 from Yu Qi Zhang  ---
Fixed in:
https://github.com/projectatomic/atomic-system-containers/commit/759decde35f539e743689020e251f9a8fcc5eb07

I have talked to Adam Miller about sponsoring. As for maintainer, technically
projectatomic as an organization will be the maintainer. I've talked to
Giuseppe and he's ok with being the listed maintainer for 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1422429] Review Request: python-junit_xml - python library to create junit compatible XML files

2017-02-21 Thread bugzilla

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



--- Comment #7 from Fedora Update System  ---
python-junit_xml-1.7-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-7b28127542

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424632] Review Request: python-iso3166 - Self-contained ISO 3166-1 country definitions

2017-02-21 Thread bugzilla

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

Athos Ribeiro  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||athoscribe...@gmail.com
   Assignee|nob...@fedoraproject.org|athoscribe...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1422429] Review Request: python-junit_xml - python library to create junit compatible XML files

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1422429] Review Request: python-junit_xml - python library to create junit compatible XML files

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
python-junit_xml-1.7-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-efff7df77e

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424851] Review Request: xplayer-plparser - Xplayer playlist parser

2017-02-21 Thread bugzilla

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

Björn "besser82" Esser  changed:

   What|Removed |Added

 Status|NEW |POST



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1422429] Review Request: python-junit_xml - python library to create junit compatible XML files

2017-02-21 Thread bugzilla

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



--- Comment #6 from Fedora Update System  ---
python-junit_xml-1.7-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-21abde9e2f

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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



--- Comment #37 from Fedora Update System  ---
nextcloud-10.0.3-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-c6a69d64d5

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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



--- Comment #36 from Fedora Update System  ---
nextcloud-10.0.3-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-5a6a9ae10c

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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



--- Comment #35 from Fedora Update System  ---
nextcloud-10.0.3-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-f83252de8c

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1409869] Review Request: perl-X10 - X10 perl module

2017-02-21 Thread bugzilla

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



--- Comment #12 from Fedora Update System  ---
perl-X10-0.04-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-650714234c

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1409869] Review Request: perl-X10 - X10 perl module

2017-02-21 Thread bugzilla

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



--- Comment #11 from Fedora Update System  ---
perl-X10-0.04-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-01dc0f6e3a

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1409869] Review Request: perl-X10 - X10 perl module

2017-02-21 Thread bugzilla

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



--- Comment #10 from Fedora Update System  ---
perl-X10-0.04-2.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-ad2ac172cd

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1409869] Review Request: perl-X10 - X10 perl module

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1409869] Review Request: perl-X10 - X10 perl module

2017-02-21 Thread bugzilla

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



--- Comment #9 from Fedora Update System  ---
perl-X10-0.04-2.el6 has been submitted as an update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-562ea1cb4d

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424851] Review Request: xplayer-plparser - Xplayer playlist parser

2017-02-21 Thread bugzilla

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

leigh scott  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|leigh123li...@googlemail.co
   ||m
  Flags||fedora-review+



--- Comment #3 from leigh scott  ---
Approved

Please fix ownership for (when you import)

%doc %{_datadir}/gtk-doc

change it to

%doc %{_datadir}/gtk-doc/html/%{name}



Package Review
==

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



= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.

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: "LGPL", "GPL (v2 or later)", "Unknown or generated", "MIT/X11
 (BSD like)", "LGPL (v2 or later)", "FSF All Permissive", "GPL (v2)".
 194 files have unknown license. Detailed output of licensecheck in
 /home/leigh/Desktop/1424851-xplayer-plparser/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/lib64/girepository-1.0,
 /usr/share/gir-1.0
[!]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/share/gtk-doc(gnome-
 desktop3-devel, libgnome-keyring-devel, libsecret-devel, gcr-devel,
 polkit-docs, p11-kit-devel, libpeas-devel, harfbuzz-devel,
 libcanberra-devel, json-glib-devel, clutter-gst3-devel, libgdata-
 devel, gnome-bluetooth-libs-devel, gtk-doc), /usr/share/gtk-doc/html
 (gnome-desktop3-devel, libgnome-keyring-devel, libsecret-devel, gcr-
 devel, polkit-docs, p11-kit-devel, libpeas-devel, harfbuzz-devel,
 libcanberra-devel, json-glib-devel, clutter-gst3-devel, libgdata-
 devel, gnome-bluetooth-libs-devel, gtk-doc)
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: The spec file handles locales properly.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 163840 bytes in 4 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]: 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]:

[Bug 1420090] Review Request: marble-subsurface - Marble Subsurface branch

2017-02-21 Thread bugzilla

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

Rex Dieter  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2017-02-21 10:52:56



--- Comment #7 from Rex Dieter  ---
imported, thanks!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1422555] Review Request: arduino-ctags - A mix of ctags and anjuta-tags for the perfect C++ ctags

2017-02-21 Thread bugzilla

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

Randy Barlow  changed:

   What|Removed |Added

 CC||gia...@gmail.com
  Flags||needinfo?(gia...@gmail.com)



--- Comment #1 from Randy Barlow  ---
There are a few things we must fix before this package can be approved:

[!]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/arduino/tools-
 builder/ctags/5.8arduino11, /usr/share/arduino, /usr/share/arduino
 /tools-builder/ctags, /usr/share/arduino/tools-builder

 Does it make sense for this package to own tools-builder? If
 not, should we request that arduino-core provide it?


[!]: Development files must be in a -devel package

 You should put the header files into a devel package.


[!]: Package obeys FHS, except libexecdir and /usr/target.

 The shared lib is arch-dependent, and so should go into
 %{_libdir}. Is make install not suffiencent for this package?


One optional suggestion from rpmlint:

arduino-ctags.x86_64: E: incorrect-fsf-address
/usr/share/licenses/arduino-ctags/COPYING

You could work with upstream to correct the fsf address in the COPYING file.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425190] Review Request: switchboard-plug-bluetooth - Switchboard Bluetooth plug

2017-02-21 Thread bugzilla

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



--- Comment #7 from Fedora Update System  ---
switchboard-plug-bluetooth-0.1-1.fc25 has been submitted as an update to Fedora
25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-ccab2c0578

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425190] Review Request: switchboard-plug-bluetooth - Switchboard Bluetooth plug

2017-02-21 Thread bugzilla

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

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424851] Review Request: xplayer-plparser - Xplayer playlist parser

2017-02-21 Thread bugzilla

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



--- Comment #2 from Björn "besser82" Esser  ---
(In reply to leigh scott from comment #1)
> .typelib should be moved from -devel to main
> 
> /usr/lib64/girepository-1.0/XplayerPlParser-1.0.typelib
> 
> 
> .gir should be moved from main to -devel
> 
> /usr/share/gir-1.0/XplayerPlParser-1.0.gir

Fixed in updated package.


> These can be removed as gnome-common requires them
> 
> BuildRequires:autoconf
> BuildRequires:automake
> BuildRequires:gettext

Explicit BRs don't hurt, I think…


> Please change this from
> 
> NOCONFIGURE=1 %{_bindir}/gnome-autogen.sh
> 
> 
> to (the commands aren't equal and miss a required macro).
> 
> NOCONFIGURE=1 ./autogen.sh

Well, it doesn't change the result of the build.  Using ./autogen.sh is just
needed for out-of-tree builds…

***

Updated package:

Spec URL: 
https://pagure.io/besser82/package-review/raw/master/f/xplayer-plparser.spec
  SRPM URL: 
https://pagure.io/besser82/package-review/raw/master/f/xplayer-plparser-1.0.2-0.2.fc26.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424634] Review Request: python-iso-639 - ISO 639 library for Python

2017-02-21 Thread bugzilla

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



--- Comment #3 from Fedora Update System  ---
python-iso-639-0.4.5-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-7f0c847df9

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424634] Review Request: python-iso-639 - ISO 639 library for Python

2017-02-21 Thread bugzilla

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



--- Comment #4 from Fedora Update System  ---
python-iso-639-0.4.5-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-7251854b8e

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424798] Review Request: xed - X-Apps [Text] Editor (Cross-DE, backward-compatible, GTK3, traditional UI)

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424798] Review Request: xed - X-Apps [Text] Editor (Cross-DE, backward-compatible, GTK3, traditional UI)

2017-02-21 Thread bugzilla

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



--- Comment #11 from Fedora Update System  ---
xed-1.2.2-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-3149a3b8d0

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1421245] Review Request: libcrush - C library to control placement in a hierarchy

2017-02-21 Thread bugzilla

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



--- Comment #9 from Loic Dachary  ---
> Wrong libdir definition for 64-bit targets.

I don't understand, the spec file contains no %{_libdir} definition, how can it
be wrong ? The zlib package also installs in %{_libdir}.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424798] Review Request: xed - X-Apps [Text] Editor (Cross-DE, backward-compatible, GTK3, traditional UI)

2017-02-21 Thread bugzilla

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



--- Comment #12 from Fedora Update System  ---
xed-1.2.2-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-72f325ca2a

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1422555] Review Request: arduino-ctags - A mix of ctags and anjuta-tags for the perfect C++ ctags

2017-02-21 Thread bugzilla

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

Randy Barlow  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ra...@electronsweatshop.com
   Assignee|nob...@fedoraproject.org|ra...@electronsweatshop.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425203] Review Request: python-speedtest-cli - Command line interface for testing internet bandwidth using speedtest.net

2017-02-21 Thread bugzilla

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



--- Comment #7 from Fedora Update System  ---
python-speedtest-cli-1.0.2-1.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-f8ab429144

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425203] Review Request: python-speedtest-cli - Command line interface for testing internet bandwidth using speedtest.net

2017-02-21 Thread bugzilla

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



--- Comment #6 from Fedora Update System  ---
python-speedtest-cli-1.0.2-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-dceb257004

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425203] Review Request: python-speedtest-cli - Command line interface for testing internet bandwidth using speedtest.net

2017-02-21 Thread bugzilla

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



--- Comment #4 from Fedora Update System  ---
python-speedtest-cli-1.0.2-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-8b9ca7bb8b

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425203] Review Request: python-speedtest-cli - Command line interface for testing internet bandwidth using speedtest.net

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424772] Review Request: bluez-tools - A set of tools to manage Bluetooth devices for Linux

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
bluez-tools-0.2.0-0.2.fc24 blueberry-1.1.10-3.fc24 cinnamon-3.2.8-14.fc24 has
been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-ba34aa36e1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424772] Review Request: bluez-tools - A set of tools to manage Bluetooth devices for Linux

2017-02-21 Thread bugzilla

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



--- Comment #4 from Fedora Update System  ---
bluez-tools-0.2.0-0.2.fc25 blueberry-1.1.10-3.fc25 cinnamon-3.2.8-14.fc25 has
been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-7b0d9bce92

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424772] Review Request: bluez-tools - A set of tools to manage Bluetooth devices for Linux

2017-02-21 Thread bugzilla

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

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425203] Review Request: python-speedtest-cli - Command line interface for testing internet bandwidth using speedtest.net

2017-02-21 Thread bugzilla

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



--- Comment #5 from Fedora Update System  ---
python-speedtest-cli-1.0.2-1.el6 has been submitted as an update to Fedora EPEL
6. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-2a78f2e493

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1424851] Review Request: xplayer-plparser - Xplayer playlist parser

2017-02-21 Thread bugzilla

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

leigh scott  changed:

   What|Removed |Added

 CC||leigh123li...@googlemail.co
   ||m



--- Comment #1 from leigh scott  ---
.typelib should be moved from -devel to main

/usr/lib64/girepository-1.0/XplayerPlParser-1.0.typelib


.gir should be moved from main to -devel

/usr/share/gir-1.0/XplayerPlParser-1.0.gir


These can be removed as gnome-common requires them

BuildRequires:autoconf
BuildRequires:automake
BuildRequires:gettext


Please change this from

NOCONFIGURE=1 %{_bindir}/gnome-autogen.sh


to (the commands aren't equal and miss a required macro).

NOCONFIGURE=1 ./autogen.sh

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1413978] Review Request: php-sabre-vobject4 - Library to parse and manipulate iCalendar and vCard objects

2017-02-21 Thread bugzilla

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

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1413978] Review Request: php-sabre-vobject4 - Library to parse and manipulate iCalendar and vCard objects

2017-02-21 Thread bugzilla

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



--- Comment #10 from Fedora Update System  ---
php-sabre-vobject4-4.1.2-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-9637bc13e0

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1413978] Review Request: php-sabre-vobject4 - Library to parse and manipulate iCalendar and vCard objects

2017-02-21 Thread bugzilla

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



--- Comment #9 from Fedora Update System  ---
php-sabre-vobject4-4.1.2-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-1ed5917b02

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425190] Review Request: switchboard-plug-bluetooth - Switchboard Bluetooth plug

2017-02-21 Thread bugzilla

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/switchboard-plug-bluetooth

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1360482] Review Request: nextcloud - a private, secure way to share and access files

2017-02-21 Thread bugzilla

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



--- Comment #34 from Didier  ---
James, thank you for the continued effort.

(Nc 10.0.3 EL7 runs very well)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1425203] Review Request: python-speedtest-cli - Command line interface for testing internet bandwidth using speedtest.net

2017-02-21 Thread bugzilla

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



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1411201] Review Request: js-jquery-file-upload - File Upload widget for jQuery

2017-02-21 Thread bugzilla

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



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/js-jquery-file-upload

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

2017-02-21 Thread bugzilla

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



--- Comment #6 from James Hogarth  ---
Small note this has been updated to owncloud 9.1.4 now that it has hit the
stable repos.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

1 2 >

1 - 100 of 134 matches

Mail list logo