[Bug 1056435] New: Review Request: python-nose-parameterized - Decorator for parameterized testing with Nose
https://bugzilla.redhat.com/show_bug.cgi?id=1056435 Bug ID: 1056435 Summary: Review Request: python-nose-parameterized - Decorator for parameterized testing with Nose Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: echevemas...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://echevemaster.fedorapeople.org/python-nose-parameterized/python-nose-parameterized.spec SRPM URL: http://echevemaster.fedorapeople.org/python-nose-parameterized/python-nose-parameterized-0.3.3-1.fc19.src.rpm Description: Library that improving nose test generators with a decorator that fixes the frequent requirement of a second function, the difficulty to separate the data from the tests and the impossibility of working with subclases of unittest.TestCase Fedora Account System Username: echevemaster Tested on koji http://koji.fedoraproject.org/koji/taskinfo?taskID=6438434 -- You are receiving this mail because: You are on the CC list 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 1056235] Review Request: nfspy - An ID-spoofing NFS client
https://bugzilla.redhat.com/show_bug.cgi?id=1056235 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #4 from Fabian Affolter m...@fabian-affolter.ch --- Michael, thanks for the review. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 961223] Review Request: ocrfeeder - A document layout analysis and optical character recognition system
https://bugzilla.redhat.com/show_bug.cgi?id=961223 Felix Schwarz fschw...@fedoraproject.org changed: What|Removed |Added CC||fschw...@fedoraproject.org --- Comment #12 from Felix Schwarz fschw...@fedoraproject.org --- I submitted a (completely untested) patch upstream to fix the pillow incompatibility: https://bugzilla.gnome.org/show_bug.cgi?id=712671 . -- 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 1005779] Review Request: uima-parent-pom - Apache UIMA Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1005779 Michael Simacek msima...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #11 from Michael Simacek msima...@redhat.com --- I'm sorry, I meant you to remove BR on maven-remote-resources-plugin, but typed just maven-resource-plugin by accident. Just remove BR on maven-remote-resources-plugin, because it is disabled and therefore not needed. That's just a simple thing, you can do it during import. -- 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 1056460] New: Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 Bug ID: 1056460 Summary: Review Request: ssdeep - Compute context triggered piecewise hashes Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: fed...@famillecollet.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw2.github.com/remicollet/remirepo/d0c5fe8be01e475c25c86eb0130d298fc9217618/ssdeep/ssdeep.spec SRPM URL: http://rpms.famillecollet.com/SRPMS/ssdeep-2.10-1.remi.src.rpm Description: ssdeep is a program for computing context triggered piecewise hashes (CTPH). Also called fuzzy hashes, CTPH can match inputs that have homologies. Such inputs have sequences of identical bytes in the same order, although bytes in between these sequences may be different in both content and length. Fedora Account System Username: remi Targets: Fedora and EPEL = 5 Copr test builds in http://copr.fedoraproject.org/coprs/remi/morephp/monitor/ -- You are receiving this mail because: You are on the CC list 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 1055391] Review Request: ocaml-easy-format - High-level and functional interface to the Format module
https://bugzilla.redhat.com/show_bug.cgi?id=1055391 Michel Alexandre Salim michel+...@sylvestre.me changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Michel Alexandre Salim michel+...@sylvestre.me --- Thank you very much for reviewing. New Package SCM Request === Package Name: ocaml-easy-format Short Description: High-level and functional interface to the Format module Owners: salimma Branches: f19 f20 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 1055419] Review Request: python-oslo-rootwrap - Oslo Rootwrap
https://bugzilla.redhat.com/show_bug.cgi?id=1055419 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System upda...@fedoraproject.org --- python-oslo-rootwrap-1.0.0-1.el6 has been pushed to the Fedora EPEL 6 testing 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 1047590] Review Request:python-gnupg - Python module for GnuPG
https://bugzilla.redhat.com/show_bug.cgi?id=1047590 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|python-gnupg-0.3.5-4.fc19 |python-gnupg-0.3.5-4.el6 --- Comment #13 from Fedora Update System upda...@fedoraproject.org --- python-gnupg-0.3.5-4.el6 has been pushed to the Fedora EPEL 6 stable 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 --- Comment #7 from Michael Schwendt bugs.mich...@gmx.net --- FYI: https://fedorahosted.org/fpc/ticket/337 -- You are receiving this mail because: You are on the CC list 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added CC||rc040...@freenet.de --- Comment #1 from Ralf Corsepius rc040...@freenet.de --- There is one minor issue with this package: The configure script peeks into the runtime system's file system and appends some directories to CPPFLAGS and LDFLAGS if it finds them. In a mock environment, this causes -I/usr/local/include and -L/usr/local/lib to be appended to GCC calls. There are many ways to fix and/or work-around this issue, e.g.: 1) A clean way would be to comment out the section (grep for for spfx) in configure and configure.ac and patch these files. (apply touch configure configure.ac -r aclocal.m4 after %patch to avoid the autotools being re-run) 2) A less clean way would be to sed out -I/usr/local/include and -L/usr/local/lib from Makefile after having run %configure. I'd go for 1) -- You are receiving this mail because: You are on the CC list 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 1005779] Review Request: uima-parent-pom - Apache UIMA Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1005779 gil cattaneo punto...@libero.it changed: What|Removed |Added Flags||fedora-cvs? --- Comment #12 from gil cattaneo punto...@libero.it --- No problems! Thanks for review! Spec URL: http://gil.fedorapeople.org/uima-parent-pom.spec SRPM URL: http://gil.fedorapeople.org/uima-parent-pom-8-3.fc19.src.rpm New Package SCM Request === Package Name: uima-parent-pom Short Description: Apache UIMA Parent POM Owners: gil Branches: f20 InitialCC: java-sig -- 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 1055108] Review Request: freecolor - Display memory information graphically
https://bugzilla.redhat.com/show_bug.cgi?id=1055108 --- Comment #4 from Jeff Backus jeff.bac...@gmail.com --- (In reply to Denis Fateyev from comment #2) Thanks for the review, Jeff. I'd recommend you to change the review's informal status into the official ;-) (get assigned with it) Wish I could! I haven't been sponsored yet. Sorry! As for the silent setup, I've just fixed that. Added el5-conditional for BuildRoot. Checked debuginfo package, it's valid. 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 --- Comment #2 from Remi Collet fed...@famillecollet.com --- Probably no effect, but fixing seems cleaner: https://github.com/remicollet/remirepo/commit/774638daf0861005ef7839cf754b04a8aab5549a Spec: https://raw2.github.com/remicollet/remirepo/774638daf0861005ef7839cf754b04a8aab5549a/ssdeep/ssdeep.spec Srpm: http://rpms.famillecollet.com/SRPMS/ssdeep-2.10-2.remi.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1055108] Review Request: freecolor - Display memory information graphically
https://bugzilla.redhat.com/show_bug.cgi?id=1055108 --- Comment #5 from Jeff Backus jeff.bac...@gmail.com --- (In reply to Denis Fateyev from comment #3) Manpage is integrated into the upstream version. License is clarified and included as a separate file. Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=6434920 (Rawhide) https://koji.fedoraproject.org/koji/taskinfo?taskID=6434914 (RHEL5) Spec URL: http://www.fateyev.com/RPMS/Fedora19/testing/freecolor.spec SRPM URL: http://www.fateyev.com/RPMS/Fedora19/testing/SRPMS/freecolor-0.9.2-1.fc19. denf.src.rpm License looks good, thanks for getting it updated and pushing the manpage upstream. As far as I can tell, everything is in order. Now we just need someone with authority to officially review 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 --- Comment #3 from Remi Collet fed...@famillecollet.com --- Notice: this library is GPLv2+ but also Perl Artistic License (for inclusion into the PECL). http://news.php.net/php.pecl.dev/7625 I don't think this exception have to be noticed in this package but rather in the pecl/ssdeep extension one. -- You are receiving this mail because: You are on the CC list 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 1005785] Review Request: snowball - Libstemmer Java bindings
https://bugzilla.redhat.com/show_bug.cgi?id=1005785 Michael Simacek msima...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|msima...@redhat.com Flags||fedora-review? --- Comment #5 from Michael Simacek msima...@redhat.com --- i dont need C libraries ... and for now i dont have intentions to package this libs Ok. It is distributed separately by upstream, so it shouldn't be a problem. The description is now ok, but I still insist on changing the name. If someone decided to package the snowball compiler or the C version of the library, there would be problem. snowball-java or libstemmer-java sounds more reasonable. Regarding that POM - why do you add dependency on junit? I don't see any unit test in there. -- 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 985051] Review Request: vcprompt - efficient program to print VCS info on your prompt
https://bugzilla.redhat.com/show_bug.cgi?id=985051
--- Comment #5 from Michael Schwendt bugs.mich...@gmx.net ---
PREFIX=/usr has to be replaced with PREFIX=%{prefix}.
Sorry for disagreeing here, but the guidelines are more lax nowadays:
https://fedoraproject.org/wiki/Packaging:Guidelines#Macros
A hardcoded /usr for this tiny package would not be a blocker. Since the
package has been changed already, that's good. One can actually rebuild it with
--define _prefix /foo, but the benefit is small.
[...]
Summary:An efficient program to print VCS information on your prompt
Even more consise, IMO:
Summary: Print VCS information on your prompt
Source0:http://hg.gerg.ca/vcprompt/archive/%{version}.tar.gz
It would be great, if upstream could add the name vcprompt to the source
tarball. Currently, it's just 1.1.tar.gz, which may be considered enough as
part of a full URL, but on local storage it's too short and confusing.
make %{?_smp_mflags}
https://fedoraproject.org/wiki/Packaging:Guidelines#Compiler_flags
This make invocation in %build ought to set PREFIX=%{_prefix} and
MANDIR=%{_mandir}/man1, too, just in case the build process substitutes those
values into any built file (such as inline help or documentation). That's a
general hint for any package. For many packages there won't be any issues, but
there are exceptions.
$ rpmls -p vcprompt-1.1-1.fc20.x86_64.rpm
-rwxr-xr-x /usr/bin/vcprompt
-rwxr-xr-x /usr/man/man1/vcprompt.1.gz
drwxr-xr-x /usr/share/doc/vcprompt
-rw-r--r-- /usr/share/doc/vcprompt/README.txt
The manual page should not be executable.
The path for the manual page is wrong, but only because the src.rpm includes an
old spec file that's missing some fixes.
If you keep the lines with Spec URL: and SRPM URL: in this ticket
up-to-date, you could point the fedora-review tool at this ticket:
fedora-review -b 985051
Running rpmlint (or rpmlint -i …) on the src.rpm and all built rpms can be
helpful and is a requirement by the review guidelines, too:
vcprompt.x86_64: W: spurious-executable-perm /usr/man/man1/vcprompt.1.gz
vcprompt.x86_64: W: manual-page-warning /usr/man/man1/vcprompt.1.gz 284: a
space character is not allowed in an escape name
vcprompt.x86_64: W: non-standard-dir-in-usr man
Makefile
It includes a check target, which executes fine here, but skips some tests.
Do you think all tests in that test-suite is suiteable for a %check section?
Then add
%check
make check
after the %install section. It may require additional BuildRequires for some of
the tests. If any of the tests require network access, that won't be possible
in the Fedora Build System, however.
--
You are receiving this mail because:
You are on the CC list 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 1053640] Review Request: python-statsmodels - Statistics in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1053640 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2014-01-22 06:53:41 -- You are receiving this mail because: You are on the CC list 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 1051901] Review Request: rubygem-rgen - Ruby Modelling and Generator Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1051901 --- Comment #6 from Sam Kottler skott...@redhat.com --- (In reply to Vít Ondruch from comment #5) * Bump release - It is good habit to bump release for each review cycle. It is easier to see I've done that this time: http://skottler.fedorapeople.org/reviews/rubygem-rgen/rubygem-rgen-0.6.6-2.fc21.src.rpm http://skottler.fedorapeople.org/reviews/rubygem-rgen/rubygem-rgen.spec changes and compare with previous version, if needed. * Don't mark test and Rakefile by %doc macro - They are not documentation, therefore they should not be marked by %doc documentation. - Some even thinks that nothing in -doc subpackage should be marked by as %doc, but I am not in that camp ;) So take it just as a remark. Done. * Remove test and Rakefile from main package - You keep them twice now. Please remove them from the main package and leave them just in -doc subpackage Whoops, fixed. * rpmlint - rpmlint complains about wrong-file-end-of-line-encoding a lot. This might be better to check with upstream. - rubygem-rgen-doc.noarch: W: file-not-utf8 /usr/share/gems/gems/rgen- 0.6.6/test/testmodel/ea_testmodel.xml error seems to be false positive, since the XML explicitly says it is windows-1252 encoded. Probably nothing we can do about it. I'll start working with the upstream to fix this. * Test suite - Please execute the test suite during build. Done. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1005792] Review Request: uima-addons - Apache UIMA Addons components
https://bugzilla.redhat.com/show_bug.cgi?id=1005792 --- Comment #1 from Michael Simacek msima...@redhat.com --- - missing BR on ant-contrib and ant-apache-regexp - the httpclient patch seems empty (changes are just whitespace) - why do you disable TikaAnnotator, Solrcas and ConfigurableFeatureExtractor? emf, tika and solr are available in Fedora - why do you remove those plugins? they are unneeded for you, but they don't do any harm and someone could make use of those OSGi artifacts - this package consists of addons, maybe it would make sense to separate artifacts into subpackages -- 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 1002157] Review Request: randomizedtesting - Java Testing Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1002157
--- Comment #4 from Michael Simacek msima...@redhat.com ---
There are no requires for randomizedtesting-junit4-ant besides jpackage-utils.
Its name suggest it should require at least ant and junit4, but the upstream
uses scopeprovided/scope everywhere. This is probably not a problem for
depending packages, because if they already use it, they had to specify those
dependencies themselves. However it looks like misuse of that scope and you
could ask upstream whether they really meant it that way.
Having '%files -f .mfiles-%{name}-parent' looks error prone. It seems better to
have just '%files -f .mfiles' and use '%mvn_package :%{name}-parent'. If the
upstream ever add something into main package, you wouldn't miss it this way.
--
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 1051771] Review Request: vdr-tvguide - a highly customizable 2D EPG viewer plugin for the VDR
https://bugzilla.redhat.com/show_bug.cgi?id=1051771
--- Comment #6 from MartinKG mgans...@alice.de ---
Spec URL: https://dl.dropboxusercontent.com/s/d1u3vl780ovey8s/vdr-tvguide.spec
SRPM URL:
https://dl.dropboxusercontent.com/s/ax6apuvhn3rl7kl/vdr-tvguide-1.0.0-8.20140117git662a17d.fc20.src.rpm
%changelog
* Wed Jan 22 2014 Martin Gansser marti...@fedoraproject.org -
1.0.0-8.20140117git662a17d
- replaced ImageMagick as requirement due big dependencies through
GraphicsMagick
- added Patch to compile with GraphicsMagick
- replaced RPM_BUILD_ROOT macro through %%{buildroot}
--
You are receiving this mail because:
You are on the CC list 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 1005792] Review Request: uima-addons - Apache UIMA Addons components
https://bugzilla.redhat.com/show_bug.cgi?id=1005792 --- Comment #2 from gil cattaneo punto...@libero.it --- (In reply to Michael Simacek from comment #1) - missing BR on ant-contrib and ant-apache-regexp Done - the httpclient patch seems empty (changes are just whitespace) strange ... - why do you disable TikaAnnotator, Solrcas and ConfigurableFeatureExtractor? emf, tika and solr are available in Fedora They use unavailable or too old build deps e.g. org.apache.uima:uimaj-examples required by ConfigurableFeatureExtractor need some eclipse emf artifacts and maven pom or depmap are unavailable use too old tika version (our tika package is imcompatible...) Lucas require lucene 2.x Solrcas solr-solrj 3.1.0 - why do you remove those plugins? they are unneeded for you, but they don't do any harm and someone could make use of those OSGi artifacts for use these plugin should be enable/built all sub modules - this package consists of addons, maybe it would make sense to separate artifacts into subpackages if possible prefer maintein the package as well as is -- 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 1005785] Review Request: snowball-java - Libstemmer Java bindings
https://bugzilla.redhat.com/show_bug.cgi?id=1005785 gil cattaneo punto...@libero.it changed: What|Removed |Added Summary|Review Request: snowball - |Review Request: |Libstemmer Java bindings|snowball-java - Libstemmer ||Java bindings --- Comment #6 from gil cattaneo punto...@libero.it --- Spec URL: http://gil.fedorapeople.org/snowball-java.spec SRPM URL: http://gil.fedorapeople.org/snowball-java-0-0.1.20130902.fc19.src.rpm - rename pkg - remove junit refs -- 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 1005779] Review Request: uima-parent-pom - Apache UIMA Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1005779 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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 1005779] Review Request: uima-parent-pom - Apache UIMA Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1005779 --- Comment #13 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 --- Comment #8 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 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 1055391] Review Request: ocaml-easy-format - High-level and functional interface to the Format module
https://bugzilla.redhat.com/show_bug.cgi?id=1055391 --- 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 1055391] Review Request: ocaml-easy-format - High-level and functional interface to the Format module
https://bugzilla.redhat.com/show_bug.cgi?id=1055391 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 1055800] Review Request: test-interface - uniform interface to Scala testing frameworks
https://bugzilla.redhat.com/show_bug.cgi?id=1055800 --- Comment #11 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 1055800] Review Request: test-interface - uniform interface to Scala testing frameworks
https://bugzilla.redhat.com/show_bug.cgi?id=1055800 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 1055802] Review Request: sbinary - typed Scala interface to binary formats
https://bugzilla.redhat.com/show_bug.cgi?id=1055802 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 1055802] Review Request: sbinary - typed Scala interface to binary formats
https://bugzilla.redhat.com/show_bug.cgi?id=1055802 --- Comment #8 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 1002157] Review Request: randomizedtesting - Java Testing Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1002157
--- Comment #5 from gil cattaneo punto...@libero.it ---
(In reply to Michael Simacek from comment #4)
There are no requires for randomizedtesting-junit4-ant besides
jpackage-utils. Its name suggest it should require at least ant and junit4,
but the upstream uses scopeprovided/scope everywhere. This is probably
not a problem for depending packages, because if they already use it, they
had to specify those dependencies themselves. However it looks like misuse
of that scope and you could ask upstream whether they really meant it that
way.
this usually is normal ...
Having '%files -f .mfiles-%{name}-parent' looks error prone. It seems better
to have just '%files -f .mfiles' and use '%mvn_package :%{name}-parent'. If
the upstream ever add something into main package, you wouldn't miss it this
way.
is not possible: mvn-package: Exactly 2 arguments are required.
Spec URL: http://gil.fedorapeople.org/randomizedtesting.spec
SRPM URL: http://gil.fedorapeople.org/randomizedtesting-2.0.9-1.fc19.src.rpm
- merge parent pom with runner sub package
--
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 1056555] New: Review Request: ioprocess - process to perform risky IO
https://bugzilla.redhat.com/show_bug.cgi?id=1056555 Bug ID: 1056555 Summary: Review Request: ioprocess - process to perform risky IO Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: smizr...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://smizrahi.fedorapeople.org/ioprocess/ioprocess.spec SRPM URL: http://smizrahi.fedorapeople.org/ioprocess/ioprocess-0.3-1.fc20.src.rpm Description: When performing IO the process goes to an unkillable state (D state) until the IO returns. This is a problem for VDSM since it might try and access storage that is unavailable and will get stuck. Since it can't be killed we can't even start a new service since the port is already taken. There is a hack to solve this issue in VDSM but we decided that making a general purpose tool to solve such problems is the correct way to go as we are not the only ones that might encounter these issues and having a single project to maintain will be simple for everyone. Fedora Account System Username: smizrahi -- You are receiving this mail because: You are on the CC list 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 1055500] Review Request: libgssh - Friendly GIO wrapper for libssh
https://bugzilla.redhat.com/show_bug.cgi?id=1055500 Colin Walters walt...@redhat.com changed: What|Removed |Added CC||walt...@redhat.com --- Comment #3 from Colin Walters walt...@redhat.com --- I should note here that while originally libgssh depended on libgsystem (a copylib), it now only depends on the libgsystem-local-alloc copylib which is a subset of libgsystem - it's about 150 lines of code. -- You are receiving this mail because: You are on the CC list 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 1004256] Review Request: pysysbot - A simple python jabber bot for getting system information
https://bugzilla.redhat.com/show_bug.cgi?id=1004256 --- Comment #11 from Fabian Affolter m...@fabian-affolter.ch --- (In reply to Christopher Meng from comment #8) libstatgrab has been rolled back as 0.90 version breaks too many stuffs. Package Review(truncated) == 1. [!]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: GPL, GPL (v3 or later), Unknown or generated. 2 files have unknown license. Detailed output of licensecheck in /home/rpmaker/Desktop/pysysbot/licensecheck.txt Since you are the author of this software, I hope you can fix them in a new version(well it also depends on if a new version is ready for publish) Thanks, there was indeed a little mess with the license statements. Should be fixed now. 2. [!]: Package requires other packages for directories it uses. Note: No known owner of /etc/pysysbot [!]: Package must own all directories that it creates. Note: Directories without known owners: /etc/pysysbot I think you can understand this. Fixed * Wed Jan 22 2014 Fabian Affolter m...@fabian-affolter.ch - 0.1.3-1 - License update - Update to new upstream release 0.1.3 Updated files: Spec URL: http://fab.fedorapeople.org/packages/SRPMS/pysysbot.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/pysysbot-0.1.3-1.fc20.src.rpm -- 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 1005779] Review Request: uima-parent-pom - Apache UIMA Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1005779 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 1005779] Review Request: uima-parent-pom - Apache UIMA Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1005779 --- Comment #14 from Fedora Update System upda...@fedoraproject.org --- uima-parent-pom-8-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/uima-parent-pom-8-3.fc20 -- 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 --- Comment #9 from Christopher Meng cicku...@gmail.com --- Another FYI https://github.com/BenHanson/lexertl/issues/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 1055802] Review Request: sbinary - typed Scala interface to binary formats
https://bugzilla.redhat.com/show_bug.cgi?id=1055802 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 1055802] Review Request: sbinary - typed Scala interface to binary formats
https://bugzilla.redhat.com/show_bug.cgi?id=1055802 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- sbinary-0.4.2-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/sbinary-0.4.2-1.fc20 -- You are receiving this mail because: You are on the CC list 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 1055800] Review Request: test-interface - uniform interface to Scala testing frameworks
https://bugzilla.redhat.com/show_bug.cgi?id=1055800 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 1055800] Review Request: test-interface - uniform interface to Scala testing frameworks
https://bugzilla.redhat.com/show_bug.cgi?id=1055800 --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- test-interface-1.0-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/test-interface-1.0-2.fc20 -- You are receiving this mail because: You are on the CC list 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 1055391] Review Request: ocaml-easy-format - High-level and functional interface to the Format module
https://bugzilla.redhat.com/show_bug.cgi?id=1055391 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- ocaml-easy-format-1.0.2-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/ocaml-easy-format-1.0.2-2.fc20 -- You are receiving this mail because: You are on the CC list 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 1055391] Review Request: ocaml-easy-format - High-level and functional interface to the Format module
https://bugzilla.redhat.com/show_bug.cgi?id=1055391 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- ocaml-easy-format-1.0.2-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/ocaml-easy-format-1.0.2-2.fc19 -- You are receiving this mail because: You are on the CC list 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 1055391] Review Request: ocaml-easy-format - High-level and functional interface to the Format module
https://bugzilla.redhat.com/show_bug.cgi?id=1055391 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 --- Comment #11 from Jerry James loganje...@gmail.com --- (In reply to Christopher Meng from comment #9) Another FYI https://github.com/BenHanson/lexertl/issues/12 Excellent. I will change the source URL and version number accordingly. 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 1037898] Review Request: python-fmn-consumer - Backend worker daemon for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037898 Patrick Uiterwijk puiterw...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||puiterw...@redhat.com Assignee|nob...@fedoraproject.org|puiterw...@redhat.com --- Comment #2 from Patrick Uiterwijk puiterw...@redhat.com --- I will review this. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1037898] Review Request: python-fmn-consumer - Backend worker daemon for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037898 --- Comment #3 from Ralph Bean rb...@redhat.com --- Upstream release Spec URL: http://threebean.org/rpm/python-fmn-consumer.spec SRPM URL: http://threebean.org/rpm/python-fmn-consumer-0.2.0-1.fc20.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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 --- Comment #10 from Jerry James loganje...@gmail.com --- (In reply to Michael Schwendt from comment #7) FYI: https://fedorahosted.org/fpc/ticket/337 Ah, thanks for the pointer. I will incorporate this idea when I import the 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 1037898] Review Request: python-fmn-consumer - Backend worker daemon for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037898 Bug 1037898 depends on bug 1037897, which changed state. Bug 1037897 Summary: Review Request: python-fmn-lib - Internal API components and model for Fedora Notifications https://bugzilla.redhat.com/show_bug.cgi?id=1037897 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list 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 1037897] Review Request: python-fmn-lib - Internal API components and model for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037897 Ralph Bean rb...@redhat.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |ERRATA Last Closed||2014-01-22 09:55:49 -- You are receiving this mail because: You are on the CC list 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 1037899] Review Request: python-fmn-web - Frontend Web Application for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037899 Bug 1037899 depends on bug 1037897, which changed state. Bug 1037897 Summary: Review Request: python-fmn-lib - Internal API components and model for Fedora Notifications https://bugzilla.redhat.com/show_bug.cgi?id=1037897 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list 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 1037899] Review Request: python-fmn-web - Frontend Web Application for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037899 Ralph Bean rb...@redhat.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |ERRATA Last Closed||2014-01-22 09:59:10 -- You are receiving this mail because: You are on the CC list 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 1037821] Review Request: python-fmn-rules - Message processing rules for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037821 Ralph Bean rb...@redhat.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |ERRATA Last Closed||2014-01-22 09:59:02 -- You are receiving this mail because: You are on the CC list 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 1037897] Review Request: python-fmn-lib - Internal API components and model for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037897 Bug 1037897 depends on bug 1037821, which changed state. Bug 1037821 Summary: Review Request: python-fmn-rules - Message processing rules for Fedora Notifications https://bugzilla.redhat.com/show_bug.cgi?id=1037821 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added Assignee|nob...@fedoraproject.org|rc040...@freenet.de Flags||fedora-review? --- Comment #4 from Ralf Corsepius rc040...@freenet.de --- Taking 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 1003188] Review Request: eclipse-pdt - PHP Development Tools (PDT) Eclipse plugin
https://bugzilla.redhat.com/show_bug.cgi?id=1003188 --- Comment #8 from Alex Villacís Lasso a_villa...@palosanto.com --- (In reply to Alexander Kurtakov from comment #7) Alex, does it happen for every file? The build process creates eclipse-pdt, eclipse-pdt-mylyn, eclipse-pdt-sdk, and eclipse-pdt-tests. I only installed eclipse-pdt for the tests. I opened an existing project that used phpeclipse, the legacy plugin for PHP. All of the PHP files trigger the error. If neither phpeclipse nor pdt are installed, the same project uses the default text editor for PHP files (without syntax highlighting). Even in new project with helloworld kind of file? Not tested. I have to test that. I'm trying to understand whether this is an upstream bug or packaging issue. Mat, how did you decided on the git sha to download? -- 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 1005785] Review Request: snowball-java - Libstemmer Java bindings
https://bugzilla.redhat.com/show_bug.cgi?id=1005785
--- Comment #7 from Michael Simacek msima...@redhat.com ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- It doesn't contain the actual license. The text in license.txt contains just
information about the license, but not the license itself. It could be used
as a notice, but you should attach the actual text of the license (there's a
link to it in the file).
- The summary deosn't reflect what it really is. It's not bindings it's a
standalone library. Something like Java stemming algorithm library should
be enough.
= 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:
Unknown or generated. 20 files have unknown license. Detailed output of
licensecheck in /home/msimacek/1005785-snowball-java/licensecheck.txt
[?]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: 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.
[x]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[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]: 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 do not use a name that already exist
[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
Java:
[x]: Bundled jar/class files should be removed before build
[x]: Packages have proper BuildRequires/Requires on jpackage-utils
Note: Maven packages do not need to (Build)Require jpackage-utils. It is
pulled in by maven-local
[x]: Javadoc documentation files are generated and included in -javadoc
subpackage
[x]: Javadoc subpackages should not have Requires: jpackage-utils
[x]: Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlink)
Maven:
[x]: If package contains pom.xml files install it (including depmaps) even
when building with ant
[x]: POM files have correct Maven mapping
[x]: Maven packages should use new style packaging
[x]: Old add_to_maven_depmap macro is not being used
[x]: Packages DO NOT have Requires(post) and Requires(postun) on jpackage-
utils for %update_maven_depmap macro
[x]: Package DOES NOT use %update_maven_depmap in %post/%postun
[x]: Packages use %{_mavenpomdir} instead of %{_datadir}/maven2/poms
= SHOULD items =
[Bug 1051920] Review Request: udisks2-lvm - LVM DBus add-on for udisks
https://bugzilla.redhat.com/show_bug.cgi?id=1051920 Patrick Uiterwijk puiterw...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |UPSTREAM Last Closed||2014-01-22 10:29:40 --- Comment #8 from Patrick Uiterwijk puiterw...@redhat.com --- This will never be put in Fedora, as it's renamed to storaged: #1054269 -- You are receiving this mail because: You are on the CC list 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 1054269] Review Request: storaged - LVM DBus add-on for udisks
https://bugzilla.redhat.com/show_bug.cgi?id=1054269 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 1054269] Review Request: storaged - LVM DBus add-on for udisks
https://bugzilla.redhat.com/show_bug.cgi?id=1054269 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- storaged-0.1.0-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/storaged-0.1.0-2.fc19 -- You are receiving this mail because: You are on the CC list 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 1054269] Review Request: storaged - LVM DBus add-on for udisks
https://bugzilla.redhat.com/show_bug.cgi?id=1054269 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- storaged-0.1.0-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/storaged-0.1.0-2.fc20 -- You are receiving this mail because: You are on the CC list 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from Ralf Corsepius rc040...@freenet.de --- APPROVED. 2 comments: - By removing the offending lines from configure in your *build-patch, you are destroying configure's line numbering. If you'd comment them out instead, line numbers would remain intact (which is useful in case of problems). - I dislike package which install headers with fairly generic names (fuzzy.h) directly into /usr/include, but this is just my personal preference. -- You are receiving this mail because: You are on the CC list 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Remi Collet fed...@famillecollet.com --- Thanks for the review ! (In reply to Ralf Corsepius from comment #5) - I dislike package which install headers with fairly generic names (fuzzy.h) directly into /usr/include, but this is just my personal preference. Me too... but without any .pc file it will make uneasy for consumer to use this library... choice is not always trivial... New Package SCM Request === Package Name: ssdeep Short Description: Compute context triggered piecewise hashes Owners: remi Branches: f20 el5 el6 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 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 1021017] Review Request: lexertl - Modular lexical analyzer generator
https://bugzilla.redhat.com/show_bug.cgi?id=1021017 --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- lexertl-2013.11.20-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/lexertl-2013.11.20-1.fc20 -- You are receiving this mail because: You are on the CC list 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 1055721] Review Request: qpid-dispatch - Dispatch router for Qpid
https://bugzilla.redhat.com/show_bug.cgi?id=1055721 --- Comment #2 from Darryl L. Pierce dpie...@redhat.com --- (In reply to Christopher Meng from comment #1) Looks like these sysmted macros are not used: https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Systemd Added them, and also will be adding them to other packages I maintain. One request, can you write those %files and %p* section below %prep~%check? Snafu... Done. Updated spec: http://mcpierce.fedorapeople.org/rpms/qpid-dispatch.spec Updated SRPM: http://mcpierce.fedorapeople.org/rpms/qpid-dispatch-0.1-1.fc20.1.src.rpm Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6440136 -- You are receiving this mail because: You are on the CC list 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 --- 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 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 1056654] Review Request: python-narcissus-hub - Hub components for Narcissus, realtime log visualization
https://bugzilla.redhat.com/show_bug.cgi?id=1056654 --- Comment #1 from Ralph Bean rb...@redhat.com --- This package built on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6440186 -- You are receiving this mail because: You are on the CC list 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 1002157] Review Request: randomizedtesting - Java Testing Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1002157
--- Comment #6 from Michael Simacek msima...@redhat.com ---
(In reply to gil cattaneo from comment #5)
Having '%files -f .mfiles-%{name}-parent' looks error prone. It seems better
to have just '%files -f .mfiles' and use '%mvn_package :%{name}-parent'. If
the upstream ever add something into main package, you wouldn't miss it this
way.
is not possible: mvn-package: Exactly 2 arguments are required.
Oh, seems like old version of javapackages-tools (f19 right?). Then it should
be possible to use '%mvn_package :%{name}-parent __default' to achieve the same
thing. mvn_package with single argument should work from f20 on.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- latest upstream version is 2.0.15
- com/carrotsearch/ant/tasks/junit4/templates/json/js directory bundles some
JavaScript:
- jquery.pathchange.js - bundled code under MIT license
- minified jQuery - this is unacceptable, quoting the guidelines If a
JavaScript library typically is shipped as minified or compiled code, it
must be compiled or minified as part of the RPM build process. Shipping
pre-minified or pre-compiled code is unacceptable in Fedora.
- it doesn't own /usr/share/java/randomizedtesting
- patch should have a comment stating what it does and why
= 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.
[?]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated, *No copyright* Apache (v2.0). 358 files have
unknown license. Detailed output of licensecheck in
/home/msimacek/1002157-randomizedtesting/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
Note: No known owner of /usr/share/java/randomizedtesting
[!]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/share/java/randomizedtesting
[!]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[!]: Sources contain only permissible code or content.
[x]: 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.
[x]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[?]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 81920 bytes in 6 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: No rpmlint messages.
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[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]: 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 do not use a name that already exist
[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
Java:
[x]: Bundled jar/class files should be removed before build
[x]: Packages have proper
[Bug 1056654] New: Review Request: python-narcissus-hub - Hub components for Narcissus, realtime log visualization
https://bugzilla.redhat.com/show_bug.cgi?id=1056654 Bug ID: 1056654 Summary: Review Request: python-narcissus-hub - Hub components for Narcissus, realtime log visualization Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: rb...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://ralph.fedorapeople.org//python-narcissus-hub.spec SRPM URL: http://ralph.fedorapeople.org//python-narcissus-hub-0.9.0.1-1.fc20.src.rpm Description: This package contains the hub components for narcissus. Narcissus is a web application that visualizes web server hits as they happen in real time. Features: * IP addresses converted to latitude/longitude, then streamed via WebSockets to `polymaps http://polymaps.org/`_. * Realtime graphs of what countries are downloading what content with `d3 http://d3js.org`_. * `Ømq (zeromq) http://www.zeromq.org/`_ on the backend. * **Fast**. No polling. You can see a demo running live at http://narcissus.rc.rit.edu It is visualizing the logs of http://mirror.rit.edu -- You are receiving this mail because: You are on the CC list 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 517507] Package Review: perl-Locale-Msgfmt - Compile .po files to .mo files
https://bugzilla.redhat.com/show_bug.cgi?id=517507 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added CC||jples...@redhat.com Flags|fedora-cvs+ |fedora-cvs? --- Comment #4 from Jitka Plesnikova jples...@redhat.com --- Package Change Request == Package Name: perl-Locale-Msgfmt New Branches: el5 el6 Owners: rlandmann I am a maintainer of devel branch, but I don't have enough time to maintain it for epel. rlandmann requested the branches and he wants to take care of them. -- 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- ssdeep-2.10-2.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/ssdeep-2.10-2.el5 -- You are receiving this mail because: You are on the CC list 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- ssdeep-2.10-2.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/ssdeep-2.10-2.el6 -- You are receiving this mail because: You are on the CC list 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 517507] Package Review: perl-Locale-Msgfmt - Compile .po files to .mo files
https://bugzilla.redhat.com/show_bug.cgi?id=517507 --- Comment #5 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- 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 517507] Package Review: perl-Locale-Msgfmt - Compile .po files to .mo files
https://bugzilla.redhat.com/show_bug.cgi?id=517507 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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 1056235] Review Request: nfspy - An ID-spoofing NFS client
https://bugzilla.redhat.com/show_bug.cgi?id=1056235 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Flags||fedora-cvs? --- Comment #5 from Fabian Affolter m...@fabian-affolter.ch --- New Package SCM Request === Package Name: nfspy Short Description: An ID-spoofing NFS client Owners: fab Branches: F19 F20 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 1056460] Review Request: ssdeep - Compute context triggered piecewise hashes
https://bugzilla.redhat.com/show_bug.cgi?id=1056460 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- ssdeep-2.10-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/ssdeep-2.10-2.fc20 -- You are receiving this mail because: You are on the CC list 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 1056235] Review Request: nfspy - An ID-spoofing NFS client
https://bugzilla.redhat.com/show_bug.cgi?id=1056235 --- Comment #6 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 1051771] Review Request: vdr-tvguide - a highly customizable 2D EPG viewer plugin for the VDR
https://bugzilla.redhat.com/show_bug.cgi?id=1051771 MartinKG mgans...@alice.de changed: What|Removed |Added Status|NEW |ASSIGNED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1056235] Review Request: nfspy - An ID-spoofing NFS client
https://bugzilla.redhat.com/show_bug.cgi?id=1056235 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 1051771] Review Request: vdr-tvguide - a highly customizable 2D EPG viewer plugin for the VDR
https://bugzilla.redhat.com/show_bug.cgi?id=1051771 MartinKG mgans...@alice.de changed: What|Removed |Added Status|ASSIGNED|NEW --- Comment #7 from MartinKG mgans...@alice.de --- Spec URL: http://martinkg.fedorapeople.org/Review/SPECS/vdr-tvguide.spec SRPM URL: http://martinkg.fedorapeople.org/Review/SRPMS/vdr-tvguide-1.0.0-9.20140122git0d3d2ef.fc20.src.rpm %changelog * Wed Jan 22 2014 Martin Gansser marti...@fedoraproject.org - 1.0.0-9.20140122git0d3d2ef - rebuild for new git release - removed GraphicsMagick patch -- You are receiving this mail because: You are on the CC list 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 1055789] Review Request: rubygem-awesome_spawn - AwesomeSpawn is a module that provides some useful features over Ruby's Kernel.spawn
https://bugzilla.redhat.com/show_bug.cgi?id=1055789 Mo Morsi mmo...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Mo Morsi mmo...@redhat.com --- 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 1006197] Review Request: stockfish - Powerful open source chess engine
https://bugzilla.redhat.com/show_bug.cgi?id=1006197 matthew gatto mattga...@gmail.com changed: What|Removed |Added CC||mattga...@gmail.com --- Comment #7 from matthew gatto mattga...@gmail.com --- Is there a downloadable stockfish RPM build on koji.fedoraproject.org? I was linking to http://kojipkgs.fedoraproject.org//work/tasks/6926/5916926/stockfish-4-1.fc20.x86_64.rpm on the PyChess engines wiki at http://code.google.com/p/pychess/wiki/ChessEngines , but that link isn't working anymore. -- 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 1037898] Review Request: python-fmn-consumer - Backend worker daemon for Fedora Notifications
https://bugzilla.redhat.com/show_bug.cgi?id=1037898 --- Comment #4 from Patrick Uiterwijk puiterw...@redhat.com --- If anyone else wants to take it, go ahead, otherwise I'll do this tomorrow. -- You are receiving this mail because: You are on the CC list 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 1055789] Review Request: rubygem-awesome_spawn - AwesomeSpawn is a module that provides some useful features over Ruby's Kernel.spawn
https://bugzilla.redhat.com/show_bug.cgi?id=1055789 Achilleas Pipinellis axill...@archlinux.gr changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Achilleas Pipinellis axill...@archlinux.gr --- Thanks! New Package SCM Request === Package Name: rubygem-awesome_spawn Short Description: AwesomeSpawn is a module that provides some useful features over Ruby's Kernel.spawn. Owners: axilleas Branches: f19 f20 -- You are receiving this mail because: You are on the CC list 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 1056291] Review Request: python-kdcproxy - MS-KKDCP (kerberos proxy) WSGI module
https://bugzilla.redhat.com/show_bug.cgi?id=1056291 --- Comment #2 from Nathaniel McCallum npmccal...@redhat.com --- Eduardo, I believe that I am following these guidelines. Can you please explain what you believe I have missed? -- You are receiving this mail because: You are on the CC list 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 1048493] Review Request: icecat - GNU version of Firefox browser
https://bugzilla.redhat.com/show_bug.cgi?id=1048493 Antonio Trande anto.tra...@gmail.com changed: What|Removed |Added Flags||needinfo?(de...@fateyev.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 1056291] Review Request: python-kdcproxy - MS-KKDCP (kerberos proxy) WSGI module
https://bugzilla.redhat.com/show_bug.cgi?id=1056291
--- Comment #3 from Eduardo Echeverria echevemas...@gmail.com ---
Sure. @Nathaniel
You should use the available conditional for build py3 packages.
%global with_python3 1
you have to do a copy of your py2 code into py3dir in order to build against
py3
cp -a . %{py3dir}
and fulfill the condition in each part of your build
%if 0%{?with_python3}
pushd %{py3dir}
.
popd
--
You are receiving this mail because:
You are on the CC list 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 1056291] Review Request: python-kdcproxy - MS-KKDCP (kerberos proxy) WSGI module
https://bugzilla.redhat.com/show_bug.cgi?id=1056291 --- Comment #4 from Nathaniel McCallum npmccal...@redhat.com --- Eduardo, this mechanism only applies if you are using the build more than once method. This method is #2 of the 3 methods for providing both python2 and python3 packages. Since I am also the upstream author and I am committing to supporting python2 and python3 in the same upstream tarball, I am following method #1. In this case, two source directories are not required. Neither is the conditional. -- You are receiving this mail because: You are on the CC list 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 1047510] Review Request: php-opencloud - PHP SDK for OpenStack/Rackspace APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1047510 --- Comment #15 from Gregor Tätzner gre...@freenet.de --- (In reply to Adam Williamson from comment #14) Would it be sensible to include upstream's samples/ and docs/ as docs? What do you mean? I'm doing that right now in doc subpkg, not? -- You are receiving this mail because: You are on the CC list 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 950171] Review Request: librecad - Computer Assisted Design (CAD) Application
https://bugzilla.redhat.com/show_bug.cgi?id=950171 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|POST|MODIFIED -- 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 950171] Review Request: librecad - Computer Assisted Design (CAD) Application
https://bugzilla.redhat.com/show_bug.cgi?id=950171 --- Comment #24 from Fedora Update System upda...@fedoraproject.org --- librecad-2.0.2-1.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/librecad-2.0.2-1.fc19 -- 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 1055799] Review Request: sbt - simple build tool
https://bugzilla.redhat.com/show_bug.cgi?id=1055799 Robert Rati rr...@redhat.com changed: What|Removed |Added CC||rr...@redhat.com Assignee|nob...@fedoraproject.org|rr...@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 950171] Review Request: librecad - Computer Assisted Design (CAD) Application
https://bugzilla.redhat.com/show_bug.cgi?id=950171 --- Comment #23 from Fedora Update System upda...@fedoraproject.org --- librecad-2.0.2-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/librecad-2.0.2-1.fc20 -- 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
