[MacPorts] #43564: girara @0.2.0
#43564: girara @0.2.0 -------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Keywords: pwmt | Port: girara -------------------------+-------------------------------- New port -- Ticket URL: <https://trac.macports.org/ticket/43564> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: girara | --------------------------+-------------------------------- Changes (by ryandesign@…): * keywords: pwmt => Comment: Thanks. Some observations: * There should be a blank line between the "`# $Id$`" line and the "`PortSystem 1.0`" line. * The license field should just be "`zlib`" (not "`zlib License`"). * The homepage variable could be used in the definition of master_sites. * The line "`distname girara-${version}`" should be removed because that is the default. * The checksums for the patchfiles should be removed because checksums are not used for local patchfiles. * The distname should be removed from the checksums line because there is only one file being checksummed. * Because you use "`use_configure no`" you should add code to ensure you're UsingTheRightCompiler, use the right `-arch` flags, and add a universal variant if possible. * The dependency on glib2 should be written so that glib2-devel could satisfy it. * The patchfile names should be of the form "patch-*.diff". In patch-Makefile: * The install_name is usually set with "`-install_name`"; if there is no special reason why you instead used "`-Wl,-dylib_install_name,`", it should be changed (in both places) * Have you reported the OS X compatibility issues that necessitated writing this patch to the developers of girara so that they can fix the build system so that these patches are no longer needed in the future? If so please let us know the URL of the bug report, mailing list post, etc. -- Ticket URL: <https://trac.macports.org/ticket/43564#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: girara | --------------------------+-------------------------------- Comment (by starkhalo@…): Replying to [comment:1 ryandesign@…]:
Thanks. Some observations:
* There should be a blank line between the "`# $Id$`" line and the "`PortSystem 1.0`" line. Done.
* The license field should just be "`zlib`" (not "`zlib License`"). Done.
* The homepage variable could be used in the definition of master_sites. Done.
* The line "`distname girara-${version}`" should be removed because that is the default. Done.
* The checksums for the patchfiles should be removed because checksums are not used for local patchfiles. Done.
* The distname should be removed from the checksums line because there is only one file being checksummed. Done.
* Because you use "`use_configure no`" you should add code to ensure you're UsingTheRightCompiler, use the right `-arch` flags, and add a universal variant if possible.
-- ''UsingTheRightCompiler'' I have checked the Makefile and it uses ${CC}, I also added the recommended lines for ''Ports with nonstandard or non-existent configure scripts'' https://trac.macports.org/wiki/UsingTheRightCompiler #nonstandard-ports -- ''build with `-arch- flags`'' Is this not handled by default? -- ''add a universal variant'' Why? §5.3.7.1 ''Configure Universal'' ''There is a default universal variant made available to all ports by MacPorts base, so redefining universal keywords '''should only be done''' to make a given port compile '''if''' the default options fail to do so.''
* The dependency on glib2 should be written so that glib2-devel could satisfy it. Done.
* The patchfile names should be of the form "patch-*.diff". Done.
In patch-Makefile:
* The install_name is usually set with "`-install_name`"; if there is
no special reason why you instead used "`-Wl,-dylib_install_name,`", it should be changed (in both places) I believe it's more compatible with older versions of XCode CLT, it's a synonym for ''--install_name'' in recent versions so it shouldn't matter.
* Have you reported the OS X compatibility issues that necessitated writing this patch to the developers of girara so that they can fix the build system so that these patches are no longer needed in the future? If so please let us know the URL of the bug report, mailing list post, etc. I'm in touch with a couple devs in their IRC channel at oftc#pwmt, my priority is to support the latest versions through macports.
-- Ticket URL: <https://trac.macports.org/ticket/43564#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: girara | --------------------------+-------------------------------- Comment (by starkhalo@…): Apparently I missed `${destroot}` in destroot.args -- Ticket URL: <https://trac.macports.org/ticket/43564#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: girara | --------------------------+-------------------------------- Comment (by starkhalo@…): Ok, added `-arch` flags and `variant universal` and tested the Portfile to work correctly. Is there anything else I should address for approval? -- Ticket URL: <https://trac.macports.org/ticket/43564#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: girara | --------------------------+-------------------------------- Comment (by starkhalo@…): This Portfile is not ready for submission, `variant universal` is not building fat binaries and `-arch` flags while it builds cleanly produces a misbehaving binary. I understand `variant universal` can be removed and worked at a later date if fat binaries are requested, but `-arch` flags are a '''requisite''' for port approval because of the `use_configure no` directive. I have to find out what is causing the bug. -- Ticket URL: <https://trac.macports.org/ticket/43564#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: girara | --------------------------+-------------------------------- Comment (by starkhalo@…): `universal variant` is working correctly now. And the library is behaving correctly. $ lipo -info /opt/local/lib/libgirara-gtk3.1.1.dylib Architectures in the fat file: /opt/local/lib/libgirara-gtk3.1.1.dylib are: x86_64 i386 Let me know if there is anything left to address. -- Ticket URL: <https://trac.macports.org/ticket/43564#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#43564: girara @0.2.0 --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: submission | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: fixed | Keywords: Port: girara | --------------------------+-------------------------------- Changes (by cal@…): * status: new => closed * resolution: => fixed Comment: Committed in r120310 with the following changes: - Added active_variants PortGroup and required `gtk3 +x11`, because girara uses gdkx.h, which isn't provided in gtk3 +quartz. - Changed category to `devel` and `gnome`, because there is not primary `gtk` category. - Added `VERBOSE=1` to `build.env` to simplify debugging of build failures. -- Ticket URL: <https://trac.macports.org/ticket/43564#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts