[MacPorts] #43614: mupdf @1.4 update
#43614: mupdf @1.4 update -------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Keywords: haspatch | Port: mupdf -------------------------+-------------------------------- I'd like to take ownership of this port too -- Ticket URL: <https://trac.macports.org/ticket/43614> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: fixed | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Changes (by sean@…): * status: new => closed * resolution: => fixed Comment: Committed in r119826. Thanks! -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: fixed | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Changes (by ryandesign@…): * cc: ryandesign@…, sean@… (added) Comment: In the future, please supply a unified diff of your portfile changes, instead of a complete new portfile. Your new version removes the `platform darwin 8` block. Did you verify that it builds correctly on Tiger without this block? If not, I can try it a little later on. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: fixed | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by starkhalo@…): Oops, sorry I forgot about the unified diff, been working on 3 new port submissions and I didn't think of it. No, I didn't verify it on Tiger, I don't have access to it. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: fixed | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by ryandesign@…): So how did you decide to remove the `platform darwin 8` block? Also, you removed the code that caused the port to use `-arch` flags, so now it doesn't anymore. That needs to be put back. And a universal variant added, if possible. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Changes (by starkhalo@…): * status: closed => reopened * resolution: fixed => Comment: Replying to [comment:4 ryandesign@…]:
So how did you decide to remove the `platform darwin 8` block? Should I put that block back? I removed it because I had no way to test it.
Also, you removed the code that caused the port to use `-arch` flags, so now it doesn't anymore. That needs to be put back. And a universal variant added, if possible. So those are the arch flags you mentioned are required for all ports without configure? I'll put them back as well as on the other port submissions then.
I'll see what I can do about the universal variant Ryan. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by ryandesign@…): Replying to [comment:5 starkhalo@…]:
Replying to [comment:4 ryandesign@…]:
So how did you decide to remove the `platform darwin 8` block? Should I put that block back? I removed it because I had no way to test it.
Well, that block was added to fix a build failure reported on Tiger in #33240. So unless something has changed, the block may still be necessary. I'll test on Tiger shortly and let you know.
Also, you removed the code that caused the port to use `-arch` flags, so now it doesn't anymore. That needs to be put back. And a universal variant added, if possible. So those are the arch flags you mentioned are required for all ports without configure? I'll put them back as well as on the other port submissions then.
All ports that build architecture-specific software (and don't just install text files) should use `-arch` flags. If a port uses an unusual build system, then you have to add code to add those flags manually, i.e. by running `[get_canonical_archflags]` and putting its value wherever the build system understands it. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by ryandesign@…): Replying to [comment:5 starkhalo@…]:
I'll see what I can do about the universal variant Ryan.
Once you reinstate the use of `[get_canonical_archflags]`, adding a universal variant could be as simple as adding the line `variant universal {}` anywhere before the first time you call `[get_canonical_archflags]`. Or it may be that the reason we hadn't done that before is that there is some problem with building universal. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by starkhalo@…): Replying to [comment:6 ryandesign@…]:
Replying to [comment:5 starkhalo@…]:
Replying to [comment:4 ryandesign@…]:
So how did you decide to remove the `platform darwin 8` block? Should I put that block back? I removed it because I had no way to test it.
Well, that block was added to fix a build failure reported on Tiger in #33240. So unless something has changed, the block may still be necessary. I'll test on Tiger shortly and let you know.
Also, you removed the code that caused the port to use `-arch` flags, so now it doesn't anymore. That needs to be put back. And a universal variant added, if possible. So those are the arch flags you mentioned are required for all ports without configure? I'll put them back as well as on the other port submissions then.
All ports that build architecture-specific software (and don't just install text files) should use `-arch` flags. If a port uses an unusual build system, then you have to add code to add those flags manually, i.e. by running `[get_canonical_archflags]` and putting its value wherever the build system understands it.
Ok, so after reading #33240 I put back the `darwin 8` code, I really doubt it's going to build without it. Also did a healthy read on Portfiles examples in the source tarballs regarding `use_configure no` and `[get_canonical_flags]`, I've put them back in the patch along with an empty `variant universal` which built fine on my end, though I still have to look at examples on it since the documentation is severely lacking. Also, the patch was made against r119826 -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by ryandesign@…): Replying to [comment:8 starkhalo@…]:
Ok, so after reading #33240 I put back the `darwin 8` code, I really doubt it's going to build without it.
I verified that `gmake` is still required for Tiger, but it still failed to build, because the build tried to use `xcrun`, which did not exist back then. This is probably simply solvable by editing the Makerules file. `xcrun` is used to find the path to command line programs when the command line tools are not installed, but MacPorts requires the command line tools to be installed, so running `xcrun` is pointless for us. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:9> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by ryandesign@…): I've tried your patch, and although the newly added universal variant succeeds to build, the resulting software is not in fact universal, so this is not sufficient. {{{ $ port -v installed mupdf The following ports are currently installed: mupdf @1.4_0+universal (active) platform='darwin 13' archs='i386 x86_64' $ lipo -info /opt/local/bin/mutool Non-fat file: /opt/local/bin/mutool is architecture: x86_64 }}} -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:10> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by starkhalo@…): Replying to [comment:9 ryandesign@…]:
Replying to [comment:8 starkhalo@…]:
Ok, so after reading #33240 I put back the `darwin 8` code, I really doubt it's going to build without it.
I verified that `gmake` is still required for Tiger, but it still failed to build, because the build tried to use `xcrun`, which did not exist back then. This is probably simply solvable by editing the Makerules file. `xcrun` is used to find the path to command line programs when the command line tools are not installed, but MacPorts requires the command line tools to be installed, so running `xcrun` is pointless for us. I'm baffled by this. `xcrun` only appears twice and only in `Makerules` at lines 52-55 and 139-142. Both times inside an unreachable block code as `ifeq "$(OS)"` prevents it.
I'll take a look and see how to fix this non-universal binary issue. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:11> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Comment (by starkhalo@…): Replying to [comment:10 ryandesign@…]:
I've tried your patch, and although the newly added universal variant succeeds to build, the resulting software is not in fact universal, so this is not sufficient.
{{{ $ port -v installed mupdf The following ports are currently installed: mupdf @1.4_0+universal (active) platform='darwin 13' archs='i386 x86_64' $ lipo -info /opt/local/bin/mutool Non-fat file: /opt/local/bin/mutool is architecture: x86_64 }}} Fixed. {{{ $ lipo -info mudraw Architectures in the fat file: mudraw are: x86_64 i386 $ lipo -info mujstest Architectures in the fat file: mujstest are: x86_64 i386 $ lipo -info mupdf-x11 Architectures in the fat file: mupdf-x11 are: x86_64 i386 $ lipo -info mupdf-x11-curl Architectures in the fat file: mupdf-x11-curl are: x86_64 i386 $ lipo -info mutool Architectures in the fat file: mutool are: x86_64 i386 }}}
Replying to [comment:9 ryandesign@…]:
Replying to [comment:8 starkhalo@…]:
Ok, so after reading #33240 I put back the `darwin 8` code, I really doubt it's going to build without it.
I verified that `gmake` is still required for Tiger, but it still failed to build, because the build tried to use `xcrun`, which did not exist back then. This is probably simply solvable by editing the Makerules file. `xcrun` is used to find the path to command line programs when the command line tools are not installed, but MacPorts requires the command line tools to be installed, so running `xcrun` is pointless for us. Fixed. I missed the patchfiles declaration and `xcrun` is gone now, it should be working on Tiger now.
Thanks for all your input Ryan, let me know is there's anything left to address. -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:12> MacPorts <http://www.macports.org/> Ports system for OS X
#43614: mupdf @1.4 update --------------------------+-------------------------------- Reporter: starkhalo@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: fixed | Keywords: haspatch Port: mupdf | --------------------------+-------------------------------- Changes (by ryandesign@…): * status: reopened => closed * resolution: => fixed Comment: r120197 -- Ticket URL: <https://trac.macports.org/ticket/43614#comment:13> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts