[Bug 1306409] Review Request: perl-Test2 - Framework for writing test tools that all work together
https://bugzilla.redhat.com/show_bug.cgi?id=1306409 Jitka Plesnikova changed: What|Removed |Added Status|NEW |ASSIGNED CC||jples...@redhat.com Assignee|nob...@fedoraproject.org|jples...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1290199] Review Request: cmake3 - Cross-platform make system
https://bugzilla.redhat.com/show_bug.cgi?id=1290199 Neal Gompa changed: What|Removed |Added CC||ngomp...@gmail.com --- Comment #7 from Neal Gompa --- Helio, Because minor versions can be backwards incompatible, I would strongly recommend you rename this to cmake34. This has been done in the past with cmake28 for EL6, and continuing that convention certainly doesn't hurt anything. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306353] Review Request: libunity - Library for integrating with Unity
https://bugzilla.redhat.com/show_bug.cgi?id=1306353
Neal Gompa changed:
What|Removed |Added
CC||ngomp...@gmail.com
--- Comment #3 from Neal Gompa ---
I did a once-over of your package and noticed a few things:
* Where's the Python 3 module? libunity does support Python 3, and the Ubuntu
package provides it as part of the "gir1.2-unity-5.0" package.
* Please do not use unversioned Python macros (like %{__python},
%{python_sitearch}, etc.). Please use the versioned ones so that unexpected
things don't happen in the future (like the target of the unversioned ones
switching on you).
* When trying to do a soversion lock, please don't use ".so.X*", as that glob
will potentially allow bad things to happen (if the soversion jumps a ton, we
might not notice). Please instead split into two: ".so.X" and ".so.X.*". That
properly locks the soversion down.
* Please check to be sure that you aren't repeating the work of file triggers
in releases of Fedora that support and use them. I believe glib-compile-schemas
falls under one of these, per this message[0]. Starting with Fedora 24, this
action will be handled automatically, so please guard these out for F24+.
* Please somehow indicate in the Version or Release field that this is the
version shipped with Ubuntu Wily. This will be extremely important as this
library will likely be upgraded as Canonical upgrades it in Ubuntu before each
release (Xenial coming up soon!).
[0]: https://lists.fedoraproject.org/pipermail/desktop/2015-August/012685.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
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301129] Review Request: hstr - Suggest box like shell history completion
https://bugzilla.redhat.com/show_bug.cgi?id=1301129 --- Comment #14 from Fedora Update System --- hstr-1.19-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-62d9e962b6 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305334] Review Request: R-inline - Functions to Inline C, C++, Fortran Function Calls from R
https://bugzilla.redhat.com/show_bug.cgi?id=1305334
--- Comment #1 from Mukundan Ragavan ---
I have a couple of questions. Please see below.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Package installs properly.
Note: Installation errors (see attachment)
See: https://fedoraproject.org/wiki/Packaging:Guidelines
---> This needs to be checked.
- Package have the default element marked as %%doc :DESCRIPTION
---> I don't see where this is. DESCRIPTION is *NOT* marked as %doc in the spec
file ...
- Package contains the mandatory BuildRequires.
Note: Missing BuildRequires on R-devel, tex(latex)
See: http://fedoraproject.org/wiki/Packaging:R
---> I think it might make more sense to use BR:R-devel instead of R-core-devel
.. Any specific reason for using R-devel?
- The package has the standard %install section.
Note: Package doesn't have the standard removal of *.o and *.so.
See: http://fedoraproject.org/wiki/Packaging:R
---> Can you add this section to the spec? Or, has it been intentionally left
out?
= 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]: 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.
---> Not included in upstream tarball. Not in package.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "Unknown or generated". 4 files have unknown license. Detailed
output of licensecheck in
/home/mukundan/ownCloud/misc_pkgs/pkg_reviews/1305334-R-inline/licensecheck.txt
---> No issues.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: 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: There are rpmlint messages (see attachment).
[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
R:
[x]: Package requires R-core.
= SHOULD items =
Generic:
[!]: Sources can be downloaded from URI in Source: tag
Note: Could not download Source0:
ftp://cran.r-project.org/pub/R/contrib/main/inline_0.3.14.tar.gz
See: http://fedoraproject.org/wiki/Packaging:Guidelines#Tags
---> This is a local issue. I have downloaded the tarball subsequently.
$ wget ftp://cran.r-project.org/pub/R/contrib/main/inline_0.3.14.tar.gz
--2016-02-10 23:12:40--
ftp://cran.r-project.org/pub/R/contrib/main/inline_0.
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #29 from awill...@redhat.com --- Forgot to mention - openqa logging is pretty light by default (info level). debug level is very chatty, but it's not the 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #28 from awill...@redhat.com --- So we can achieve fairly simple logging to the journal simply by setting the log 'file' setting to nothing at all in config: file = or making it undefined it in the code block that defines the 'defaults'. this causes Mojo to log to stderr, and the journal picks that up. The log message levels aren't picked up by journald when you do it this way, but it's good enough for now I think. A sysadmin can still easily log to file by setting `file` in openqa.ini. I've written a simple patch to change the default in this way and sent it upstream for discussion, and done a 4.3-10 build with the change: https://github.com/os-autoinst/openQA/pull/541 https://www.happyassassin.net/reviews/openqa/openqa.spec https://www.happyassassin.net/reviews/openqa/openqa-4.3-10.fc23.src.rpm https://www.happyassassin.net/reviews/openqa/4.3-9_4.3-10.diff https://koji.fedoraproject.org/koji/taskinfo?taskID=12934037 I wasn't citing Wordpress the project, but wordpress the package ;) It's one of our older and more heavily used webapp packages, so I tend to consider it a reasonable example spec. Yeah, I believe that avoiding nesting should avoid the attack vector. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #27 from Upstream Release Monitoring --- adamwill's scratch build of openqa-4.3-10.fc23.src.rpm for f23 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12934037 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882
--- Comment #26 from Zbigniew Jędrzejewski-Szmek ---
(In reply to Ludwig Nussel from comment #22)
> If you prefer logging to the journal ie stdout feel free let your openQA
> package do that instead.
That would seem like the best option, unless openqa generates huge amounts of
logs. How much logs does openqa generate? logrotate is annoying, the journal is
much nicer to use.
(In reply to awill...@redhat.com from comment #17)
> I still don't entirely grok that 'subdirectory ownership' thing, but I asked
> on #yum, and mls said:
>
> 06:09:12> adamw: I guess this is about subdirectories which include
> other files/directories also packaged in rpm
> 06:10:26> I think the security issue is that the non-root user can
> modify the directory while rpm messes (as root) with the directory contents
> 06:10:48> e.g. creates symlinks and the like.
Oh, OK, I think I get it now. Let's say that we have user-owned /var/a in
%files, and then /var/a/b/file in %files. The user can rename /var/a/b to
/var/a/b.old, create /var/a/b, and e.g. symlink /var/a/b/file → /etc/passwd.
When rpm updates /var/a/file during package upgrade it will trash /etc/passwd.
Similar considerations would hold for a subdirectory inside a user-owned
directory. At least it allows the user to cause rpm to write files to arbitrary
directories in the filesystem. I'm not sure if it's possible to carry out the
attack with just one level of nesting. Maybe, it probably depends on the order
in which rpm does operations and whether it uses O_EXCL.
Anyway, I think that the last version is OK, only leaf directories are owned by
geekotest.
> I'll note that the Wordpress package has something very similar to what this
> now has:
>
> %dir %attr(0775,apache,ftp) %{wp_content}/plugins
> %dir %attr(0775,apache,ftp) %{wp_content}/themes
> %dir %attr(0775,apache,ftp) %{wp_content}/upgrade
> %dir %attr(0775,apache,ftp) %{wp_content}/uploads
>
> so either it's not really a problem or every Fedora wordpress instance in
> the world is vulnerable to it :)
In this snippet there is no nesting, so it's not relevant. But I wouldn't use
wordpress to prove security anyway ;)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #25 from awill...@redhat.com --- The apache case is a bit different because IIRC Apache starts as root and creates missing log files as root before it drops privs; /var/log/httpd is owned by root. The reason not to create /var/log/openqa owned by geekotest is given by Ludwig in #c22. I like to try and 'do things right' as far as reasonably possible and not second-guess the user/sysadmin; I don't think it's gonna be super common to remove the package, but I still want to handle it as correctly as possible. I'm still technically on vacation, though, so I haven't had enough time to sit down and work through all the possibilities yet... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304040] Review Request: docker-anaconda-addon - Anaconda kickstart support for Docker
https://bugzilla.redhat.com/show_bug.cgi?id=1304040
David Shea changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #4 from David Shea ---
Looks good. Here goes the checklist:
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "LGPL (v2.1 or later)", "Unknown or generated". 26 files have
unknown license. Detailed output of licensecheck in
/home/dshea/1304040-docker-anaconda-addon/licensecheck.txt
(For these two, /usr/share/anaconda/addons is fixed in anaconda, and for the
other stuff I guess I get to file a bug against filesystem. Or ignore it!)
[x]: Package requires other packages for directories it uses.
Note: No known owner of /usr/share/anaconda/addons,
/usr/share/locale/kw_GB/LC_MESSAGES,
/usr/share/locale/kw@kkcor/LC_MESSAGES,
/usr/share/locale/kw@uccor/LC_MESSAGES,
/usr/share/locale/wba/LC_MESSAGES, /usr/share/locale/kw@uccor,
/usr/share/locale/kw_GB, /usr/share/locale/kw@kkcor,
/usr/share/locale/wba
[x]: Package must own all directories that it creates.
Note: Directories without known owners:
/usr/share/locale/kw@uccor/LC_MESSAGES,
/usr/share/locale/wba/LC_MESSAGES, /usr/share/locale/kw@uccor,
/usr/share/locale/kw@kkcor, /usr/share/locale/kw_GB/LC_MESSAGES,
/usr/share/locale/kw_GB, /usr/share/anaconda/addons,
/usr/share/locale/wba, /usr/share/locale/kw@kkcor/LC_MESSAGES
[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]: The spec file handles locales properly.
[x]: Package consistently uses macros (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: 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 does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: SourceX tarball generation or download is documented.
Note: Pac
[Bug 1306353] Review Request: libunity - Library for integrating with Unity
https://bugzilla.redhat.com/show_bug.cgi?id=1306353 --- Comment #2 from Kevin Kofler --- Shouldn't the description say something like "Library for integrating with Unity and Plasma"? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1294839] Review Request: kf5-libkipi - Common plugin infrastructure for KDE image applications
https://bugzilla.redhat.com/show_bug.cgi?id=1294839 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #13 from Fedora Update System --- kf5-libkipi-15.12.1-1.fc23, spectacle-15.12.1-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-ce83cb6a02 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1297174] Review Request: spectacle - Screenshot capture utility
https://bugzilla.redhat.com/show_bug.cgi?id=1297174 Fedora Update System changed: What|Removed |Added Status|CLOSED |ON_QA Resolution|RAWHIDE |--- Keywords||Reopened --- Comment #11 from Fedora Update System --- kf5-libkipi-15.12.1-1.fc23, spectacle-15.12.1-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-ce83cb6a02 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304967] Review Request: perl-Tie-Hash-Method - Tied hash with specific methods overridden by callbacks
https://bugzilla.redhat.com/show_bug.cgi?id=1304967 --- Comment #7 from Fedora Update System --- perl-Tie-Hash-Method-0.02-2.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-9cd786629a -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304967] Review Request: perl-Tie-Hash-Method - Tied hash with specific methods overridden by callbacks
https://bugzilla.redhat.com/show_bug.cgi?id=1304967 --- Comment #6 from Fedora Update System --- perl-Tie-Hash-Method-0.02-2.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-fbe71c51b8 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304522] Review Request: python-pg8000 - Pure Python PostgreSQL Driver
https://bugzilla.redhat.com/show_bug.cgi?id=1304522 --- Comment #16 from Fedora Update System --- python-pg8000-1.10.3-2.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-db7ed9df43 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306407] Review Request: ansible1.9 - SSH-based configuration management, deployment, and task execution system
https://bugzilla.redhat.com/show_bug.cgi?id=1306407 --- Comment #2 from Toshio Ernie Kuratomi --- * This package is not for rawhide as there's no need to maintain backwards compatibility there * This package is mostly for EPEL6 and 7 users so they can switch to it while fixing any playbooks to work with ansible-2.0. * This package is intended to have a limited lifetime in EPEL and eventually it will be dropped and people will have to migrate (ansible upstream won't maintain 1.9.x forever). * It's undecided whether this will also go to fedora 22 and 23. Waiting to whether the porting work from 1.9.x playbooks to 2.0.x playbooks can be brought down with newer 2.0.x releases. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306409] New: Review Request: perl-Test2 - Framework for writing test tools that all work together
https://bugzilla.redhat.com/show_bug.cgi?id=1306409 Bug ID: 1306409 Summary: Review Request: perl-Test2 - Framework for writing test tools that all work together Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: ppi...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://ppisar.fedorapeople.org/perl-Test2/perl-Test2.spec SRPM URL: https://ppisar.fedorapeople.org/perl-Test2/perl-Test2-0.25-1.fc24.src.rpm Description: Test2 is a new Perl testing framework produced by forking Test::Builder, completely refactoring it, adding many new features and capabilities. Fedora Account System Username: ppisar -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306407] Review Request: ansible1.9 - SSH-based configuration management, deployment, and task execution system
https://bugzilla.redhat.com/show_bug.cgi?id=1306407 Kevin Fenzi changed: What|Removed |Added Status|NEW |ASSIGNED CC||ke...@scrye.com Assignee|nob...@fedoraproject.org|ke...@scrye.com Flags||fedora-review? --- Comment #1 from Kevin Fenzi --- I'll look at reviewing this soon... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306407] New: Review Request: ansible1.9 - SSH-based configuration management, deployment, and task execution system
https://bugzilla.redhat.com/show_bug.cgi?id=1306407 Bug ID: 1306407 Summary: Review Request: ansible1.9 - SSH-based configuration management, deployment, and task execution system Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: a.bad...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://toshio.fedorapeople.org/ansible/ansible1.9.spec SRPM URL: https://toshio.fedorapeople.org/ansible/ansible1.9-1.9.4-1.fc23.src.rpm Description: Ansible is a radically simple model-driven configuration management, multi-node deployment, and remote task execution system. Ansible works over SSH and does not require any software or daemons to be installed on remote nodes. Extension modules can be written in any language and are transferred to managed machines automatically. This package contains the older ansible-1.9.x package. Some people will have trouble upgrading from 1.9.x to 2.0 so providing ansible-1.9.x in this package as an interim measure. Fedora Account System Username: toshio -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1267009] Review Request: ola - a framework for controlling entertainment lighting equipment
https://bugzilla.redhat.com/show_bug.cgi?id=1267009
--- Comment #25 from Michael Schwendt ---
> %post devel -p /sbin/ldconfig
> %postun devel -p /sbin/ldconfig
As I've seen such scriptlets in -devel packages several times in late 2015,
where does this come from?
It is very unusual for -devel packages to run ldconfig, because ldconfig is a
tool for the runtime linker cache whereas -devel packages only contain symlinks
and files for development-time. If a -devel package doesn't install any runtime
lib, it is superfluous to run ldconfig.
> %package data
> Group: Development/Libraries
> Summary:data for OLA
Sounds like the wrong group for the files in that package. The same group as
the base package would be more sane. ( /usr/share/doc/rpm/GROUPS )
> %check
> export LD_LIBRARY_PATH="%buildroot%{_libdir}"
> make check %{!?el6:%{?_smp_mflags}} #el6 has problems with parralel building
> find %buildroot -name "*\.la" -delete
> find %buildroot -name "*\.a" -delete
>
> %install
Note that %check is executed _after_ %install, so moving the section below the
%install section would be more logical. Prior to %install, %buildroot would be
empty, btw.
> %files data
> %{_datadir}/olad
> %dir %{_datadir}/ola
> %{_datadir}/ola/pids
As a hint here, increase readability of a %files section with explicit trailing
'/' on directories:
%files data
%{_datadir}/olad/
%dir %{_datadir}/ola
%{_datadir}/ola/pids/
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304522] Review Request: python-pg8000 - Pure Python PostgreSQL Driver
https://bugzilla.redhat.com/show_bug.cgi?id=1304522 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #15 from Fedora Update System --- python-pg8000-1.10.3-2.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-7ed3155955 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304040] Review Request: docker-anaconda-addon - Anaconda kickstart support for Docker
https://bugzilla.redhat.com/show_bug.cgi?id=1304040 --- Comment #3 from Brian Lane --- Spec URL: https://bcl.fedorapeople.org/docker-anaconda-addon/docker-anaconda-addon.spec SRPM URL: https://bcl.fedorapeople.org/docker-anaconda-addon/docker-anaconda-addon-0.2-1.fc24.src.rpm This update removes pylint requirement, I didn't add %check since it always gets run when doing the release and in Jenkins. The python3-devel is required by the packaging guidelines (and I think it make sure the right python version is in the buildroot) so I left it. The Makefile wasn't including the ./po/*po files :) So I fixed that and added the necessary bits to the spec to pick up the translations. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #24 from John Dulaney --- I note that the httpd rpm install /var/log/httpd (the folder), but no log files are populated until the httpd server is started. At that point, it is populated. However, uninstalling httpd leaves the log files in place. That said, why would you be uninstalling openqa once you had it on a system? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #8 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7721aaae83 --- Comment #9 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 --- Comment #10 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7721aaae83 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304967] Review Request: perl-Tie-Hash-Method - Tied hash with specific methods overridden by callbacks
https://bugzilla.redhat.com/show_bug.cgi?id=1304967 --- Comment #5 from Fedora Update System --- perl-Tie-Hash-Method-0.02-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-acbd5affee -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #8 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7721aaae83 --- Comment #9 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 --- Comment #10 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7721aaae83 --- Comment #11 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #8 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7721aaae83 --- Comment #9 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #11 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #13 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-fed83465dd -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #11 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 --- Comment #12 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-06a0ba7060 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #13 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-fed83465dd --- Comment #14 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el6 has been submitted as an update to Fedora EPEL 6. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-e0c8bb15b2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #8 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7721aaae83 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #15 from Fedora Update System --- perl-File-Edit-Portable-1.18-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-fed83465dd -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304967] Review Request: perl-Tie-Hash-Method - Tied hash with specific methods overridden by callbacks
https://bugzilla.redhat.com/show_bug.cgi?id=1304967 --- Comment #4 from Fedora Update System --- perl-Tie-Hash-Method-0.02-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-484ed6038a -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306353] Review Request: libunity - Library for integrating with Unity
https://bugzilla.redhat.com/show_bug.cgi?id=1306353 --- Comment #1 from Upstream Release Monitoring --- rdieter's scratch build of libunity-7.1.4-1.fc23.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12929997 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1218758] Review Request: flmsg - Fast Light Message Amateur Radio Forms Manager
https://bugzilla.redhat.com/show_bug.cgi?id=1218758 Richard Shaw changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|i...@cicku.me |nob...@fedoraproject.org Flags|fedora-review? | --- Comment #16 from Richard Shaw --- Review stalled. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306353] Review Request: libunity - Library for integrating with Unity
https://bugzilla.redhat.com/show_bug.cgi?id=1306353 Rex Dieter changed: What|Removed |Added Blocks||1135103 (plasma5) Alias||libunity Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1135103 [Bug 1135103] Plasma 5 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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306353] New: Review Request: libunity - Library for integrating with Unity
https://bugzilla.redhat.com/show_bug.cgi?id=1306353 Bug ID: 1306353 Summary: Review Request: libunity - Library for integrating with Unity Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: rdie...@math.unl.edu QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://rdieter.fedorapeople.org/rpms/libunity/libunity.spec SRPM URL: https://rdieter.fedorapeople.org/rpms/libunity/libunity-7.1.4-1.fc23.src.rpm Description: Library for integrating with Unity Fedora Account System Username: rdieter Being packaged primarily to provide extra features/support for plasma5 per http://blog.broulik.de/2016/01/on-being-more-convenient/ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304040] Review Request: docker-anaconda-addon - Anaconda kickstart support for Docker
https://bugzilla.redhat.com/show_bug.cgi?id=1304040 --- Comment #2 from David Shea --- (In reply to David Shea from comment #1) > - Related to that, po/Makefile doesn't use $(DESTDIR) Never mind, I see $RPM_BUILD_ROOT in there now. > - Huh, nothing owns /usr/share/anaconda/addons. I guess we oughta fix that > in anaconda. https://github.com/rhinstaller/anaconda/pull/500 fixes 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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306299] Review Request: perl-Time-Moment - Represents a date and time of day with an offset from UTC
https://bugzilla.redhat.com/show_bug.cgi?id=1306299 Petr Šabata changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com Assignee|nob...@fedoraproject.org|psab...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303432] Review Request: python-click-threading - Multithreaded support for python click apps
https://bugzilla.redhat.com/show_bug.cgi?id=1303432 --- Comment #5 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/python-click-threading -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1285518] Review Request: perl-File-Edit-Portable - Read and write files while keeping the original line-endings intact
https://bugzilla.redhat.com/show_bug.cgi?id=1285518 --- Comment #7 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/perl-File-Edit-Portable -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol
https://bugzilla.redhat.com/show_bug.cgi?id=749885 Raphael Groner changed: What|Removed |Added CC||projects...@smart.ms Alias|iris| --- Comment #40 from Raphael Groner --- Removing alias to allow general search for 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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305893] Review Request: nodejs-is-utf8 - Detect if a buffer is utf8 encoded
https://bugzilla.redhat.com/show_bug.cgi?id=1305893 --- Comment #5 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-is-utf8 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305696] Review Request: nodejs-repeat-element - Create an array by repeating the given value n times
https://bugzilla.redhat.com/show_bug.cgi?id=1305696 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-repeat-element -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305720] Review Request: nodejs-expand-brackets - Expand POSIX bracket expressions (character classes) in glob patterns
https://bugzilla.redhat.com/show_bug.cgi?id=1305720 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-expand-brackets -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305723] Review Request: nodejs-success-symbol - Cross-platform success symbol
https://bugzilla.redhat.com/show_bug.cgi?id=1305723 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-success-symbol -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305722] Review Request: nodejs-is-extglob - Returns true if a string has an extglob
https://bugzilla.redhat.com/show_bug.cgi?id=1305722 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-is-extglob -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305724] Review Request: nodejs-ansi-wrap - Create ansi colors by passing the open and close codes
https://bugzilla.redhat.com/show_bug.cgi?id=1305724 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-ansi-wrap -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305726] Review Request: nodejs-is-extendable - True if a value is array, regexp, plain object, function or date
https://bugzilla.redhat.com/show_bug.cgi?id=1305726 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-is-extendable -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305846] Review Request: nodejs-for-in - Iterate over the own and inherited enumerable properties of an object
https://bugzilla.redhat.com/show_bug.cgi?id=1305846 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-for-in -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305864] Review Request: nodejs-dotfile-regex - Regular expression for matching dotfiles
https://bugzilla.redhat.com/show_bug.cgi?id=1305864 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-dotfile-regex -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305883] Review Request: nodejs-is-primitive - Returns `true` if the value is a primitive
https://bugzilla.redhat.com/show_bug.cgi?id=1305883 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-is-primitive -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1305899] Review Request: nodejs-md5-o-matic - Fast and simple MD5 hashing utility with zero dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=1305899 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-md5-o-matic -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303430] Review Request: python-click-log - Logging integration for python-click
https://bugzilla.redhat.com/show_bug.cgi?id=1303430 Charalampos Stratakis changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #6 from Charalampos Stratakis --- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819 --- Comment #15 from Fedora Update System --- python-tinydb-3.1.2-3.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-afeb226343 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306299] New: Review Request: perl-Time-Moment - Represents a date and time of day with an offset from UTC
https://bugzilla.redhat.com/show_bug.cgi?id=1306299 Bug ID: 1306299 Summary: Review Request: perl-Time-Moment - Represents a date and time of day with an offset from UTC Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: de...@fateyev.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://www.fateyev.com/RPMS/Fedora22/perl-Time-Moment.spec SRPM URL: http://www.fateyev.com/RPMS/Fedora22/perl-Time-Moment-0.38-1.fc24.src.rpm Description: Time::Moment is an immutable object representing a date and time of day with an offset from UTC in the ISO 8601 calendar system. Fedora Account System Username: dfateyev Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=12927509 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819 --- Comment #14 from Fedora Update System --- python-tinydb-3.1.2-3.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-f0ffd18314 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1189015] Review Request: python-MDAnalysis - Analyze and manipulate molecular dynamics trajectories
https://bugzilla.redhat.com/show_bug.cgi?id=1189015
--- Comment #18 from Antonio Trande ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Permissions on files are set properly.
Note: See rpmlint output
See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions
- doc/sphinx/source/logos is released under CC-BY-ND license.
I guess -doc subpackage must contain an own License tag:
License: CC-BY-ND and GPLv2+
- -doc subpackage does not provide its own doc files
(LICENSE AUTHORS CHANGELOG README SUMMARY.txt)
- The libxdr code for accessing Gromacs trajectories including our
modifications is published under the Lesser GNU Public Licence
(LGPL).
LGPLv2 license is missing.
- Package does not provide python2-MDAnalysis (see Provides list)
Since, sooner or later, this software will support Python3, I think it's
better to package a python2-MDAnalysis already now.
http://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file
- python_provide macro is missing
- Please, fix non-standard-executable-perm and unused-direct-shlib-dependency
errors (see rpmlint output)
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
Note: Unversioned so-files in private %_libdir subdirectory (see
attachment). Verify they are not in ld path.
[x]: Header files in -devel subpackage, if present.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "BSD (3 clause)", "GPL (v2 or later)", "Unknown or generated".
261 files have unknown license. Detailed output of licensecheck in
/home/sagitter/FedoraReview/1189015-python-MDAnalysis/licensecheck.txt
[!]: License file installed when any subpackage combination is installed.
[x]: If the package is under multiple licenses, the licensing breakdown
must be documented in the spec.
[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.
[-]: 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.
[x]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 71680 bytes in 4 files.
[!]: 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]: 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
[Bug 1277484] Review Request: php-nette - Nette Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1277484 Bug 1277484 depends on bug 1277478, which changed state. Bug 1277478 Summary: Review Request: php-nette-deprecated - APIs and features removed from Nette Framework https://bugzilla.redhat.com/show_bug.cgi?id=1277478 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1277478] Review Request: php-nette-deprecated - APIs and features removed from Nette Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1277478 Remi Collet changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |RAWHIDE Last Closed||2016-02-10 08:19:00 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190735] Review Request: mycila-pom - Mycila parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1190735 --- Comment #3 from Nicolas Lécureuil --- i added : %pom_xpath_remove "pom:extension[pom:artifactId[text()='wagon-webdav-jackrabbit']]" and removed BuildRequires: maven-wagon-webdav-jackrabbit it builds fine. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190735] Review Request: mycila-pom - Mycila parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1190735 Nicolas Lécureuil changed: What|Removed |Added CC||mageia-java@nicolaslecureui ||l.fr --- Comment #2 from Nicolas Lécureuil --- we would need to make jackrabbit back: ERROR] Unresolveable build extension: Plugin org.apache.maven.wagon:wagon-webdav-jackrabbit:SYSTEM or one of its dependencies could not be resolved: Cannot access central (https://repo.maven.apache.org/maven2) in offline mode and the artifact org.apache.maven.wagon:wagon-webdav-jackrabbit:jar:SYSTEM has not been downloaded from it before. -> [Help 2] -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819 --- Comment #13 from Denis Fateyev --- Looks fine now ;-) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306258] Review Request: python-Lektor - A static content management system
https://bugzilla.redhat.com/show_bug.cgi?id=1306258 Charalampos Stratakis changed: What|Removed |Added Priority|medium |unspecified -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1306258] New: Review Request: python-Lektor - A static content management system
https://bugzilla.redhat.com/show_bug.cgi?id=1306258 Bug ID: 1306258 Summary: Review Request: python-Lektor - A static content management system Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: cstra...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://copr-dist-git.fedorainfracloud.org/cgit/cstratak/python-Lektor/python-Lektor.git/plain/python-Lektor.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/cstratak/python-Lektor/fedora-rawhide-x86_64/00159006-python-Lektor/python-Lektor-1.1-1.fc24.src.rpm Description: Lektor is a static website generator. It builds out an entire project from static files into many individual HTML pages and has a built-in admin UI and minimal desktop app. Fedora Account System Username: cstratak -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819 --- Comment #12 from Sundeep Anand --- Done. SPEC: https://bitbucket.org/sundeep_co_in/python-pkgs/downloads/python-tinydb.spec SRPM: https://bitbucket.org/sundeep_co_in/python-pkgs/downloads/python-tinydb-3.1.2-3.fc23.src.rpm Koji scratch builds: http://koji.fedoraproject.org/koji/taskinfo?taskID=12926250 (rawhide) http://koji.fedoraproject.org/koji/taskinfo?taskID=12926265 (epel7) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1298165] Review Request: golang-github-hashicorp-go-memdb - Golang in-memory database built on immutable radix trees
https://bugzilla.redhat.com/show_bug.cgi?id=1298165 --- Comment #8 from Fedora Update System --- golang-github-hashicorp-go-memdb-0-0.2.git9ea975b.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-b5b9dcba62 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1277478] Review Request: php-nette-deprecated - APIs and features removed from Nette Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1277478 --- Comment #15 from Fedora Update System --- php-nette-deprecated-2.3.2-1.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-b80cee8936 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301129] Review Request: hstr - Suggest box like shell history completion
https://bugzilla.redhat.com/show_bug.cgi?id=1301129 --- Comment #13 from Fedora Update System --- hstr-1.19-1.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-909a7951af -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303649] Review Request: perl-Tie-Sub - Tying a subroutine, function or method to a hash
https://bugzilla.redhat.com/show_bug.cgi?id=1303649 --- Comment #8 from Fedora Update System --- perl-Tie-Sub-1.001-1.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-9fad327947 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1277478] Review Request: php-nette-deprecated - APIs and features removed from Nette Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1277478 --- Comment #14 from Fedora Update System --- php-nette-deprecated-2.3.2-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-5163d91411 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1286119] Review Request: python-binaryornot - A pure Python package to check if a file is binary or text
https://bugzilla.redhat.com/show_bug.cgi?id=1286119 --- Comment #13 from Fedora Update System --- python-binaryornot-0.4.0-2.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-d302423db2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1277478] Review Request: php-nette-deprecated - APIs and features removed from Nette Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1277478 --- Comment #13 from Fedora Update System --- php-nette-deprecated-2.3.2-1.el6 has been pushed to the Fedora EPEL 6 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-54356eb14f -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819
--- Comment #11 from Denis Fateyev ---
Looks good. Only small corrections:
- BRs `python2-devel` and `pytest` can be also wrapped into `0%{?with_python2}`
conditions (there's no py2 devel packages requirement if no py2 build
provided);
- The same idea to `python2-%{pypi_name}` package declaration (it won't be
built anyway). But this is optional since won't affect anything;
- `rm -fr %{buildroot}%{python2_sitelib}/tests` should go into the conditions
one line above.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1296974] Review Request: python-beanbag - A helper module for accessing REST APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1296974 --- Comment #12 from Fedora Update System --- python-beanbag-1.9.2-2.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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1277476] Review Request: php-nette-bootstrap - Nette Bootstrap
https://bugzilla.redhat.com/show_bug.cgi?id=1277476 --- Comment #23 from Fedora Update System --- php-nette-bootstrap-2.3.4-1.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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1294278] Review Request: python-usbtmc - Python implementation of the USBTMC protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1294278 --- Comment #26 from Fedora Update System --- python-usbtmc-0.7-1.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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301285] Review Request: perl-Mock-Sub - Mock package, object and standard subroutines, with unit testing in mind
https://bugzilla.redhat.com/show_bug.cgi?id=1301285 --- Comment #17 from Fedora Update System --- perl-Mock-Sub-1.06-1.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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301284] Review Request: perl-File-Tempdir - Object interface for File::Temp module
https://bugzilla.redhat.com/show_bug.cgi?id=1301284 --- Comment #19 from Fedora Update System --- perl-File-Tempdir-0.02-1.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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301395] Review Request: perl-Sort-MergeSort - Merge pre-sorted input streams
https://bugzilla.redhat.com/show_bug.cgi?id=1301395 --- Comment #20 from Fedora Update System --- perl-Sort-MergeSort-0.31-2.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://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304774] Review Request: perl-Function-Parameters - Subroutine definitions with parameter lists
https://bugzilla.redhat.com/show_bug.cgi?id=1304774 Petr Pisar changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-Function-Parameters-1. ||0703-1.fc24 Resolution|--- |RAWHIDE Last Closed||2016-02-10 06:04:26 --- Comment #3 from Petr Pisar --- I removed the redundant "perl(warnings)" and added the dependency on gcc. Although I think the gcc dependency is not needed because no stdlib header are included and gcc is called by ExtUtils::CBuilder. 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1277476] Review Request: php-nette-bootstrap - Nette Bootstrap
https://bugzilla.redhat.com/show_bug.cgi?id=1277476 --- Comment #22 from Fedora Update System --- php-nette-bootstrap-2.3.4-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301395] Review Request: perl-Sort-MergeSort - Merge pre-sorted input streams
https://bugzilla.redhat.com/show_bug.cgi?id=1301395 --- Comment #19 from Fedora Update System --- perl-Sort-MergeSort-0.31-2.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301284] Review Request: perl-File-Tempdir - Object interface for File::Temp module
https://bugzilla.redhat.com/show_bug.cgi?id=1301284 --- Comment #18 from Fedora Update System --- perl-File-Tempdir-0.02-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1301285] Review Request: perl-Mock-Sub - Mock package, object and standard subroutines, with unit testing in mind
https://bugzilla.redhat.com/show_bug.cgi?id=1301285 --- Comment #16 from Fedora Update System --- perl-Mock-Sub-1.06-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303430] Review Request: python-click-log - Logging integration for python-click
https://bugzilla.redhat.com/show_bug.cgi?id=1303430 --- Comment #5 from Michele Baldessari --- Ack. Spec URL: http://acksyn.org/files/rpms/python-click-log/python-click-log.spec SRPM URL: http://acksyn.org/files/rpms/python-click-log/python-click-log-0.1.3-3.fc23.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1302283] Review Request: perl-Number-Misc - Handy utilities for numbers
https://bugzilla.redhat.com/show_bug.cgi?id=1302283 --- Comment #10 from Fedora Update System --- perl-Number-Misc-1.2-1.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1302249] Review Request: perl-MemHandle - Supply memory-based FILEHANDLE methods
https://bugzilla.redhat.com/show_bug.cgi?id=1302249 --- Comment #13 from Fedora Update System --- perl-MemHandle-0.07-1.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1266429] Review Request: cmark - CommonMark parsing and rendering
https://bugzilla.redhat.com/show_bug.cgi?id=1266429 --- Comment #34 from Fedora Update System --- cmark-0.23.0-3.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1302285] Review Request: perl-String-Util - String processing utilities
https://bugzilla.redhat.com/show_bug.cgi?id=1302285 --- Comment #10 from Fedora Update System --- perl-String-Util-1.24-1.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303819] Review Request: python-tinydb - A tiny, document oriented database
https://bugzilla.redhat.com/show_bug.cgi?id=1303819 --- Comment #10 from Sundeep Anand --- Thanks for the pointers. SPEC and package updated, links: SPEC: https://bitbucket.org/sundeep_co_in/python-pkgs/downloads/python-tinydb.spec SRPM: https://bitbucket.org/sundeep_co_in/python-pkgs/downloads/python-tinydb-3.1.2-3.fc23.src.rpm Koji scratch builds: http://koji.fedoraproject.org/koji/taskinfo?taskID=12925188 (rawhide) http://koji.fedoraproject.org/koji/taskinfo?taskID=12925185 (epel7) if they look good, will push them to branches. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303432] Review Request: python-click-threading - Multithreaded support for python click apps
https://bugzilla.redhat.com/show_bug.cgi?id=1303432 Charalampos Stratakis changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Charalampos Stratakis --- Seems 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #23 from Ludwig Nussel --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #13) > > "Strictly speaking, the generator is wrong, because generators cannot rely > > on /var being mounted. It will not operate correctly if someone has a system > > with separate /var partitions." > > > > Is there a fix or change you can recommend? Honestly this just came from the > > SUSE spec, I don't even know what it does. > > It adds openqa-worker@.service instance for every item in > /var/lib/openqa/pool/[0-9]* to the openqa-worker.target. The annoying thing > is that things will break in a confusing way if /var is separate: after > initial installation things will be fine, and then after a reboot the units > will not be generated as expected. But maybe that's not that much of a > problem, after all this package will not be installed by too many random > people. The generator was an attempt to make things more convenient, so one doesn't have to configure the number of workers explicitly. It's true that it doesn't work in all deployment scenarios (we use tmpfs for the pool for example where it also doesn't work). If it didn't achieve that goal we may just as well drop it. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1304882] Review Request: openqa - OS-level automated test framework and web UI
https://bugzilla.redhat.com/show_bug.cgi?id=1304882 Ludwig Nussel changed: What|Removed |Added CC||ludwig.nus...@suse.de --- Comment #22 from Ludwig Nussel --- (In reply to awill...@redhat.com from comment #20) > so sadly, just ditching the log file stuff doesn't work, because geekotest > of course can't create /var/log/openqa (as it doesn't have the necessary > permissions on /var/log). d'oh. So effectively we have the same issue > openSUSE has. It's not a bug, it's a feature :-) Daemon users with full access to the directory they are logging to hit logrotate badly in the past: http://openwall.com/lists/oss-security/2011/03/04/16 The safe choice is to have the directory owned by root, so the daemon can only access it's log file. Therefore one has to create an empty file when installing the package. If %post+%ghost is undesirable that could be also achieved with %install and %verify(not md5 size mtime) I guess. If you prefer logging to the journal ie stdout feel free let your openQA package do that instead. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1300698] Review Request: perl-Fuse - Write filesystems in Perl using FUSE
https://bugzilla.redhat.com/show_bug.cgi?id=1300698 Jitka Plesnikova changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-Fuse-0.16.1-2.fc24 Resolution|--- |RAWHIDE Last Closed||2016-02-10 05:07:07 --- Comment #6 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1303430] Review Request: python-click-log - Logging integration for python-click
https://bugzilla.redhat.com/show_bug.cgi?id=1303430 --- Comment #4 from Charalampos Stratakis --- A %licence tag should be included in the SPEC file. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
