Bug 453465
Summary: | Review Request: transbot - An irc bot designed to translate between languages on separate channels | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | John McLean <mclean.jb> |
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, mtasaka, notting |
Target Milestone: | --- | Flags: | mtasaka:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-08-12 13:29:11 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
John McLean
2008-06-30 19:29:59 UTC
for a starter; - ditch the name, version and release macros from the first 5 lines, as they are identical to the ones used below - the URL link should point to the place where one can found the project that you are packaging, not to your [empty] page in fedora's wiki. - a better %source value would be https://fedorahosted.org/lingobot/browser/transbot-0.1-1.src.rpm?format=raw - the changelog must contain at least one entry -the macros in the first 5 lines are gone. -the url link now points to http://git.fedorahosted.org/git/lingobot.git -the %source you suggested won't work properly because of the '?format=raw' tagged onto the end. The spec file searches for a file called 'transbot-0.1-1.src.rpm?format=raw'. Therefore, I've simply hosted the project on another site that does have a direct link to the source tarball. It should work fine. -The changelog now has an entry. Please post the new URLs of your spec/srpm when you modify them. Please make it sure that you change the release number of your srpm every time you modify your spec/srpm (if the version number does not change). I just fixed a small bug in the program itself. I've incremented the release number and uploaded the new srpm and spec. The spec url is the same. SRPM: http://jesusfreak91.googlepages.com/transbot-0.1-2.fc9.src.rpm Added utf-8 character support. new srpm url: http://jesusfreak91.googlepages.com/transbot-0.1-3.fc9.src.rpm For 1.0-3: * License - (As you seem to be the upstream developer) unless you mention in the tarball that this package is licensed under GPLv2+, we regard the license of this package to be GPL+ (from section 9) * Source - The tarball which can be downloaded from the URL differs from the one in the tarball. When you modify the tarball itself, please change the version number and upload the new one. * %defattr - We now recommend %defattr(-,root,root,-) * General rpmlint issue -------------------------------------------------------------- transbot.noarch: E: non-executable-script /usr/lib/python2.5/site-packages/trans/transbot.py 0644 transbot.noarch: E: standard-dir-owned-by-package /usr/share/man/man1 -------------------------------------------------------------- - If the permission of transbot.py is correct (as 0644), then this script should not have shebang. - This package must not own /usr/share/man/man1 (owned by filesystem) -I've fixed the licensing issues. It is now simply GPL+ -The latest source tarball has been uploaded at the same url. I simply forgot to upload it last time I made changes. I'm still forming habits concerning uploading changes and bumping the release numbers. -%defattr is fixed. -The rpmlint issues should be cleared up. The shebang was left over from some testing. The package shouldn't own /usr/share/man/man1 anymore. forgot to add this to my last comment: new srpm: http://jesusfreak91.googlepages.com/transbot-0.1-4.fc9.src.rpm Well, * Currently I don't know how to use this, however when I just type $ transbot, I get a traceback as --------------------------------------------------------------------------- $ transbot Traceback (most recent call last): File "/usr/bin/transbot", line 37, in <module> channel_list = conf_file.readline().split ( ':' ).lower() [ 1 ] AttributeError: 'list' object has no attribute 'lower' ---------------------------------------------------------------------------- Is this expected? (In reply to comment #7) > -The latest source tarball has been uploaded at the same url. I simply forgot > to upload it last time I made changes. I'm still forming habits concerning > uploading changes and bumping the release numbers. * What I asked you is to change the _version_ number when you modify the tarball itself (release number is rpm specific). Changing tarball without changing version may confuse people who are trying to use this package. (In reply to comment #9) > Well, > > * Currently I don't know how to use this, however when I just type $ transbot, > I get a traceback as > --------------------------------------------------------------------------- > $ transbot > Traceback (most recent call last): > File "/usr/bin/transbot", line 37, in <module> > channel_list = conf_file.readline().split ( ':' ).lower() [ 1 ] > AttributeError: 'list' object has no attribute 'lower' > ---------------------------------------------------------------------------- > Is this expected? > The traceback came from a typo. I'll need to go back and fix that. > (In reply to comment #7) > > -The latest source tarball has been uploaded at the same url. I simply forgot > > to upload it last time I made changes. I'm still forming habits concerning > > uploading changes and bumping the release numbers. > > * What I asked you is to change the _version_ number when you modify the > tarball > itself (release number is rpm specific). > > Changing tarball without changing version may confuse people who are trying > to > use this package. Alright. I guess I read your comment too quickly and equated version with release in my mind. My apologies. For future reference, should the version number be changed even for one-line corrections? uploaded new srpm: http://jesusfreak91.googlepages.com/transbot-0.2-1.fc9.src.rpm (In reply to comment #10) > > * What I asked you is to change the _version_ number when you modify the > > tarball > > itself (release number is rpm specific). > > > > Changing tarball without changing version may confuse people who are trying > > to > > use this package. > > Alright. I guess I read your comment too quickly and equated version with > release in my mind. My apologies. For future reference, should the version > number be changed even for one-line corrections? If you want to modify the tarball itself and wants to publish a new tarball _formally_, please. (Note: some upstream developer and also Fedora package submitter doesn't publish new tarball formally until the review request he/she submitted is passed) Well, * Now this package itself is okay * You are currently in need of sponsor. In this case you have to either submit another review request or do pre-review of other person's review request as described in http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored You have 2 other review requests, which seem good to some extent (they need fixing on some points... however it is not sure if I have enough time to review your other review requests at the moment) -------------------------------------------------------------------------------------- This package (transbot) is APPROVED by mtasaka -------------------------------------------------------------------------------------- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". At a point a mail should be sent to sponsor members which notifies that you need a sponsor. At the stage, please also write on this bug for confirmation that you requested for sponsorship and your FAS (Fedora Account System) name. Then I will sponsor you. If you want to import this package into Fedora 8/9, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask me. I've requested sponsorship. My FAS name is mcleanj. Confirmed. Now I am sponsoring you. Please follow "Join" wiki again. Removing NEEDSPONSOR. New Package CVS Request ======================= Package Name: transbot Short Description: IRC translation bot Owners: mcleanj Branches: F-8 F-9 InitialCC: mcleanj Cvsextras Commits: yes cvs done. Please make it sure that the EVR (Epoch-Version-Release) of rpm has the order devel >= F-9 >= F-8 Currently F-8 transbot has transbot-0.2-3.fc8 so rawhide (F-10) transbot must have higher EVR than 0.2-3.fc8. Please bump the release number of F-10/9 rpms. Note: When you want to fix F-9/8 rpm while you don't have to modify F-10 rpm, you can use "Release: 1%{?dist}.1" for example because 0.2-1.fc10 > 0.2-1.fc8.1 . |