[Bug 1255179] Review Request: runc - CLI for running Open Containers
https://bugzilla.redhat.com/show_bug.cgi?id=1255179 --- Comment #13 from Nalin Dahyabhai na...@redhat.com --- Looks pretty good to me. Here's the checklist from fedora-review, with a few items that I'd like to know more about (search for [ ] and [!]): Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = C/C++: [X]: Package does not contain kernel modules. [X]: Package contains no static executables. [x]: 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. Package license is ASL 2.0. [X]: License field in the package spec file matches the actual license. No patches are applied, and the bundled license text is that of ASL 2.0, which is in line with the opencontainers charter. [X]: License file installed when any subpackage combination is installed. Except for the debuginfo, of course. [!]: Package requires other packages for directories it uses. Subdirectories under /usr/share/gocode that aren't owned by golang or the opencontainer-specs package should be owned by this package so that its removal doesn't leave empties around. Note: No known owner of /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/system, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/cgroups/systemd, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/selinux, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/seccomp, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/configs, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/stacktrace, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/cgroups/fs, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/label, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/netlink, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/cgroups, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/devices, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/configs/validate, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/utils, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/nsenter, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/criurpc, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/xattr, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/user, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/integration, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/apparmor [!]: Package must own all directories that it creates. Note: Directories without known owners: /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/system, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/utils, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/configs/validate, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/nsenter, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/cgroups/systemd, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/selinux, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/label, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/netlink, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/criurpc, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/configs, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/seccomp, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/xattr, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/stacktrace, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/integration, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/cgroups/fs, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/user, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/cgroups, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/devices, /usr/share/gocode/src/github.com/opencontainers/runc/libcontainer/apparmor [ ]: %build honors applicable compiler flags or justifies otherwise. Might want to add a comment about the -B flag here, for people like me who didn't know that it tells the go linker to embed a build ID note in the binary. [X]: Package contains no bundled libraries
[Bug 1255505] Review Request: dnf-plugin-system-upgrade - DNF plugin to handle system upgrades
https://bugzilla.redhat.com/show_bug.cgi?id=1255505 --- Comment #4 from Will Woods wwo...@redhat.com --- For new packages, see: https://copr.fedoraproject.org/coprs/wwoods/dnf-plugin-system-upgrade/builds/ -- You are receiving this mail because: You are on the CC list 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 1219540] Review Request: ahc-tools - Tools for RDO-manager automatic health checks
https://bugzilla.redhat.com/show_bug.cgi?id=1219540 John Trowbridge jtrow...@redhat.com changed: What|Removed |Added Fixed In Version||ahc-tools-0.2.0-1.fc22 -- You are receiving this mail because: You are on the CC list 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 1255179] Review Request: runc - CLI for running Open Containers
https://bugzilla.redhat.com/show_bug.cgi?id=1255179 Nalin Dahyabhai na...@redhat.com changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | --- Comment #14 from Nalin Dahyabhai na...@redhat.com --- Jan is already in the packagers group, removing this from the list of FE-NEEDSPONSOR bugs. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1248100] Review Request: python-qpid - The Apache Qpid Python client library for AMQP.
https://bugzilla.redhat.com/show_bug.cgi?id=1248100 Irina Boverman ibove...@redhat.com changed: What|Removed |Added Flags|needinfo?(limburgher@gmail. | |com)| -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1248100] Review Request: python-qpid - The Apache Qpid Python client library for AMQP.
https://bugzilla.redhat.com/show_bug.cgi?id=1248100 Irina Boverman ibove...@redhat.com changed: What|Removed |Added Flags|fedora-cvs? | -- You are receiving this mail because: You are on the CC list 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 1257329] New: Review Request: openstack-ironic-python-agent - A python agent for provisioning and deprovisioning Bare Metal servers.
https://bugzilla.redhat.com/show_bug.cgi?id=1257329 Bug ID: 1257329 Summary: Review Request: openstack-ironic-python-agent - A python agent for provisioning and deprovisioning Bare Metal servers. Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: jtrow...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://trown.fedorapeople.org/openstack-ironic-python-agent.spec SRPM URL: https://trown.fedorapeople.org/openstack-ironic-python-agent-0.1-0.1gite6439ca1d06a50e72b97975da9b3963ba04ba841.src.rpm Description: A python agent for provisioning and deprovisioning Bare Metal servers. Fedora Account System Username: trown I was unable to build on koji[1], but it builds fine locally[2]. [1] https://kojipkgs.fedoraproject.org//work/tasks/388/10840388/build.log [2] https://trown.fedorapeople.org/openstack-ironic-python-agent-0.1-0.1gite6439ca1d06a50e72b97975da9b3963ba04ba841.noarch.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 1213184] Review Request: grantlee-qt5 - Qt string template engine based on the Django template system
https://bugzilla.redhat.com/show_bug.cgi?id=1213184 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@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 1256435] Review Request: metapixel - photomosaic generator
https://bugzilla.redhat.com/show_bug.cgi?id=1256435 Neil Horman nhor...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-cvs? --- Comment #4 from Neil Horman nhor...@redhat.com --- Package Change Request == Package Name: metapixel New Branches: master Owners: nhorman please unretire this package, I have fixes to allow it to build again. -- You are receiving this mail because: You are on the CC list 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 1252918] Review Request: php-tecnickcom-tc-lib-color - PHP library to manipulate various color representations
https://bugzilla.redhat.com/show_bug.cgi?id=1252918 --- Comment #16 from Fedora Update System upda...@fedoraproject.org --- php-tecnickcom-tc-lib-color-1.5.1-1.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update php-tecnickcom-tc-lib-color'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14189 -- You are receiving this mail because: You are on the CC list 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 1252924] Review Request: php-tecnickcom-tc-lib-barcode - PHP library to generate linear and bidimensional barcodes
https://bugzilla.redhat.com/show_bug.cgi?id=1252924 --- Comment #16 from Fedora Update System upda...@fedoraproject.org --- php-tecnickcom-tc-lib-barcode-1.4.1-2.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update php-tecnickcom-tc-lib-barcode'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14207 -- You are receiving this mail because: You are on the CC list 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 1256435] Review Request: metapixel - photomosaic generator
https://bugzilla.redhat.com/show_bug.cgi?id=1256435 Neil Horman nhor...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NOTABUG Last Closed||2015-08-26 15:39:23 --- Comment #3 from Neil Horman nhor...@redhat.com --- sorry about that, I didn't even think to check dist-git. I'll just unretire it. Thanks -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1234208] Review Request: PyPDF2 - A Pure-Python library built as a PDF toolkit
https://bugzilla.redhat.com/show_bug.cgi?id=1234208 Major Hayden ma...@mhtx.net changed: What|Removed |Added CC||itsme_...@yahoo.com Flags||needinfo?(itsme_410@yahoo.c ||om) --- Comment #14 from Major Hayden ma...@mhtx.net --- Ah, okay. I certainly didn't intend to trample on your previous work -- only to assist. I gave you some other pointers (see comment 11) to improve the package a bit and make it easier for someone who is a package sponsor to review. With that said, if you'd like me to own the package, I have no issues there. If you'd like to own it, I'll gladly step back and let you take 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 1248100] Review Request: python-qpid - The Apache Qpid Python client library for AMQP.
https://bugzilla.redhat.com/show_bug.cgi?id=1248100 Irina Boverman ibove...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list 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 1247328] Review Request: sshrc - Bring your .bashrc, .vimrc etc. with you when you ssh
https://bugzilla.redhat.com/show_bug.cgi?id=1247328
--- Comment #8 from William Moreno williamjmore...@gmail.com ---
Package Review
==
Hi
Sorry for the late feedback, this is a simple package, the spec looks good,
but there is some things than can improve the user experience:
1- Can you add a manpage? You can install the maneditor package in Fedora
and write a manpage, even a minimal manpage can improve the user experience,
if you write a manpage please ask upstream to merge it.
2- This package provides 2 binaries, sshrc and moshrc, I am not sure about
whow users will find the second binari based in the name and description
of the package, maybe if you write manpage for sshrc you can refer than
moshrc it is avaible too, note than you will need a man page for each binari.
The missing manpage is not a bloquer, but I want to ask if you can work in it.
= MUST items =
Generic:
PASS: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
PASS: License field in the package spec file matches the actual license.
PASS: Package contains no bundled libraries without FPC exception.
PASS: Changelog in prescribed format.
PASS: Sources contain only permissible code or content.
NA: Package contains desktop file if it is a GUI application.
NA: Development files must be in a -devel package
PASS: Package uses nothing in %doc for runtime.
PASS: Package consistently uses macros (instead of hard-coded directory
names).
PASS: Package is named according to the Package Naming Guidelines.
PASS: Package does not generate any conflict.
PASS: Package obeys FHS, except libexecdir and /usr/target.
PASS: If the package is a rename of another package, proper Obsoletes and
Provides are present.
PASS: Requires correct, justified where necessary.
PASS: Spec file is legible and written in American English.
PASS: Package contains systemd file(s) if in need.
PASS: Package is not known to require an ExcludeArch tag.
PASS: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
PASS: Package complies to the Packaging Guidelines
PASS: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
PASS: Package installs properly.
PASS: Rpmlint is run on all rpms the build produces.
PASS: 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.
PASS: Package requires other packages for directories it uses.
PASS: Package must own all directories that it creates.
PASS: Package does not own files or directories owned by other packages.
PASS: All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
PASS: Package uses either %{buildroot} or $RPM_BUILD_ROOT
PASS: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
PASS: Macros in Summary, %description expandable at SRPM build time.
PASS: Dist tag is present.
PASS: Package does not contain duplicates in %files.
PASS: Permissions on files are set properly.
NA: Package use %makeinstall only when make install DESTDIR=... doesn't
work.
PASS: Package is named using only allowed ASCII characters.
PASS: Package does not use a name that already exists.
PASS: Package is not relocatable.
PASS: Sources used to build the package match the upstream source, as
provided in the spec URL.
PASS: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
PASS: File names are valid UTF-8.
PASS: Packages must not store files under /srv, /opt or /usr/local
= SHOULD items =
Generic:
NA: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
PASS: Final provides and requires are sane (see attachments).
PASS: Latest version is packaged.
PASS: Package does not include license text files separate from upstream.
PASS: Patches link to upstream bugs/comments/lists or are otherwise
justified.
PASS: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
PASS: Package should compile and build into binary rpms on all supported
architectures.
NA: %check is present and all tests pass.
PASS: Packages should try to preserve timestamps of original installed
files.
PASS: Packager, Vendor, PreReq, Copyright tags should not be in spec file
PASS: Sources can be downloaded from URI in Source: tag
PASS: Reviewer should test that the package builds in mock.
PASS: Buildroot is not present
PASS: Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
PASS: No file requires outside
[Bug 1015868] Review Request: python-qutepart - Source code text editor component based on Qt
https://bugzilla.redhat.com/show_bug.cgi?id=1015868 --- Comment #33 from Yajo yajo@gmail.com --- Can I link to a spec from copr? -- You are receiving this mail because: You are on the CC list 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 1246192] Review Request: python-pymod2pkg - python module to package map
https://bugzilla.redhat.com/show_bug.cgi?id=1246192 Jakub Ruzicka jruzi...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list 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 1230872] Review Request: python-trollius-redis - Redis client for the Python event loop PEP3156 for Trollius.
https://bugzilla.redhat.com/show_bug.cgi?id=1230872 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|0.1.4-2.fc21|0.1.4-2.fc22 -- You are receiving this mail because: You are on the CC list 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 1257302] New: Review Request: nodejs-module-not-found-error - Create a module not found error
https://bugzilla.redhat.com/show_bug.cgi?id=1257302 Bug ID: 1257302 Summary: Review Request: nodejs-module-not-found-error - Create a module not found error Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: t...@compton.nu QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://download.compton.nu/tmp/nodejs-module-not-found-error.spec SRPM URL: http://download.compton.nu/tmp/nodejs-module-not-found-error-1.0.1-1.fc22.src.rpm Fedora Account System Username: tomh Description: Create a module not found error. -- You are receiving this mail because: You are on the CC list 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 1257302] Review Request: nodejs-module-not-found-error - Create a module not found error
https://bugzilla.redhat.com/show_bug.cgi?id=1257302 Tom Hughes t...@compton.nu changed: What|Removed |Added Blocks||956806 (nodejs-reviews), ||1223413 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=956806 [Bug 956806] Node.js Review Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1223413 [Bug 1223413] nodejs-proxyquire-1.7.1 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1219807] Review Request: python-appdirs - Python module for determining platform-specific directories
https://bugzilla.redhat.com/show_bug.cgi?id=1219807 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||1.4.0-2.fc22 Resolution|--- |NEXTRELEASE Last Closed||2015-08-26 13:50:01 -- You are receiving this mail because: You are on the CC list 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 1230872] Review Request: python-trollius-redis - Redis client for the Python event loop PEP3156 for Trollius.
https://bugzilla.redhat.com/show_bug.cgi?id=1230872 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- python-trollius-redis-0.1.4-2.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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- anka-coder-fonts-1.100-0.3.20130409hg4348cf4ec395.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14353 -- You are receiving this mail because: You are on the CC list 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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 Fedora Update System upda...@fedoraproject.org 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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- anka-coder-fonts-1.100-0.3.20130409hg4348cf4ec395.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14352 -- You are receiving this mail because: You are on the CC list 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 1230872] Review Request: python-trollius-redis - Redis client for the Python event loop PEP3156 for Trollius.
https://bugzilla.redhat.com/show_bug.cgi?id=1230872 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|0.1.4-2.el7 |0.1.4-2.fc21 -- You are receiving this mail because: You are on the CC list 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 1230872] Review Request: python-trollius-redis - Redis client for the Python event loop PEP3156 for Trollius.
https://bugzilla.redhat.com/show_bug.cgi?id=1230872 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- python-trollius-redis-0.1.4-2.fc21 has been pushed to the Fedora 21 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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- anka-coder-fonts-1.100-0.3.20130409hg4348cf4ec395.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14354 -- You are receiving this mail because: You are on the CC list 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 1256492] Review Request: python-libnuma - Python bindings for the numactl library
https://bugzilla.redhat.com/show_bug.cgi?id=1256492 --- Comment #4 from Guy Streeter stree...@redhat.com --- The combined specfile would have to be edited and checked in to build each of the packages, one at a time. I don't see an advantage to that. Keeping them separate will also allow fixes and updates to be made independently for the two packages. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 Lukas Bezdicka lbezd...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |NEXTRELEASE Last Closed||2015-08-26 13:06:12 -- You are receiving this mail because: You are on the CC list 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 1252924] Review Request: php-tecnickcom-tc-lib-barcode - PHP library to generate linear and bidimensional barcodes
https://bugzilla.redhat.com/show_bug.cgi?id=1252924 --- Comment #17 from Fedora Update System upda...@fedoraproject.org --- php-tecnickcom-tc-lib-barcode-1.4.1-2.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update php-tecnickcom-tc-lib-barcode'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14206 -- You are receiving this mail because: You are on the CC list 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 1252918] Review Request: php-tecnickcom-tc-lib-color - PHP library to manipulate various color representations
https://bugzilla.redhat.com/show_bug.cgi?id=1252918 --- Comment #17 from Fedora Update System upda...@fedoraproject.org --- php-tecnickcom-tc-lib-color-1.5.1-1.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update php-tecnickcom-tc-lib-color'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14191 -- You are receiving this mail because: You are on the CC list 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 1180378] Review Request: rubygem-sigdump - ruby signal handler which dumps backtrace of running threads and number of allocated objects per class
https://bugzilla.redhat.com/show_bug.cgi?id=1180378 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- rubygem-sigdump-0.2.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.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update rubygem-sigdump'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14171 -- You are receiving this mail because: You are on the CC list 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 1251034] Review Request: php-zendframework-zendpdf - Zend Framework ZendPdf component
https://bugzilla.redhat.com/show_bug.cgi?id=1251034 --- Comment #16 from Fedora Update System upda...@fedoraproject.org --- php-zendframework-zendpdf-2.0.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.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update php-zendframework-zendpdf'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14193 -- You are receiving this mail because: You are on the CC list 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 1219807] Review Request: python-appdirs - Python module for determining platform-specific directories
https://bugzilla.redhat.com/show_bug.cgi?id=1219807 --- Comment #15 from Fedora Update System upda...@fedoraproject.org --- python-appdirs-1.4.0-2.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 1247177] Review Request: python-os-client-config - OpenStack Client Configuation Library
https://bugzilla.redhat.com/show_bug.cgi?id=1247177 Bug 1247177 depends on bug 1219807, which changed state. Bug 1219807 Summary: Review Request: python-appdirs - Python module for determining platform-specific directories https://bugzilla.redhat.com/show_bug.cgi?id=1219807 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |NEXTRELEASE -- You are receiving this mail because: You are on the CC list 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 1251034] Review Request: php-zendframework-zendpdf - Zend Framework ZendPdf component
https://bugzilla.redhat.com/show_bug.cgi?id=1251034 --- Comment #15 from Fedora Update System upda...@fedoraproject.org --- php-zendframework-zendpdf-2.0.2-1.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update php-zendframework-zendpdf'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14195 -- You are receiving this mail because: You are on the CC list 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 1256492] Review Request: python-libnuma - Python bindings for the numactl library
https://bugzilla.redhat.com/show_bug.cgi?id=1256492 --- Comment #6 from Guy Streeter stree...@redhat.com --- I promise I'm not trying to be difficult. If that's the correct way to set it up, I'll change it. Perhaps there's something about the way packages are built in the distro I don't understand. I assumed a src.rpm was submitted to the the build system. Is that incorrect? Not being a maintainer, I haven't ever participated in that process. I've looked for information about how you get from source code to built package in the repo, but I can't find it. If I knew that, I could make sure my source is prepared for it. Do you have a link to that? I'd be really happy to see something that says this is how your source tree should be laid out. thanks, --Guy -- You are receiving this mail because: You are on the CC list 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 1213184] Review Request: grantlee-qt5 - Qt string template engine based on the Django template system
https://bugzilla.redhat.com/show_bug.cgi?id=1213184 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Jan Grulich jgrul...@redhat.com --- 1) Typo in BuildRequires: qt5-tools-devel, should be qt5-qttools-devel 2) Use %license macro instead of %doc for the license file 3) Some python files are also under BSD license so maybe the license should be LGPLv2+ and BSD Otherwise looks good, all mentioned issues are not blockers and can be simply fixed during import. 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 1257336] Review Request: nodejs-async-queue - Simple FIFO queue to execute async functions linear
https://bugzilla.redhat.com/show_bug.cgi?id=1257336 Tom Hughes t...@compton.nu changed: What|Removed |Added Blocks||1114313, 956806 ||(nodejs-reviews) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=956806 [Bug 956806] Node.js Review Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1114313 [Bug 1114313] nodejs-tilelive-5.8.2 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256492] Review Request: python-libnuma - Python bindings for the numactl library
https://bugzilla.redhat.com/show_bug.cgi?id=1256492 --- Comment #5 from Jiri Kastner jkast...@redhat.com --- hey man, do you want to get package reviewed? :) one specfile means one review, less mess. i'm not aware of any package maintained in way you want to go. i would understand your attitude if resulting specfile would look like kernel specfile, but that is exception. with two specfiles and two srpms in case of abandoning python2 in future of fedora means retiring packages. i would also understand your attitude if you will have two separated repositories, but you you have code in one repo. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1242886] Review Request: openstack-ironic-inspector - Hardware introspection service for OpenStack Ironic
https://bugzilla.redhat.com/show_bug.cgi?id=1242886 bigswitch rhosp-bugs-inter...@bigswitch.com changed: What|Removed |Added CC||rhosp-bugs-internal@bigswit ||ch.com --- Comment #1 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Hi Dmitry, I see there's 'oslo.log' as requires on pypi, but its missing in the requires section in the spec file. Is that intentional (some other oslo package includes it)? Otherwise it should be added. Everything else looks good. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1257338] New: Review Request: nodejs-progress-stream - Read the progress of a stream
https://bugzilla.redhat.com/show_bug.cgi?id=1257338 Bug ID: 1257338 Summary: Review Request: nodejs-progress-stream - Read the progress of a stream Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: t...@compton.nu QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://download.compton.nu/nodejs/nodejs-progress-stream.spec SRPM URL: http://download.compton.nu/nodejs/nodejs-progress-stream-1.1.1-1.fc22.src.rpm Fedora Account System Username: tomh Description: Read the progress of a stream. Supports speed and eta. Gets the lengths of the stream automatically if you're using the request or http module. You can also pass the length on initiation. Progress-stream will also check to see if the stream already have a length property. -- You are receiving this mail because: You are on the CC list 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 1257338] Review Request: nodejs-progress-stream - Read the progress of a stream
https://bugzilla.redhat.com/show_bug.cgi?id=1257338 Tom Hughes t...@compton.nu changed: What|Removed |Added Blocks||1114313, 956806 ||(nodejs-reviews) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=956806 [Bug 956806] Node.js Review Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1114313 [Bug 1114313] nodejs-tilelive-5.8.2 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1257340] Review Request: nodejs-single-line-log - Keep writing to the same line in the terminal
https://bugzilla.redhat.com/show_bug.cgi?id=1257340 Tom Hughes t...@compton.nu changed: What|Removed |Added Blocks||1257338, 956806 ||(nodejs-reviews) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=956806 [Bug 956806] Node.js Review Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1257338 [Bug 1257338] Review Request: nodejs-progress-stream - Read the progress of a stream -- You are receiving this mail because: You are on the CC list 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 1257340] New: Review Request: nodejs-single-line-log - Keep writing to the same line in the terminal
https://bugzilla.redhat.com/show_bug.cgi?id=1257340 Bug ID: 1257340 Summary: Review Request: nodejs-single-line-log - Keep writing to the same line in the terminal Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: t...@compton.nu QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://download.compton.nu/nodejs/nodejs-single-line-log.spec SRPM URL: http://download.compton.nu/nodejs/nodejs-single-line-log-1.0.0-1.fc22.src.rpm Fedora Account System Username: tomh Description: Node.js module that keeps writing to the same line in the console (or a stream). Very useful when you write progress bars, or a status message during longer operations. Supports multilines. -- You are receiving this mail because: You are on the CC list 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 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin
https://bugzilla.redhat.com/show_bug.cgi?id=1254828 bigswitch rhosp-bugs-inter...@bigswitch.com changed: What|Removed |Added Flags||needinfo?(karlthered@gmail. ||com) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256492] Review Request: python-libnuma - Python bindings for the numactl library
https://bugzilla.redhat.com/show_bug.cgi?id=1256492 Jiri Kastner jkast...@redhat.com changed: What|Removed |Added CC||jkast...@redhat.com Assignee|nob...@fedoraproject.org|jkast...@redhat.com --- Comment #1 from Jiri Kastner jkast...@redhat.com --- instead of 2 specfiles, ther should be one and not in rpm/SPECS folder but in toplevel directory (good for tito for example and for rpmbuild too). -- You are receiving this mail because: You are on the CC list 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 1015868] Review Request: python-qutepart - Source code text editor component based on Qt
https://bugzilla.redhat.com/show_bug.cgi?id=1015868 Jens Lody fed...@jenslody.de changed: What|Removed |Added CC||fed...@jenslody.de --- Comment #35 from Jens Lody fed...@jenslody.de --- (In reply to Yajo from comment #33) Can I link to a spec from copr? In recent copr you can click on Builds, then chose a build by clicking on the build id, then chose the appropriate dist git source, click on the tree-link, then chose the spec-file and use the link for plain view. -- You are receiving this mail because: You are on the CC list 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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com --- Awesome! Thanks for the review Parag! New Package SCM Request === Package Name: anka-coder-fonts Short Description: A mono spaced, courier-width font Upstream URL: http://code.google.com/p/anka-coder-fonts/ Owners: ankursinha Branches: f21 f22 f23 InitialCC: fonts-sig -- You are receiving this mail because: You are on the CC list 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 1256367] Review Request: kf5-kcalendarutils - The KCalendarUtils Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256367 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from Jan Grulich jgrul...@redhat.com --- 1) License should be LGPLv2+ Otherwise looks good, just fix the mentioned issue during import. 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 1015868] Review Request: python-qutepart - Source code text editor component based on Qt
https://bugzilla.redhat.com/show_bug.cgi?id=1015868
Yajo yajo@gmail.com changed:
What|Removed |Added
Flags|needinfo?(yajo@gmail.co |
|m) |
--- Comment #34 from Yajo yajo@gmail.com ---
(In reply to Raphael Groner from comment #29)
= MUST items =
C/C++:
[!]: 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.
How to do this?
Generic:
Python:
[!]: Package meets the Packaging Guidelines::Python
= Please consider to build for Python3.
Seems not possible:
https://github.com/hlamer/qutepart#building-and-installation-on-linux
= SHOULD items =
Generic:
[?]: Package functions as described.
It does.
[?]: Latest version is packaged.
I'll update ASAP to 2.2.2.
[!]: %check is present and all tests pass.
= Maybe consider to execute the provided tests in subfolder.
I added these lines:
%check
%{__python2} tests/run_all.py
Now, when building, I get this:
Ejecutando(%check): /bin/sh -e /var/tmp/rpm-tmp.WGauMV
+ umask 022
+ cd /home/yajo/rpmbuild/BUILD
+ cd qutepart-2.2.0
+ /usr/bin/python2 tests/run_all.py
run_all.py: cannot connect to X server
Seems like I need an X server for testing. How to do that? For now, I removed
%check.
Rpmlint
---
python-qutepart.x86_64: I: enchant-dictionary-not-found es
= Ignore. https://fedorahosted.org/autoqa/ticket/239
I removed the Spanish translation. I guess it will be easier to maintain and
understand for others.
python-qutepart.x86_64: W: incoherent-version-in-changelog 2.2.0-6
['2.2.0-7.1', '2.2.0-7.1']
See comment #30.
= Please fix.
python-qutepart.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/qutepart/syntax/cParser.so 775
= Please fix.
Fixed
Diff spec file in url and in SRPM
-
[...]
Also see comment #30.
SPEC URL:
https://doc-14-44-docs.googleusercontent.com/docs/securesc/ha0ro937gcuc7l7deffksulhg5h7mbp1/qc6jbhsdf33ma0e29lrdv0blr4i1e19m/144056880/12408635912512098830/*/0B6L4jqW88ytdWDZKanBRa0FEbEk?e=download
SRPM URL:
https://doc-0s-44-docs.googleusercontent.com/docs/securesc/ha0ro937gcuc7l7deffksulhg5h7mbp1/abmvj2ng3qfbs27qet3p6rphcie4c1s3/144056880/12408635912512098830/*/0B6L4jqW88ytdMjhpWm5LMjQ2aGs?e=download
--
You are receiving this mail because:
You are on the CC list 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 1251951] Review Request: python-discover - Test discovery for unittest. Backported from Python 2.7 for Python 2.4+
https://bugzilla.redhat.com/show_bug.cgi?id=1251951 --- Comment #2 from Chandan Kumar chkumar...@gmail.com --- Hello Javier, Thanks for reviewing the package. Here is the updated spec file: https://chandankumar.fedorapeople.org/python-discover.spec SRPM: https://chandankumar.fedorapeople.org/python-discover-0.4.0-1.fc22.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10836365 Thanks, Chandan Kumar -- You are receiving this mail because: You are on the CC list 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 1229903] Review Request: NetworkManager-sstp - NetworkManager VPN plugin for SSTP
https://bugzilla.redhat.com/show_bug.cgi?id=1229903 --- Comment #3 from Marcin Zajaczkowski msz...@wp.pl --- You can always become a co-maintainer of NetworkManager-sstp and have the another package to (co-)maintain :). -- You are receiving this mail because: You are on the CC list 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 1256389] Review Request: kf5-kontactinterface - The KontactInterface Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256389 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@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 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin
https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Matthias Runge mru...@redhat.com changed: What|Removed |Added CC||mru...@redhat.com Summary|Review Request: |Review Request: |python-networking-bigswitc |python-networking-bigswitch |h - big switch neutron|- big switch neutron plugin |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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256358] Review Request: kf5-gpgmepp - C++ wrapper and Qt integreation for GpgMe library
https://bugzilla.redhat.com/show_bug.cgi?id=1256358 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@redhat.com Flags||fedora-review+ --- Comment #1 from Jan Grulich jgrul...@redhat.com --- 1) License should be LGPLv2+ Otherwise looks good, just fix the mentioned issue during import. 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 1256367] Review Request: kf5-kcalendarutils - The KCalendarUtils Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256367 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@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 1256389] Review Request: kf5-kontactinterface - The KontactInterface Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256389 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from Jan Grulich jgrul...@redhat.com --- 1) License should be LGPLv2+ Otherwise looks good, just fix the mentioned issue during import. 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 1256365] Review Request: kf5-kcalendarcore - The KCalendarCore library
https://bugzilla.redhat.com/show_bug.cgi?id=1256365 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@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 1256365] Review Request: kf5-kcalendarcore - The KCalendarCore library
https://bugzilla.redhat.com/show_bug.cgi?id=1256365 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from Jan Grulich jgrul...@redhat.com --- 1) License should be LGPLv2+ and GPLv3+ (src/versit/vcc.c is GPLv3+) Otherwise looks good, just fix the mentioned issue during import. 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 1203528] Review Request: rubygem-sensu-settings - The Sensu settings library, loader and validator
https://bugzilla.redhat.com/show_bug.cgi?id=1203528 Daniel Kopeček dkope...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|dkope...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611043] Review Request: python-billiard - Multiprocessing Pool Extensions
https://bugzilla.redhat.com/show_bug.cgi?id=611043 Richard Marko rma...@redhat.com changed: What|Removed |Added CC||rma...@redhat.com Flags|fedora-cvs+ |fedora-cvs? --- Comment #14 from Richard Marko rma...@redhat.com --- Package Change Request == Package Name: python-billiard New Branches: epel7 Owners: fab rmarko InitialCC: -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1218410] Review Request: python-APSscheduler - In-process task scheduler with Cron-like capabilities
https://bugzilla.redhat.com/show_bug.cgi?id=1218410 --- Comment #9 from Fabien Boucher fbouc...@redhat.com --- Hi Tristan, I've tested a build on Koji (https://koji.fedoraproject.org/koji/taskinfo?taskID=10837030) and packages produced on it does not have the same issue like below: rpm -qp --provides ../RPMS/noarch/python-APScheduler-2.1.2-1.fc22.noarch.rpm python-APScheduler = 2.1.2-1.fc22 rpm -qp --requires ../RPMS/noarch/python-APScheduler-doc-2.1.2-1.fc22.noarch.rpm | grep python python-APScheduler(x86-64) = 2.1.2-1.fc22 So for the packaging part, I'm ok with it. Nevertheless it seems there is like a random issue that appears during unit tests like here: http://koji.fedoraproject.org/koji/taskinfo?taskID=10837176 So maybe it is nice to have a look and/or check with upstream about that, or even deactivate that specific test. I won't block on that and let you decide. Fabien -- You are receiving this mail because: You are on the CC list 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 1257178] Review Request: python-yaql - Yet Another Query Language
https://bugzilla.redhat.com/show_bug.cgi?id=1257178 Marcos marcos.fermin.l...@cern.ch changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 --- Comment #8 from Lukas Bezdicka lbezd...@redhat.com --- SRCRPM URL: https://social.fedorapeople.org/python-editor-0.4-1.fc22.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 1218410] Review Request: python-APSscheduler - In-process task scheduler with Cron-like capabilities
https://bugzilla.redhat.com/show_bug.cgi?id=1218410 Fabien Boucher fbouc...@redhat.com changed: What|Removed |Added 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 1257178] New: Review Request: python-yaql - Yet Another Query Language
https://bugzilla.redhat.com/show_bug.cgi?id=1257178 Bug ID: 1257178 Summary: Review Request: python-yaql - Yet Another Query Language Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: marcos.fermin.l...@cern.ch QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://mferminl.web.cern.ch/mferminl/fedorapkg/python-yaql/python-yaql.spec SRPM URL: http://mferminl.web.cern.ch/mferminl/fedorapkg/python-yaql/python-yaql-0.2.7-1.fc23.src.rpm Description: YAQL library is a query language library. YAQL library has a out of the box large set of commonly used functions. But they are not built-in. All the functions and operators (which are also function: `a + b = operator_+(a, b)` etc) are user-supplied. User is free to add other functions that could be used in expressions and to remove standard ones. Fedora Account System Username: mflobo This library is part of the OpenStack project and it's a requirement for other components not yet available in Fedora/EPEL, as python-muranoclient or openstack-murano. This is my first package, and I am looking for a sponsor. I have some experience reviewing, developing and deploying OpenStack components. See https://launchpad.net/~marcos-fermin-lobo Successful koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10836848 -- You are receiving this mail because: You are on the CC list 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 1254851] Review Request: python-os-testr : A testr wrapper to provide functionality for OpenStack projects
https://bugzilla.redhat.com/show_bug.cgi?id=1254851 Chandan Kumar chkumar...@gmail.com changed: What|Removed |Added Flags|fedora-review+ fedora-cvs? |fedora-review? --- Comment #3 from Chandan Kumar chkumar...@gmail.com --- Dropping review flags -- You are receiving this mail because: You are on the CC list 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 1256393] Review Request: kf5-syndication - The Syndication Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256393 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@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 1256393] Review Request: kf5-syndication - The Syndication Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256393 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from Jan Grulich jgrul...@redhat.com --- 1) License should be LGPLv2+ and BSD Otherwise looks good, just fix the mentioned issue during import. 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 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin
https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #17 from Ihar Hrachyshka ihrac...@redhat.com --- I think there are several issues with the service files I see at: https://github.com/xinwu/python-networking-bigswitch/blob/rpm-master/ Specifically, 1. they read ovs_neutron_agent.ini. Why not the agent specific file? do you really use ovs in bigswitch plugin? 2. they don't read configuration from /etc/neutron/conf.d/common and /etc/neutron/conf.d/neutron-service-name directories. The latter directory should be created as part of build process in the spec file, and added to corresponding (sub)packages. Please refer to existing neutron services to get an idea. 3. please rename neutron-bsn-agent.log into just bsn-agent.log since the path already suggests it's neutron. Do we have a Delorean pull request for 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 1244977] Review Request: libfabric -Open Fabric Interfaces
https://bugzilla.redhat.com/show_bug.cgi?id=1244977 gil cattaneo punto...@libero.it changed: What|Removed |Added CC||punto...@libero.it --- Comment #1 from gil cattaneo punto...@libero.it --- hi seem missing src rpm ... available newer release 1.1.0 this package use hadoop? regards -- You are receiving this mail because: You are on the CC list 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 1244977] Review Request: libfabric -Open Fabric Interfaces
https://bugzilla.redhat.com/show_bug.cgi?id=1244977 --- Comment #2 from gil cattaneo punto...@libero.it --- (In reply to gil cattaneo from comment #1) available newer release 1.1.0 https://github.com/ofiwg/libfabric/tags -- You are receiving this mail because: You are on the CC list 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 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 Chandan Kumar chkumar...@gmail.com changed: What|Removed |Added 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 1256392] Review Request: kf5-ktnef - The KTNEF Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256392 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from Jan Grulich jgrul...@redhat.com --- 1) License should be LGPLv2+ Otherwise looks good, just fix the mentioned issue during import. 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 1256377] Review Request:java-mersenne-twister - Mersenne Twister random number generator in Java
https://bugzilla.redhat.com/show_bug.cgi?id=1256377 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Jonathan Underwood jonathan.underw...@gmail.com --- New Package SCM Request === Package Name: java-mersenne-twister Short Description: Mersenne Twister random number generator in Java Upstream URL: http://www.cs.gmu.edu/~sean/research/ Owners: jgu Branches: f21 f22 f23 InitialCC: -- You are receiving this mail because: You are on the CC list 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 1256392] Review Request: kf5-ktnef - The KTNEF Library
https://bugzilla.redhat.com/show_bug.cgi?id=1256392 Jan Grulich jgrul...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@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 1203961] Review Request: rubygem-sensu-em - Ruby/EventMachine library
https://bugzilla.redhat.com/show_bug.cgi?id=1203961 Daniel Kopeček dkope...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|dkope...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1203046] Review Request: rubygem-effin_utf8 - Enforces default internal and external encodings to be UTF-8 on Ruby 1.9
https://bugzilla.redhat.com/show_bug.cgi?id=1203046 Daniel Kopeček dkope...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|dkope...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1209299] Review Request: rubygem-msgpack - MessagePack, a binary-based efficient data interchange format
https://bugzilla.redhat.com/show_bug.cgi?id=1209299 Daniel Kopeček dkope...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|dkope...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256377] Review Request:java-mersenne-twister - Mersenne Twister random number generator in Java
https://bugzilla.redhat.com/show_bug.cgi?id=1256377 --- Comment #5 from Jonathan Underwood jonathan.underw...@gmail.com --- (In reply to Jerry James from comment #4) Just to satisfy my curiosity, what was the point of changing the BR: jpackage-utils to BR: javapackages-tools? It doesn't appear to me that any of the tools are actually used. Simply that jpackage-utils was renamed to javapackages-tools which Provides jpackage-utils (afaics anyway). It's a little odd that the -javadoc subpackage has a Group tag when the main package does not. I suggest dropping the Group tag in -javadoc. OK, will do. There are two SHOULD items that you don't have to address, but, well, should. - Ask upstream to include a license file. Well... as you know upstream doesn't distribute these as a packaged tarball or anything, simply making the two files available for download, so I'm not sure it makes much sense to ask. I'll mention it anyway, when I send the patch. - Write a %check script Will give that some thought. Many thanks for the review, and the work on the original package! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 --- Comment #7 from Lukas Bezdicka lbezd...@redhat.com --- New scratch build with fixed rpmlint issues: ScratchBuild: http://koji.fedoraproject.org/koji/taskinfo?taskID=10837750 -- You are receiving this mail because: You are on the CC list 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 1254851] Review Request: python-os-testr : A testr wrapper to provide functionality for OpenStack projects
https://bugzilla.redhat.com/show_bug.cgi?id=1254851 Chandan Kumar chkumar...@gmail.com changed: What|Removed |Added CC||chkumar...@gmail.com Flags|fedora-review? |fedora-review+ fedora-cvs? --- Comment #2 from Chandan Kumar chkumar...@gmail.com --- New Package SCM Request === Package Name: python-os-testr Short Description: A testr wrapper to provide functionality for OpenStack projects Upstream URL: http://git.openstack.org/cgit/openstack/python-editor Owners: chandankumar Branches: f22 f23 InitialCC: apevec -- You are receiving this mail because: You are on the CC list 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 1254851] Review Request: python-os-testr : A testr wrapper to provide functionality for OpenStack projects
https://bugzilla.redhat.com/show_bug.cgi?id=1254851 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from Parag AN(पराग) panem...@gmail.com --- As I have already had a brief look at this package and provided suggestions in comment#1, I am going to accept this now. Thanks for quick update :) 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 1256492] Review Request: python-libnuma - Python bindings for the numactl library
https://bugzilla.redhat.com/show_bug.cgi?id=1256492 --- Comment #2 from Guy Streeter stree...@redhat.com --- Jiri, Thank you for reviewing this package. The source is used to build two separate packages, python-libnuma and python3-libnuma. That's the reason there are 2 specfiles. I don't know what tito is. Is that something I need? Why does rmpbuild need the specfiles in the top-level directory? The package is designed to build and install from setup.py for platforms that don't use rpm. It seems like packaging-specific files should be in their own directory. tuna, python-linux-procfs, and python-schedutils all have this directory layout. The top-level Makefile has a target to build rpms. thanks again, --Guy -- You are receiving this mail because: You are on the CC list 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 844740] Review Request: qmmp-plugin-pack - A set of extra plugins for Qmmp
https://bugzilla.redhat.com/show_bug.cgi?id=844740 Karel Volný kvo...@redhat.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #16 from Karel Volný kvo...@redhat.com --- Package Change Request == Package Name: qmmp New Branches: epel7 Owners: kvolny InitialCC: nucleo see bug #1242361 -- You are receiving this mail because: You are on the CC list 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 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks|1245848 | Depends On||1245848 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1245848 [Bug 1245848] python-alembic-0.8.2 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1254851] Review Request: python-os-testr : A testr wrapper to provide functionality for OpenStack projects
https://bugzilla.redhat.com/show_bug.cgi?id=1254851 Chandan Kumar chkumar...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Chandan Kumar chkumar...@gmail.com --- New Package SCM Request === Package Name: python-os-testr Short Description: A testr wrapper to provide functionality for OpenStack projects Upstream URL: http://git.openstack.org/cgit/openstack/python-editor Owners: chandankumar Branches: f22 f23 InitialCC: apevec -- You are receiving this mail because: You are on the CC list 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 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||1245848 Depends On|1245848 | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1245848 [Bug 1245848] python-alembic-0.8.2 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 280751] Review Request: qmmp - Qt-based multimedia player
https://bugzilla.redhat.com/show_bug.cgi?id=280751 Karel Volný kvo...@redhat.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #33 from Karel Volný kvo...@redhat.com --- Package Change Request == Package Name: qmmp New Branches: epel7 Owners: kvolny InitialCC: nucleo see bug #1242361 -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1231427] Review Request: COPASI - Biochemical network simulator
https://bugzilla.redhat.com/show_bug.cgi?id=1231427 --- Comment #22 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list 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 1231427] Review Request: COPASI - Biochemical network simulator
https://bugzilla.redhat.com/show_bug.cgi?id=1231427 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list 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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 --- Comment #7 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list 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 949954] Review Request: anka-coder-fonts - A mono spaced, courier-width font
https://bugzilla.redhat.com/show_bug.cgi?id=949954 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list 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 1256377] Review Request:java-mersenne-twister - Mersenne Twister random number generator in Java
https://bugzilla.redhat.com/show_bug.cgi?id=1256377 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list 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 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 --- Comment #11 from Lukas Bezdicka soc...@v3.sk --- New Package SCM Request === Package Name: python-editor Short Description: Programmatically open an editor, capture the result. Upstream URL: https://github.com/fmoo/python-editor Owners: social apevec Branches: f22 f23 epel7 InitialCC: -- You are receiving this mail because: You are on the CC list 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 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353 --- Comment #6 from Lukas Bezdicka lbezd...@redhat.com --- Updated to 0.4 and created koji build. ScratchBuild: http://koji.fedoraproject.org/koji/taskinfo?taskID=10836458 -- You are receiving this mail because: You are on the CC list 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 1254851] Review Request: python-os-testr : A testr wrapper to provide functionality for OpenStack projects
https://bugzilla.redhat.com/show_bug.cgi?id=1254851 --- Comment #4 from Chandan Kumar chkumar...@gmail.com --- Here is the updated spec and srpms with fixes: SPEC : https://chandankumar.fedorapeople.org/python-os-testr.spec SRPM: https://chandankumar.fedorapeople.org/python-os-testr-0.3.0-1.fc22.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10838865 -- You are receiving this mail because: You are on the CC list 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 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.
https://bugzilla.redhat.com/show_bug.cgi?id=1256353
--- Comment #9 from Chandan Kumar chkumar...@gmail.com ---
Looks good now.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[ ]: 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: Unknown or generated. 2 files have unknown license. Detailed
output of licensecheck in /home/chandankumar/review-python-
editor/licensecheck.txt
[ ]: License file installed when any subpackage combination is installed.
[ ]: Package contains no bundled libraries without FPC exception.
[ ]: Changelog in prescribed format.
[ ]: 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
[ ]: Package uses nothing in %doc for runtime.
[ ]: Package consistently uses macros (instead of hard-coded directory
names).
[ ]: Package is named according to the Package Naming Guidelines.
[ ]: Package does not generate any conflict.
[ ]: Package obeys FHS, except libexecdir and /usr/target.
[ ]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[ ]: Requires correct, justified where necessary.
[ ]: Spec file is legible and written in American English.
[ ]: Package contains systemd file(s) if in need.
[ ]: 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.
[ ]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
Python:
[ ]: Python eggs must not download any dependencies during the build
process.
[ ]: A package which is used by another package via an egg interface should
provide egg info.
[ ]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[ ]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[ ]: Final provides and requires are sane (see attachments).
[ ]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
python3-editor
[ ]: Package functions as described.
[ ]: Latest version is packaged.
[ ]: Package does not include license text files separate from upstream.
[ ]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[ ]: Package should compile and build into binary rpms on all supported
architectures.
[ ]: %check is present and all tests pass.
[ ]: Packages should try to preserve timestamps of original installed
files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should
