Spec URL: http://kiilerich.com/tortoisehg.spec SRPM URL: http://kiilerich.com/tortoisehg-0.8-3.src.rpm Description: TortoiseHg is "TortoiseSVN for Hg" ... if you happen to know TortoiseSVN ... It consists of a gui layer on top of hg, available from the commandline through the hgtk command. The commands can also be made available in the context menu in Nautilus. I created the rpm package for upstream and would like to have it properly included in fedora. This is my first package, so I need a sponsor.
$ rpmlint tortoisehg-0.8-3.fc11.src.rpm tortoisehg-0.8-3.fc11.i586.rpm tortoisehg-nautilus-0.8-3.fc11.i586.rpm tortoisehg.i586: E: no-binary tortoisehg.i586: E: non-executable-script /usr/lib/python2.6/site-packages/hggtk/hgtk.py 0644 /usr/bin/env tortoisehg-nautilus.i586: W: only-non-binary-in-usr-lib tortoisehg-nautilus.i586: W: no-documentation 3 packages and 0 specfiles checked; 2 errors, 2 warnings. Yes, there are no binaries, but it can't be noarch because of nautilus-python extension folder name ... hgtk.py - patch has been submitted and applied upstream. Not worth patching here. Yes, the python nautilus-python extension folder should be somewhere else, but this is how it is. There _are_ no docs for tortoisehg, especially not for the nautilus extension. The package can be build with mock and has been successfully build on x86 and x64_64.
Just some comments after a quick look at your spec file: - There are translations, gettext is missing https://fedoraproject.org/wiki/Packaging:Guidelines#Handling_Locale_Files - You can drop '-n tortoisehg-%{version}' from the %setup, this is the default The licensing situation is a bit wired. PKG-INFO says GPLv2, hggtk/logview/colormap.py says GPLv2+, and contrib/nautilus-thg.py says GPL.
Thanks for the comments. Previous tar balls didn't contain po files but only mo files, so there were no need for gettext on build time. Upstream has changed that now and I added gettext as build requirement. I just _knew_ the license was GPLv2 and failed to review it... Upstream has clarified that it is GPLv2 - see http://n2.nabble.com/-thg-dev--TortoiseHg-license-tp3286436p3286246.html and http://n2.nabble.com/-thg-dev--TortoiseHg-license-tp3286436p3289824.html . A copy of iniparse has been dropped. A few files are under the more permissive GPLv2+, but this project is GPLv2 anyway. I have updated the spec to temporarily use the development branch where these changes are included. http://kiilerich.com/tortoisehg.spec http://kiilerich.com/tortoisehg-0.8-4.6da01818c9ea.fc11.src.rpm
* Fri Jul 24 2009 Mads Kiilerich <mads> 0.8.1-1 - New upstream release where minor fixes has been applied - Remove workarounds no longer needed http://kiilerich.com/tortoisehg.spec http://kiilerich.com/tortoisehg-0.8.1-1.fc11.src.rpm
Latest package seems 0.8.3. Would you update your srpm?
Updated to snapshot of development version which will be released before I get sponsored ;-) http://kiilerich.com/tortoisehg.spec http://kiilerich.com/tortoisehg-0.9-0.0.hg7d91c4a48d37.fc12.src.rpm tortoisehg.i686: E: no-binary tortoisehg-nautilus.i686: W: only-non-binary-in-usr-lib tortoisehg-nautilus.i686: W: no-documentation 3 packages and 1 specfiles checked; 1 errors, 2 warnings.
Well, * Redundant (Build)Requires - "BuildRequires: python" is redundant because python-devel requires python. - "Requires python >= 2.4" is redundant because python is required by pygtk2 and so on, and on supported Fedora branch python is at least 2.5.2. - "Requires: pycairo, pygobject2" is redundant because these are required by pygtk2 - And ">= 2.10" in "Requires: pygtk2" is redundant because on currently supported Fedora pygtk2 is 2.13.0. ! Note: - It seems that hgtk is a GUI application. Usually when packaging GUI application a corresponding desktop file should be installed altogether: https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files However is seems that hgtk requires at least one argument invoking "hgtk" command only does not make sense. So for now I don't request to install a desktop file for hgtk. Then: ------------------------------------------------------------- NOTE: Before being sponsored: This package will be accepted with another few work. But before I accept this package, someone (I am a candidate) must sponsor you. Once you are sponsored, you have the right to review other submitters' review requests and approve the packages formally. For this reason, the person who want to be sponsored (like you) are required to "show that you have an understanding of the process and of the packaging guidelines" as is described on : http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored Usually there are two ways to show this. A. submit other review requests with enough quality. B. Do a "pre-review" of other person's review request (at the time you are not sponsored, you cannot do a formal review) When you have submitted a new review request or have pre-reviewed other person's review request, please write the bug number on this bug report so that I can check your comments or review request. Fedora package collection review requests which are waiting for someone to review can be checked on my wiki page: http://fedoraproject.org/wiki/User:Mtasaka#B._Review_request_tickets (Check "No one is reviewing") Review guidelines are described mainly on: http://fedoraproject.org/wiki/Packaging/ReviewGuidelines http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ScriptletSnippets ------------------------------------------------------------
> * Redundant (Build)Requires Ok, too much "Explicit is better than implicit" ;-) > ! Note: > - It seems that hgtk is a GUI application. Usually when packaging GUI > application a corresponding desktop file should be installed altogether: > https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files > > However is seems that hgtk requires at least one argument invoking > "hgtk" command only does not make sense. So for now I don't request to > install a desktop file for hgtk. Yes, that's the reasoning. hgtk is a graphical command line tool. > NOTE: Before being sponsored: Thanks for the guidance. As qualified (! ;-)) applicant I know the process in theory, but knowing that someone are listening will motivate me to get more done. http://kiilerich.com/tortoisehg.spec http://kiilerich.com/tortoisehg-0.9-0.1.hgdc0d0231f39a.fc12.src.rpm
Okay, now tortoisehg itself is in good shape, so I will wait for your pre-review or your another review request.
Mr Tasaka, I would like to hand in some homework: A new package with gwsmhg, Bug 531391 A real review of asn1c, Bug 459872, Comment #9 A informal but thorough preview of python-ZODB3, Bug 476600, Comment #1
Okay. --------------------------------------------------------- This package (tortoisehg) is APPROVED by mtasaka --------------------------------------------------------- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". After you request for sponsorship a mail will be sent to sponsor members automatically (which is invisible for you) which notifies that you need a sponsor. After that, 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 10/11/12, 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. Note that after getting sponsored, you can review package-request packages formally (unless the submitter needs sponsor). I hope that you review bug 476600 formally.
Thanks! FAS said: kiilerix has applied to packager tortoisehg-0.9 was scheduled for release tomorrow but has been delayed for a week, so I'm a bit too early here ;-)
Okay, now I am sponsoring you. Please follow "Join" wiki again.
ping?
New Package CVS Request ======================= Package Name: tortoisehg Short Description: Mercurial gui tools and nautilus plugin Owners: kiilerix Branches: F-11 F-12 InitialCC:
CVS done.
Closing.
tortoisehg-0.9-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
tortoisehg-0.9-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
tortoisehg-0.9-1.fc12.x86_64 wants mercurial >= 1.4 which is present in version 1.3.1 in F12
Yes, I pushed tortoisehg to updates too early; I hadn't noticed that the push request for mercurial 1.4 package only was to updates-testing. Mercurial 1.4 was requested pushed to updates friday (3 days ago) but is still waiting in the release engineering queue. The only solution is to wait. As a workaround you can get mercurial 1.4 from updates-testing.