#35852: new port: ccnx ------------------------------------------+--------------------------------- Reporter: alexander.afanasyev@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Keywords: | Port: ccnx ------------------------------------------+--------------------------------- Changes (by ryandesign@…): * cc: alexander.afanasyev@… (removed) * port: => ccnx Comment: Some notes: * The epoch and revision should start at 0. * The license versions should be indicated. * Multiple configure.env-append and build.env-append invocations should be combined. * Possibly you may want to append to rather than overwrite configure.cflags. * The reinplace that you are doing in a pre-build block should probably be done in a post-patch or post-configure block, so that, if the user were to interrupt and repeat the build phase, the second try would not try to re-run the reinplace * "xinstall -d" should be used to create directories instead of "system "mkdir"" * "file attributes -permissions" should be used to set permissions instead of "system "chown"", or when creating a directory you can use "xinstall" to do that too * You should not register a logfile to the port (by installing it into the destroot); the logfile would be deleted when the port is upgraded or uninstalled, which the user would not expect. If the software cannot live without an existing logfile, then you can create it directly in ${prefix} (i.e. not in ${destroot}${prefix}) in a post-activate block, if it doesn't already exist. * The portfile uses our standard modeline, which is good, but then the portfile's whitespace should conform to it: indentations should occur at multiples of 4 spaces -- Ticket URL: <https://trac.macports.org/ticket/35852#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS