[MacPorts] #44535: CrystFEL software suite
#44535: CrystFEL software suite ------------------------+-------------------------------- Reporter: rkirian@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Keywords: | Port: crystfel ------------------------+-------------------------------- Hello, I have not submitted anything before, but I would like to make the software suite CrystFEL available for installation through macports. I have tested the portfile locally on a clean install of macports on Mavericks and it so far has worked fine. This software suite is for the analysis of x-ray crystallographic diffraction data. It should be under the "science" category. It is GPLv3 licensed. More information can be found here: http://www.desy.de/~twhite/crystfel/ -- Ticket URL: <https://trac.macports.org/ticket/44535> MacPorts <http://www.macports.org/> Ports system for OS X
#44535: CrystFEL software suite -------------------------+-------------------------------- Reporter: rkirian@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: crystfel | -------------------------+-------------------------------- Comment (by ryandesign@…): Thanks. Some observations: * the `license` should be "`GPL-3`" (or possibly "`GPL-3+`"; I haven't checked). See `port lint`: {{{ $ port lint ---> Verifying Portfile for crystfel Error: invalid license 'GPLv3': missing hyphen before version }}} * the `maintainers` line must be your valid email address, in our obfuscated host:user format * the pango dependency should be written so that pango-devel could satisfy it; see other ports that depend on pango for an example * pkgconfig and gtk-doc are typically only used at build time and should therefore be `depends_build` not `depends_lib` * for better logging, `--disable-silent-rules` should be added to `configure.args` * the `post-destroot` block has numerous problems: * "`file copy`" can be more succinctly written as "`copy`" * you're attempting to copy files directly into `${prefix}`, bypassing `${destroot}`; MacPorts sandboxing prevents this from succeeding; this ''should'' cause a permission denied error to be printed and the destroot phase to be interrupted at that point, but that's not happening; I don't understand why * if you fix that, then on the preceding line you're pre-creating the directory, which should cause the `copy` to exit with an error that the directory already exists * the Makefile already installs these files in ${prefix}/share/doc/crystfel/scripts; you could `move` them into ${prefix}/share/examples/crystfel to better conform to `portheir(7)`. -- Ticket URL: <https://trac.macports.org/ticket/44535#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#44535: CrystFEL software suite -------------------------+-------------------------------- Reporter: rkirian@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: crystfel | -------------------------+-------------------------------- Comment (by rkirian@…): Replying to [comment:2 ryandesign@…]:
Cc Me!
Thank you for the many corrections and suggestions. I've done nearly everything suggested, and I removed the unnecessary `post-destroot` block. There is one exception: I don't know what to do about pango/pango-devel. I don't even know how to find a listing of Portfiles that depend on pango. I'll start digging into this, but further guidance would be greatly appreciated. Rick -- Ticket URL: <https://trac.macports.org/ticket/44535#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#44535: CrystFEL software suite -------------------------+-------------------------------- Reporter: rkirian@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: crystfel | -------------------------+-------------------------------- Comment (by larryv@…): Replying to [comment:3 rkirian@…]:
There is one exception: I don't know what to do about pango/pango-devel. I don't even know how to find a listing of Portfiles that depend on pango. I'll start digging into this, but further guidance would be greatly appreciated.
You can get a workable list with `port echo depends:pango`. -- Ticket URL: <https://trac.macports.org/ticket/44535#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#44535: CrystFEL software suite -------------------------+-------------------------------- Reporter: rkirian@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: crystfel | -------------------------+-------------------------------- Comment (by rkirian@…): Replying to [comment:4 larryv@…]:
You can get a workable list with `port echo depends:pango`.
Thank you. Following a few examples, I added `path:lib/pkgconfig/pango.pc:pango` to `depends_lib`. The installation appears to be successful. -- Ticket URL: <https://trac.macports.org/ticket/44535#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#44535: CrystFEL software suite -------------------------+-------------------------- Reporter: rkirian@… | Owner: ryandesign@… Type: submission | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: crystfel | -------------------------+-------------------------- Changes (by ryandesign@…): * status: new => assigned * cc: ryandesign@… (removed) * owner: macports-tickets@… => ryandesign@… -- Ticket URL: <https://trac.macports.org/ticket/44535#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#44535: CrystFEL software suite -------------------------+-------------------------- Reporter: rkirian@… | Owner: ryandesign@… Type: submission | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: fixed | Keywords: Port: crystfel | -------------------------+-------------------------- Changes (by ryandesign@…): * status: assigned => closed * resolution: => fixed Comment: Thanks. Added in r123124 with a few more changes, including fixing the livecheck. -- Ticket URL: <https://trac.macports.org/ticket/44535#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts