[Bug 828626] Review Request: saslwrapper - Ruby and Python wrappers for the Cyrus SASL library
https://bugzilla.redhat.com/show_bug.cgi?id=828626 Dan Callaghan changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-07-26 02:58:54 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842509] Review Request: libdbusmenu - A helper library for libindicator
https://bugzilla.redhat.com/show_bug.cgi?id=842509 --- Comment #7 from Michael Scherer --- The explicit requires on gtk3 is IMHO unwarranted, since rpm already detect it fine. I also do not see why the main rpm requires vala, since that's a run time library. The same goes for gobject-introspection, who is pulled twice by -devel ( one time explicitely, one time by the main rpm pulled by -devel ). The Requires pkgconfig is already detected on newer rpm ( ie > EL 5 ), so can be dropped. ( like rm -rf $RPM_BUILD_ROOT in %install, btw ). The multiple license of the tarbal requires more comments ( http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Multiple_Licensing_Scenarios ) And are the ldconfig call still needed ? ( real question, i think they were taken care by some automated script nowadays but maybe I am just dreaming ) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 841003] Review Request: mate-corba - CORBA Object Request Broker for MATE Desktop
https://bugzilla.redhat.com/show_bug.cgi?id=841003 Dan Mashal changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821224] tntnet - A web application server for web applications
https://bugzilla.redhat.com/show_bug.cgi?id=821224 MartinKG changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821224] tntnet - A web application server for web applications
https://bugzilla.redhat.com/show_bug.cgi?id=821224 --- Comment #26 from MartinKG --- New Package SCM Request === Package Name: tntnet Short Description: A web application server for web applications. Owners: martinkg Branches: f16 f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842570] Review Request: maven-script-interpreter - Maven Script Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=842570 gil cattaneo changed: What|Removed |Added Status|NEW |ASSIGNED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830664] Review Request: Add64 - an additive synthesizer for JACK
https://bugzilla.redhat.com/show_bug.cgi?id=830664 --- Comment #7 from Fedora Update System --- Add64-1.2.2-2.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/Add64-1.2.2-2.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830664] Review Request: Add64 - an additive synthesizer for JACK
https://bugzilla.redhat.com/show_bug.cgi?id=830664 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 767649] Review Request: lcm - Lightweight communications and marshaling
https://bugzilla.redhat.com/show_bug.cgi?id=767649 Dan Callaghan changed: What|Removed |Added Flags||fedora-cvs? --- Comment #33 from Dan Callaghan --- Package Change Request == Package Name: lcm New Branches: f16 f17 el5 el6 Owners: dcallagh Nelson has deprecated this package, although really he meant to just orphan it because he is no longer contributing to Fedora. I will take ownership of it so that his hard work is not wasted. I'm also requesting two new branches, f16 and f17. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 829971] Review Request: samplv1 -A polyphonic sampler synthesizer with stereo fx
https://bugzilla.redhat.com/show_bug.cgi?id=829971 --- Comment #7 from Fedora Update System --- samplv1-0.0.8-0.1.svn759.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/samplv1-0.0.8-0.1.svn759.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 829971] Review Request: samplv1 -A polyphonic sampler synthesizer with stereo fx
https://bugzilla.redhat.com/show_bug.cgi?id=829971 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836403] Review Request: jbossxb - JBoss XML Binding
https://bugzilla.redhat.com/show_bug.cgi?id=836403 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System --- jbossxb-2.0.3-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830155] Review Request: unlambda - An interpreter of the Unlambda language
https://bugzilla.redhat.com/show_bug.cgi?id=830155 --- Comment #11 from Fedora Update System --- unlambda-0.1-2.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid
https://bugzilla.redhat.com/show_bug.cgi?id=842296 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System --- kde-plasma-akonadi-calendars-0.1.1-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 825450] Review Request: snmpcheck - An utility to get information via SNMP protocols
https://bugzilla.redhat.com/show_bug.cgi?id=825450 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-07-26 00:00:39 --- Comment #13 from Fedora Update System --- snmpcheck-1.8-4.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836940] Review Request: sshj - SSHv2 library for Java
https://bugzilla.redhat.com/show_bug.cgi?id=836940 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System --- sshj-0.8.1-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830155] Review Request: unlambda - An interpreter of the Unlambda language
https://bugzilla.redhat.com/show_bug.cgi?id=830155 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-07-25 23:59:51 --- Comment #10 from Fedora Update System --- unlambda-0.1-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842778] Review Request: avgtime - Time a command and print average, standard deviation
https://bugzilla.redhat.com/show_bug.cgi?id=842778 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System --- avgtime-0-0.1.git20120724.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 767985] Review Request: man2html - Convert man pages to HTML
https://bugzilla.redhat.com/show_bug.cgi?id=767985 --- Comment #24 from Fedora Update System --- man2html-1.6-8.g.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842067] Review Request: will-crash - Set of crashing executables written in various languages
https://bugzilla.redhat.com/show_bug.cgi?id=842067 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System --- will-crash-0.2-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 767985] Review Request: man2html - Convert man pages to HTML
https://bugzilla.redhat.com/show_bug.cgi?id=767985 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-07-25 23:56:25 --- Comment #23 from Fedora Update System --- man2html-1.6-7.g.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 828626] Review Request: saslwrapper - Ruby and Python wrappers for the Cyrus SASL library
https://bugzilla.redhat.com/show_bug.cgi?id=828626 --- Comment #12 from Fedora Update System --- saslwrapper-0.16-1.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 820729] Review Request: mingw-cximage - MinGW Windows CxImage manipulation library
https://bugzilla.redhat.com/show_bug.cgi?id=820729 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-07-25 23:55:39 --- Comment #12 from Fedora Update System --- mingw-cximage-600-6.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821247] Review Request: fcitx-table-other - Other tables for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=821247 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #13 from Fedora Update System --- fcitx-table-other-0.1.0-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824057] Review Request: fcitx-hangul - Hangul Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=824057 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System --- fcitx-hangul-0.1.1-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 828626] Review Request: saslwrapper - Ruby and Python wrappers for the Cyrus SASL library
https://bugzilla.redhat.com/show_bug.cgi?id=828626 --- Comment #11 from Fedora Update System --- saslwrapper-0.16-1.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842509] Review Request: libdbusmenu - A helper library for libindicator
https://bugzilla.redhat.com/show_bug.cgi?id=842509 --- Comment #6 from Jef Spaleta --- (In reply to comment #4) Let me be clear, I'm not currently working towards packaging "unity" so I don't see the point of a tracker bug. My first goal is to get libunity packaged, as its libunity that applications are going to be building against to get access to the unity apis... regardless of whether unity the desktop shell built on top of compiz is running or not. libdbusmenu is the only missing package that needs to get into the repository ahead of submitting libunity for review. The other deps just need to be revved in rawhide which I now have access to do from Adam. The first step towards packaging unity will be for someone to make the commitment to maintain compiz in Fedora. That person will not be me. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842891] Review Request: stickshift-broker - StickShift broker components
https://bugzilla.redhat.com/show_bug.cgi?id=842891 --- Comment #4 from Troy Dawson --- Thank you for being so thorough. It helps when I have so many of these that I am working on. ridealong - That should be having a review request in by the end of the week. Ridealong also has it it's own selinux policy and selinux setup. So you are correct, that part should not be in this rpm. I see what you are saying about mongodb-server and bind. I will talk with the developer. Especially with bind, since that is a dependancy of uplift. I didn't even look in the gemfile, since this package isn't technically a gem. Didn't see those dependancies. oddjob - yep, need to fix that, should be doing mcollective. systemd and startup scripts. I'll talk to the developer about that. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - OpenID authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 --- Comment #9 from Fedora Update System --- python-django-authopenid-1.0.1-4.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/python-django-authopenid-1.0.1-4.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - OpenID authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 --- Comment #8 from Fedora Update System --- python-django-authopenid-1.0.1-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/python-django-authopenid-1.0.1-4.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - OpenID authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756465] Review Request: usbsoftrock - Command line utility for interfacing with Si570 based SDR kits
https://bugzilla.redhat.com/show_bug.cgi?id=756465 Lucian Langa changed: What|Removed |Added CC||kwiz...@gmail.com Component|Package Review |oyranos Assignee|nob...@fedoraproject.org|kwiz...@gmail.com --- Comment #6 from Lucian Langa --- Updated package: Spec URL: http://lucilanga.fedorapeople.org/usbsoftrock.spec SRPM URL: http://lucilanga.fedorapeople.org/usbsoftrock-1.0.2-3.fc18.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842509] Review Request: libdbusmenu - A helper library for libindicator
https://bugzilla.redhat.com/show_bug.cgi?id=842509 --- Comment #5 from Jef Spaleta --- sorry about that guys. added the one missing BR. mock f17 run completes without error now for me mock rawhide run completes without error now for me ive pushed corrected spec and srpms to the same url with the one additional BR. Please redownload. -jef -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842891] Review Request: stickshift-broker - StickShift broker components
https://bugzilla.redhat.com/show_bug.cgi?id=842891 --- Comment #3 from Michael Scherer --- And it need to have a systemd file, I could write one but I am not sure. Stickshift-broker should be started separately, with a separate apache + passenger ( + mod_ssl ) instance running on port 8080 ? ( also, why is there a requirement on oddjob since oddjob is deprecated upstream, replaced by mcollective, and oddjob do not seems to be used in the code ). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842891] Review Request: stickshift-broker - StickShift broker components
https://bugzilla.redhat.com/show_bug.cgi?id=842891 --- Comment #2 from Michael Scherer --- Also, can the unit test be run in %check ( and so that would require rcov at built time) ? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842481] Review Request: jhighlight - An embeddable pure Java syntax highlighting library
https://bugzilla.redhat.com/show_bug.cgi?id=842481 Mikolaj Izdebski changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|mizde...@redhat.com Flags||fedora-review? --- Comment #4 from Mikolaj Izdebski --- I am taking this review. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842891] Review Request: stickshift-broker - StickShift broker components
https://bugzilla.redhat.com/show_bug.cgi?id=842891 Michael Scherer changed: What|Removed |Added CC||m...@zarb.org --- Comment #1 from Michael Scherer --- Hi, mod_ridealong is not packaged ( in f17, didn't check in rawhide ), and neither is the ruby module ( I assume that's a fork of passenger ? ) And I think that it work with a remote mongodb server or bind, so i am not sure this is good to pull mongodb-server, bind ( especially if it use a plugin to modify dns, like uplift-bind ). For the selinux part, wouldn't it be easier to push them directly in selinux-policy ( I am not sure that's good idea to modify boolean from %post without warning ) Among the various deps, there is rcov, but this is not listed as useful for runtime, just for testing ( like cucumber, who is not in Requires ). Shouldn't it be left out ? There is also no mention of multimap in code, and the following gem are not listed in the spec while they are in the Gemfile : gem 'rack' gem 'regin' gem 'open4' Could you check with upstream the requirement ? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 722713] Review Request: pyace - Optimal route search in a complete graph
https://bugzilla.redhat.com/show_bug.cgi?id=722713 Mauricio Cleveland changed: What|Removed |Added Version|16 |17 --- Comment #24 from Mauricio Cleveland --- I am working with a mentor. I am making the RPM for new version. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 838756] Review Request: maven-indexer - Standard for producing indexes of Maven repositories
https://bugzilla.redhat.com/show_bug.cgi?id=838756 Gerard Ryan changed: What|Removed |Added Flags||fedora-cvs? --- Comment #9 from Gerard Ryan --- New Package SCM Request === Package Name: maven-indexer Short Description: Standard for producing indexes of Maven repositories Owners: galileo Branches: f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 838756] Review Request: maven-indexer - Standard for producing indexes of Maven repositories
https://bugzilla.redhat.com/show_bug.cgi?id=838756 --- Comment #8 from Mikolaj Izdebski --- Tested on Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4329789 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 838756] Review Request: maven-indexer - Standard for producing indexes of Maven repositories
https://bugzilla.redhat.com/show_bug.cgi?id=838756 Mikolaj Izdebski changed: What|Removed |Added Status|ASSIGNED|POST Flags|fedora-review? |fedora-review+ --- Comment #7 from Mikolaj Izdebski --- (In reply to comment #6) > (In reply to comment #5) > > The LICENSE file must be included in the SRPM and installed as %doc. > > This is required by the point 4. of the license. > > It's already being installed as %doc, and by extension is included in the > SRPM, right? My bad. You were right. > Anyway, the uber-jar is no longer being installed, so this no longer applies > :) Nice :) [x]: MUST Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: MUST Package contains no bundled libraries. [x]: MUST Package complies to the Packaging Guidelines rpmlint output: maven-indexer.noarch: I: enchant-dictionary-not-found en_US maven-indexer.src: W: invalid-url Source0: maven-indexer-4.1.2-source-release.zip 3 packages and 0 specfiles checked; 0 errors, 1 warnings. No more issues. * ** APPROVED** * -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842481] Review Request: jhighlight - An embeddable pure Java syntax highlighting library
https://bugzilla.redhat.com/show_bug.cgi?id=842481 --- Comment #3 from Gerard Ryan --- Fixed! :) Spec URL: http://galileo.fedorapeople.org/jhighlight/1.0-3/jhighlight.spec SRPM URL: http://galileo.fedorapeople.org/jhighlight/1.0-3/jhighlight-1.0-3.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842013] Review Request: nfsometer - NFS Performance Framework Tool
https://bugzilla.redhat.com/show_bug.cgi?id=842013
--- Comment #5 from Michael Scherer ---
+ /usr/bin/python setup.py build
Error importing numpy - Make sure numpy is installed
seems numpy is missing :)
In fact to make it build, I have added the following ( setup.py check runtime
deps at build time ) :
--- /home/misc/checkout/git/FedoraReview/nfsometer/nfsometer.spec2012-07-25
15:09:22.0 +0200
+++ SPECS/nfsometer.spec2012-07-25 22:24:33.383177536 +0200
@@ -9,6 +9,9 @@
Source0:
http://www.linux-nfs.org/~dros/nfsometer/releases/%{name}-%{version}.tar.gz
BuildRequires: python-setuptools
+BuildRequires: numpy
+BuildRequires: python-matplotlib
+BuildRequires: python-mako
Requires: nfs-utils
Requires: python-matplotlib
Requires: numpy
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842509] Review Request: libdbusmenu - A helper library for libindicator
https://bugzilla.redhat.com/show_bug.cgi?id=842509 --- Comment #4 from Michael Scherer --- Btw, could a tracker bug be added, so we can see the progression of unity packaging ? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 796953] Review Request: trac-vatar-plugin - Add gravatar icons to various points around trac
https://bugzilla.redhat.com/show_bug.cgi?id=796953 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-07-25 16:04:33 --- Comment #11 from Fedora Update System --- trac-vatar-plugin-1.6-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810010] Review Request: genders - file based database for cluster managment
https://bugzilla.redhat.com/show_bug.cgi?id=810010 --- Comment #25 from Fedora Update System --- genders-1.18-6.el5 has been pushed to the Fedora EPEL 5 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810010] Review Request: genders - file based database for cluster managment
https://bugzilla.redhat.com/show_bug.cgi?id=810010 --- Comment #24 from Fedora Update System --- genders-1.18-6.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 841766] Review Request: erlang-riak_search - Full-text search engine based on Riak
https://bugzilla.redhat.com/show_bug.cgi?id=841766 Bug 841766 depends on bug 652629, which changed state. Bug 652629 Summary: Review Request: erlang-riak_kv - Riak Key/Value Store https://bugzilla.redhat.com/show_bug.cgi?id=652629 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 838756] Review Request: maven-indexer - Standard for producing indexes of Maven repositories
https://bugzilla.redhat.com/show_bug.cgi?id=838756
--- Comment #6 from Gerard Ryan ---
(In reply to comment #5)
> The LICENSE file must be included in the SRPM and installed as %doc.
> This is required by the point 4. of the license.
It's already being installed as %doc, and by extension is included in the SRPM,
right?
> /usr/share/java/maven-indexer-core-cli.jar contains bundled libraries.
> Please remove them. See:
> http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
Done. The entire jar was bundled libs. Turns out that's what maven-shade-plugin
does...I guess maven-shade-plugin shouldn't be used in any fedora packages in
this case.
> Packaging guidelines says "If the number of provided JAR files exceeds
> two, you MUST place them into a sub-directory named %{name}."
> Please obey this rule. See:
> http://fedoraproject.org/wiki/Packaging:Java#Installation_directory
Ah yeah, I (incorrectly) was looking at the draft guidelines, where the MUST is
substituted for a CAN:
https://fedoraproject.org/w/index.php?title=User:Akurtakov/JavaPackagingDraftUpdate#Installation_directory
Anyway, the uber-jar is no longer being installed, so this no longer applies :)
Spec URL:
http://galileo.fedorapeople.org/maven-indexer/4.1.2-3/maven-indexer.spec
SRPM URL:
http://galileo.fedorapeople.org/maven-indexer/4.1.2-3/maven-indexer-4.1.2-3.fc17.src.rpm
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 841003] Review Request: mate-corba - CORBA Object Request Broker for MATE Desktop
https://bugzilla.redhat.com/show_bug.cgi?id=841003 --- Comment #30 from Dan Mashal --- New Package SCM Request === Package Name: mate-corba Short Description: CORBA Object Request Broker for MATE Desktop Owners: vicodan rdieter Branches: f16 f17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 841003] Review Request: mate-corba - CORBA Object Request Broker for MATE Desktop
https://bugzilla.redhat.com/show_bug.cgi?id=841003 Rex Dieter changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #29 from Rex Dieter --- OK, latest -8 at http://vicodan.fedorapeople.org/matespec/mate-corba.spec Looks good. APPROVED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842509] Review Request: libdbusmenu - A helper library for libindicator
https://bugzilla.redhat.com/show_bug.cgi?id=842509 Adam Williamson changed: What|Removed |Added CC||awill...@redhat.com --- Comment #3 from Adam Williamson --- Yup, confirmed misc's result, looks like missing BRs. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842570] Review Request: maven-script-interpreter - Maven Script Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=842570 --- Comment #5 from gil cattaneo --- *** APPROVED *** -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836403] Review Request: jbossxb - JBoss XML Binding
https://bugzilla.redhat.com/show_bug.cgi?id=836403 --- Comment #5 from Fedora Update System --- jbossxb-2.0.3-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/jbossxb-2.0.3-1.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836403] Review Request: jbossxb - JBoss XML Binding
https://bugzilla.redhat.com/show_bug.cgi?id=836403 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842570] Review Request: maven-script-interpreter - Maven Script Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=842570 --- Comment #4 from gil cattaneo --- hi >rpmlint maven-script-interpreter-1.0-1.fc18.noarch.rpm >maven-script-interpreter.noarch: W: spelling-error %description -l en_US >>beanshell -> bean shell, bean-shell, beans hell >1 packages and 0 specfiles checked; 0 errors, 1 warnings. >rpmlint maven-script-interpreter-1.0-1.fc18.src.rpm >maven-script-interpreter.src: W: spelling-error %description -l en_US >beanshell >-> bean shell, bean-shell, beans hell >1 packages and 0 specfiles checked; 0 errors, 1 warnings. can you change beanshell in BeanShell and groovy in Groovy? thanks -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842570] Review Request: maven-script-interpreter - Maven Script Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=842570
gil cattaneo changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #3 from gil cattaneo ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[ ]: MUST Package requires other packages for directories it uses.
[ ]: MUST Package uses nothing in %doc for runtime.
[ ]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf would be needed if support for EPEL5 is required
[ ]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST 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.
[ ]: MUST License field in the package spec file matches the actual license.
[ ]: MUST License file installed when any subpackage combination is installed.
[ ]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package is named according to the Package Naming Guidelines.
[ ]: MUST Package does not generate any conflict.
[ ]: MUST Package obeys FHS, except libexecdir and /usr/target.
[ ]: MUST Package must own all directories that it creates.
[ ]: MUST Package does not own files or directories owned by other packages.
[ ]: MUST Package installs properly.
[ ]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.
rpmlint maven-script-interpreter-javadoc-1.0-1.fc18.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint maven-script-interpreter-1.0-1.fc18.noarch.rpm
maven-script-interpreter.noarch: W: spelling-error %description -l en_US
beanshell -> bean shell, bean-shell, beans hell
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
rpmlint maven-script-interpreter-1.0-1.fc18.src.rpm
maven-script-interpreter.src: W: spelling-error %description -l en_US beanshell
-> bean shell, bean-shell, beans hell
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
maven-script-interpreter-1.0-source-release.zip :
MD5SUM this package : b7d1859dd80e8c37e97132d19e439e1e
MD5SUM upstream package : b7d1859dd80e8c37e97132d19e439e1e
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[ ]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[ ]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[ ]: SHOULD 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]: SHOULD Dist tag is present.
[ ]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[ ]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[ ]: SHOULD Package functions as described.
[ ]: SHOULD Latest version is packaged.
[ ]: SHOULD Package does not include license text files separate from
upstream.
[ ]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
justified.
[x]: SHOULD SourceX / PatchY prefixed with %{name}.
[x]: SHOULD Sou
[Bug 821224] tntnet - A web application server for web applications
https://bugzilla.redhat.com/show_bug.cgi?id=821224 Michel Alexandre Salim changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #25 from Michel Alexandre Salim --- Everything looks fine; I'm approving this package, but could you add a new line between every changelog entry? Makes them easier to read. Sorry for the delay (traveling for the past week) APPROVED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 767649] Review Request: lcm - Lightweight communications and marshaling
https://bugzilla.redhat.com/show_bug.cgi?id=767649 --- Comment #32 from Nelson Marques --- Dan, I don't know what to do from here, and this what I did: - on pkgdb retired the package; - removed myself from all groups I was subscrived (packaging, CLA, etc); - filed a ticket with infra-structure to disable my account, if possible permanently; - closed as WONTFIX all requests I had pending. If any steps are needed to comply with a proper drop-out, let me know so I can proceed into them and try to exit gracefully. Thanks everyone for the time spent; you guys were awesome. NM -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842570] Review Request: maven-script-interpreter - Maven Script Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=842570 gil cattaneo changed: What|Removed |Added CC||punto...@libero.it Assignee|nob...@fedoraproject.org|punto...@libero.it Flags||fedora-review? --- Comment #2 from gil cattaneo --- take this review. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 843153] New: Review Request: rubygem-sfcc - Ruby bindings for SBLIM client library
https://bugzilla.redhat.com/show_bug.cgi?id=843153 Bug ID: 843153 QA Contact: extras...@fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: nott...@redhat.com, package-review@lists.fedoraproject.org Assignee: nob...@fedoraproject.org Summary: Review Request: rubygem-sfcc - Ruby bindings for SBLIM client library Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: mimi...@redhat.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://www.minami.cz/public/rubygem-sfcc.spec SRPM URL: http://www.minami.cz/public/rubygem-sfcc-0.8.1-1.fc16.src.rpm Description: Ruby bindings for SBLIM Small Foot CIM Client (SFCC) library used to directly access any CIMOM supporting CIM-XML. Fedora Account System Username: miminar -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824057] Review Request: fcitx-hangul - Hangul Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=824057 --- Comment #10 from Fedora Update System --- fcitx-hangul-0.1.1-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/fcitx-hangul-0.1.1-1.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824057] Review Request: fcitx-hangul - Hangul Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=824057 --- Comment #9 from Fedora Update System --- fcitx-hangul-0.1.1-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/fcitx-hangul-0.1.1-1.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824057] Review Request: fcitx-hangul - Hangul Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=824057 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821247] Review Request: fcitx-table-other - Other tables for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=821247 --- Comment #12 from Fedora Update System --- fcitx-table-other-0.1.0-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/fcitx-table-other-0.1.0-1.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821247] Review Request: fcitx-table-other - Other tables for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=821247 --- Comment #11 from Fedora Update System --- fcitx-table-other-0.1.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/fcitx-table-other-0.1.0-1.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821247] Review Request: fcitx-table-other - Other tables for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=821247 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824052] Review Request: fcitx-m17n - M17n Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=824052 Liang Suilong changed: What|Removed |Added Flags|needinfo?(liangsuilong@gmai | |l.com) | --- Comment #7 from Liang Suilong --- SPEC: http://liangsuilong.fedorapeople.org/fcitx/fcitx-m17n.spec SRPM: http://liangsuilong.fedorapeople.org/fcitx/fcitx-m17n-0.1.2-1.fc16.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4328987 I upload a new SRPM and SPEC. Please continue to review it. Thank you. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824061] Review Request: fcitx-unikey - Vietnamese Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=824061 Liang Suilong changed: What|Removed |Added Flags|needinfo?(liangsuilong@gmai | |l.com) | --- Comment #11 from Liang Suilong --- SPEC: http://liangsuilong.fedorapeople.org/fcitx/fcitx-unikey.spec SRPM: http://liangsuilong.fedorapeople.org/fcitx/fcitx-hangul-0.1.0-1.fc16.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4328977 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842778] Review Request: avgtime - Time a command and print average, standard deviation
https://bugzilla.redhat.com/show_bug.cgi?id=842778 Richard W.M. Jones changed: What|Removed |Added Depends On||843133 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842778] Review Request: avgtime - Time a command and print average, standard deviation
https://bugzilla.redhat.com/show_bug.cgi?id=842778 --- Comment #9 from Richard W.M. Jones --- FTBFS in Rawhide. Appears to be a D compiler problem: http://kojipkgs.fedoraproject.org//work/tasks/8950/4328950/build.log + ldc2 avgtime.d /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_setopt' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_global_cleanup' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_init' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_slist_append' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_pause' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_slist_free_all' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_duphandle' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_perform' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_cleanup' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_global_init' /usr/lib/gcc/x86_64-redhat-linux/4.7.1/../../../../lib64/libphobos-ldc.so: undefined reference to `curl_easy_strerror' Error: linking failed: status: 1 collect2: error: ld returned 1 exit status -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842778] Review Request: avgtime - Time a command and print average, standard deviation
https://bugzilla.redhat.com/show_bug.cgi?id=842778 --- Comment #8 from Fedora Update System --- avgtime-0-0.1.git20120724.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/avgtime-0-0.1.git20120724.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842778] Review Request: avgtime - Time a command and print average, standard deviation
https://bugzilla.redhat.com/show_bug.cgi?id=842778 Fedora Update System changed: What|Removed |Added Status|NEW |MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749673] eclipse-mylyn: Build Eclipse Mylyn code review integration
https://bugzilla.redhat.com/show_bug.cgi?id=749673 Krzysztof Daniel changed: What|Removed |Added CC||jerb...@gmail.com, ||swagi...@redhat.com Component|Package Review |eclipse-mylyn Summary|Review Request: |eclipse-mylyn: Build |eclipse-mylyn-reviews - |Eclipse Mylyn code review |Eclipse Mylyn code review |integration |integration | --- Comment #8 from Krzysztof Daniel --- Review here is not required. eclipse-mylyn package reflects now the upstream build and it is one big package for entire mylyn. The source for reviews is there, but it is not built. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842197] Review Request: deltacloud-client-java - Deltacloud java client
https://bugzilla.redhat.com/show_bug.cgi?id=842197 Mikolaj Izdebski changed: What|Removed |Added CC||mizde...@redhat.com --- Comment #1 from Mikolaj Izdebski --- Some issues: 1) incorrect license tag -- should be ASL 2.0 and (EPL or LGPLv2+ or GPLv2+ or ASL 2.0 or BSD 2) missing javadoc package Please fix the above issues first. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842481] Review Request: jhighlight - An embeddable pure Java syntax highlighting library
https://bugzilla.redhat.com/show_bug.cgi?id=842481 Mikolaj Izdebski changed: What|Removed |Added CC||mizde...@redhat.com --- Comment #2 from Mikolaj Izdebski --- jhighlight-1.0/lib in SRPM contains some bundled libraries. Notably jflex-1.4.1.jar is licensed under GPLv2, but there are no sources included. Please remove bundled jars from the SRPM before I can take the review. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 838756] Review Request: maven-indexer - Standard for producing indexes of Maven repositories
https://bugzilla.redhat.com/show_bug.cgi?id=838756
--- Comment #5 from Mikolaj Izdebski ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
Generic
[!]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
[!]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[!]: MUST Package complies to the Packaging Guidelines
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST 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]: MUST License field in the package spec file matches the actual license.
[x]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
names).
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: MUST Package contains systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[!]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
upstream.
[x]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
justified.
[x]: SHOULD SourceX / PatchY prefixed with %{name}.
[x]: SHOULD SourceX is a working URL.
[x]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[?]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
files.
[x]: SHOULD Spec use %global instead of %define.
Java
[x]: MUST If source tarball includes bundled jar/class files these need to be
removed prior to building
[x]: MUST Packages have proper BuildRequires/Requires on jpackage-utils
[x]: MUST Fully versioned dependency in subpackages, if present.
[x]: MUST Javadoc documentation files are generated and included in -javadoc
subpackage
[x]: MUST Javadoc subpackages have Requires: jpackage-utils
[x]: MUST Javadocs are placed in %{_javadocdir}/%{name} (no -%{version}
symlink)
[x]: SHOULD Package has BuildArch: noarch (if possible)
[x]: SHOULD Package
[Bug 842067] Review Request: will-crash - Set of crashing executables written in various languages
https://bugzilla.redhat.com/show_bug.cgi?id=842067 --- Comment #6 from Fedora Update System --- will-crash-0.2-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/will-crash-0.2-1.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842067] Review Request: will-crash - Set of crashing executables written in various languages
https://bugzilla.redhat.com/show_bug.cgi?id=842067 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830784] Review Request: leiningen - Clojure project automation tool
https://bugzilla.redhat.com/show_bug.cgi?id=830784 Bug 830784 depends on bug 830933, which changed state. Bug 830933 Summary: maven2 needs to provide maven-artifact as a subpackage https://bugzilla.redhat.com/show_bug.cgi?id=830933 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |WONTFIX -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid
https://bugzilla.redhat.com/show_bug.cgi?id=842296 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid
https://bugzilla.redhat.com/show_bug.cgi?id=842296 --- Comment #13 from Fedora Update System --- kde-plasma-akonadi-calendars-0.1.1-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/kde-plasma-akonadi-calendars-0.1.1-1.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 838756] Review Request: maven-indexer - Standard for producing indexes of Maven repositories
https://bugzilla.redhat.com/show_bug.cgi?id=838756 --- Comment #4 from Gerard Ryan --- Bundled jars removed from the source zip in the srpm so that the terms of their licenses are not violated Spec URL: http://galileo.fedorapeople.org/maven-indexer/4.1.2-2/maven-indexer.spec SRPM URL: http://galileo.fedorapeople.org/maven-indexer/4.1.2-2/maven-indexer-4.1.2-2.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 784145] Review Request: libomxil-bellagio - OpenMAX Integration Layer
https://bugzilla.redhat.com/show_bug.cgi?id=784145 Niels de Vos changed: What|Removed |Added Assignee|nob...@fedoraproject.org|nde...@redhat.com Flags||needinfo?(kwiz...@gmail.com ||) --- Comment #9 from Niels de Vos --- Jon, uhm, right, sure. I assumed it was assigned to me already :-/ Nicolas, I've built 0.9.3-3 in mock (f17, x86_64) and now also in koji where it failed: - http://koji.fedoraproject.org/koji/taskinfo?taskID=4328662 From the build.log: omxregister.c:249:13: error: ignoring return value of 'fwrite', declared with attribute warn_unused_result [-Werror=unused-result] omxregister.c:250:13: error: ignoring return value of 'fwrite', declared with attribute warn_unused_result [-Werror=unused-result] omxregister.c:325:14: error: ignoring return value of 'fwrite', declared with attribute warn_unused_result [-Werror=unused-result] cc1: all warnings being treated as errors make[3]: *** [omxregister_bellagio-omxregister.o] Error 1 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842013] Review Request: nfsometer - NFS Performance Framework Tool
https://bugzilla.redhat.com/show_bug.cgi?id=842013
--- Comment #4 from Steve Dickson ---
(In reply to comment #3)
> Build in mock fail :
>
> Exécution_de(%build): /bin/sh -e /var/tmp/rpm-tmp.E1njxS
> + umask 022
> + cd /builddir/build/BUILD
> + cd nfsometer-1.1
> + LANG=C
> + export LANG
> + unset DISPLAY
> + /usr/bin/python setup.py build
> Traceback (most recent call last):
> File "setup.py", line 15, in
> from setuptools.command.install import install as _install
> ImportError: No module named setuptools.command.install
>
>
> I think a BR is missing ( like python2-devel, or python3-devel )
python-setuptools was missing...
>
> Also, I think the COPYING file should be in %doc, as well as README ( the
> former for legal reason, the later for usabilty )
Done!
>
> As a side note, I usually recommend to have 1 line for each requires, this
> permit to have IMHO better diff ( ie, +/- show the only change )
Good idea...
Here the diff:
diff .old/nfsometer.spec nfsometer.spec
--- .old/nfsometer.spec2012-07-20 21:13:39.911420286 +
+++ nfsometer.spec2012-07-25 13:09:22.960228234 +
@@ -1,6 +1,6 @@
Name:nfsometer
Version: 1.1
-Release: 0%{?dist}
+Release: 1%{?dist}
Summary: NFS Performance Framework Tool
Group: System Tools
@@ -8,7 +8,12 @@ License: GPLv2
URL: http://wiki.linux-nfs.org/wiki/index.php/NFSometer
Source0:
http://www.linux-nfs.org/~dros/nfsometer/releases/%{name}-%{version}.tar.gz
-Requires: nfs-utils python-matplotlib numpy python-mako filebench
+BuildRequires: python-setuptools
+Requires: nfs-utils
+Requires: python-matplotlib
+Requires: numpy
+Requires: python-mako
+Requires: filebench
%description
NFSometer is a performance measurement framework for running workloads and
@@ -30,8 +35,11 @@ NFS client implementations.
#For noarch packages: sitelib
%{python_sitelib}/*
-%doc
+%doc COPYING README
%changelog
+* Wed Jul 25 2012 Steve Dickson 1.1-1
+- Incorporated review comments.
+
* Thu Jul 19 2012 Steve Dickson 1.1-0
- Inital commit.
http://steved.fedorapeople.org/nfsometer/ has been updated
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - OpenID authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 --- Comment #7 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746215] Review Request: perl-RT-Authen-ExternalAuth - RT Authentication using External Sources
https://bugzilla.redhat.com/show_bug.cgi?id=746215 Bug 746215 depends on bug 809142, which changed state. Bug 809142 Summary: mod_perl fails to rebuild in rawhide https://bugzilla.redhat.com/show_bug.cgi?id=809142 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842778] Review Request: avgtime - Time a command and print average, standard deviation
https://bugzilla.redhat.com/show_bug.cgi?id=842778 --- Comment #7 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid
https://bugzilla.redhat.com/show_bug.cgi?id=842296 --- Comment #12 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842067] Review Request: will-crash - Set of crashing executables written in various languages
https://bugzilla.redhat.com/show_bug.cgi?id=842067 --- Comment #5 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - OpenID authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 Ankur Sinha (FranciscoD) changed: What|Removed |Added Summary|Review Request: (Rename |Review Request: (Rename |request)|request) |python-django-authopenid - |python-django-authopenid - |Openid authentication |OpenID authentication |application for Django |application for Django Flags||fedora-cvs? --- Comment #6 from Ankur Sinha (FranciscoD) --- New Package SCM Request === Package Name: python-django-authopenid Short Description: OpenID authentication application for Django Owners: ankursinha Branches: f16 f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836403] Review Request: jbossxb - JBoss XML Binding
https://bugzilla.redhat.com/show_bug.cgi?id=836403 --- Comment #4 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - Openid authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 --- Comment #5 from Ankur Sinha (FranciscoD) --- (In reply to comment #4) > Yes, that thing on my page is not clear enough - the initial release of > python-django-authopenid package should have already been 4, because the > release of obsoleted package was 3, see [1]. > So please bump your release to 4 before commiting to dist-git :) And don't > forget to retire django-authopenid properly. Thanks! Ah. Done: http://ankursinha.fedorapeople.org/python-django-authopenid/python-django-authopenid-1.0.1-4.fc17.src.rpm http://ankursinha.fedorapeople.org/python-django-authopenid/python-django-authopenid.spec > > [1] > http://fedoraproject.org/wiki/Packaging:Guidelines#Renaming. > 2FReplacing_Existing_Packages > > This package is APPROVED. Thanks for the quick review Bohuslav :) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 784145] Review Request: libomxil-bellagio - OpenMAX Integration Layer
https://bugzilla.redhat.com/show_bug.cgi?id=784145 --- Comment #8 from Jon Ciesla --- Git done (by process-git-requests). Niels, please take ownership of review BZs. Thanks! -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 841001] Review Request: libmatewnck - MATE Desktop Window Navigator Construction Kit libraries
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #4 from raveit65 ---
a few suggestions :)
1. You have double entries of BuildRequires !
2. A empty row before %changelog would be nice.
3. We need only this as BuildRequires
BuildRequires: glib2-devel
BuildRequires: gtk2-devel
BuildRequires: startup-notification-devel
BuildRequires: libXres-devel
BuildRequires: intltool
BuildRequires: gobject-introspection-devel
BuildRequires: mate-common
BuildRequires: gtk-doc
BuildRequires: cairo-gobject-devel
For XRes support in configure results we need libXres-devel, not libX11-devel
4. Reminder: no need of mate-common in Requires
5. startup-notification needs to be in Requires.
6. IHMO no need for gtk-doc in Requires, not shure
let's wait what Rex said.
In result of 4/5/6
Requires: startup-notification
7. Why configure flag --enable-gtk-doc-html ?
I use --enable-gtk-doc
8. I remove unecessary .la files with that line
find $RPM_BUILD_ROOT -name '*.la' -exec rm -f {} ';'
in %install section after make install.
9. IHMO gobject-introspection files needs to be split in main and devel
Package, not shure.
%{_datadir}/gir-1.0/Matewnck-1.0.gir in -devel
%files -f %{name}.lang
%doc AUTHORS COPYING README NEWS
%{_bindir}/matewnck-urgency-monitor
%{_bindir}/matewnckprop
%{_libdir}/lib*.so.*
%{_libdir}/girepository-1.0/Matewnck-1.0.typelib
%files devel
%{_libdir}/lib*.so
%{_libdir}/pkgconfig
%{_includedir}/libmatewnck/libmatewnck
%{_datadir}/gir-1.0/Matewnck-1.0.gir
%doc %{_datadir}/gtk-doc
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842481] Review Request: jhighlight - An embeddable pure Java syntax highlighting library
https://bugzilla.redhat.com/show_bug.cgi?id=842481 --- Comment #1 from Gerard Ryan --- The lgpl text that was in this package, not from the tarball, has been removed. Spec URL: http://galileo.fedorapeople.org/jhighlight/1.0-2/jhighlight.spec SRPM URL: http://galileo.fedorapeople.org/jhighlight/1.0-2/jhighlight-1.0-2.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842067] Review Request: will-crash - Set of crashing executables written in various languages
https://bugzilla.redhat.com/show_bug.cgi?id=842067 Richard Marko changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Richard Marko --- Thanks for the review! I've did one more update - removing AM_PATH_PYTHON as this package doesn't need python to be built. Spec URL: http://rmarko.fedorapeople.org/packages/will-crash.spec SRPM URL: http://rmarko.fedorapeople.org/packages/will-crash-0.2-1.fc17.src.rpm New Package SCM Request === Package Name: will-crash Short Description: Set of crashing executables written in various languages Owners: rmarko Branches: f16 f17 el6 InitialCC: rmarko -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - Openid authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 Bohuslav "Slavek" Kabrda changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Bohuslav "Slavek" Kabrda --- Yes, that thing on my page is not clear enough - the initial release of python-django-authopenid package should have already been 4, because the release of obsoleted package was 3, see [1]. So please bump your release to 4 before commiting to dist-git :) And don't forget to retire django-authopenid properly. Thanks! [1] http://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages This package is APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - Openid authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633 Ankur Sinha (FranciscoD) changed: What|Removed |Added Assignee|nob...@fedoraproject.org|bkab...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 842633] Review Request: (Rename request) python-django-authopenid - Openid authentication application for Django
https://bugzilla.redhat.com/show_bug.cgi?id=842633
--- Comment #3 from Ankur Sinha (FranciscoD) ---
(In reply to comment #2)
> - Probably just a typo: Your VR is 1.0.1-1, but you obsolete
> django-authopenid < 1.0.1-4. That should be 1.0.1-1, right?
Hey Bohuslav,
Not sure about this:
The django-authopenid package (that I'm renaming to python-django-authopenid)
had a current running package VR of 1.0.1-3. According to the example here[1],
the obsolete is correct:
python-django-authopeind-1.0.1-2 (latest spec) obsoletes django-authopenid <
1.0.1-4 (so, 1.0.1-{1,2,3})
The comment in the example says: "# Obsolete anything that is lower than the
_bumped release_ of the renamed package. Obsolete with hardcoded values of
renamed package"
Do have a look and let me know? I'll make the change if required, it's really a
tiny one.
> - You should remove the egg-info directory in the %prep section, so that it
> gets recreated by python setup.py build.
Done.
>
> Otherwise the package looks good, so once you correct these nits, I will
> approve it.
New spec/srpm:
http://ankursinha.fedorapeople.org/python-django-authopenid/python-django-authopenid.spec
http://ankursinha.fedorapeople.org/python-django-authopenid/python-django-authopenid-1.0.1-2.fc17.src.rpm
* Wed Jul 25 2012 Ankur Sinha 1.0.1-2
- Remove egg-info to ensure that setup.py builds it #842633
[1] https://fedoraproject.org/wiki/User:Bkabrda/Django_rename#django-bar
Thanks,
Ankur
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
