Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#64011 closed defect (fixed)

ports that use use_xmkmf yes fail with make Makefiles failure

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone: MacPorts 2.7.2
Component: base Version: 2.7.1
Keywords: Cc:
Port:

Description

Ports that use use_xmkmf yes fail to build. For example:

$ sudo port -n build tgif
--->  Cleaning tgif
--->  Computing dependencies for tgif
--->  Fetching distfiles for tgif
--->  Verifying checksums for tgif
--->  Extracting tgif
--->  Applying patches to tgif
--->  Configuring tgif
Error: Failed to configure tgif: make Makefiles failure: can't read "cd /opt/local/var/macports/build/_Users_rschmidt_macports_macports-ports-ryandesign-fork_graphics_tgif/tgif/work/tgif-QPL-4.2.5 && make Makefiles.dir": no such variable
Error: See /opt/local/var/macports/logs/_Users_rschmidt_macports_macports-ports-ryandesign-fork_graphics_tgif/tgif/main.log for details.
Error: Follow https://guide.macports.org/#project.tickets if you believe there is a bug.
Error: Processing of port tgif failed

This has the same root cause as #63931: as of [da3524e8a3006a9b9a2370b66fa91bb554dd768b/macports-base] (which introduced build progress bars) which was included for the first time in MacPorts 2.7.0, command_exec requires flags to be specified before the command but not all uses of command_exec were adjusted to match this new requirement; the invocation of command_exec for running make Makefiles when using xmkmf was not.

In addition, command_exec no longer honors the -varprefix flag which use_xmkmf yes requires.

Change History (2)

comment:1 Changed 3 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: assignedclosed

In c8ff808191a16032509ff5bf5b2df35faa9ed4c7/macports-base (master):

Fix ports that use "use_xmkmf yes"

This was broken by da3524e which changed command_exec to require flags
to be specified before the command without updating all uses of
command_exec to conform to the new requirement.

Closes: #64011

comment:2 Changed 3 years ago by jmroot (Joshua Root)

Milestone: MacPorts FutureMacPorts 2.7.2
Note: See TracTickets for help on using tickets.