#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)
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)
Change available for review at https://github.com/macports/macports-base/pull/13.
comment:4 Changed 8 years ago by raimue (Rainer Müller)
Version: | 2.3.5 → 2.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: | new → closed |
comment:7 Changed 8 years ago by jmroot (Joshua Root)
Milestone: | → MacPorts 2.4.0 |
---|---|
Version: | 2.3.99 → 2.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.
Cf. #17182