[Bug 1418429] Review Request: python-cachez - memoization cache decorator

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

  Flags||needinfo?(ehar...@redhat.co
   ||m)



--- Comment #4 from Alan Pevec  ---
Please include following patch before the initial import to allow building for
EL7:

 %{?python_provide:%python_provide python2-%{pypi_name}}

 BuildRequires:  python2-devel
+BuildRequires:  python-setuptools

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

[Bug 1418429] Review Request: python-cachez - memoization cache decorator

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Alan Pevec  ---
APPROVED

Package Review
==

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

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Apache (v2.0)", "*No copyright* Apache", "*No copyright*
 Apache (v2)", "Unknown or generated", "*No copyright* Apache (v2.0)".
 6 files have unknown license. Detailed output of licensecheck below.
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[-]: Fully versioned dependency in subpackages if applicable.
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package 

[Bug 1386563] Review Request: python-ruamel-yaml - YAML 1.2 loader/ dumper package for Python

[bugzilla]

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

Parag AN(पराग)  changed:

   What|Removed |Added

  Flags|needinfo?(panem...@gmail.co |
   |m)  |



--- Comment #12 from Parag AN(पराग)  ---
Hi Soctt,
I just checked this
https://admin.fedoraproject.org/pkgdb/package/rpms/python-ruamel-yaml and found
that I am not the owner of this package. From that URL it looks like
"chandankumar" is the only person who owns this package. 
   I now checked https://admin.fedoraproject.org/accounts/user/view/sdodson and
found that you have all the required group membership and part of the packager
group. Then I checked
https://bodhi.fedoraproject.org/updates/FEDORA-2016-84c28903b4 and found that
you have not provided any feedback comment or karma on that update to help it
move further.
   Now how can this update becomes stable? Either Chandakumar need to push it
himself or that update need to get +3 karma.

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

[Bug 1420605] New: Review Request: nodejs-grunt-wrap - A Grunt plugin for wrapping project text files

[bugzilla]

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

Bug ID: 1420605
   Summary: Review Request: nodejs-grunt-wrap - A Grunt plugin for
wrapping project text files
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ra...@electronsweatshop.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://bowlofeggs.fedorapeople.org/nodejs-grunt-wrap.spec
SRPM URL:
https://bowlofeggs.fedorapeople.org/nodejs-grunt-wrap-0.3.0-1.fc26.src.rpm
Description: A Grunt plugin for wrapping project text files.
Fedora Account System Username: bowlofeggs

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

[Bug 1387669] Review Request: kio-gdrive - KDE Dolphin Google Drive access

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2017-02-08 23:21:42



--- Comment #25 from Fedora Update System  ---
kio-gdrive-1.1.0-1.fc25 has been pushed to the Fedora 25 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420568] New: Review Request: docker-distribution - A container for the docker-distribution package

[bugzilla]

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

Bug ID: 1420568
   Summary: Review Request: docker-distribution - A container for
the docker-distribution package
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ra...@electronsweatshop.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://bowlofeggs.fedorapeople.org/docker-distribution/Dockerfile
SRPM URL: N/A
Description: A container for the docker-distribution package
Fedora Account System Username: bowlofeggs

Note that this is a container package review, not an RPM package review:

https://fedoraproject.org/wiki/Container:Guidelines

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

[Bug 1419226] Review Request: prelude-correlator

[bugzilla]

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



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
s/SystemD/systemd/

+ package name is OK
+ license is acceptable (GPLv2)
- license is specified correctly:
  not, it's GPLv2+, see the headers in the files

Why /etc/prelude-correlator/rules? That python code does not look like
something that is configurable, wouldn't it be better to put it somewhere in
/usr/?

Does the binary fork anything? If there's just one process, you can do away
with the pid file, which would obviate the need for a directory in /run and
simplify things.

Could this daemon run as a normal user? What kind of privileges does is
require?

Hm, why do the python subpackage require prelude-correlator.rpm?
prelude-correlator includes a service file which will not work without
python3-p-c, so it seems prelude-correlator.rpm should require
python3-prelude-correlator. I'm pretty sure that 'dnf install
prelude-correlator' should yield a working setup.

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

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

[bugzilla]

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



--- Comment #15 from Orion Poplawski  ---
My bad - I got confused with too many different versions at the same time.

I've been playing around with trying to build a versioned clang with the
versioned llvm packages (you've seen my commits to llvm3.7 towards that).  I'm
starting to think it would be better to install into a prefix (%_libdir/%name)
and then move/symlink from there rather than the reverse in order to get the
output of llvm-config right and satisfy some assumptions of the clang 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

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

[bugzilla]

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



--- Comment #5 from James Hogarth  ---
Just to clear up in the PHP bit ... these are the owncloud packages we support
on Fedora and that we support on the PHP shipped with Fedora - nothing special.

It's best not to get wrapped up in that as if there were issues we'd be dealing
with them in Fedora anyway (eg we will have PHP71 in F26 but that needs some
maintainer love to backport a patch from upstream, which I already do and
support, since they don't intend to support PHP71 until owncloud 10
themselves).

Ignore PHP versions here ... that's up to our talented package maintainers in
fedora ;)

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

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

[bugzilla]

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



--- Comment #4 from Josh Berkus  ---
UPDATE:

Discussed structure of container on IRC.   Additional components *are*
separated.  This more points to the need for additional metadata in dockerfiles
for the build service.  Will be following up with suggestions.

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

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

[bugzilla]

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



--- Comment #3 from Josh Berkus  ---
Ah, I misunderstood the test SQLite instance as the real database.

So the other obvious part is the file storage for WebDAV documents.  When I was
looking at contianerizing OwnCloud myself I identified a couple other
components; let me find the writeup and I'll post it here.

There's also Fedora's limitation to unsupported PHP, which is going to be a
problem here ...

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

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

[bugzilla]

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



--- Comment #2 from James Hogarth  ---
This is just to sort of talk I'm hoping to engage with given the early
developments of the container review process.

In case you are not aware I am the owncloud maintainer in fedora/epel so am
intimately aware of what it involves.

This container is explicitly httpd+mod_php+mod_ssl as a minimal base, which is
the predominant deployment I've found.

This explicitly does not include a database (other than a test sqlite instance)
but directs the admin to have a database served out of another host/container
and that's an exercise for them.

This is just one component - though the one most people expect - and is similar
to the way it's packaged as a container upstream (not identical as it tries to
take advantage of Fedora features and infrastructure).

Rather than:

> Official Fedora containers need to be examples of good containerized 
> application design.

and

> This container should be sent back and broken into several services.

I'd appreciate some constructive feedback (which is the point of reviews after
all) on where you think a better split or an improved Dockerfile would worked
out - particularly given the lack of guidelines at present.

I care about these packages for some bizarre reason and I care about those who
consume my packages.

It was a crying shame when I had to retire owncloud from EPEL6 and point users
elsewhere, when the time soon comes to retire from EPEL7 I'd really like to be
able to point them to our containers to migrate to.

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

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

[bugzilla]

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

Josh Berkus  changed:

   What|Removed |Added

 CC||jber...@redhat.com



--- Comment #1 from Josh Berkus  ---
In addition to the other issues mentioned, this is a terrible container. 
Owncloud is a multi-component platform, which should be broken into 3 or more
separate containers, not lumped together in one container like porting a VM. 
Official Fedora containers need to be examples of good containerized
application design.

This container should be sent back and broken into several services.

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

[Bug 1394193] Review Request: arduino-builder - A command line tool for compiling Arduino sketches

[bugzilla]

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

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

[Bug 1394193] Review Request: arduino-builder - A command line tool for compiling Arduino sketches

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
arduino-builder-1.0.5-3.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-9e3f59cacf

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

[Bug 1394193] Review Request: arduino-builder - A command line tool for compiling Arduino sketches

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
arduino-builder-1.0.5-3.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-e26cf562b7

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

[Bug 1356625] Review Request: python-cotyledon - Framework for defining long-running services

[bugzilla]

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

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

[Bug 1356625] Review Request: python-cotyledon - Framework for defining long-running services

[bugzilla]

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



--- Comment #25 from Fedora Update System  ---
python-cotyledon-1.6.7-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-907b21eb77

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

[Bug 1349578] Review Request: python-pifpaf - Suite of fixtures to manage daemons

[bugzilla]

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

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

[Bug 1349578] Review Request: python-pifpaf - Suite of fixtures to manage daemons

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
python-pifpaf-0.24.1-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-a49c5c32ff

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

[Bug 1420512] New: Review Request: clang3.7 - A C language family front-end for LLVM

[bugzilla]

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

Bug ID: 1420512
   Summary: Review Request: clang3.7 - A C language family
front-end for LLVM
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: or...@cora.nwra.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://www.cora.nwra.com/~orion/fedora/clang3.7.spec
SRPM URL: https://www.cora.nwra.com/~orion/fedora/clang3.7-3.7.1-1.fc26.src.rpm
Description:
clang: noun
1. A loud, resonant, metallic sound.
2. The strident call of a crane or goose.
3. C-language family front-end toolkit.

The goal of the Clang project is to create a new C, C++, Objective C
and Objective C++ front-end for the LLVM compiler. Its tools are built
as libraries and designed to be loosely-coupled and extensible.

Fedora Account System Username: orion

This is a versioned clang package to allow parallel installs of different
versions of clang.

https://koji.fedoraproject.org/koji/taskinfo?taskID=17678231

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

[Bug 1420099] Review Request: perl-Mail-Message - MIME message handling

[bugzilla]

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

Tom "spot" Callaway  changed:

   What|Removed |Added

 Blocks|182235 (FE-Legal)   |



--- Comment #2 from Tom "spot" Callaway  ---
New SPEC: https://spot.fedorapeople.org/perl-Mail-Message.spec
New SRPM: https://spot.fedorapeople.org/perl-Mail-Message-3.000-2.fc26.src.rpm

Opened upstream ticket for the test files, removed them in %setup. Added 40
billion BuildRequires.

Lifting FE-Legal.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=182235
[Bug 182235] Fedora Legal 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1406881] Review Request: vint - Lint tool for Vim script Language

[bugzilla]

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

Filip Szymański  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2017-02-08 14:51:05



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

[Bug 1281313] Review Request: podget - Podcast aggregator/ downloader optimized for cron

[bugzilla]

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

Filip Szymański  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2017-02-08 14:50:33



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

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

[bugzilla]

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



--- Comment #4 from Luya Tshimbalanga  ---
Here is the updated files with latest git snapshot

Spec URL: https://luya.fedorapeople.org/packages/SPECS/YafaRay.spec
SRPM URL:
https://luya.fedorapeople.org/packages/SRPMS/YafaRay-3.2.0-0.1.git20170131.fc25.src.rpm

I notice usptream added qt-gui support disabled by default.

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

[Bug 1386563] Review Request: python-ruamel-yaml - YAML 1.2 loader/ dumper package for Python

[bugzilla]

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

Scott Dodson  changed:

   What|Removed |Added

  Flags||needinfo?(panem...@gmail.co
   ||m)



--- Comment #11 from Scott Dodson  ---
What's the status of this? Looks like the review is approved and it's been
sitting in testing for 4 months, does it just need to be manually flipped to
stable or are there still outstanding problems?

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

[Bug 1419035] Review Request: perl-Types-Path-Tiny - Path:: Tiny types and coercions for Moose and Moo

[bugzilla]

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



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/perl-Types-Path-Tiny

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

[Bug 1419043] Review Request: perl-Sub-Exporter-Lexical - Export lexically-available subs with Sub:: Exporter

[bugzilla]

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



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/perl-Sub-Exporter-Lexical

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

[Bug 1419035] Review Request: perl-Types-Path-Tiny - Path:: Tiny types and coercions for Moose and Moo

[bugzilla]

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



--- Comment #2 from Ralf Corsepius  ---
Thanks, apparently I missed this approval :-)

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

[Bug 1327635] Review Request: openstack-congress - OpenStack Congress Service

[bugzilla]

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

Javier Peña  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2017-02-08 12:27:30



--- Comment #18 from Javier Peña  ---
openstack-congress builds are now in the RDO Trunk repos.

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

[Bug 1417392] Review Request: python-semver - Python helper for Semantic Versioning

[bugzilla]

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



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

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

[Bug 1418933] Review Request: abrt-server-info-page - Webpage for servers containing ABRT's services

[bugzilla]

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



--- Comment #7 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/abrt-server-info-page

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

[Bug 1411502] Review Request: flr - Fedora RelEng python libraries and command line tools

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
flr-0.0.2-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-6fdcd2790e

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

[Bug 1411502] Review Request: flr - Fedora RelEng python libraries and command line tools

[bugzilla]

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

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

[Bug 1411502] Review Request: flr - Fedora RelEng python libraries and command line tools

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
flr-0.0.2-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-b86629264d

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

[Bug 1411502] Review Request: flr - Fedora RelEng python libraries and command line tools

[bugzilla]

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



--- Comment #11 from Fedora Update System  ---
flr-0.0.2-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-640794de99

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

[Bug 1402590] Review Request: ecryptfs-simple - A CLI front end to ecryptfs that works with normal user account

[bugzilla]

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



--- Comment #6 from Rex Dieter  ---
for item 4, in case you're unfamiliar with %attr usage, see:
http://rpm.org/max-rpm-snapshot/s1-rpm-inside-files-list-directives.html

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

[Bug 1419043] Review Request: perl-Sub-Exporter-Lexical - Export lexically-available subs with Sub:: Exporter

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Jitka Plesnikova  ---
> BuildRequires
> FIX: Please remove duplicity 'perl(strict)' in BR.
> FIX: Please change the version restriction for perl(ExtUtils::MakeMaker) to
>  6.76 or higher, because NO_PACKLIST option (spec file line 37) was
>  added in this version.
-BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.30
+BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.76
 BuildRequires:  perl(Lexical::Sub)
-BuildRequires:  perl(strict)
Ok.


> FIX: Do you want to distribute the file 'snippet.pl'? It was put into
>  /usr/share/perl5/vendor_perl/Sub/Exporter. The dependencies for the file
>  are not generated, because 'file' detects it as 'ASCII text' and
> rpm-build 
>  does not process it by perl-generators.
> 
$ rpm -qpl perl-Sub-Exporter-Lexical-0.092292-2.fc26.noarch.rpm 
/usr/share/doc/perl-Sub-Exporter-Lexical
/usr/share/doc/perl-Sub-Exporter-Lexical/Changes
/usr/share/doc/perl-Sub-Exporter-Lexical/README
/usr/share/licenses/perl-Sub-Exporter-Lexical
/usr/share/licenses/perl-Sub-Exporter-Lexical/LICENSE
/usr/share/man/man3/Sub::Exporter::Lexical.3pm.gz
/usr/share/perl5/vendor_perl/Sub
/usr/share/perl5/vendor_perl/Sub/Exporter
/usr/share/perl5/vendor_perl/Sub/Exporter/Lexical.pm

Ok. snippet.pl is not distributed. 

The package looks good.
Approved

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

[Bug 1419043] Review Request: perl-Sub-Exporter-Lexical - Export lexically-available subs with Sub:: Exporter

[bugzilla]

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



--- Comment #2 from Ralf Corsepius  ---
I believe to have reflected all your remarks.

Update:
Spec URL:
https://corsepiu.fedorapeople.org/packages/perl-Sub-Exporter-Lexical.spec
SRPM URL:
https://corsepiu.fedorapeople.org/packages/perl-Sub-Exporter-Lexical-0.092292-2.fc26.src.rpm

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

[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Blocks||1420384




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420384
[Bug 1420384] Review Request: phpcov - CLI frontend for PHP_CodeCoverage
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420374] Review Request: php-phpunit-php-code-coverage5 - PHP code coverage information

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Blocks||1420384




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420384
[Bug 1420384] Review Request: phpcov - CLI frontend for PHP_CodeCoverage
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420384] Review Request: phpcov - CLI frontend for PHP_CodeCoverage

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Depends On||1420381, 1420374




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420374
[Bug 1420374] Review Request: php-phpunit-php-code-coverage5 - PHP code
coverage information
https://bugzilla.redhat.com/show_bug.cgi?id=1420381
[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420384] New: Review Request: phpcov - CLI frontend for PHP_CodeCoverage

[bugzilla]

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

Bug ID: 1420384
   Summary: Review Request: phpcov - CLI frontend for
PHP_CodeCoverage
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/master/php/phpunit/phpcov/phpcov.spec
SRPM URL: http://rpms.remirepo.net/SRPMS/phpcov-4.0.0-1.remi.src.rpm
Description: 
phpcov is a command-line frontend for the PHP_CodeCoverage library.


Fedora Account System Username: remi

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

[Bug 1420382] Review Request: php-phpunit-dbunit3 - PHPUnit extension for database interaction testing

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Depends On||1420381




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420381
[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Blocks||1420382




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420382
[Bug 1420382] Review Request: php-phpunit-dbunit3 - PHPUnit extension for
database interaction testing
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420382] New: Review Request: php-phpunit-dbunit3 - PHPUnit extension for database interaction testing

[bugzilla]

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

Bug ID: 1420382
   Summary: Review Request: php-phpunit-dbunit3 - PHPUnit
extension for database interaction testing
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/master/php/phpunit/php-phpunit-dbunit3/php-phpunit-dbunit3.spec
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-phpunit-dbunit3-3.0.0-1.remi.src.rpm
Description: 
PHPUnit extension for database interaction testing.

Fedora Account System Username: remi

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

[Bug 1420378] Review Request: php-phpunit-mock-objects4 - ock Object library for PHPUnit

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Blocks||1420381




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420381
[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420374] Review Request: php-phpunit-php-code-coverage5 - PHP code coverage information

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Blocks||1420381




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420381
[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420381] Review Request: phpunit6 - The PHP Unit Testing framework

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

 Depends On||1420378, 1420374




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420374
[Bug 1420374] Review Request: php-phpunit-php-code-coverage5 - PHP code
coverage information
https://bugzilla.redhat.com/show_bug.cgi?id=1420378
[Bug 1420378] Review Request: php-phpunit-mock-objects4 - ock Object
library for PHPUnit
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420381] New: Review Request: phpunit6 - The PHP Unit Testing framework

[bugzilla]

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

Bug ID: 1420381
   Summary: Review Request: phpunit6 - The PHP Unit Testing
framework
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/master/php/phpunit/phpunit6/phpunit6.spec
SRPM URL: http://rpms.remirepo.net/SRPMS/phpunit6-6.0.6-2.remi.src.rpm
Description: 
PHPUnit is a programmer-oriented testing framework for PHP.
It is an instance of the xUnit architecture for unit testing frameworks.

Fedora Account System Username: remi

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

[Bug 1420378] New: Review Request: php-phpunit-mock-objects4 - ock Object library for PHPUnit

[bugzilla]

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

Bug ID: 1420378
   Summary: Review Request: php-phpunit-mock-objects4 - ock Object
library for PHPUnit
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/master/php/phpunit/php-phpunit-mock-objects4/php-phpunit-mock-objects4.spec
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-phpunit-mock-objects4-4.0.0-1.remi.src.rpm
Description: 
Mock Object library for PHPUnit

Fedora Account System Username: remi

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

[Bug 1420374] New: Review Request: php-phpunit-php-code-coverage5 - PHP code coverage information

[bugzilla]

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

Bug ID: 1420374
   Summary: Review Request: php-phpunit-php-code-coverage5 - PHP
code coverage information
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/master/php/phpunit/php-phpunit-php-code-coverage5/php-phpunit-php-code-coverage5.spec
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-phpunit-php-code-coverage5-5.0.0-1.remi.src.rpm
Description: 
Library that provides collection, processing, and rendering functionality
for PHP code coverage information.


Fedora Account System Username: remi

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

[Bug 1420153] Review Request: js-jquery-jstree - A jQuery tree plugin

[bugzilla]

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



--- Comment #1 from Randy Barlow  ---
Thanks Yanko!

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

[Bug 720818] Review Request: python-tw2-core - Web widget creation toolkit based on TurboGears widgets

[bugzilla]

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



--- Comment #21 from Ralph Bean  ---
Package request has been approved:
https://admin.stg.fedoraproject.org/pkgdb/package/modules/memcached

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

[Bug 1419152] Review Request: rubygem-memfs - fake file system that can be used for tests

[bugzilla]

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



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

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

[Bug 1420336] New: Review Request: libcbor - CBOR protocol implementation

[bugzilla]

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

Bug ID: 1420336
   Summary: Review Request: libcbor - CBOR protocol implementation
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: jvyma...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://pagure.io/libcbor_package_review/raw/master/f/libcbor.spec
SRPM URL:
https://pagure.io/libcbor_package_review/raw/master/f/libcbor-0.5.0-1.fc25.src.rpm
Description: Libcbor is a C library for parsing and generating CBOR, the
general-purpose schema-less binary data format.
Fedora Account System Username:jvymazal

https://koji.fedoraproject.org/koji/taskinfo?taskID=17674162

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

[Bug 1420329] New: Review Request: python-django-cors-headers - A Django application for handling the CORS headers

[bugzilla]

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

Bug ID: 1420329
   Summary: Review Request: python-django-cors-headers - A Django
application for handling the CORS headers
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: rb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://ralph.fedorapeople.org//python-django-cors-headers.spec
SRPM URL:
http://ralph.fedorapeople.org//python-django-cors-headers-1.1.0-2.fc25.src.rpm

Description:
django-cors-headers is a Django application for handling the server headers
required for Cross-Origin Resource Sharing (CORS).

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

[Bug 1420329] Review Request: python-django-cors-headers - A Django application for handling the CORS headers

[bugzilla]

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



--- Comment #1 from Ralph Bean  ---
This package built on koji: 
https://koji.fedoraproject.org/koji/taskinfo?taskID=17674411

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

[Bug 1419152] Review Request: rubygem-memfs - fake file system that can be used for tests

[bugzilla]

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



--- Comment #2 from Athos Ribeiro  ---
Hello Eduardo,

Thank you for your review!

About the issues: I will fix the rpmlint warning, for the summary text.

About the other issue, I believe this is a problem with a fedora-review regex
(I will check it): The guidelines say we are not supposed to use
Requires: ruby(release),
but says nothing about BuildRequires.

Since guidelines say to explicitly list all build requirements and that the
most important packaged rubygems, like rails [1] do BuildRequires:
ruby(release), I believe I should keep that BuildRequires there and patch
fedora-review.

Any thoughts?

[1]
http://pkgs.fedoraproject.org/cgit/rpms/rubygem-rails.git/tree/rubygem-rails.spec?id=ed97cb3b6b775f6759ef604ce58e82d06903e8e4

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

[Bug 1379432] Review Request: odoo - Suite of web based open source business apps

[bugzilla]

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

Alec Leamas  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks||201449 (FE-DEADREVIEW)
 Resolution|--- |NOTABUG
Last Closed||2017-02-08 08:10:29



--- Comment #10 from Alec Leamas  ---
Closing as per policy for stalled reviews. I'm open to resurrect it.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=201449
[Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter
response should be blocking this bug.
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1415686] Review Request: php-phpmyadmin-sql-parser - A validating SQL lexer and parser with a focus on MySQL dialect

[bugzilla]

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



--- Comment #3 from Remi Collet  ---
Updated to 4.1.1

Spec:
https://raw.githubusercontent.com/remicollet/remirepo/a85c9292862c4b3a7bdea22502f17fc4317ab4ee/php/php-phpmyadmin-sql-parser/php-phpmyadmin-sql-parser.spec
Srpm:
http://rpms.remirepo.net/SRPMS/php-phpmyadmin-sql-parser-4.1.1-1.remi.src.rpm

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

[Bug 1402445] Review Request: myman - text mode videogame

[bugzilla]

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



--- Comment #3 from Jiří Vymazal  ---
Thanks Raphael :)

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

[Bug 1297011] Review Request: rubygem-mimemagic - Fast mime detection by extension or content

[bugzilla]

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



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

I am taking this review.

The gem does not build:

gem build mimemagic.gemspec
WARNING:  See http://guides.rubygems.org/specification-reference/ for help
ERROR:  While executing gem ... (Gem::InvalidSpecificationException)
[".gitignore", ".travis.yml", ".yardopts", "Gemfile", "Rakefile",
"script/freedesktop.org.xml", "script/generate-mime.rb"] are not files

Will you fix that so we can proceed with this review? Also, version 0.3.2 is
out, so you might want to update 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1297011] Review Request: rubygem-mimemagic - Fast mime detection by extension or content

[bugzilla]

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

Athos Ribeiro  changed:

   What|Removed |Added

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



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

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

[bugzilla]

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

Bug ID: 1420275
   Summary: Container Review Request : owncloud - self hosted file
storage and collaboration platform
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: james.hoga...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Description: 

ownCloud gives you universal access to your files through a web interface or
WebDAV. It also provides a platform to easily view & sync your contacts,
calendars and bookmarks across all your devices and enables basic editing right
on the web. ownCloud is extendable via a simple but powerful API for
applications and plugins.

This is a container that utilises httpd and mod_ssl, it can be used standalone
with the embedded sqlite database but it's suggested in production use to be
connected to a postgresql or mariadb database by whichever method the admin
prefers.

Volumes are provided for the httpd config (so that ssl certs can trivially be
added by an admin), owncloud config and owncloud data locations.

To test the container:

docker build -t owncloud .
docker run -d P --name owncloud owncloud
docker port owncloud

Open browser to https://localhost:/owncloud/ and follow the
prompts to add an admin user and point to a database (or just use the built in
sqlite for testing).

Dockerfile: https://fedorapeople.org/~jhogarth/owncloud_container/Dockerfile

Fedora Account System Username: jhogarth

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

[Bug 1394193] Review Request: arduino-builder - A command line tool for compiling Arduino sketches

[bugzilla]

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



--- Comment #12 from Gianluca Sforna  ---
I didn't push it yet because, while I was using the current package in testing
to try out the updated arduino, I found a couple missing files for correct
operations.

I'm spinning up a new update, I will put there a lower karma requirement so if
you want to help we can push it to stable earlier.

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

[Bug 1420099] Review Request: perl-Mail-Message - MIME message handling

[bugzilla]

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

Petr Pisar  changed:

   What|Removed |Added

 Blocks||182235 (FE-Legal)
Summary|Review Request: |Review Request:
   |perl-Mail-Message   |perl-Mail-Message - MIME
   ||message handling



--- Comment #1 from Petr Pisar  ---
The standalone and the packaged spec file differ in white space only. I will
use the standalone one for this review.

URL and Source0 addresses are Ok.
Source archive (SHA-256:
3a70553d4898e783c0ae8645d6bb3c8fa1ed66f997627685a72119b02dd0be1d) is Ok.
Summary is Ok.
Description is Ok.
License verified from lib/Mail/Message.pod and other POD files. Ok.

FIX: The t/203-mlfolder.mbox and t/204-sgfolder.mbox files contain e-mail
messages whose license is not very likely "GPL or Artistic". Since Fedora does
not modify them and I believe their authors are content with their
redistribution (because they were sent to public mailing lists), I think their
existence in Fedora source packages does not violate copyright law. But they
still can be unacceptable for Fedora because Fedora requires a permission for
modification.
Please add a notice into the spec file about their existence and negotiate
their license with the Mail-Message author and their permissibility with Fedora
legal department.

No XS code, noarch BuildArch is Ok.

FIX: Build-require `perl' because you execute it the spec file on lines 32 and
55.
FIX: Build-require `coreutils' (perl-Mail-Message.spec:55).
FIX: Build-require `make' (perl-Mail-Message.spec:56).
FIX: Build-require `findutils' (perl-Mail-Message.spec:60).
FIX: Build-require `glibc-common' (perl-Mail-Message.spec:66).

TODO: Constrain `perl(URI)' dependency with `>= 1.23' (META.json:47).

FIX: Build-require `perl(IO::Handle)' (Makefile.PL:5).
FIX: Build-require `perl(warnings)' (Makefile.PL:96).
FIX: Build-require `perl(base)' (lib/Mail/Message.pm:12).
FIX: Build-require `perl(Carp)' (lib/Mail/Message.pm:22).
FIX: Build-require `perl(Cwd)' (lib/Mail/Message/Test.pm:17).
FIX: Build-require `perl(Email::Simple)'
(lib/Mail/Message/Convert/EmailSimple.pm:20).
FIX: Build-require `perl(Encode::Alias)' (lib/Mail/Message/Body/Encode.pm:24).
FIX: Build-require `perl(Exporter)' (lib/Mail/Message/Test.pm:11).
FIX: Build-require `perl(File::Basename)' (lib/Mail/Message/Body.pm:20).
FIX: Build-require `perl(File::Copy)' (lib/Mail/Message/Test.pm:13).
FIX: Build-require `perl(File::Temp)' (lib/Mail/Message/Body/File.pm:18).
FIX: Build-require `perl(HTML::FormatText)'
(lib/Mail/Message/Convert/HtmlFormatPS.pm:17).
FIX: Build-require `perl(HTML::TreeBuilder)'
(lib/Mail/Message/Convert/HtmlFormatPS.pm:16).
FIX: Build-require `perl(integer)' (lib/Mail/Box/FastScalar.pm:13).
FIX: Build-require `perl(IO::File)' (lib/Mail/Box/Parser/Perl.pm:16).
FIX: Build-require `perl(IO::Handle)' (lib/Mail/Message/Field.pm:17).
FIX: Build-require `perl(IO::Lines)' (lib/Mail/Message/Body/Lines.pm:15).
FIX: Build-require `perl(List::Util)' (lib/Mail/Box/Parser/Perl.pm:15).
FIX: Build-require `perl(Mail::Header)'
(lib/Mail/Message/Convert/EmailSimple.pm:15).
FIX: Build-require `perl(Mail::Identity)'
(lib/Mail/Message/Field/Address.pm:12).
FIX: Build-require `perl(Mail::Internet)'
(lib/Mail/Message/Convert/EmailSimple.pm:14).
FIX: Build-require `perl(MIME::Parser)'
(lib/Mail/Message/Convert/MimeEntity.pm:16).
FIX: Build-require `perl(MIME::QuotedPrint)'
(lib/Mail/Message/Field/Full.pm:16).
FIX: Build-require `perl(overload)' (lib/Mail/Message/Body.pm:27).
FIX: Build-require `perl(POSIX)' (lib/Mail/Message/Field/Date.pm:14).
FIX: Build-require `perl(Storable)' (lib/Mail/Message/Field/Full.pm:17).
FIX: Build-require `perl(strict)' (lib/Mail/Message.pm:5).
FIX: Build-require `perl(Text::Autoformat)'
(lib/Mail/Message/Convert/TextAutoformat.pm:15).
FIX: Build-require `perl(User::Identity::Collection::Emails)'
(lib/Mail/Message/Field/AddrGroup.pm:12).
FIX: Build-require `perl(utf8)' (lib/Mail/Message/Field/Full.pm:14).
FIX: Build-require `perl(vars)' (lib/Mail/Message.pm:9).

FIX: Build-require `perl(Data::Dumper)' (t/314encode-charset.t:13).
TODO: Build-require `perl(Email::Abstract)' (t/423msgconv-emabs.t:14).
TODO: Build-require `perl(Email::Simple)' (t/422msgconv-emsimp.t:16).

FIX: Run-require `perl(Mail::Transport::Send)' (lib/Mail/Message.pm:164).

FIX: Tests fail for me:

t/322bodyconv-htmlps.t  Can't locate Font/Metrics/TimesRoman.pm in @INC
(you may need to install the Font::Metrics::TimesRoman module) (@INC contains:
/home/test/rpmbuild/BUILD/Mail-Message-3.000/blib/lib
/home/test/rpmbuild/BUILD/Mail-Message-3.000/blib/arch /usr/local/lib64/perl5
/usr/local/share/perl5 /usr/lib64/perl5/vendor_perl
/usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at

[Bug 1420153] Review Request: js-jquery-jstree - A jQuery tree plugin

[bugzilla]

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

Yanko Kaneti  changed:

   What|Removed |Added

 CC||yan...@declera.com
Summary|Review Request:  -|tree plugin



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

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

[bugzilla]

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



--- Comment #14 from Milan Bouchet-Valat  ---
Are you sure? I just downloaded the SRPM again, and the .spec file calls
%cmake, not ./configure.

Update on progress: the patch is still being reviewed upstream. If that takes
too long, I'll push the package with the current patch, but for now there's no
hurry (at least for me).

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

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

[bugzilla]

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

James Hogarth  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|james.hoga...@gmail.com
  Flags||fedora-review?



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

[Bug 1420099] Review Request: perl-Mail-Message

[bugzilla]

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

Petr Pisar  changed:

   What|Removed |Added

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

[Bug 1419226] Review Request: prelude-correlator

[bugzilla]

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



--- Comment #2 from Thomas Andrejak  ---
Hello,

Here are the new files :
SPEC : https://fedorapeople.org/~totol/prelude-correlator.spec
SRPM : https://fedorapeople.org/~totol/prelude-correlator-3.1.0-1.fc26.src.rpm

(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)
> No python 3 support? Please note that python3 support MUST be enabled if
> possible.

=> I had a patch to have it :)

> 
> "install init script" → not to be picky, but it's neither init nor a script
> ;)

Done

> 
> In one place you use install -D, in others the directories are created by
> hand.

Done

> 
> It looks all good, only cosmetic issues, apart from the one big question —
> python3.

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

[Bug 1419942] Review Request: perl-HTTP-Thin - Thin Wrapper around HTTP:: Tiny to play nice with HTTP::Message

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-HTTP-Thin-0.006-1.fc26
 Resolution|--- |RAWHIDE
Last Closed||2017-02-08 03:26:21



--- Comment #3 from Jitka Plesnikova  ---
Thank you for the review and the repository.

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