[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #39 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 Jorge A Gallegos k...@blegh.net changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #38 from Jorge A Gallegos k...@blegh.net --- Package Change Request == Package Name: uwsgi New Branches: el6 epel7 Owners: kad Requesting EPEL 6 and 7 branches for existing package uwsgi. I am the owner. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #33 from Mitch Davis mjd+red...@afork.com --- Created attachment 597512 -- https://bugzilla.redhat.com/attachment.cgi?id=597512action=edit Spec file for uwsgi. Not suitable for production. -- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #34 from Mitch Davis mjd+red...@afork.com --- Created attachment 597513 -- https://bugzilla.redhat.com/attachment.cgi?id=597513action=edit initd script for uwsgi Should really be a systemd script, but I couldn't get that to work. -- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #35 from Mitch Davis mjd+red...@afork.com --- Created attachment 597514 -- https://bugzilla.redhat.com/attachment.cgi?id=597514action=edit .ini file for uwsgi. Gets patched at package build 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #36 from Mitch Davis mjd+red...@afork.com --- Jorge, Not sure what the status of this package is at the moment, but I've been fiddling with it for my own needs. In particular, I've given it an initd script (albeit not a very good one), and changed the spec file so that the different plugins can be turned on and off. Me, I only need the python2 plugin, and I have to compile on the 32-bit server where I need to run it. I don't want to pull in all the devel stuff onto that server, just gcc and python-devel. I'm also starting it in emperor mode. This way, new uwsgi services can be created just by putting a service file into /etc/uwsgo/vassals - no modification of uwsgi's config file is necessary. Please find attached the .spec file, initd script, and .ini script.I'm not saying any of these are sutiable for production, but you might consider making some of these changes. Mitch. -- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #37 from Jorge A Gallegos k...@blegh.net --- Mitch, As you can see in the bug details, this got completed a while back and uWSGI is available in Fedora 17. I just pushed v1.2.4 and has been sitting in updates-testing for a couple of days (see https://admin.fedoraproject.org/updates/FEDORA-2012-10444/uwsgi-1.2.4-1.fc17 ) if you want to test/add karma would be much appreciated. I am working on the systemd integration for f18. Regards (In reply to comment #36) Jorge, Not sure what the status of this package is at the moment, but I've been fiddling with it for my own needs. In particular, I've given it an initd script (albeit not a very good one), and changed the spec file so that the different plugins can be turned on and off. Me, I only need the python2 plugin, and I have to compile on the 32-bit server where I need to run it. I don't want to pull in all the devel stuff onto that server, just gcc and python-devel. I'm also starting it in emperor mode. This way, new uwsgi services can be created just by putting a service file into /etc/uwsgo/vassals - no modification of uwsgi's config file is necessary. Please find attached the .spec file, initd script, and .ini script.I'm not saying any of these are sutiable for production, but you might consider making some of these changes. Mitch. -- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #31 from Jorge A Gallegos k...@blegh.net --- This was intended to go with f17, however there is a test koji build for f16 I did before submitting it for f17. http://koji.fedoraproject.org/koji/buildinfo?buildID=301059 You can also just rebuild the .src.rpm, since I did all the testing in a f16 box -- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #32 from Mitch Davis mjd+red...@afork.com --- I fetched the .src.rpm and it rebuilt under F16 very easily. There's no systemd/initd support. I have been working on that for myself. I gave up on systemd because I couldn't get it to work. Initd is close to working in emperor mode. Anyone like work work on this with 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 techto...@gmail.com changed: What|Removed |Added CC||techto...@gmail.com --- Comment #30 from techto...@gmail.com 2012-05-19 08:30:29 EDT --- Where is the package for F16? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Terje Røsten terje...@phys.ntnu.no changed: What|Removed |Added CC||terje...@phys.ntnu.no --- Comment #29 from Terje Røsten terje...@phys.ntnu.no 2012-03-26 16:39:26 EDT --- Any chance of a F16 build and bodhi push? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #28 from Ewoud Kohl van Wijngaarden ewoud+red...@kohlvanwijngaarden.nl 2012-03-22 07:44:30 EDT --- I'd like to use uwsgi on EL6 but I'm unsure if I should request this here or in a separate bug. Would you be interested in maintaining it for EL6 as well? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||uwsgi-1.0.4-1.fc17 Resolution||ERRATA Last Closed||2012-03-13 14:29:55 --- Comment #27 from Fedora Update System upda...@fedoraproject.org 2012-03-13 14:29:55 EDT --- uwsgi-1.0.4-1.fc17 has been pushed to the Fedora 17 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #26 from Fedora Update System upda...@fedoraproject.org 2012-02-23 17:29:36 EST --- uwsgi-1.0.4-1.fc17 has been pushed to the Fedora 17 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #22 from Jon Ciesla limburg...@gmail.com 2012-02-22 08:28:38 EST --- Git done (by process-git-requests). Also, a word of advice, don't build RPMs as root. Even if you wrote every line of code in the tarball. :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #24 from Jorge A Gallegos k...@blegh.net 2012-02-23 02:05:38 EST --- Ok, I believe is all ready to go for f17, unless I'm missing something? thanks a lot guys, you rock! PS: Jon, I believe I didn't build my packages as root... I used mock on my machine and tested via koji later, did you see any warning wrt root file ownership? If so I'd like to know to fix my setup :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #23 from Fedora Update System upda...@fedoraproject.org 2012-02-23 02:01:49 EST --- uwsgi-1.0.4-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/uwsgi-1.0.4-1.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Jorge A Gallegos k...@blegh.net changed: What|Removed |Added CC||osk...@saarenmaa.fi --- Comment #25 from Jorge A Gallegos k...@blegh.net 2012-02-23 02:08:16 EST --- *** Bug 682704 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Jorge A Gallegos k...@blegh.net changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | Flag||fedora-cvs? --- Comment #20 from Jorge A Gallegos k...@blegh.net 2012-02-22 01:28:39 EST --- New Package SCM Request === Package Name: uwsgi Short Description: Fast, self-healing, application container server Owners: kad Branches: f16, f17 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #21 from Jorge A Gallegos k...@blegh.net 2012-02-22 01:31:47 EST --- Thanks a lot Steven! Ok, I hope I changed all of the right flags here, I'm following https://fedoraproject.org/wiki/Package_SCM_admin_requests#New_Packages and saw a bunch of results for SCM request bundled in the corresponding package review ticket, hope that is all ok, otherwise I can open a new ticket for that. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #16 from Jorge A Gallegos k...@blegh.net 2012-02-19 13:50:03 EST --- Ok put a new .src.rpm and .spec addressing the issues in the feedback, I also went ahead and updated to the latest stable version: SRPM: http://kad.fedorapeople.org/packages/uwsgi/uwsgi-1.0.4-1.fc16.src.rpm SPEC: http://kad.fedorapeople.org/packages/uwsgi/uwsgi.spec (or https://gist.github.com/629491ed695cc4004831 if you are interested in the history) Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3803298 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Steven Dake sd...@redhat.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #18 from Steven Dake sd...@redhat.com 2012-02-19 14:48:01 EST --- PACKAGE APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #17 from Steven Dake sd...@redhat.com 2012-02-19 14:47:17 EST --- Official review: [PASS] MUST: rpmlint must be run on the source rpm and all binary rpms the build produces. The output should be posted in the review. [root@beast SRPMS]# rpmlint uwsgi-1.0.4-1.fc16.src.rpm uwsgi.src: W: spelling-error %description -l en_US interprocess - inter process, inter-process, intercessors uwsgi.src: W: spelling-error %description -l en_US preforking - preforming, preferring uwsgi.src: W: spelling-error %description -l en_US evented - evened, vented, e vented uwsgi.src: W: spelling-error %description -l en_US uGreen - u Green, Green, green uwsgi.src: W: spelling-error %description -l en_US xml - XML, ml, x ml uwsgi.src: W: spelling-error %description -l en_US ini - uni, in, ii uwsgi.src: W: spelling-error %description -l en_US yaml - yam, yams, yawl 1 packages and 0 specfiles checked; 0 errors, 7 warnings. [PASS] MUST: The package must be named according to the Package Naming Guidelines. [root@beast x86_64]# rpmlint uwsgi-plugin-ruby-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-ruby.x86_64: W: summary-not-capitalized C uWSGI - Plugin for Ruby support uwsgi-plugin-ruby.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-rack-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-rack.x86_64: W: summary-not-capitalized C uWSGI - Ruby rack plugin uwsgi-plugin-rack.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-python3-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-python3.x86_64: W: summary-not-capitalized C uWSGI - Plugin for Python 3.2 support uwsgi-plugin-python3.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-python-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-python.x86_64: W: summary-not-capitalized C uWSGI - Plugin for Python support uwsgi-plugin-python.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-psgi-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-psgi.x86_64: W: summary-not-capitalized C uWSGI - Plugin for PSGI support uwsgi-plugin-psgi.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-nagios-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-nagios.x86_64: W: summary-not-capitalized C uWSGI - Plugin for Nagios support uwsgi-plugin-nagios.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-lua-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-lua.x86_64: W: summary-not-capitalized C uWSGI - Plugin for LUA support uwsgi-plugin-lua.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-greenlet-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-greenlet.x86_64: W: summary-not-capitalized C uWSGI - Plugin for Python Greenlet support uwsgi-plugin-greenlet.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-fastrouter-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-fastrouter.x86_64: W: summary-not-capitalized C uWSGI - Plugin for FastRouter support uwsgi-plugin-fastrouter.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-common-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-common.x86_64: W: summary-not-capitalized C uWSGI - Common plugins for uWSGI uwsgi-plugin-common.x86_64: W: spelling-error %description -l en_US uGreen - u Green, Green, green uwsgi-plugin-common.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 3 warnings. [root@beast x86_64]# rpmlint uwsgi-plugin-admin-1.0.4-1.fc16.x86_64.rpm uwsgi-plugin-admin.x86_64: W: summary-not-capitalized C uWSGI - Plugin for Admin support uwsgi-plugin-admin.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-devel-1.0.4-1.fc16.x86_64.rpm uwsgi-devel.x86_64: W: summary-not-capitalized C uWSGI - Development header files and libraries uwsgi-devel.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [root@beast x86_64]# rpmlint uwsgi-1.0.4-1.fc16.x86_64.rpm uwsgi.x86_64: W: spelling-error %description -l en_US interprocess - inter process, inter-process, intercessors uwsgi.x86_64: W: spelling-error %description -l en_US preforking - preforming, preferring uwsgi.x86_64: W: spelling-error %description -l en_US evented - evened, vented, e vented uwsgi.x86_64: W: spelling-error %description -l en_US uGreen - u Green, Green, green uwsgi.x86_64: W: spelling-error %description -l en_US xml -
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #12 from Jorge A Gallegos k...@blegh.net 2012-02-18 13:48:36 EST --- (In reply to comment #10) Python2 packages should buildrequires python2-devel python3 packages should buildrequires python3-devel I am not an expert in the plugins you have developed - could you please address whether these build requires make more sense then the python deps you have now? Hi Steven, Both python-devel and python3-devel are in the .spec, although at the top along with all the other plugin BuildRequires, should I move them down to each subpackage definition? Also, I'm somewhat confused if it should be python-devel or python2-devel, currently in my fedora 16 box there's no python2-devel but I think that just changed on this release. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=784156
--- Comment #13 from Jorge A Gallegos k...@blegh.net 2012-02-18 14:03:16 EST
---
(In reply to comment #11)
Gal
couple comments inline
(In reply to comment #9)
I've been asked to publish a full review report.
[PASS] MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.
The review process requires that the output of rpmlint on all binaries and
source rpm files be posted in the review.
[PASS] MUST: The package must be named according to the Package Naming
Guidelines.
[PASS] MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption.
[PASS] MUST: The package must meet the Packaging Guidelines.
[PASS] MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines.
[PASS] MUST: The License field in the package spec file must match the
actual
license.
[PASS] 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 must be included in %doc.
This is only done for the main package. It should be done for every package
(devel, plugin-*, so this requirement FAILS.
from
http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Subpackage_Licensing:
If a subpackage is dependent (either implicitly or explicitly) upon a base
package (where a base package is defined as a resulting binary package from the
same source RPM which contains the appropriate license texts as %doc), it is
not necessary for that subpackage to also include those license texts as %doc.
The -devel package is depending on the main uwsgi package, and all -plugin
packages depend on -plugin-common, which depends on the main uwsgi package.
[PASS] MUST: The spec file must be written in American English.
[PASS] MUST: The spec file for the package MUST be legible.
[PASS] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this
task.
If no upstream URL can be specified for this package, please see the Source
URL
Guidelines for how to deal with this.
Typically you would show the sha256sum in the review. For example:
[root@beast SOURCES]# sha256sum uwsgi-1.0.2.1.tar.gz
78280b57a970db7842e4481f8b00f13d011f27b340c869dc1ad28d564d716439
uwsgi-1.0.2.1.tar.gz
[root@beast SPECS]# wget
http://projects.unbit.it/downloads/uwsgi-1.0.2.1.tar.gz
--2012-02-14 09:06:35--
http://projects.unbit.it/downloads/uwsgi-1.0.2.1.tar.gz
Resolving projects.unbit.it... 81.174.68.52
Connecting to projects.unbit.it|81.174.68.52|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 465250 (454K) [application/x-gzip]
Saving to: “uwsgi-1.0.2.1.tar.gz”
100%[==] 465,250 93.5K/s in 5.9s
2012-02-14 09:06:41 (76.9 KB/s) - “uwsgi-1.0.2.1.tar.gz” saved [465250/465250]
[root@beast SPECS]# sha256sum uwsgi-1.0.2.1.tar.gz
78280b57a970db7842e4481f8b00f13d011f27b340c869dc1ad28d564d716439
uwsgi-1.0.2.1.tar.gz
[PASS] MUST: The package MUST successfully compile and build into binary
rpms
on at least one primary architecture.
[IRRELEVANT] MUST: If the package does not successfully compile, build or
work
on an architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed
in
bugzilla, describing the reason that the package does not
compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
[PASS] MUST: All build dependencies must be listed in BuildRequires, except
for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
[IRRELEVANT] MUST: The spec file MUST handle locales properly. This is done
by
using the %find_lang macro. Using %{_datadir}/locale/* is strictly
forbidden.
[IRRELEVANT] MUST: Every binary RPM package (or subpackage) which stores
shared
library files (not just symlinks) in any of the dynamic linker's default
paths,
must call ldconfig in %post and %postun.
rather then irrelevant, NA (not applicable) makes more sense here.
[PASS] MUST: Packages must NOT bundle copies of system libraries.
[IRRELEVANT] MUST: If the package is designed to be relocatable, the
packager
must state this fact in the request for review, along with the
rationalization
for relocation of that specific package. Without this, use of Prefix: /usr
is
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #14 from Steven Dake sd...@redhat.com 2012-02-18 19:13:57 EST --- (In reply to comment #12) (In reply to comment #10) Python2 packages should buildrequires python2-devel python3 packages should buildrequires python3-devel I am not an expert in the plugins you have developed - could you please address whether these build requires make more sense then the python deps you have now? Hi Steven, Both python-devel and python3-devel are in the .spec, although at the top along with all the other plugin BuildRequires, should I move them down to each subpackage definition? Also, I'm somewhat confused if it should be python-devel or python2-devel, currently in my fedora 16 box there's no python2-devel but I think that just changed on this release. The python2-devel and python3-devel should be in the in the top (the package is only built once, so subpackages doesn't make any sense). I missed them in the original spec, but they were present. Take care to change python-devel to python2-devel. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=784156
--- Comment #15 from Steven Dake sd...@redhat.com 2012-02-18 19:16:46 EST ---
(In reply to comment #13)
(In reply to comment #11)
Gal
couple comments inline
(In reply to comment #9)
I've been asked to publish a full review report.
[PASS] MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.
The review process requires that the output of rpmlint on all binaries and
source rpm files be posted in the review.
[PASS] MUST: The package must be named according to the Package Naming
Guidelines.
[PASS] MUST: The spec file name must match the base package %{name}, in
the
format %{name}.spec unless your package has an exemption.
[PASS] MUST: The package must meet the Packaging Guidelines.
[PASS] MUST: The package must be licensed with a Fedora approved license
and
meet the Licensing Guidelines.
[PASS] MUST: The License field in the package spec file must match the
actual
license.
[PASS] 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 must be included in %doc.
This is only done for the main package. It should be done for every package
(devel, plugin-*, so this requirement FAILS.
from
http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Subpackage_Licensing:
If a subpackage is dependent (either implicitly or explicitly) upon a base
package (where a base package is defined as a resulting binary package from
the
same source RPM which contains the appropriate license texts as %doc), it is
not necessary for that subpackage to also include those license texts as
%doc.
The -devel package is depending on the main uwsgi package, and all -plugin
packages depend on -plugin-common, which depends on the main uwsgi package.
Thanks your right - learning experience for everyone ;)
[PASS] MUST: The spec file must be written in American English.
[PASS] MUST: The spec file for the package MUST be legible.
[PASS] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this
task.
If no upstream URL can be specified for this package, please see the
Source URL
Guidelines for how to deal with this.
Typically you would show the sha256sum in the review. For example:
[root@beast SOURCES]# sha256sum uwsgi-1.0.2.1.tar.gz
78280b57a970db7842e4481f8b00f13d011f27b340c869dc1ad28d564d716439
uwsgi-1.0.2.1.tar.gz
[root@beast SPECS]# wget
http://projects.unbit.it/downloads/uwsgi-1.0.2.1.tar.gz
--2012-02-14 09:06:35--
http://projects.unbit.it/downloads/uwsgi-1.0.2.1.tar.gz
Resolving projects.unbit.it... 81.174.68.52
Connecting to projects.unbit.it|81.174.68.52|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 465250 (454K) [application/x-gzip]
Saving to: “uwsgi-1.0.2.1.tar.gz”
100%[==] 465,250 93.5K/s in 5.9s
2012-02-14 09:06:41 (76.9 KB/s) - “uwsgi-1.0.2.1.tar.gz” saved
[465250/465250]
[root@beast SPECS]# sha256sum uwsgi-1.0.2.1.tar.gz
78280b57a970db7842e4481f8b00f13d011f27b340c869dc1ad28d564d716439
uwsgi-1.0.2.1.tar.gz
[PASS] MUST: The package MUST successfully compile and build into binary
rpms
on at least one primary architecture.
[IRRELEVANT] MUST: If the package does not successfully compile, build or
work
on an architecture, then those architectures should be listed in the spec
in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug
filed in
bugzilla, describing the reason that the package does not
compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
[PASS] MUST: All build dependencies must be listed in BuildRequires,
except for
any that are listed in the exceptions section of the Packaging Guidelines
;
inclusion of those as BuildRequires is optional. Apply common sense.
[IRRELEVANT] MUST: The spec file MUST handle locales properly. This is
done by
using the %find_lang macro. Using %{_datadir}/locale/* is strictly
forbidden.
[IRRELEVANT] MUST: Every binary RPM package (or subpackage) which stores
shared
library files (not just symlinks) in any of the dynamic linker's default
paths,
must call ldconfig in %post and %postun.
rather then irrelevant, NA (not applicable) makes more sense here.
[PASS] MUST: Packages must NOT bundle copies of system libraries.
[IRRELEVANT] MUST: If the
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=784156
--- Comment #9 from Gal Hammer gham...@redhat.com 2012-02-14 09:48:06 EST ---
I've been asked to publish a full review report.
[PASS] MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.
[PASS] MUST: The package must be named according to the Package Naming
Guidelines.
[PASS] MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption.
[PASS] MUST: The package must meet the Packaging Guidelines.
[PASS] MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines.
[PASS] MUST: The License field in the package spec file must match the actual
license.
[PASS] 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 must be included in %doc.
[PASS] MUST: The spec file must be written in American English.
[PASS] MUST: The spec file for the package MUST be legible.
[PASS] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
If no upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
[PASS] MUST: The package MUST successfully compile and build into binary rpms
on at least one primary architecture.
[IRRELEVANT] MUST: If the package does not successfully compile, build or work
on an architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
[PASS] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
[IRRELEVANT] MUST: The spec file MUST handle locales properly. This is done by
using the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[IRRELEVANT] MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's default paths,
must call ldconfig in %post and %postun.
[PASS] MUST: Packages must NOT bundle copies of system libraries.
[IRRELEVANT] MUST: If the package is designed to be relocatable, the packager
must state this fact in the request for review, along with the rationalization
for relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
[PASS] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[PASS] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)
[PASS] MUST: Permissions on files must be set properly. Executables should be
set with executable permissions, for example.
[PASS] MUST: Each package must consistently use macros.
[PASS] MUST: The package must contain code, or permissable content.
[IRRELEVANT] MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity). [18]
MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present.
[IRRELEVANT] MUST: Static libraries must be in a -static package.
[PASS] MUST: Development files must be in a -devel package. [20]
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release}
[PASS] MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built.
[IRRELEVANT] MUST: Packages containing GUI applications must include a
%{name}.desktop file, and that file must be properly installed with
desktop-file-install in the %install section. If you feel that your packaged
GUI application does not need a .desktop file, you must put a comment in the
spec file with your explanation.
[PASS] MUST: Packages must not own files or directories already owned by other
packages. The rule of thumb here is that the first package to be installed
should own the files or directories that other packages may rely upon. This
means, for example, that no package in Fedora should ever
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #10 from Steven Dake sd...@redhat.com 2012-02-14 11:02:19 EST --- Python2 packages should buildrequires python2-devel python3 packages should buildrequires python3-devel I am not an expert in the plugins you have developed - could you please address whether these build requires make more sense then the python deps you have now? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=784156
--- Comment #11 from Steven Dake sd...@redhat.com 2012-02-14 11:13:46 EST ---
Gal
couple comments inline
(In reply to comment #9)
I've been asked to publish a full review report.
[PASS] MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.
The review process requires that the output of rpmlint on all binaries and
source rpm files be posted in the review.
[PASS] MUST: The package must be named according to the Package Naming
Guidelines.
[PASS] MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption.
[PASS] MUST: The package must meet the Packaging Guidelines.
[PASS] MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines.
[PASS] MUST: The License field in the package spec file must match the actual
license.
[PASS] 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 must be included in %doc.
This is only done for the main package. It should be done for every package
(devel, plugin-*, so this requirement FAILS.
[PASS] MUST: The spec file must be written in American English.
[PASS] MUST: The spec file for the package MUST be legible.
[PASS] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this
task.
If no upstream URL can be specified for this package, please see the Source
URL
Guidelines for how to deal with this.
Typically you would show the sha256sum in the review. For example:
[root@beast SOURCES]# sha256sum uwsgi-1.0.2.1.tar.gz
78280b57a970db7842e4481f8b00f13d011f27b340c869dc1ad28d564d716439
uwsgi-1.0.2.1.tar.gz
[root@beast SPECS]# wget
http://projects.unbit.it/downloads/uwsgi-1.0.2.1.tar.gz
--2012-02-14 09:06:35--
http://projects.unbit.it/downloads/uwsgi-1.0.2.1.tar.gz
Resolving projects.unbit.it... 81.174.68.52
Connecting to projects.unbit.it|81.174.68.52|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 465250 (454K) [application/x-gzip]
Saving to: “uwsgi-1.0.2.1.tar.gz”
100%[==] 465,250 93.5K/s in 5.9s
2012-02-14 09:06:41 (76.9 KB/s) - “uwsgi-1.0.2.1.tar.gz” saved [465250/465250]
[root@beast SPECS]# sha256sum uwsgi-1.0.2.1.tar.gz
78280b57a970db7842e4481f8b00f13d011f27b340c869dc1ad28d564d716439
uwsgi-1.0.2.1.tar.gz
[PASS] MUST: The package MUST successfully compile and build into binary rpms
on at least one primary architecture.
[IRRELEVANT] MUST: If the package does not successfully compile, build or work
on an architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work
on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
[PASS] MUST: All build dependencies must be listed in BuildRequires, except
for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
[IRRELEVANT] MUST: The spec file MUST handle locales properly. This is done by
using the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[IRRELEVANT] MUST: Every binary RPM package (or subpackage) which stores
shared
library files (not just symlinks) in any of the dynamic linker's default
paths,
must call ldconfig in %post and %postun.
rather then irrelevant, NA (not applicable) makes more sense here.
[PASS] MUST: Packages must NOT bundle copies of system libraries.
[IRRELEVANT] MUST: If the package is designed to be relocatable, the packager
must state this fact in the request for review, along with the rationalization
for relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
[PASS] MUST: A package must own all directories that it creates. If it does
not
create a directory that it uses, then it should require a package which does
create that directory.
are you sure about this? %{_libdir}/%name doesn't appear to be owned by any
package although it is used by a variety of packages. A recommendation on what
package should own this directory would be helpful for the packager as well.
[PASS] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)
[PASS] MUST: Permissions on files must be set properly. Executables should be
set with executable
[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #8 from Jorge A Gallegos k...@blegh.net 2012-02-07 01:38:48 EST --- (In reply to comment #6) Gal, Regarding ldconfig in %post section: From guidelines: Whenever possible (and feasible), Fedora Packages containing libraries should build them as shared libraries. In addition, every binary RPM package which contains shared library files (not just symlinks) in any of the dynamic linker's default paths The key determinate on ldconfig being run in %post section is if the shared object is in the default linker paths. From man ldconfig: ldconfig creates the necessary links and cache to the most recent shared libraries found in the directories specified on the command line, in the file /etc/ld.so.conf, and in the trusted directories (/lib and /usr/lib). The cache is used by the run-time linker, ld.so or ld- linux.so. ldconfig checks the header and filenames of the libraries it encounters when determining which versions should have their links updated. Thanks for the feedback, I just fixed the unstripped-binary-or-object warning, according to http://fedoraproject.org/wiki/Common_Rpmlint_issues#unstripped-binary-or-object Solution:- Make sure binaries are executable. - So I basically just made all .so files 0755. This warning does not appear anymore Updated SPEC is http://kad.fedorapeople.org/packages/uwsgi/uwsgi.spec Updated SRPM is http://kad.fedorapeople.org/packages/uwsgi/uwsgi-1.0.2.1-2.fc16.src.rpm You can check the changes to the spec at https://gist.github.com/629491ed695cc4004831 and the koji scratch build results at http://koji.fedoraproject.org/koji/taskinfo?taskID=3768034 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Gal Hammer gham...@redhat.com changed: What|Removed |Added CC||gham...@redhat.com --- Comment #5 from Gal Hammer gham...@redhat.com 2012-02-05 07:56:05 EST --- I am not in the package reviewer group, but have provided this review as part of my sponsorship process. As a result, this notifies future reviewers that the review should be verified. $ rpmlint uwsgi.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint uwsgi-1.0.2.1-1.fc16.src.rpm uwsgi.src: W: spelling-error %description -l en_US interprocess - inter process, inter-process, intercessors uwsgi.src: W: spelling-error %description -l en_US preforking - preforming, preferring uwsgi.src: W: spelling-error %description -l en_US evented - evened, vented, e vented uwsgi.src: W: spelling-error %description -l en_US coroutine - co routine, co-routine, routine uwsgi.src: W: spelling-error %description -l en_US uGreen - u Green, Green, green uwsgi.src: W: spelling-error %description -l en_US xml - XML, ml, x ml uwsgi.src: W: spelling-error %description -l en_US ini - uni, in, ii uwsgi.src: W: spelling-error %description -l en_US yaml - yam, yams, yawl 1 packages and 0 specfiles checked; 0 errors, 8 warnings. There are few more warning when running rpmlint on the uwsgi-*.rpm files. For example: uwsgi-plugin-common.x86_64: W: unstripped-binary-or-object /usr/lib64/uwsgi/cgi_plugin.so uwsgi-plugin-fastrouter.x86_64: W: no-documentation === I'm not sure about it. Do you need to run ldconfig in the %post section (http://fedoraproject.org/wiki/Packaging/Guidelines#Shared_Libraries)? === That's it. For now. :-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #6 from Steven Dake sd...@redhat.com 2012-02-05 11:34:34 EST --- Gal, Regarding ldconfig in %post section: From guidelines: Whenever possible (and feasible), Fedora Packages containing libraries should build them as shared libraries. In addition, every binary RPM package which contains shared library files (not just symlinks) in any of the dynamic linker's default paths The key determinate on ldconfig being run in %post section is if the shared object is in the default linker paths. From man ldconfig: ldconfig creates the necessary links and cache to the most recent shared libraries found in the directories specified on the command line, in the file /etc/ld.so.conf, and in the trusted directories (/lib and /usr/lib). The cache is used by the run-time linker, ld.so or ld- linux.so. ldconfig checks the header and filenames of the libraries it encounters when determining which versions should have their links updated. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #7 from Steven Dake sd...@redhat.com 2012-02-05 11:37:49 EST --- Gal, As part of your sponsorship process please use the example review contained below as a template. The checklist is here: http://fedoraproject.org/wiki/Packaging:ReviewGuidelines An example review is here: https://bugzilla.redhat.com/show_bug.cgi?id=781831 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Steven Dake sd...@redhat.com changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #4 from Steven Dake sd...@redhat.com 2012-02-01 10:40:10 EST --- Jorge, Thanks for the prompt review of Bug #772608. You did a really nice job of picking out the issues and demonstrating you can execute a review. I'll give Gal until Feb 6 to review this package at which point I'll review and then add you to the packagers group once the package has met packaging guidelines. Regards -steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Steven Dake sd...@redhat.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|sd...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Steven Dake sd...@redhat.com changed: What|Removed |Added CC||sd...@redhat.com Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Steven Dake sd...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #3 from Steven Dake sd...@redhat.com 2012-01-30 09:11:05 EST --- Jorge, I will sponsor you as a packager after meeting certain criteria. First you must demonstrate that you are able to submit packages for review (this package looks pretty good on first glance) and also that you are able to execute the review process. Please execute the review process on: https://bugzilla.redhat.com/show_bug.cgi?id=772608 The checklist is here: http://fedoraproject.org/wiki/Packaging:ReviewGuidelines An example review is here: https://bugzilla.redhat.com/show_bug.cgi?id=781831 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #1 from Jorge A Gallegos k...@blegh.net 2012-01-23 21:44:32 EST --- Opening a new ticket as per this comment in this req https://bugzilla.redhat.com/show_bug.cgi?id=682704#c9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 --- Comment #2 from Jorge A Gallegos k...@blegh.net 2012-01-23 21:57:29 EST --- Forgot to add the link to the koji scratch build, here it is: http://koji.fedoraproject.org/koji/taskinfo?taskID=3725888 I'm also starting the habit of keeping track of pre-approval spec changes in gist, like this: https://gist.github.com/629491ed695cc4004831 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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
