Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

configure: replace $# shell syntax #15584

Closed
wants to merge 1 commit into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Nov 14, 2024

With a more portable alternative.

Fixes (seen on macOS):

../configure: line 47131: 1: command not found

Ref: https://github.com/curl/curl/actions/runs/11846071276/job/33012894013#step:7:635

Follow-up to e244d50 #15550

@vszakats vszakats added the build label Nov 14, 2024
@vszakats vszakats changed the title configure: replace $# syntax configure: replace $# shell syntax Nov 14, 2024
@vszakats vszakats marked this pull request as draft November 14, 2024 22:34
@vszakats vszakats force-pushed the configure-ac-shell-fixup branch from fd9d1fa to c38a60f Compare November 14, 2024 22:35
@vszakats vszakats marked this pull request as ready for review November 14, 2024 22:37
@vszakats vszakats force-pushed the configure-ac-shell-fixup branch from c38a60f to 452cba2 Compare November 14, 2024 22:40
@vszakats vszakats closed this in cb7558a Nov 15, 2024
@vszakats vszakats deleted the configure-ac-shell-fixup branch November 15, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant