Ticket #27479 (closed defect: fixed)
mozplugger builds universal even when the universal variant was not requested
| Reported by: | ryandesign@… | Owned by: | jeremyhu@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 1.9.2 |
| Keywords: | Cc: | ||
| Port: | mozplugger |
Description
$ sudo port install mozplugger -universal Password: ---> Fetching mozplugger ---> Attempting to fetch mozplugger-1.14.2.tar.gz from http://distfiles.macports.org/mozplugger ---> Verifying checksum(s) for mozplugger ---> Extracting mozplugger ---> Configuring mozplugger ---> Building mozplugger ---> Staging mozplugger into destroot ---> Installing mozplugger @1.14.2_0 ---> Activating mozplugger @1.14.2_0 ---> Cleaning mozplugger $ port -v installed mozplugger The following ports are currently installed: mozplugger @1.14.2_0 (active) platform='darwin 10' archs='x86_64' $ port -q contents mozplugger | grep /bin/ | xargs lipo -info Architectures in the fat file: /opt/local/bin/mozplugger-controller are: x86_64 i386 Architectures in the fat file: /opt/local/bin/mozplugger-helper are: x86_64 i386 Architectures in the fat file: /opt/local/bin/mozplugger-linker are: x86_64 i386 $
r73886 caused this, because it unconditionally uses ${configure.universal_whatever}. These should only be used if the universal variant is selected. And their non-universal counterparts should be used in the other case, to support changing the build_arch.
Attachments
Change History
comment:1 Changed 2 years ago by jeremyhu@…
- Status changed from new to closed
- Resolution set to fixed
comment:2 Changed 2 years ago by ryandesign@…
- Status changed from closed to reopened
- Resolution fixed deleted
They're always set.
You also need to actually create a universal variant, as in the attached patch.
comment:3 Changed 2 years ago by ryandesign@…
- Status changed from reopened to closed
- Resolution set to fixed
Ah, it's openmaintainer; I fixed it in r73905.
comment:4 in reply to: ↑ description Changed 2 years ago by ryandesign@…
Replying to ryandesign@…:
And their non-universal counterparts should be used in the other case, to support changing the build_arch.
Fixed this in r73906.
Note: See
TracTickets for help on using
tickets.


I thought those were only set if we were +universal...
r73898