Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#53169 closed defect (fixed)

PortIndex contains expanded prefix

Reported by: jmroot (Joshua Root) Owned by: neverpanic (Clemens Lang)
Priority: Normal Milestone: MacPorts 2.4.0
Component: base Version: 2.3.5
Keywords: Cc: neverpanic (Clemens Lang)
Port:

Description

Somewhere along the way, portindex lost its code for replacing the prefix with the static string {${prefix}}. That means you end up with this sort of thing:

% port -v deps --index BiggerSQL
Full Name: BiggerSQL @1.3.9_0
Library Dependencies: path:/var/tmp/mprsyncup/opt/local/lib/postgresql84:postgresql84

Change History (8)

comment:1 Changed 8 years ago by jmroot (Joshua Root)

Cf. #17182

comment:2 Changed 8 years ago by neverpanic (Clemens Lang)

Yep, that was probably me in [0e21180e3d1b197d077fde37a394ef67d8e486d1/macports-base], since I didn't understand the purpose of set prefix {\${prefix}} and it didn't have a comment explaining it.

comment:3 Changed 8 years ago by neverpanic (Clemens Lang)

comment:4 Changed 8 years ago by raimue (Rainer Müller)

Version: 2.3.52.3.99

I checked and this bug does not exist in 2.3.5, but only on 2.3.99 master.

comment:5 Changed 8 years ago by jmroot (Joshua Root)

How come it's in the PortIndex on the rsync server then?

comment:6 Changed 8 years ago by neverpanic (Clemens Lang)

Owner: set to neverpanic
Resolution: fixed
Status: newclosed

In 9a23db2/macports-base:

portindex: Restore special $prefix handling

This avoids expanded values for ${prefix} in the PortIndex.

Closes: #53169
See: #17182

comment:7 Changed 8 years ago by jmroot (Joshua Root)

Milestone: MacPorts 2.4.0
Version: 2.3.992.3.5

I've confirmed the problem exists in 2.3.5.

comment:8 Changed 8 years ago by raimue (Rainer Müller)

Sorry, that was misleading. The problem exists in 2.3.5, but the commit linked earlier was never merged to the release-2.3 branch. I tried to bisect it but the problem was definitely there back in 2.3.0, but earlier releases do not work on macOS 10.12 Sierra.

Note: See TracTickets for help on using tickets.