[Bug 1357724] Review Request: qcint - general GTO integrals for quantum chemistry

[bugzilla]

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



--- Comment #16 from Susi Lehtola  ---
(In reply to Dave Love from comment #15)
> (In reply to Susi Lehtola from comment #14)
> > Yeah, so the default ATLAS package on x86_64 is non-SSE3, but there's a SSE3
> > variant available, as well.
> 
> Yes, and it's the ld.so.conf mechanism I think is relevant.

Yeah, I'd just rather avoid the confusion of being able to install both
simultaneously and then have to hack the alternatives configuration to figure
out which one is actually in use.

> > Packaging both within the same srpm is not allowed per the packaging
> > guidelines, as there needs to be a spec per tarball.
> 
> I don't see where it says that, though you can only have one URL.

I am 99.9% certain this is a rule, i.e., that you can't bundle different
projects together into one spec file. And this is exactly the case here: we
have the original version and a fork of it, that's SSE3-only.

> > I believe the ABIs are shared, though.
> 
> Apologies, I was confusing libcint and libint, which already has two
> versions.
> [If they're from the same place, I'd have thought it would be more of an
> argument for bundling source with libcint, but I'm not reviewing it.]

No, libcint and libint are totally different projects. They have similar
purpose, though.

> Obviously I should stick to pointing out the way hardware-specific versions
> have been handled.  That's relevant for those of us who have to support
> such things.  The way ATLAS does it at least can support heterogeneous HPC
> systems using a single image on stateless nodes, where the image system
> can make node-dependent mods to the configuration files.  [Not that you
> should use ATLAS on x86, given OpenBLAS, and you could perhaps handle that 
> HPC case
> differently.]

Well, that's true. But most compute clusters have local operating systems, and
it's trivial to pick qcint or libcint on a node basis.

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

[Bug 1079064] Review Request: btbuilder - Role-playing game construction set in the style of the Bard' s Tale Construction Set

[bugzilla]

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

Ben Rosser  changed:

   What|Removed |Added

   Assignee|ti...@math.uh.edu   |rosser@gmail.com



--- Comment #35 from Ben Rosser  ---
Taking; I'll review this one, too, once you've imported and built SDL_mng for
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1227972] Review Request: SDL_mng - Library to load MNG files for SDL

[bugzilla]

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

Ben Rosser  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #13 from Ben Rosser  ---
Great, package is APPROVED!

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: "MIT/X11 (BSD like)", "LGPL (v2.1 or later)", "GPL (v2 or
 later)", "Unknown or generated". 6 files have unknown license.
 Detailed output of licensecheck in
 /home/bjr/Programming/fedora/reviews/1227972-SDL_mng/licensecheck.txt

Note for review purposes: this is fine, the non-LGPL files are scripts
that aren't compiled or installed.

[x]: License file installed when any subpackage combination is installed.
[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]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: 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]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in SDL_mng-
 debuginfo
[x]: Package functions as 

[Bug 1367598] Review Request: gap-pkg-guava - Computing with error-correcting codes

[bugzilla]

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

Paulo Andrade  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Paulo Andrade  ---
Looks like the parallel command is broken in rawhide:
$ head -1 /usr/bin/parallel 
##!/usr/bin/perl
note the two '#'.
Fixing it manually and building the package I do not see
any other issue.
You might want to either check what is broken with the
parallel package, or not use it... See the sed command in
http://pkgs.fedoraproject.org/cgit/rpms/parallel.git/commit/?id=b941a86552fa42ca018a58bffc3ba35c428b0d07
(I just fedpkg co'd parallel and it is indeed broken).

  Otherwise, the package is now approved. All the issues
have been fixed!

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

[Bug 1227972] Review Request: SDL_mng - Library to load MNG files for SDL

[bugzilla]

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



--- Comment #12 from Dennis Payne  ---
Spec URL: https://raw.githubusercontent.com/dulsi/SDL_mng/master/SDL_mng.spec
SRPM URL: http://identicalsoftware.com/btbuilder/SDL_mng-0.2.6-2.fc24.src.rpm

Fixed the unused shlib dependency.

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

[Bug 1227972] Review Request: SDL_mng - Library to load MNG files for SDL

[bugzilla]

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

Ben Rosser  changed:

   What|Removed |Added

 CC||rosser@gmail.com
   Assignee|ti...@math.uh.edu   |rosser@gmail.com



--- Comment #11 from Ben Rosser  ---
Sorry this (and btbuilder) sat in the queue for so long. I noticed it when
compiling the games tracker bug, and it looked easy enough to review... so here
I am.

I have one general piece of packaging feedback that isn't actually a blocker:
In the files list, you should be aware that using "%{_libdir}/lib*.so.*" is
potentially risky.

In this case, it may be fine because you appear to be upstream for SDL_mng, but
generally this is bad practice because it means that when you bump the soname
(if you ever bump the soname), you won't notice when rebuilding the package,
which means you may forget to e.g. mail the devel list when the soname is being
bumped.

Anyway, you may already know that but I figured I'd bring it up.

Package Review
==

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


Issues
==

- You have an unused direct shlib dependency, according to rpmlint, on
/usr/lib64/libSDL_mng.so.0.0.0 /lib64/libz.so.1. See the following for
ways you can fix this:
https://fedoraproject.org/wiki/Common_Rpmlint_issues#unused-direct-shlib-dependency

- The following rpmlint output is easy enough to fix (but hardly a review
blocker). :)
SDL_mng-devel.x86_64: W: summary-ended-with-dot C Libraries and includes
for SDL MNG development.

If you fix the direct shlib dependency, I'll approve the package.

= 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: "MIT/X11 (BSD like)", "LGPL (v2.1 or later)", "GPL (v2 or
 later)", "Unknown or generated". 6 files have unknown license.
 Detailed output of licensecheck in
 /home/bjr/Programming/fedora/reviews/1227972-SDL_mng/licensecheck.txt

Note for review purposes: this is fine, the non-LGPL files are scripts
that aren't compiled or installed.

[x]: License file installed when any subpackage combination is installed.
[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]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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



--- Comment #5 from Fedora Update System  ---
python-dict-sorted-1.0.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-444c7206c8

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

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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



--- Comment #4 from Fedora Update System  ---
python-dict-sorted-1.0.0-1.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-5091ba8f2d

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

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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

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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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



--- Comment #3 from Fedora Update System  ---
python-dict-sorted-1.0.0-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-e22bc2ff30

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

[Bug 1370644] New: Review Request: python-django-picklefield - A pickled object field for Django

[bugzilla]

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

Bug ID: 1370644
   Summary: Review Request: python-django-picklefield - A pickled
object field for Django
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-django-picklefield.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-django-picklefield-0.3.2-1.fc26.src.rpm
Description: A pickled object field for Django.
Fedora Account System Username: ignatenkobrain

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

[Bug 1370644] Review Request: python-django-picklefield - A pickled object field for Django

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||python-django-picklefield



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

[Bug 1368790] Review Request: perl-App-PFT - Hacker friendly static blog generator, command line utilities

[bugzilla]

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



--- Comment #10 from Giovanni  ---
Updates: using tangerine for Perl dependencies

SPEC:
http://copr-dist-git.fedorainfracloud.org/cgit/dacav/pft/perl-App-PFT.git/plain/perl-App-PFT.spec
SRPM:
https://copr-be.cloud.fedoraproject.org/results/dacav/pft/fedora-rawhide-x86_64/00446660-perl-App-PFT/perl-App-PFT-1.0.5-4.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1367569] Review Request: perl-PFT - Hacker friendly static blog generator, core library

[bugzilla]

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



--- Comment #9 from Giovanni  ---
Hello Parag

In reply to Comment 7:

1) Done. I found a bit weird the fact that rpmdev-bumpspec is not using the
angular bracket. However I've fixed it manually.

2) I've fixed the README files upstream, however the SPEC file is pointing with
Source0 to the archive of version 1.0.3. In order to enable the new README I
should bump the version of the whole package and probably re-publish on CPAN
too. The same applies to APP::PFT 1.0.5.

3) Thanks for the suggestion. I've used tangerine as suggested, and you will
find the updated SPEC (links follow). Just out of curiosity I've checked with
mock if the RPM was constructed with the original set of `BuildRequires:`, and
it turned out to be feasible nevertheless (the missing `BuildRequires:` were
not installed as transitive dependencies!). I think however it's good to be
conservative and to rely on the tangerine tool.

Finally, I'm currently involved in the review of someone else's package:
https://bugzilla.redhat.com/show_bug.cgi?id=1307271

SPEC:
http://copr-dist-git.fedorainfracloud.org/cgit/dacav/pft/perl-PFT.git/plain/perl-PFT.spec
SRPM:
https://copr-be.cloud.fedoraproject.org/results/dacav/pft/fedora-rawhide-x86_64/00446658-perl-PFT/perl-PFT-1.0.3-4.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370449] Review Request: rtv - A simple terminal viewer for Reddit ( Reddit Terminal Viewer)

[bugzilla]

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



--- Comment #1 from Germano Massullo  ---
Could you please add, into the spec file, a few commented lines about
rtv-1.11.0-mailcap_fix.patch
?
Thank you

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

[Bug 1370621] Review Request: python-django-ipware - Returns user' s real IP address in Django

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||python-django-ipware



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

[Bug 1370621] New: Review Request: python-django-ipware - Returns user' s real IP address in Django

[bugzilla]

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

Bug ID: 1370621
   Summary: Review Request: python-django-ipware - Returns user's
real IP address in Django
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-django-ipware.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-django-ipware-1.1.5-1.fc26.src.rpm
Description: Django utility application that returns client's real IP address.
Fedora Account System Username: ignatenkobrain

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

[Bug 1343814] Review Request: vagrant-sshfs - A Vagrant synced folder plugin that mounts folders via SSHFS.

[bugzilla]

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



--- Comment #20 from Dusty Mabe  ---
(In reply to Vít Ondruch from comment #18)
> (In reply to Dusty Mabe from comment #17)
> Ha, now I really checked what is going on and that it is not exactly your
> package fault to carry the fonts. That is the generated documentation and
> this is reported as bug 1224715.
> 
> In this case, I would recommend to drop the "bundled" notices and carry on,
> since this issue applies to all rubygem- and vagrant- packages :/

Thanks Vit, I will do that

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

[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent with the requests package

[bugzilla]

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



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

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

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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



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

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

[Bug 1355819] Review Request: python-functools32 - Backport of the functools module from Python 3.2 for use on 2.7

[bugzilla]

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



--- Comment #11 from Jon Ciesla  ---
(In reply to Jon Ciesla from comment #8)
> Thanks, I just found that. :)
> 
> Do you think they'll conflict?  If not, I'll package it.

Haïkel, have you had a chance to look at 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1355819] Review Request: python-functools32 - Backport of the functools module from Python 3.2 for use on 2.7

[bugzilla]

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

Haïkel Guémar  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
  Flags|needinfo?(karlthered@gmail. |
   |com)|
Last Closed||2016-08-26 15:05:30



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

[Bug 1268358] Review Request: rubygem-underscore-rails - underscore.js asset pipeline provider/wrapper

[bugzilla]

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



--- Comment #1 from Ilya Gradina  ---
New Spec:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-underscore-rails.spec
New SRPM:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-underscore-rails-1.8.3-2.fc24.src.rpm

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

[Bug 1370611] Review Request: python-serpy - Ridiculously fast object serialization

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||python-serpy



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

[Bug 1370611] New: Review Request: python-serpy - Ridiculously fast object serialization

[bugzilla]

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

Bug ID: 1370611
   Summary: Review Request: python-serpy - Ridiculously fast
object serialization
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/python-serpy.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-serpy-0.1.1-1.fc26.src.rpm
Description:
serpy is a super simple object serialization framework built for speed.
serpy serializes complex datatypes (Django Models, custom classes, ...)
to simple native types (dicts, lists, strings, ...). The native types can
easily be converted to JSON or any other format needed.

The goal of serpy is to be able to do this simply, reliably, and quickly.
Since serializers are class based, they can be combined, extended and
customized with very little code duplication. Compared to other popular
Python serialization frameworks like marshmallow or Django Rest Framework
Serializers serpy is at least an order of magnitude faster.
Fedora Account System Username: ignatenkobrain

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

[Bug 1267458] Review Request: rubygem-faraday_middleware - Various middleware for Faraday

[bugzilla]

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



--- Comment #1 from Ilya Gradina  ---
New Spec:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-faraday_middleware.spec
New SRPM:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-faraday_middleware-0.10.0-2.fc24.src.rpm

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

[Bug 1370607] Review Request: rubygem-rash - simple extension to Hashie:: Mash for rubyified keys

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1267458




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1267458
[Bug 1267458] Review Request: rubygem-faraday_middleware - Various
middleware for Faraday
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1267458] Review Request: rubygem-faraday_middleware - Various middleware for Faraday

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Depends On||1370607




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1370607
[Bug 1370607] Review Request: rubygem-rash - simple extension to
Hashie::Mash for rubyified keys
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370607] New: Review Request: rubygem-rash - simple extension to Hashie::Mash for rubyified keys

[bugzilla]

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

Bug ID: 1370607
   Summary: Review Request: rubygem-rash - simple extension to
Hashie::Mash for rubyified keys
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ilya.grad...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-rash.spec
SRPM URL:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-rash-0.4.0-1.fc24.src.rpm
Description: simple extension to Hashie::Mash for rubyified keys, all keys are
converted to
underscore to eliminate horrible camelCasing.
Fedora Account System Username: ilgrad

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

[Bug 1268360] Review Request: rubygem-simple_oauth - Simply builds and verifies OAuth headers

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Blocks||1267458




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1267458
[Bug 1267458] Review Request: rubygem-faraday_middleware - Various
middleware for Faraday
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1267458] Review Request: rubygem-faraday_middleware - Various middleware for Faraday

[bugzilla]

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

Ilya Gradina  changed:

   What|Removed |Added

 Depends On||1268360




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1268360
[Bug 1268360] Review Request: rubygem-simple_oauth - Simply builds and
verifies OAuth headers
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1267448] Review Request: rubygem-email_validator - An email validator for Rails 3+

[bugzilla]

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



--- Comment #1 from Ilya Gradina  ---
New Spec:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-email_validator.spec
New SRPM:
https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-email_validator-1.6.0-2.fc24.src.rpm

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

[Bug 1264546] Review Request: soletta - A framework for making IoT devices

[bugzilla]

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



--- Comment #62 from Gustavo Lima Chaves  ---
New spec URL:
https://raw.githubusercontent.com/solettaproject/soletta-packaging/v11/rpm/soletta.spec

New SRPM URL:
https://github.com/solettaproject/soletta-packaging/releases/download/v11/soletta-1-1.fc24.src.rpm

koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=15389537

Hi, Paulo. Finally, we have come to this -- v1 both on our side and on the RPM
spec. Sorry for taking longer than needed, but a lot of development took place
in order for this to happen. We're now a stable API, at least, and hope to
benefit many Fedora users with the ease of packaging.

There are close to 0 rpmlint complaints, I hope we're good now. The
libmicrohttpd issue has been solved by pushing its usage to future releases,
when the lib version in the system has been bumped.

Let me know if there are any issues left.

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

[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent with the requests package

[bugzilla]

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

Fabio Alessandro Locati  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Fabio Alessandro Locati  ---
Package is APPROVED

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: "BSD (3 clause)", "MIT/X11 (BSD like)", "Unknown or generated".
 18 files have unknown license. Detailed output of licensecheck in
 /tmp/1370473-python-fauxquests/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[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.
[x]: 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]: 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 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

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.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python2-fauxquests , python3-fauxquests
[x]: Package functions as described.
[-]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is 

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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

Fabio Alessandro Locati  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Fabio Alessandro Locati  ---
This package is APPROVED.

To Do:

- Add email address to changelog

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: "BSD (3 clause)", "Unknown or generated". 12 files have unknown
 license. Detailed output of licensecheck in /tmp/1370469-python-dict-
 sorted/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[ ]: 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.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

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

= SHOULD items =

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

[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning

[bugzilla]

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



--- Comment #82 from Fedora Update System  ---
rdkit-2016.03.2-6.fc24 has been pushed to the Fedora 24 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-0a6d8b86c1

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

[Bug 1366881] Review Request: ispc - C-based SPMD programming language compiler

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #24 from Fedora Update System  ---
ispc-1.9.1-7.fc24 has been pushed to the Fedora 24 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-01fee06612

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

[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning

[bugzilla]

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



--- Comment #81 from Fedora Update System  ---
rdkit-2016.03.2-6.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-1903e53093

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

[Bug 1334611] Review Request: python-cvss - CVSS2/ 3 library with interactive calculator

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1334611] Review Request: python-cvss - CVSS2/ 3 library with interactive calculator

[bugzilla]

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



--- Comment #18 from Fedora Update System  ---
python-cvss-1.3-2.el6 has been submitted as an update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-c8d5d75f85

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

[Bug 1334611] Review Request: python-cvss - CVSS2/ 3 library with interactive calculator

[bugzilla]

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



--- Comment #19 from Fedora Update System  ---
python-cvss-1.3-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-8a2d144b6e

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

[Bug 1250025] Review Request: rocksdb - A Persistent Key-Value Store for Flash and RAM Storage

[bugzilla]

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



--- Comment #2 from Matej Mužila  ---
I've updated the package

Spec URL: https://mmuzila.fedorapeople.org/rocksdb.spec
SRPM URL: https://mmuzila.fedorapeople.org/rocksdb-4.11-1.fc22.src.rpm

Description: 
Rocksdb is a library that forms the core building block for a fast key value 
server, especially suited for storing data on flash drives. It has a
Log-Structured-Merge-Database (LSM) design with flexible tradeoffs between 
Write-Amplification-Factor (WAF), Read-Amplification-Factor (RAF) and 
Space-Amplification-Factor (SAF). It has multi-threaded compactions, making it
specially suitable for storing multiple terabytes of data in a single database.

Fedora Account System Username: mmuzila

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

[Bug 1369224] Review Request: jackson-modules-base - Jackson modules: Base

[bugzilla]

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



--- Comment #5 from gil cattaneo  ---
Spec URL: https://gil.fedorapeople.org/jackson-modules-base.spec
SRPM URL:
https://gil.fedorapeople.org/jackson-modules-base-2.7.6-1.fc24.src.rpm

Submit https://github.com/FasterXML/jackson-modules-base/pull/19
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=15389344

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

[Bug 1368611] Review Request: typetype-molot-fonts - Display sans-serif fonts

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+
   |needinfo?(puntogil@libero.i |
   |t)  |



--- Comment #12 from gil cattaneo  ---
(In reply to Luya Tshimbalanga from comment #11)
> Would you like continue reviewing the spec file as the requested update is
> done?

Approved

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

[Bug 1364618] Review Request: embree - Collection of high-performance ray tracing kernels developed at Intel

[bugzilla]

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



--- Comment #24 from Fedora Update System  ---
embree-2.10.0-7.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-208750d7bc

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

[Bug 1364618] Review Request: embree - Collection of high-performance ray tracing kernels developed at Intel

[bugzilla]

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

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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1364618] Review Request: embree - Collection of high-performance ray tracing kernels developed at Intel

[bugzilla]

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



--- Comment #23 from Fedora Update System  ---
embree-2.10.0-7.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-dfb58eee59

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

[Bug 1368611] Review Request: typetype-molot-fonts - Display sans-serif fonts

[bugzilla]

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

Luya Tshimbalanga  changed:

   What|Removed |Added

  Flags|needinfo?(luya@fedoraprojec |needinfo?(puntogil@libero.i
   |t.org)  |t)



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

[Bug 1368611] Review Request: typetype-molot-fonts - Display sans-serif fonts

[bugzilla]

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

Luya Tshimbalanga  changed:

   What|Removed |Added

 CC||l...@fedoraproject.org
  Flags||needinfo?(luya@fedoraprojec
   ||t.org)



--- Comment #11 from Luya Tshimbalanga  ---
Would you like continue reviewing the spec file as the requested update is
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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1154218] Review Request: graphite-api - Graphite-web, without the interface. Just the rendering HTTP API

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2015-10-23 13:20:56 |2016-08-26 11:49:20



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

[Bug 1154218] Review Request: graphite-api - Graphite-web, without the interface. Just the rendering HTTP API

[bugzilla]

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



--- Comment #37 from Fedora Update System  ---
graphite-api-1.1.3-3.el7 has been pushed to the Fedora EPEL 7 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1365535] Review Request: entr - Run arbitrary commands when files change

[bugzilla]

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



--- Comment #2 from Igor Gnatenko  ---
Created attachment 1194406
  --> https://bugzilla.redhat.com/attachment.cgi?id=1194406=edit
suggested patch for spec

I found some number of issues in spec so I fixed them for you.

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

[Bug 1365535] Review Request: entr - Run arbitrary commands when files change

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #1 from Igor Gnatenko  ---
Suggested patch for spec is coming.

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

[Bug 1365535] Review Request: entr - Run arbitrary commands when files change

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|ignate...@redhat.com



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

[Bug 1370533] New: Review Request: python-django-pgjson - PostgreSQL json field support for Django

[bugzilla]

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

Bug ID: 1370533
   Summary: Review Request: python-django-pgjson - PostgreSQL json
field support for Django
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-django-pgjson.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-django-pgjson-0.3.1-1.fc26.src.rpm
Description: PostgreSQL json field support for Django.
Fedora Account System Username: ignatenkobrain

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

[Bug 1370533] Review Request: python-django-pgjson - PostgreSQL json field support for Django

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||python-django-pgjson



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

[Bug 1370291] Review Request: python-tenacity - Tenacity is a general purpose retrying python library

[bugzilla]

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



--- Comment #1 from Pradeep Kilambi  ---
rebased to 3.0

Spec URL:
https://pkilambi.fedorapeople.org/python-tenacity/python-tenacity.spec
SRPM URL:
https://pkilambi.fedorapeople.org/python-tenacity/python-tenacity-3.0.0-1.fc24.src.rpm

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

[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent with the requests package

[bugzilla]

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

Fabio Alessandro Locati  changed:

   What|Removed |Added

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



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

[Bug 1366741] Review Request: xcb-util-xrm - XCB utility functions for the X resource manager

[bugzilla]

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



--- Comment #5 from Fedora Update System  ---
xcb-util-xrm-1.0-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-7c9a1bba3d

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

[Bug 1366741] Review Request: xcb-util-xrm - XCB utility functions for the X resource manager

[bugzilla]

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



--- Comment #6 from Fedora Update System  ---
xcb-util-xrm-1.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-de5c7d599c

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

[Bug 1366741] Review Request: xcb-util-xrm - XCB utility functions for the X resource manager

[bugzilla]

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

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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1368626] Review Request: python-whichcraft - Cross-python shutil.which functionality

[bugzilla]

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



--- Comment #5 from Fedora Update System  ---
python-whichcraft-0.4.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-4beb444a5f

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

[Bug 1368626] Review Request: python-whichcraft - Cross-python shutil.which functionality

[bugzilla]

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

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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1368626] Review Request: python-whichcraft - Cross-python shutil.which functionality

[bugzilla]

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



--- Comment #4 from Fedora Update System  ---
python-whichcraft-0.4.0-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-bea2db5393

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

[Bug 1264546] Review Request: soletta - A framework for making IoT devices

[bugzilla]

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



--- Comment #61 from Gustavo Lima Chaves  ---
v1.0 ack-ed on our side, with rpmlint happy for all assets. Now going for
Fedora's infra back again (koji build).

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

[Bug 1343814] Review Request: vagrant-sshfs - A Vagrant synced folder plugin that mounts folders via SSHFS.

[bugzilla]

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

Parag AN(पराग)  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |
   Assignee|panem...@gmail.com  |admil...@redhat.com



--- Comment #19 from Parag AN(पराग)  ---
Removing myself from the review assignee as I have not reviewed this package
and assigning Adam as he has approved this package.

Also, Removing FE-NEEDSPONSOR from this review as Adam has already sponsored
Dusty.


Referenced Bugs:

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

[Bug 1366687] Review Request: gnutls30 - A TLS protocol implementation

[bugzilla]

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

Jon Ciesla  changed:

   What|Removed |Added

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



--- Comment #16 from Jon Ciesla  ---
It's mandatory, if it's el6-only, EOL the master branch.

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

[Bug 1366687] Review Request: gnutls30 - A TLS protocol implementation

[bugzilla]

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

Nikos Mavrogiannopoulos  changed:

   What|Removed |Added

 CC||limburg...@gmail.com
  Flags||needinfo?(limburgher@gmail.
   ||com)



--- Comment #15 from Nikos Mavrogiannopoulos  ---
Shouldn't not contain the devel branch? This is an el6-only package.

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

[Bug 1350143] Review Request: fmt - Small, safe and fast formating library for C++

[bugzilla]

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



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

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

[Bug 1366713] Review Request: nunit2 - unit-testing framework for .Net/ mono

[bugzilla]

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



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

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

[Bug 1368626] Review Request: python-whichcraft - Cross-python shutil.which functionality

[bugzilla]

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



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

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

[Bug 1366687] Review Request: gnutls30 - A TLS protocol implementation

[bugzilla]

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



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

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

[Bug 1366741] Review Request: xcb-util-xrm - XCB utility functions for the X resource manager

[bugzilla]

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



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

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

[Bug 1343814] Review Request: vagrant-sshfs - A Vagrant synced folder plugin that mounts folders via SSHFS.

[bugzilla]

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



--- Comment #18 from Vít Ondruch  ---
(In reply to Dusty Mabe from comment #17)
Ha, now I really checked what is going on and that it is not exactly your
package fault to carry the fonts. That is the generated documentation and this
is reported as bug 1224715.

In this case, I would recommend to drop the "bundled" notices and carry on,
since this issue applies to all rubygem- and vagrant- packages :/

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

[Bug 1367033] Review Request: dumb-init - entrypoint wrapper for docker that pass signal and handle zombies

[bugzilla]

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



--- Comment #31 from Muayyad Alsadi  ---
here is the upstream issue for fixing the tests

https://github.com/Yelp/dumb-init/issues/115

how should we continue?

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

[Bug 1367033] Review Request: dumb-init - entrypoint wrapper for docker that pass signal and handle zombies

[bugzilla]

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



--- Comment #30 from Muayyad Alsadi  ---
the test assert than when sh is killed by a signal it should return status code
= 128 + signal_code

does fedora sh have this behavior or is it debian specific?

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

[Bug 1367033] Review Request: dumb-init - entrypoint wrapper for docker that pass signal and handle zombies

[bugzilla]

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



--- Comment #29 from Muayyad Alsadi  ---
when I used mock it gave me 4 test errors, so I believe we should stop here,
and leave it to the upstream to fix his tests. (I'll open a ticket for him)

https://paste.fedoraproject.org/414308/22152031/

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

[Bug 1366687] Review Request: gnutls30 - A TLS protocol implementation

[bugzilla]

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



--- Comment #13 from Jan Včelák  ---
I've also replaced the patches with the upstream ones. Thanks for fixing these
issues so fast.

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

[Bug 1343814] Review Request: vagrant-sshfs - A Vagrant synced folder plugin that mounts folders via SSHFS.

[bugzilla]

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



--- Comment #17 from Dusty Mabe  ---
hi vit,

I also have this comment in there:

```
# Don't know why this is required - should try to figure this out and 
# remove bundled fonts in the future.
Provides: bundled(lato-fonts)
# Using OFL license https://www.google.com/fonts/specimen/Source+Code+Pro
Provides: bundled(sourcecodepro-fonts)
```

Can you help me figure out how to remove them? I really don't know anything
about fonts or bundled fonts and I don't know why that would have been in my
package. 

Dusty

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

[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent with the requests package

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks||1370239




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1370239
[Bug 1370239] Review Request: python-ansible-tower-cli - A CLI tool for
Ansible Tower
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370239] Review Request: python-ansible-tower-cli - A CLI tool for Ansible Tower

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Depends On||1370473 (python-fauxquests)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1370473
[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent
with the requests package
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks||1370473 (python-fauxquests)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1370473
[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent
with the requests package
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370473] Review Request: python-fauxquests - Mock HTTP requests sent with the requests package

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Depends On||1370469
   ||(python-dict-sorted)
  Alias||python-fauxquests




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1370469
[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by
key or by comparison function
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370473] New: Review Request: python-fauxquests - Mock HTTP requests sent with the requests package

[bugzilla]

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

Bug ID: 1370473
   Summary: Review Request: python-fauxquests - Mock HTTP requests
sent with the requests package
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-fauxquests.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-fauxquests-1.1-1.fc26.src.rpm
Description: Mock HTTP requests sent with the requests package.
Fedora Account System Username: ignatenkobrain

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

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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

Fabio Alessandro Locati  changed:

   What|Removed |Added

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



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

[Bug 1370469] Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||python-dict-sorted



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

[Bug 1370469] New: Review Request: python-dict-sorted - Dictionaries sorted by key or by comparison function

[bugzilla]

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

Bug ID: 1370469
   Summary: Review Request: python-dict-sorted - Dictionaries
sorted by key or by comparison function
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-dict-sorted.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-dict-sorted-1.0.0-1.fc26.src.rpm
Description: Dictionaries sorted by key or by comparison function.
Fedora Account System Username: ignatenkobrain

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

[Bug 1370449] Review Request: rtv - A simple terminal viewer for Reddit ( Reddit Terminal Viewer)

[bugzilla]

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

Bug 1370446 Summary: Review Request: python-praw - PRAW, an acronym for `Python 
Reddit API Wrapper`, is a python package that allows for simple access to 
reddit's API
https://bugzilla.redhat.com/show_bug.cgi?id=1370446

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG



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

[Bug 1370446] Review Request: python-praw - PRAW, an acronym for `Python Reddit API Wrapper`, is a python package that allows for simple access to reddit's API

[bugzilla]

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

Germano Massullo  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
  Flags|fedora-review?  |fedora-review-
Last Closed||2016-08-26 08:16:45



--- Comment #1 from Germano Massullo  ---
Package already exists:
https://admin.fedoraproject.org/pkgdb/package/rpms/python-praw/

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

[Bug 1350257] Review Request: petsc - Portable Extensible Toolkit for Scientific Computation

[bugzilla]

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



--- Comment #20 from Antonio Trande  ---
SPEC:
http://copr-dist-git.fedorainfracloud.org/cgit/sagitter/petsc/petsc.git/plain/petsc.spec?id=79c67a625efcd1f39a24705c8ce881977f2bb0b1

SRPM:
https://copr-be.cloud.fedoraproject.org/results/sagitter/petsc/fedora-rawhide-x86_64/00446542-petsc/petsc-3.7.2-11.fc26.src.rpm

SuperLU support works with superlu-2.5.0, see previous build:
http://copr-fe.cloud.fedoraproject.org/coprs/sagitter/petsc/build/446367/

> --known-endian=$(%{__python2} -c "import sys;print sys.byteorder") \

Is this option totally useless?

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

[Bug 1366687] Review Request: gnutls30 - A TLS protocol implementation

[bugzilla]

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



--- Comment #12 from Jan Včelák  ---
(In reply to Nikos Mavrogiannopoulos from comment #11) 
> I see it only in x86 (not x86-64).

My fault. You are right. I will disable HW acceleration on %{ix86} then.

Btw error appears even on Fedora:

% rpmlint ./gnutls-3.5.3-1.fc26.i686.rpm   
gnutls.i686: E: shlib-with-non-pic-code /usr/lib/libgnutls.so.30.9.0
1 packages and 0 specfiles checked; 1 errors, 0 warnings.

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

[Bug 1370444] Review Request: python-decorator - Better living through Python with decorators

[bugzilla]

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

Germano Massullo  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review-



--- Comment #1 from Germano Massullo  ---
Package already exists
https://admin.fedoraproject.org/pkgdb/package/rpms/python-decorator/

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

[Bug 1370449] Review Request: rtv - A simple terminal viewer for Reddit ( Reddit Terminal Viewer)

[bugzilla]

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

Bug 1370444 Summary: Review Request: python-decorator - Better living through 
Python with decorators
https://bugzilla.redhat.com/show_bug.cgi?id=1370444

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG



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

[Bug 1370444] Review Request: python-decorator - Better living through Python with decorators

[bugzilla]

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

Fabio Alessandro Locati  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2016-08-26 08:01:55



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

[Bug 1366713] Review Request: nunit2 - unit-testing framework for .Net/ mono

[bugzilla]

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

Claudio Rodrigo Pereyra DIaz  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|claudiorodrigo@pereyradiaz.
   ||com.ar



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

[Bug 1370451] Review Request: - nodejs-grunt-contrib-copy - Copy files and folders

[bugzilla]

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

Matthias Runge  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1370446] Review Request: python-praw - PRAW, an acronym for `Python Reddit API Wrapper`, is a python package that allows for simple access to reddit's API

[bugzilla]

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

Germano Massullo  changed:

   What|Removed |Added

 CC||germano.massu...@gmail.com
   Assignee|nob...@fedoraproject.org|germano.massu...@gmail.com
  Flags||fedora-review?



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

[Bug 1370449] Review Request: rtv - A simple terminal viewer for Reddit ( Reddit Terminal Viewer)

[bugzilla]

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

Germano Massullo  changed:

   What|Removed |Added

 CC||germano.massu...@gmail.com
   Assignee|nob...@fedoraproject.org|germano.massu...@gmail.com
  Flags||fedora-review?



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

[Bug 1370444] Review Request: python-decorator - Better living through Python with decorators

[bugzilla]

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

Germano Massullo  changed:

   What|Removed |Added

 CC||germano.massu...@gmail.com
   Assignee|nob...@fedoraproject.org|germano.massu...@gmail.com
 QA Contact|extras...@fedoraproject.org |
  Flags||fedora-review?



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

[Bug 1370451] New: Review Request: - nodejs-grunt-contrib-copy - Copy files and folders

[bugzilla]

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

Bug ID: 1370451
   Summary: Review Request:  - nodejs-grunt-contrib-copy - Copy
files and folders
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mru...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://www.matthias-runge.de/fedora/nodejs-grunt-contrib-copy.spec
SRPM URL:
http://www.matthias-runge.de/fedora/nodejs-grunt-contrib-copy-1.0.0-1.fc24.src.rpm

This is a re-review, of the package after being retired

Description: Copy files and folders. 

Fedora Account System Username: mrunge


Original review was this here:
https://bugzilla.redhat.com/show_bug.cgi?id=1115671
the package is still in f24:
https://apps.fedoraproject.org/packages/nodejs-grunt-contrib-copy/bugs/

This review also fixes both bugs.

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