Bug 478683
| Summary: | Review Request: perl-Net-Google-AuthSub - Provides interface to interact with sites that implement Google style AuthSub | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Andy Shevchenko <andy> |
| Component: | Package Review | Assignee: | Marcela Mašláňová <mmaslano> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | andy.shevchenko, cweyl, fedora-package-review, jan.klepek, notting |
| Target Milestone: | --- | Flags: | mmaslano:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | perl-Net-Google-AuthSub-0.5-1.fc11 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2010-02-18 22:21:05 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Andy Shevchenko
2009-01-03 14:58:32 UTC
smile.org.ua is not resolving for me currently. Please clear the whiteboard field if the site returns. I relocated sources to following site: http://starua.com/~andy/Fedora/Fedora.html So, the direct links are: http://starua.com/~andy/Fedora/perl-Net-Google-AuthSub.spec http://starua.com/~andy/Fedora/perl-Net-Google-AuthSub-0.4-1.sh7.src.rpm Marking as blocking FE-NEEDSPONSOR. (In reply to comment #4) > Marking as blocking FE-NEEDSPONSOR. Sorry, I probably lost new information about sponsorship. At least 8 packages have being already maintained by me and I am member of Fedora Packager CVS Commit Group. What else should I do to get sponsorship nowadays? Thank you. This whole thing is confusing. If you already maintain packages in Fedora, you don't require sponsorship. I'm not sure why Chris blocked FE-NEEDSPONSOR. Your FAS ID is andriy, isn't it? Could you consider updating to 0.5? *sigh* sorry about that -- your bugzilla email is different than your FAS email (andy.shevchenko) so my reviewing script flagged you as unsponsored. I'm going to drop this out of the review queue; Andy, please clear the whiteboard if you have a 0.5 package or info on why 0.4 should go in instead. any progress on packaging 0.5 version? If you are not interested anymore, let me know, I will take over this package. (In reply to comment #9) > any progress on packaging 0.5 version? If you are not interested anymore, let > me know, I will take over this package. Please, look at http://starua.com/~andy/Fedora/perl-Net-Google-AuthSub.spec and http://starua.com/~andy/Fedora/perl-Net-Google-AuthSub-0.5-1.sh7.src.rpm Any interest here? OK rpmlint on every package.
OK name of package accords to the Naming Guidelines.
OK spec file name match the base package %{name}.
OK package meet the Packaging Guidelines.
OK package has a good license.
OK spec file is written in American English.
OK spec file for the package must be legible.
OK The sources used to build the package must match the upstream source.
OK successful koji compilation.
http://koji.fedoraproject.org/koji/taskinfo?taskID=1937848
OK correct BuildRequires, Requires.
OK proper use of %find_lang macro.
OK shared library must call ldconfig in %post and %postun.
OK relocatable package must state this fact.
OK package must own their directories.
OK permissions on files must be set properly.
use rather %{__fixperms} then chmod in build section
OK %clean section with rm -rf %{buildroot}.
OK consistent use of macros.
OK package contains code, or permissable content.
OK large documentation go in a -doc subpackage.
OK %doc must not affect the runtime of the application.
OK header files must be in a -devel package.
OK static libraries must be in a -static package.
OK pkgconfig(.pc) files must 'Requires: pkgconfig'.
OK library files with a suffix (.so) must go in -devel.
OK usually devel packages must require the base package.
OK Remove .la libtool archives.
OK GUI applications must include a %{name}.desktop file.
OK %install section starts with rm -rf %{buildroot}.
Thank you.
Does it mean the package passes review? ('cause I don't see any flags, like fedora-review is set).
Thanks for reminder. ACCEPTED New Package CVS Request ======================= Package Name: perl-Net-Google-AuthSub Short Description: Provides interface to interact with sites that implement Google style AuthSub Owners: andriy Branches: F-11 F-12 InitialCC: perl-sig (In reply to comment #14) > Thanks for reminder. ACCEPTED Thank you for review! CVS done (by process-cvs-requests.py). perl-Net-Google-AuthSub-0.5-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/perl-Net-Google-AuthSub-0.5-1.fc12 perl-Net-Google-AuthSub-0.5-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Net-Google-AuthSub-0.5-1.fc11 perl-Net-Google-AuthSub-0.5-1.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-Net-Google-AuthSub'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2010-1651 perl-Net-Google-AuthSub-0.5-1.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-Net-Google-AuthSub'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-1671 perl-Net-Google-AuthSub-0.5-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. perl-Net-Google-AuthSub-0.5-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report. |