[MacPorts] #28956: TortoiseHg Port Submission
#28956: TortoiseHg Port Submission ----------------------------------------+----------------------------------- Reporter: guidolorenz@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: mercurial | Port: ----------------------------------------+----------------------------------- This is a Portfile for [http://tortoisehg.bitbucket.org/ TortoiseHg] 2.0.2, a PyQt4-based GUI for Mercurial. -- Ticket URL: <https://trac.macports.org/ticket/28956> MacPorts <http://www.macports.org/> Ports system for Mac OS
#28956: TortoiseHg Port Submission ----------------------------------------+----------------------------------- Reporter: guidolorenz@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: mercurial | Port: ----------------------------------------+----------------------------------- Comment(by shiena.jp@…): I added the config for the locale, the icon, and the copyright. Because it cannot be displayed that it is a config of default. In addition, "/opt/local/bin/thg" and the html document are added. -- Ticket URL: <https://trac.macports.org/ticket/28956#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#28956: TortoiseHg Port Submission ----------------------------------------+----------------------------------- Reporter: guidolorenz@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: mercurial | Port: ----------------------------------------+----------------------------------- Comment(by guidolorenz@…): Hi Shiena, thanks for your input. I had a look at your changes and have a few questions and comments: 1. Copying the license file and building the english documentation is a good idea. 1. Also building czech and japanese documentation seems a bit too much to me. I'm not sure, though -- is there a MacPorts best practice for dealing with localized documentation? 1. Why do you manually create a symlink for thg in ${prefix}/bin? For me, the python26 port group already created this symlink automatically. 1. I'm not a big fan of patch files -- instead of patching doc/Makefile, what do you think of calling make like this: {{{make SPHINXBUILD=sphinx- build-2.6 html}}} 1. Why do we need config.py? To me it seemed like the auto-detection mechanisms worked fine. If we want to use a custom config.py, we must replace the hardcoded "/opt/local" by the value of ${prefix}. Cheers, Guido -- Ticket URL: <https://trac.macports.org/ticket/28956#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#28956: TortoiseHg Port Submission ----------------------------------------+----------------------------------- Reporter: guidolorenz@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: mercurial | Port: ----------------------------------------+----------------------------------- Comment(by shiena.jp@…): Hi Guido Thank you for seeing my file.
1. Copying the license file and building the english documentation is a good idea. Thank you.
2. Also building czech and japanese documentation seems a bit too much to me. I'm not sure, though -- is there a MacPorts best practice for dealing with localized documentation? There is especially no intention though I did all documents in the build.
3. Why do you manually create a symlink for thg in ${prefix}/bin? For me, the python26 port group already created this symlink automatically. It is my mistake. I misunderstood it while a variety of trying. A symlink need not be made by manually.
4. I'm not a big fan of patch files -- instead of patching doc/Makefile, what do you think of calling make like this: {{{make SPHINXBUILD=sphinx-build-2.6 html}}} I am this feeling. Your method is good. The method had been forgotten.
5. Why do we need config.py? To me it seemed like the auto-detection mechanisms worked fine. If we want to use a custom config.py, we must replace the hardcoded "/opt/local" by the value of ${prefix}. I explain the content of config.py respectively.
bin_path This is added to PATH for the hg command. I set for the time being "/opt/local/bin". However, "${prefix}/bin" is better as you say. license_path This specifies the text displayed in the license dialog. When path is wrong, it becomes blank. locale_path If this passing is correct, it comes to be able to select UI Language of the TortoiseHG Setting dialog. I want to select the mother tongue (Japanese). icon_path This specifies passing icon as well as locale_path. However, the Qt library seems to detect the setting of default automatically. nofork This is the same as default. I corrected Portfile and config.py from your questions. -- Ticket URL: <https://trac.macports.org/ticket/28956#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#28956: TortoiseHg Port Submission ----------------------------------------+----------------------------------- Reporter: guidolorenz@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: mercurial | Port: ----------------------------------------+----------------------------------- Comment(by guidolorenz@…): Shiena, thanks for the updated Portfile. I changed the version number to 2.0.3, because it has been released in the meantime. To me, the port seems ready to be committed now. -- Ticket URL: <https://trac.macports.org/ticket/28956#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#28956: TortoiseHg Port Submission ----------------------------------------+----------------------------------- Reporter: guidolorenz@… | Owner: ryandesign@… Type: submission | Status: assigned Priority: Normal | Milestone: Component: ports | Version: Keywords: mercurial | Port: tortoisehg ----------------------------------------+----------------------------------- Changes (by ryandesign@…): * status: new => assigned * owner: macports-tickets@… => ryandesign@… * port: => tortoisehg -- Ticket URL: <https://trac.macports.org/ticket/28956#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#28956: TortoiseHg Port Submission -----------------------------------------+---------------------------------- Reporter: guidolorenz@… | Owner: ryandesign@… Type: submission | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: mercurial Port: tortoisehg | -----------------------------------------+---------------------------------- Changes (by ryandesign@…): * status: assigned => closed * resolution: => fixed Comment: Committed in r78450 with some changes: * removed md5 checksum (we're gradually removing it from MacPorts) * changed runtime dependencies to library dependencies (e.g. "sudo port build" failed without py26-pyqt4 installed) * did patching in post-patch and copying extra files in post-extract, instead of both in pre-build * added "supported_archs noarch" since this port doesn't seem to install any architecture-specific files; removed "universal_variant no" since "supported_archs noarch" implies it I see 2.0.4 is already available; if you'd like to update to that version, please supply a diff in a new ticket. -- Ticket URL: <https://trac.macports.org/ticket/28956#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts