Skip to content

CI: Remove code made redundant by the dependency caching rewrite - #774

Closed
notroj wants to merge 2 commits into
apache:trunkfrom
notroj:ci-script-cleanup
Closed

notroj wants to merge 2 commits into
apache:trunkfrom
notroj:ci-script-cleanup

Conversation

@notroj

@notroj notroj commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

Removes code from the Linux CI scripts which the gha-resolve-deps.sh cache keys have made dead:

  • The perl -V check of ~/perl5 in travis_before_linux.sh: the cache key includes $PERLID with no restore-keys, so restored modules always match the running perl.
  • resolve_apx()/fetch_apx(): install_apx() fetches the tag, or the commit gha-resolve-deps.sh already resolved, directly. It no longer fetches the APR source for an APR-util build, since $ROOTID covers both and the cache is saved only once both have built.
  • The nghttp2 marker file, as the version is in $ROOTID.
  • TEST_MOD_TLS/rustls support: modules/tls is gone.
  • The unset LD_LIBRARY_PATH/LD_RUN_PATH, left over from before OpenSSL 3 builds used an RPATH via LDFLAGS.
  • The unused TRAVIS_* variables.

TEST_APR_TARBALL support is kept, as 2.4.x uses it. The first run after this rebuilds the CPAN module cache once, since its key hashes travis_before_linux.sh.

🤖 Generated with Claude Code

notroj and others added 2 commits September 28, 2026 16:38
* test/travis_before_linux.sh: Drop the Travis variables and the
  "perl -V" check of ~/perl5.  Remove TEST_MOD_TLS support.
  (resolve_apx, fetch_apx): Removed.
  (install_apx): Fetch the tag, or the commit resolved by
  gha-resolve-deps.sh, directly.  Don't fetch the APR source for an
  APR-util build.
  Test for the nghttp2 install rather than a marker file.

* test/travis_run_linux.sh: Remove TEST_MOD_TLS support and the unset
  of LD_LIBRARY_PATH and LD_RUN_PATH.

* test/gha-resolve-deps.sh: Drop RUSTLS_VERSION from $ROOTID.

* test/README.ci: Update to match.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
  test/gha-*.sh. [skip ci]

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant