New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #27479 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

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

mozplugger-universal.diff (218 bytes) - added by ryandesign@… 3 years ago.
proposed patch

Change History

comment:1 Changed 3 years ago by jeremyhu@…

  • Status changed from new to closed
  • Resolution set to fixed

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

r73898

comment:2 Changed 3 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.

Changed 3 years ago by ryandesign@…

proposed patch

comment:3 Changed 3 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 3 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.