[Bug 1008799] New: Review Request: python-beanstalkc - A simple beanstalkd client library for Python
https://bugzilla.redhat.com/show_bug.cgi?id=1008799 Bug ID: 1008799 Summary: Review Request: python-beanstalkc - A simple beanstalkd client library for Python Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: dkho...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: https://bitbucket.org/dhiru/packages/raw/master/python-beanstalkc/python-beanstalkc.spec SRPM URL: https://bitbucket.org/dhiru/packages/raw/master/python-beanstalkc/python-beanstalkc-0.3.0-1.fc19.src.rpm Description: beanstalkc is a simple beanstalkd client library for Python. beanstalkd is a fast, distributed, in-memory workqueue service. Fedora Account System Username: halfie Koji Link: http://koji.fedoraproject.org/koji/taskinfo?taskID=5943933 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JGvCuepFqMa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008473] Review Request: perl-Class-Tiny - Minimalist class construction
https://bugzilla.redhat.com/show_bug.cgi?id=1008473 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags||fedora-cvs? --- Comment #6 from Jitka Plesnikova jples...@redhat.com --- New Package SCM Request === Package Name: perl-Class-Tiny Short Description: Minimalist class construction Owners: jplesnik ppisar psabata Branches: f20 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PAizUCedLka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008809] New: Review Request: perl-Path-IsDev - Determine if a given Path resembles a development source tree
https://bugzilla.redhat.com/show_bug.cgi?id=1008809 Bug ID: 1008809 Summary: Review Request: perl-Path-IsDev - Determine if a given Path resembles a development source tree Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jples...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://jplesnik.fedorapeople.org/perl-Path-IsDev/perl-Path-IsDev.spec SRPM URL: http://jplesnik.fedorapeople.org/perl-Path-IsDev/perl-Path-IsDev-0.3.0-1.fc21.src.rpm Description: This module is more or less a bunch of heuristics for determining if a given path is a development tree root of some kind. Fedora Account System Username: jplesnik -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9dbRn74KgGa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008809] Review Request: perl-Path-IsDev - Determine if a given Path resembles a development source tree
https://bugzilla.redhat.com/show_bug.cgi?id=1008809 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added Depends On||1008473 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=qem7whOvcGa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008816] New: Review Request: libkkc-data - Language model data for libkkc
https://bugzilla.redhat.com/show_bug.cgi?id=1008816 Bug ID: 1008816 Summary: Review Request: libkkc-data - Language model data for libkkc Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: du...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://ueno.fedorapeople.org//libkkc-data.spec SRPM URL: http://ueno.fedorapeople.org//libkkc-data-0.2.7-1.fc21.src.rpm Description: The libkkc-data package contains the language model data that libkkc uses at run time. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=sJAxPAuxSea=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008799] Review Request: python-beanstalkc - A simple beanstalkd client library for Python
https://bugzilla.redhat.com/show_bug.cgi?id=1008799 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com --- + is ok - is needs attention Review: + Package builds find in mock rawhide - rpmlint on rpms gave python-beanstalkc.src: W: spelling-error Summary(en_US) beanstalkd - beanstalks, beanstalk, beanstalk d python-beanstalkc.src: W: spelling-error %description -l en_US beanstalkd - beanstalks, beanstalk, beanstalk d python-beanstalkc.src: W: spelling-error %description -l en_US workqueue - work queue, work-queue, workhouse python-beanstalkc.noarch: W: spelling-error Summary(en_US) beanstalkd - beanstalks, beanstalk, beanstalk d python-beanstalkc.noarch: W: spelling-error %description -l en_US beanstalkd - beanstalks, beanstalk, beanstalk d python-beanstalkc.noarch: W: spelling-error %description -l en_US workqueue - work queue, work-queue, workhouse python-beanstalkc.noarch: W: no-documentation python-beanstalkc.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/beanstalkc.py 0644L /usr/bin/env 2 packages and 0 specfiles checked; 1 errors, 7 warnings. == Fix non-executable-script error by adding at the end of %prep section sed -i -e '/^#!\//, 1d' beanstalkc.py + Source verified with upstream as (sha256sum) srpm tarball : 45c3915ed876f80712fcafadee0feaaf0628e5c297e33aaea7f7b229deb80467 upstream tarball : 45c3915ed876f80712fcafadee0feaaf0628e5c297e33aaea7f7b229deb80467 + License is valid and included in actual source file header APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=1TDG1CdPuYa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008816] Review Request: libkkc-data - Language model data for libkkc
https://bugzilla.redhat.com/show_bug.cgi?id=1008816 --- Comment #1 from Daiki Ueno du...@redhat.com --- This is split from the libkkc package to reduce the build time of the package. libkkc-data takes some time to build, libkkc doesn't need it at build time, and libkkc-data upstream is less frequently updated than libkkc. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0fuV6bdUPWa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008816] Review Request: libkkc-data - Language model data for libkkc
https://bugzilla.redhat.com/show_bug.cgi?id=1008816 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=rE7qR81u0ja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008809] Review Request: perl-Path-IsDev - Determine if a given Path resembles a development source tree
https://bugzilla.redhat.com/show_bug.cgi?id=1008809 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added Blocks||1008828 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Fmqbn2jTBFa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008828] Review Request: perl-Path-FindDev - Find a development path somewhere in an upper hierarchy
https://bugzilla.redhat.com/show_bug.cgi?id=1008828 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added Depends On||1008809 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Ltd1gWtYQDa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008828] New: Review Request: perl-Path-FindDev - Find a development path somewhere in an upper hierarchy
https://bugzilla.redhat.com/show_bug.cgi?id=1008828 Bug ID: 1008828 Summary: Review Request: perl-Path-FindDev - Find a development path somewhere in an upper hierarchy Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jples...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://jplesnik.fedorapeople.org/perl-Path-FindDev/perl-Path-FindDev.spec SRPM URL: http://jplesnik.fedorapeople.org/perl-Path-FindDev/perl-Path-FindDev-0.2.0-1.fc21.src.rpm Description: This package is mostly a glue layer around Path::IsDev with a few directory walking tricks. Fedora Account System Username: jplesnik -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MuJqVAmksxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008856] New: Review Request: perl-File-ShareDir-ProjectDistDir - Simple set-and-forget using of a '/share' directory in your projects root
https://bugzilla.redhat.com/show_bug.cgi?id=1008856 Bug ID: 1008856 Summary: Review Request: perl-File-ShareDir-ProjectDistDir - Simple set-and-forget using of a '/share' directory in your projects root Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jples...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://jplesnik.fedorapeople.org/perl-File-ShareDir-ProjectDistDir/perl-File-ShareDir-ProjectDistDir.spec SRPM URL: http://jplesnik.fedorapeople.org/perl-File-ShareDir-ProjectDistDir/perl-File-ShareDir-ProjectDistDir-0.5.1-1.fc21.src.rpm Description: This module allows to use a directory name other than 'share' ( assuming you make sure when you install that, you specify the different directory there also ). Fedora Account System Username: jplesnik -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9KgMpuKGU9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008856] Review Request: perl-File-ShareDir-ProjectDistDir - Simple set-and-forget using of a '/share' directory in your projects root
https://bugzilla.redhat.com/show_bug.cgi?id=1008856 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added Blocks||1008407 Depends On||1008809, 1008828 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nAizgSs9gka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008828] Review Request: perl-Path-FindDev - Find a development path somewhere in an upper hierarchy
https://bugzilla.redhat.com/show_bug.cgi?id=1008828 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added Blocks||1008856 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=WwMLXdQOcea=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008809] Review Request: perl-Path-IsDev - Determine if a given Path resembles a development source tree
https://bugzilla.redhat.com/show_bug.cgi?id=1008809 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Blocks||1008856 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bZKXiAr0Wra=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 990544] Review Request: mangler - VOIP client capable of connecting to Ventrilo 3.x servers
https://bugzilla.redhat.com/show_bug.cgi?id=990544 Palle Ravn ravn...@gmail.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #7 from Palle Ravn ravn...@gmail.com --- Just updated them, all files can be found at http://zom.dk/mangler/ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Q330aKXtiGa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 912960] Review Request: rubygem-gdk3 - Ruby binding of GDK-3.x
https://bugzilla.redhat.com/show_bug.cgi?id=912960 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #42 from Miroslav Suchý msu...@redhat.com --- My update: I would accept those headers on that place, since this is probably first package where this happen. But please consider bringing up this discussion to ruby-sig and find some final and better solution. I would accept that waive of these, but only if: * you put there in comment why it is currently waived out (does not work on F21 because of...) * if you put there in comment link to upstream report of that issue. And once those tests are functional again in future, remove that || echo. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=HW1nqLi3Qda=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 839056] Review Request: python-flake8 - code checking using pep8 and pyflakes
https://bugzilla.redhat.com/show_bug.cgi?id=839056 Matthias Runge mru...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #18 from Matthias Runge mru...@redhat.com --- Once that python3-pyflakes is in f20, IMHO we can continue here. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ku50opXhgna=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 816991] Review Request: rubygem-backports - Backports of Ruby 1.8.7+ for older ruby
https://bugzilla.redhat.com/show_bug.cgi?id=816991 Vít Ondruch vondr...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags||fedora-cvs? --- Comment #11 from Vít Ondruch vondr...@redhat.com --- Thanks for the review. Good catch with the gem install. Not sure about the README though. It seems to be some f-r false positive? New Package SCM Request === Package Name: rubygem-backports Short Description: Backports of Ruby features for older Ruby Owners: vondruch axilleas Branches: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vppum4MfPla=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] New: Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 Bug ID: 1008962 Summary: Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: mgold...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://goldmann.fedorapeople.org/package_review/jboss-concurrency-1.0-api/1/jboss-concurrency-1.0-api.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/jboss-concurrency-1.0-api/1/jboss-concurrency-1.0-api-1.0.0-0.1.CR1.fc19.src.rpm Fedora Account System Username: goldmann Description: JSR-000236 Concurrency Utilities API Classes Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5944788 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=TfcAoz9bdaa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 Marek Goldmann mgold...@redhat.com changed: What|Removed |Added Blocks||652183 (FE-JAVASIG) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=yZvBgxf9dwa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 Marek Goldmann mgold...@redhat.com changed: What|Removed |Added Blocks||1008614 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=knDJe6Xqlxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 901365] Review Request: python-mongoengine - A Python Document-Object Mapper for working with MongoDB
https://bugzilla.redhat.com/show_bug.cgi?id=901365 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4PViuPWc3xa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 971243] Review Request: sugar-geogebra - Dynamic Mathematics for Everyone
https://bugzilla.redhat.com/show_bug.cgi?id=971243 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||cicku...@gmail.com --- Comment #4 from Christopher Meng cicku...@gmail.com --- I think we need this: http://www.geogebra.org/download/?os=unix Any news here? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=URnOd128Cra=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008799] Review Request: python-beanstalkc - A simple beanstalkd client library for Python
https://bugzilla.redhat.com/show_bug.cgi?id=1008799 Dhiru Kholia dkho...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #2 from Dhiru Kholia dkho...@redhat.com --- Fixed now. Thanks! Spec URL: https://bitbucket.org/dhiru/packages/raw/master/python-beanstalkc/python-beanstalkc.spec SRPM URL: https://bitbucket.org/dhiru/packages/raw/master/python-beanstalkc/python-beanstalkc-0.3.0-2.fc19.src.rpm Koji Task: http://koji.fedoraproject.org/koji/taskinfo?taskID=5944930 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=gNBd5fVBmWa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 Michal Srb m...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||m...@redhat.com Assignee|nob...@fedoraproject.org|m...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JiYuijcfUsa=cc_unsubscribe ___ 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 Björn besser82 Esser bjoern.es...@gmail.com changed: What|Removed |Added CC||bjoern.es...@gmail.com, ||package-review@lists.fedora ||project.org, ||rc040...@freenet.de --- Comment #5 from Björn besser82 Esser bjoern.es...@gmail.com --- (In reply to Christopher Meng from comment #4) Because of too many bugs now, we have rebuilt libstatgrab to 0.17 with epoch 1. I cannot find any FPC / FESCo ticket for bumping epoch on libstatgrab? Who granted permission to do so? What's going on there? Sorry for the inconvenience, if possible please try again after today's bodhi pushing. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2mOCkvJEsca=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008767] hydra
https://bugzilla.redhat.com/show_bug.cgi?id=1008767 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added CC|mreyno...@redhat.com, |package-review@lists.fedora |nho...@redhat.com, |project.org |nkin...@redhat.com, | |rmegg...@redhat.com | Component|389-admin |Package Review Assignee|rmegg...@redhat.com |nob...@fedoraproject.org -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Nch0LUXwwQa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008778] Metasploit
https://bugzilla.redhat.com/show_bug.cgi?id=1008778 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added CC|mreyno...@redhat.com, |package-review@lists.fedora |nho...@redhat.com, |project.org |nkin...@redhat.com, | |rmegg...@redhat.com | Component|389-admin |Package Review Assignee|rmegg...@redhat.com |nob...@fedoraproject.org -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cmgoVxqfsNa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008828] Review Request: perl-Path-FindDev - Find a development path somewhere in an upper hierarchy
https://bugzilla.redhat.com/show_bug.cgi?id=1008828 Petr Pisar ppi...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Status|NEW |ASSIGNED CC||ppi...@redhat.com Assignee|nob...@fedoraproject.org|ppi...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0ejvjK96lka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962
Michal Srb m...@redhat.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #1 from Michal Srb m...@redhat.com ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable
= 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.
[x]: License file installed when any subpackage combination is installed.
[x]: If the package is under multiple licenses, the licensing breakdown must
be documented in the spec.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[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.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %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]: Each %files section contains %defattr if rpm 4.4
[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]: Packages have proper BuildRequires/Requires on jpackage-utils
[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)
[x]: Bundled jar/class files should be removed before build
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 DOES 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 =
Generic:
[x]: Dist tag is present (not strictly required in GL).
[x]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in jboss-
concurrency-1.0-api-javadoc -- this is OK
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 Marek Goldmann mgold...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #2 from Marek Goldmann mgold...@redhat.com --- Thanks, I'll add the missing 's' :) New Package SCM Request === Package Name: jboss-concurrency-1.0-api Short Description: Concurrency Utilities for JavaEE Owners: goldmann Branches: f20 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=577GmZxomxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 --- Comment #3 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. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=R3AlFpAlN7a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 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. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pn8axf2nFKa=cc_unsubscribe ___ 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 #6 from Christopher Meng cicku...@gmail.com --- (In reply to Björn besser82 Esser from comment #5) (In reply to Christopher Meng from comment #4) Because of too many bugs now, we have rebuilt libstatgrab to 0.17 with epoch 1. I cannot find any FPC / FESCo ticket for bumping epoch on libstatgrab? Who granted permission to do so? What's going on there? Sorry for the inconvenience, if possible please try again after today's bodhi pushing. Hi, you should look at git repo and see who is doing this, although I'm the owner, I never touch it still. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wYq6mp2F0ja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009059] New: Review Request: appdata-tools - Tools for AppData files
https://bugzilla.redhat.com/show_bug.cgi?id=1009059 Bug ID: 1009059 Summary: Review Request: appdata-tools - Tools for AppData files Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: rhug...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://people.freedesktop.org/~hughsient/temp/appdata-tools.spec SRPM URL: http://people.freedesktop.org/~hughsient/temp/appdata-tools-0.1.0-1.fc19.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5945327 Description: appdata-tools contains a command line program designed to validate AppData application descriptions for standards compliance and to the style guide. Fedora Account System Username: rhughes $ rpmlint */appdata*.* 3 packages and 1 specfiles checked; 0 errors, 0 warnings. Note: This is an upstream project designed and package because Fedora and upstream contributors wanted a way to validate the appdata files installed for gnome-software, which is a technical preview in F20. This package only has the appdata-validate command at the moment but in the next version will also do useful things like install the its file for translation and the RELAX NG schema. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=dfMXE0fHYDa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009054] New: Review Request: main package name here - short summary here
https://bugzilla.redhat.com/show_bug.cgi?id=1009054 Bug ID: 1009054 Summary: Review Request: main package name here - short summary here Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: rhug...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://people.freedesktop.org/~hughsient/temp/appdata-tools.spec SRPM URL: http://people.freedesktop.org/~hughsient/temp/appdata-tools-0.1.0-1.fc19.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5945327 Description: Tools for AppData files Fedora Account System Username: rhughes $ rpmlint */appdata*.* 3 packages and 1 specfiles checked; 0 errors, 0 warnings. Note: This is an upstream project designed and package because Fedora and upstream contributors wanted a way to validate the appdata files installed for gnome-software, which is a technical preview in F20. This package only has the appdata-validate command at the moment but in the next version will also do useful things like install the its file for translation and the RELAX NG schema. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=BjibLlLL1ca=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008476] Review Request: perl-Date-HolidayParser - Parser for .holiday-files
https://bugzilla.redhat.com/show_bug.cgi?id=1008476 Susi Lehtola susi.leht...@iki.fi changed: What|Removed |Added Priority|medium |unspecified Severity|urgent |unspecified -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fAf9comaGYa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008476] Review Request: perl-Date-HolidayParser - Parser for .holiday-files
https://bugzilla.redhat.com/show_bug.cgi?id=1008476 Susi Lehtola susi.leht...@iki.fi changed: What|Removed |Added CC||susi.leht...@iki.fi --- Comment #2 from Susi Lehtola susi.leht...@iki.fi --- dayplanner has been in Fedora for 5+ years, so a missing dependency is doubtful. It may be necessary for an upgrade, but the same situation is very common for other packages as well. Please don't usurp the severity, otherwise the whole review queue will quickly turn red. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=RreVEbkGhxa=cc_unsubscribe ___ 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 Michael Schwendt bugs.mich...@gmx.net changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #7 from Michael Schwendt bugs.mich...@gmx.net --- Who granted permission to do so? It doesn't need permission from the FPC, and FESCo has nothing to do with it at all. https://fedoraproject.org/wiki/Packaging:Guidelines#Use_of_Epochs Why not introduce a libstatgrab6 (6 for the SONAME major version) package to install alongside the upgraded libstatgrab? ... although I'm the owner, I never touch it still. Is that disinterest? Or are you happy with the Epoch? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=T7QOiBFmtVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009054] Review Request: appdata-tools - Tools for AppData files
https://bugzilla.redhat.com/show_bug.cgi?id=1009054 Mario Blättermann mario.blaetterm...@gmail.com changed: What|Removed |Added Summary|Review Request: main |Review Request: |package name here - short |appdata-tools - Tools for |summary here |AppData files Alias||appdata-tools -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=K9HGqKW0tta=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008767] hydra
https://bugzilla.redhat.com/show_bug.cgi?id=1008767 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | CC||athma...@gmail.com, ||m...@fabian-affolter.ch, ||re...@seznam.cz Component|Package Review |hydra Version|el6 |rawhide Assignee|nob...@fedoraproject.org|athma...@gmail.com Product|Fedora EPEL |Fedora -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MJKsJhnoCwa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1007478] Review Request: compress-lzf - Basic LZF codec, compatible with standard C LZF package
https://bugzilla.redhat.com/show_bug.cgi?id=1007478 Will Benton wi...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #1 from Will Benton wi...@redhat.com --- Created attachment 798956 -- https://bugzilla.redhat.com/attachment.cgi?id=798956action=edit review notes Thanks for your hard work, Gil. I've reached out to upstream about the lack of licensing info in source files here but the maintainer doesn't like license/copyright comments. Other than that, the package is clean. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=OMtmReDD2Na=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] New: Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153 Bug ID: 1009153 Summary: Review Request: javolution - Real-time Java library Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: pmack...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org This project provides a Java library for real-time applications. It is maven-based and can be used to build multi-platform real-time applications. It forms an important dependency for a forthcoming review request (Hive). Note that this review request is for version 5.5.1, the latest is actually 6.0 as of August 18. However, 6.0 is heavily dependent on OSGi. The version of OSGi currently available in Fedora is R4 (Felix JBoss) which is incompatible with being built using the OpenJDK 7 javac. See the following: http://stackoverflow.com/questions/10911231/how-to-compile-mavenized-osgi-4-3-bundle-with-openjdk-7 http://blog.osgi.org/2012/10/43-companion-code-for-java-7.html Spec URL: http://pmackinn.fedorapeople.org/javolution/javolution.spec SRPM URL: http://pmackinn.fedorapeople.org/javolution/javolution-5.5.1-0.1.fc19.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=6cSyhbsGrla=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009059] Review Request: appdata-tools - Tools for AppData files
https://bugzilla.redhat.com/show_bug.cgi?id=1009059 Pierre-Yves Luyten p...@luyten.fr changed: What|Removed |Added Assignee|nob...@fedoraproject.org|p...@luyten.fr Flags||fedora-review+ --- Comment #1 from Pierre-Yves Luyten p...@luyten.fr --- I'm fine with every item, package approved. (A little surprised with the output on random example : start tag 'name' not allowed from section 'application' isn't name a valid node? Anyway this point is rather a question / bug report than part of the package review) appdata-tools = /usr/share/doc/appdata-tools /usr/bin/appdata-validate /usr/share/doc/appdata-tools/AUTHORS /usr/share/doc/appdata-tools/COPYING /usr/share/doc/appdata-tools/NEWS /usr/share/doc/appdata-tools/README /usr/share/locale/en_GB/LC_MESSAGES/appdata-tools.mo /usr/share/man/man1/appdata-validate.1.gz -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=yXPKekL0m5a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009178] New: Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178 Bug ID: 1009178 Summary: Review Request: python-sockjs-tornado - SockJS server implementation for Tornado Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: gho...@fedoraproject.org QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://gholms.fedorapeople.org/review/python-sockjs-tornado-1.0.0-1.spec SRPM URL: http://gholms.fedorapeople.org/review/python-sockjs-tornado-1.0.0-1.src.rpm Build URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=5947221 Description: SockJS-tornado is a Python server side counterpart of the SockJS-client browser library running on top of the Tornado framework. Fedora Account System Username: gholms -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=61geMM0z08a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 769919] Review Request: hydra - Very fast network log-on cracker
https://bugzilla.redhat.com/show_bug.cgi?id=769919 Athmane Madjoudj athma...@gmail.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #42 from Athmane Madjoudj athma...@gmail.com --- Package Change Request == Package Name: hydra New Branches: el6 Owners: athmane -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PBeObXFhTta=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009178] Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178 --- Comment #1 from Garrett Holmstrom gho...@fedoraproject.org --- Updated with line terminator fixes: Spec URL: http://gholms.fedorapeople.org/review/python-sockjs-tornado-1.0.0-2.spec SRPM URL: http://gholms.fedorapeople.org/review/python-sockjs-tornado-1.0.0-2.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=R168kxEloOa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009178] Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178 Matt Spaulding mspauldin...@gmail.com changed: What|Removed |Added CC||mspauldin...@gmail.com Assignee|nob...@fedoraproject.org|mspauldin...@gmail.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Rn4bISn7vxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009178] Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178
Matt Spaulding mspauldin...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Matt Spaulding mspauldin...@gmail.com ---
Everything looks good. Please note that I marked the license included item as a
Fail since it's not actually included. Spec file has a comment showing that a
pull request has been submitted to add the LICENSE file upstream.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[X]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[!]: 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]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated. 26 files have unknown license. Detailed output of
licensecheck in /home/mspaulding/1009178-python-sockjs-
tornado-1.0.0-2/licensecheck.txt
[X]: Package contains no bundled libraries without FPC exception.
[X]: Changelog in prescribed format.
[X]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[X]: Package uses nothing in %doc for runtime.
[X]: Package consistently uses macros (instead of hard-coded directory names).
[X]: Package is named according to the Package Naming Guidelines.
[X]: Package does not generate any conflict.
[X]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[X]: Requires correct, justified where necessary.
[X]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[X]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 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: No rpmlint messages.
[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]: Each %files section contains %defattr if rpm 4.4
[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]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
Python:
[X]: Python eggs must not download any dependencies during the build process.
[-]: A package which is used by another package via an egg interface should
provide egg info.
[X]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[X]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[X]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[X]: Latest version is packaged.
[X]: Package does not include license text files separate from upstream.
[!]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[?]: Package should compile and build into binary rpms on all supported
architectures.
[-]: %check is present and all tests pass. (NO TESTS)
[X]: Packages should try to preserve timestamps of original installed
[Bug 1009178] Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178 Matt Spaulding mspauldin...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fLxzF9SGKga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009178] Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178 Garrett Holmstrom gho...@fedoraproject.org changed: What|Removed |Added Flags||fedora-cvs? --- Comment #3 from Garrett Holmstrom gho...@fedoraproject.org --- Thanks, Matt! New Package SCM Request === Package Name: python-sockjs-tornado Short Description: SockJS server implementation for Tornado Owners: gholms madsa Branches: f18 f19 f20 el6 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=iCwbCd0W4Sa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009247] New: Review Request: minimodem - General-purpose software audio FSK modem
https://bugzilla.redhat.com/show_bug.cgi?id=1009247 Bug ID: 1009247 Summary: Review Request: minimodem - General-purpose software audio FSK modem Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: kg4...@gmail.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://kg4sgp.com/specs/minimodem.spec SRPM URL: http://kg4sgp.com/srpms/minimodem-0.18-1.fc19.src.rpm Description: General-purpose software audio FSK modem Fedora Account System Username: kg4sgp -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=yQdxsMM2dwa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009247] Review Request: minimodem - General-purpose software audio FSK modem
https://bugzilla.redhat.com/show_bug.cgi?id=1009247 Jim kg4...@gmail.com changed: What|Removed |Added Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=gfSXaUYWdxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009247] Review Request: minimodem - General-purpose software audio FSK modem
https://bugzilla.redhat.com/show_bug.cgi?id=1009247 Ricky Elrod rel...@redhat.com changed: What|Removed |Added CC||rel...@redhat.com Flags|fedora-review? | -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=I4bf6FFVvKa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009247] Review Request: minimodem - General-purpose software audio FSK modem
https://bugzilla.redhat.com/show_bug.cgi?id=1009247 Ricky Elrod rel...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|rel...@redhat.com Flags||fedora-review? --- Comment #1 from Ricky Elrod rel...@redhat.com --- Taking for review. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Y28BYHWOSFa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008962] Review Request: jboss-concurrency-1.0-api - Concurrency Utilities for JavaEE
https://bugzilla.redhat.com/show_bug.cgi?id=1008962 Marek Goldmann mgold...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Status|ASSIGNED|CLOSED Blocks|652183 (FE-JAVASIG) | Resolution|--- |RAWHIDE Last Closed||2013-09-18 01:01:48 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bFNvu5jInCa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009247] Review Request: minimodem - General-purpose software audio FSK modem
https://bugzilla.redhat.com/show_bug.cgi?id=1009247
Ricky Elrod rel...@redhat.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Ricky Elrod rel...@redhat.com ---
This package is APPROVED.
I have sponsored you into the packager group, as well. Welcome aboard!
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.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package complies to the Packaging Guidelines
[x]: License field in the package spec file matches the actual license.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[-]: Large documentation must go in a -doc subpackage.
Note: Documentation size is 51200 bytes in 6 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Each %files section contains %defattr if rpm 4.4
[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]: Fully versioned dependency in subpackages, if present.
[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 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
[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).
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
architectures.
[x]: %check is present and all tests pass.
[x]: 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 test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
[x]: Dist tag is present.
[x]: No file requires
