openvax / vaxrank / 24402225647
96%
master: 88%

LAST BUILD BRANCH: v3.1.8
DEFAULT BRANCH: master
Ran
Jobs 0
Files 0
Run time –
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
24402225647

push

github

web-flow 
Fix WeasyPrint on macOS arm64 (test skip + deploy.sh DYLD export + README note) (#225)

* Make WeasyPrint PDF test + release scripts work on macOS arm64 (#224)

Three stacked fixes so the release flow stops tripping on Apple Silicon:

- tests/test_shell_script.py: add a _skip_if_pdf_backend_unavailable
  helper that does a trial weasyprint render (so the native pango/cairo
  load is actually exercised) and pytest.skip()s with an actionable
  message if OSError/ImportError. Same helper handles pdfkit by checking
  for wkhtmltopdf on PATH.

- deploy.sh + test.sh: re-export DYLD_FALLBACK_LIBRARY_PATH=/opt/homebrew/lib
  inside each script on Darwin. macOS SIP strips DYLD_* when exec'ing
  /bin/bash, so a parent-shell export doesn't survive the `#!/usr/bin/env
  bash` hop. Re-setting it inside the script is the only reliable fix
  short of switching to a non-SIP-protected bash.

- README.md: add a one-liner under the WeasyPrint install notes telling
  Apple Silicon users to add DYLD_FALLBACK_LIBRARY_PATH to their shell
  profile (Intel macOS doesn't need it; /usr/local/lib is already in
  dyld's default fallback path).

Verified with ./test.sh (DYLD unset in parent): 253 passed.

* Guard DYLD_FALLBACK_LIBRARY_PATH export against repeated prepends

Review feedback on #225: when deploy.sh calls test.sh, both scripts
prepend /opt/homebrew/lib, so the path would end up as
  /opt/homebrew/lib:/opt/homebrew/lib:…
dyld doesn't care, but it looks sloppy and would only get longer if
other scripts grew the same stanza.

- Skip the export when /opt/homebrew/lib is already on the path.
- Use `${VAR:+:$VAR}` so we don't synthesize a trailing empty entry
  when DYLD was unset to start with.

Verified with a 3-case harness (unset / existing-other-path / already-
contains-homebrew) plus full ./test.sh (253 passed).
Source Files on build 24402225647
Detailed source file information is not available for this build.