#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 7 years ago by jmroot (Joshua Root)
comment:2 Changed 7 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 7 years ago by neverpanic (Clemens Lang)
Change available for review at https://github.com/macports/macports-base/pull/13.
comment:4 Changed 7 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 7 years ago by jmroot (Joshua Root)
How come it's in the PortIndex on the rsync server then?
comment:6 Changed 7 years ago by neverpanic (Clemens Lang)
| Owner: | set to neverpanic |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:7 Changed 7 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 7 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