Skip to content

build: if no perl, fix to use the pre-built hugehelp, if present#18118

Closed
vszakats wants to merge 17 commits into
curl:masterfrom
vszakats:cm-opt
Closed

build: if no perl, fix to use the pre-built hugehelp, if present#18118
vszakats wants to merge 17 commits into
curl:masterfrom
vszakats:cm-opt

Conversation

@vszakats

@vszakats vszakats commented Jul 31, 2025

Copy link
Copy Markdown
Member

Also:

  • GHA/distcheck: verify if the pre-built hugehelp makes it into curl.
  • GHA/distcheck: streamline disabling perl with autotools.
  • GHA/distcheck: --disable-docs no longer required to build without perl.
    Passing it also implicitly disables the curl manual, which is
    undesired in these tests.
  • cmake: drop redundant HAVE_MANUAL_TOOLS interim variable.
  • cmake: move two set() lines close to their counterparts.

Follow-up to 25119fb #18104


w/o ws https://github.com/curl/curl/pull/18118/files?w=1

  • autotools: also try fixing.

Tested OK in CI:
https://github.com/curl/curl/actions/runs/16671747516?pr=18118
also locally using 8 combinations (with autotools).

@vszakats
vszakats marked this pull request as draft July 31, 2025 18:45
@github-actions github-actions Bot added the CI Continuous Integration label Aug 1, 2025
@vszakats
vszakats marked this pull request as ready for review August 1, 2025 01:23
@vszakats vszakats changed the title cmake: if no perl, fix to use the pre-built hugehelp, if present build: if no perl, fix to use the pre-built hugehelp, if present Aug 1, 2025
@vszakats vszakats closed this in 54da6f5 Aug 1, 2025
@vszakats
vszakats deleted the cm-opt branch August 1, 2025 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant