Opened 5 years ago

Closed 5 years ago

#58532 closed defect (fixed)

pypy @7.1.1: didn't catch all references to /sw and /opt/local

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: danchr (Dan Villiom Podlaski Christiansen)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: pypy

Description

Build fails:

DEBUG: Executing proc-post-org.macports.patch-patch-0
Error: Failed to patch pypy: didn't catch all references to /sw and /opt/local:
/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_lang_pypy/pypy/work/pypy2.7-v7.1.1-src/pypy/doc/image/bpnn_update_detail.png
DEBUG: Error code: NONE

Change History (6)

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

How could a PNG image possibly contain a relevant path reference? At worst it's just in the metadata, and it could easily just be /sw coincidentally occurring somewhere in the binary data.

comment:2 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)

Yes. The point is, the build failed, so maybe that check needs to be refined to only check text files.

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

In aa585153068a6336839c106cd2004f603913ed4d/macports-ports (master):

pypy: don't abort when '/sw' is in a binary file

See: #58532

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

I changed it to a warning for now, just to fix the build failure. I'll leave further refinements to the maintainer, such as restricting the check to appropriate file types, and a maintainer mode variable that can be set to turn the warning into an error.

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

Ping

comment:6 Changed 5 years ago by danchr (Dan Villiom Podlaski Christiansen)

Resolution: fixed
Status: assignedclosed

In a2875252600ac2b443dc3f0a1a98d043a11ed51c/macports-ports (master):

pypy: bump to 7.2.0

While at it, I restricted the check for hardcoded paths to text files,
which should fix https://github.com/macports/macports-ports/pull/58532.

Closes: #58532

Note: See TracTickets for help on using tickets.