[MacPorts] #35171: mogenerator Portfile
#35171: mogenerator Portfile -----------------------------------------+---------------------------------- Reporter: public@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.1 Keywords: | Port: -----------------------------------------+---------------------------------- this is a Portfile for mogenerator, a quite useful tool for Mac/iOS developers. I http://rentzsch.github.com/mogenerator/ -- Ticket URL: <https://trac.macports.org/ticket/35171> MacPorts <http://www.macports.org/> Ports system for Mac OS
#35171: mogenerator Portfile -----------------------------------------+---------------------------------- Reporter: public@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.1 Keywords: | Port: mogenerator -----------------------------------------+---------------------------------- Changes (by ryandesign@…): * port: => mogenerator Comment: Some initial observations: * Delete the name and version lines; that's part of what the github.setup line does for you automatically * As "port lint" will tell you, the maintainers line should be obfuscated (host:user) * As "port lint" will tell you, "configure {}" should be changed to "use_configure no" * As "port livecheck" will tell you, "git" is not a valid livecheck.type * Does this port really use git-core at build time and at runtime? * Have you considered using the xcode portgroup? * Please change the whitespace of the portfile to conform to the modeline (indentations at multiples of 4 spaces) -- Ticket URL: <https://trac.macports.org/ticket/35171#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#35171: mogenerator Portfile -----------------------------------------+---------------------------------- Reporter: public@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.1 Keywords: | Port: mogenerator -----------------------------------------+---------------------------------- Comment(by public@…):
* Delete the name and version lines; that's part of what the github.setup line does for you automatically * As "port lint" will tell you, the maintainers line should be obfuscated (host:user) * As "port lint" will tell you, "configure {}" should be changed to "use_configure no" * As "port livecheck" will tell you, "git" is not a valid livecheck.type * Please change the whitespace of the portfile to conform to the modeline (indentations at multiples of 4 spaces) * Have you considered using the xcode portgroup?
done I guess the "lint" part should be mentioned on the docs? http://guide.macports.org/chunked/project.contributing.html
* Does this port really use git-core at build time and at runtime?
The port only needs git-core for cloning out of github, but because of your comment I assume that the github porgroup also takes care of that I have just attached Portfile.2 to the ticket There is also one extra modification I made, which should not be a big deal. In the first line, along the Vim parameters I set the portfile syntax as filetype, and not "tcl" -- Ticket URL: <https://trac.macports.org/ticket/35171#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#35171: mogenerator Portfile -----------------------------------------+---------------------------------- Reporter: public@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.1 Keywords: | Port: mogenerator -----------------------------------------+---------------------------------- Comment(by ryandesign@…): If you set fetch.type to git, then MacPorts base (not the github portgroup) automatically adds port:git as a fetch dependency. However in this port the fetch.type is not set to git, and git is not used at all to fetch the source (nor should it be). This port fetches a regular tarball, like other normal ports. It just happens to fetch that tarball from a hosting service called github. -- Ticket URL: <https://trac.macports.org/ticket/35171#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#35171: mogenerator Portfile -----------------------------------------+---------------------------------- Reporter: public@… | Owner: ryandesign@… Type: submission | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.1.1 Keywords: | Port: mogenerator -----------------------------------------+---------------------------------- Changes (by ryandesign@…): * status: new => assigned * owner: macports-tickets@… => ryandesign@… -- Ticket URL: <https://trac.macports.org/ticket/35171#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#35171: mogenerator Portfile ------------------------------------------+--------------------------------- Reporter: public@… | Owner: ryandesign@… Type: submission | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.1 Resolution: fixed | Keywords: Port: mogenerator | ------------------------------------------+--------------------------------- Changes (by ryandesign@…): * status: assigned => closed * resolution: => fixed Comment: Thanks; I committed it in r97120. I removed "use_configure no" because that is the default when using the xcode portgroup. I rearranged the lines of the portfile to match other ports. I added code to the destroot block to [wiki:PortfileRecipes#doc install the README file]. I guess the Contributing page in the Guide assumes you already have a complete Portfile submission or update to contribute, and assumes you have also read the rest of the Guide; lint is mentioned in the Guide [http://guide.macports.org/chunked/using.html#using.port.lint here]. -- Ticket URL: <https://trac.macports.org/ticket/35171#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts