Skip to content

r-zip: update to 3.0.0, drop obsolete libr-zip.patch#367

Merged
starsareintherose merged 1 commit into
BioArchLinux:masterfrom
bshor:r-zip-drop-patch
Jun 24, 2026
Merged

r-zip: update to 3.0.0, drop obsolete libr-zip.patch#367
starsareintherose merged 1 commit into
BioArchLinux:masterfrom
bshor:r-zip-drop-patch

Conversation

@bshor

@bshor bshor commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Fixes #366. cc @pekkarr (maintainer)

Updates r-zip to 3.0.0 and removes libr-zip.patch, which no longer applies to the rewritten 3.0.0 source (now bundles a Mbed TLS subset for WinZip AES crypto + new source files). That stale patch was the cause of the prepare() failure that kept the package pinned at 2.3.3.

Why dropping the patch is safe

  • The miniz symbol rename is obsolete. The mz_r_mz_ rename worked around symbol collisions (upstream issue Refresh depends for all r pkgs #98). zip 3.0.0's Makevars builds with PKG_CFLAGS = $(C_VISIBILITY) (-fvisibility=hidden), which hides the bundled-miniz symbols at link time — the modern fix for that collision.
  • Nothing consumes the exported libr-zip.so. r-zip is the only package in the repo that referenced libr-zip; no other PKGBUILD links /usr/lib/libr-zip.so. The package now ships the standard usr/lib/R/library/zip/libs/zip.so like every other R package.

Other changes

  • r-cli stays in makedepends (CRAN LinkingTo: cli, added in r-zip: add r-cli makedepend for CRAN 3.0.0 #362) and is also added to optdepends (it's in 3.0.0 Suggests).
  • optdepends refreshed to 3.0.0 Suggests: added r-callr, r-curl, r-webfakes; dropped r-covr (no longer suggested).
  • checkdepends / repo_makedepends: added r-curl and r-webfakes — the new test-http.R range-request tests need them (the suite's skip_if_not_installed only skips under _R_CHECK_FORCE_SUGGESTS_=false, so they run and require the deps present).

Verification

Built locally with makepkg:

  • Compiles cleanly (mbedtls subset + cli headers), produces zip.so.
  • Test suite: FAIL 0 | PASS 520 | SKIP 5 (skips are Windows-only / long-running).
  • Packaged contents: usr/lib/R/library/zip/libs/zip.so + usr/share/licenses/r-zip/LICENSE, no libr-zip.so.

@starsareintherose starsareintherose merged commit 2973521 into BioArchLinux:master Jun 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

r-zip: 3.0.0 build fails, libr-zip.patch no longer applies to Mbed-TLS'd source

2 participants