Skip to content

tests: drop BUNDLE_SRC variable#17692

Closed
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:t-am-bundle-src
Closed

tests: drop BUNDLE_SRC variable#17692
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:t-am-bundle-src

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Jun 21, 2025

Derive it from $BUNDLE instead. autotools seems to be already relying
on $BUNDLE_SRC being equal to $BUNDLE.c. (I haven't realized this
before aaebb45.)

Also drop redundant nodist_<target>_SOURCE lines in tunits and units.

Follow-up to aaebb45 #17688
Follow-up to 2c27a67 #17590

Derive it from `$BUNDLE` instead. autotools seems to be already relying
on `$BUNDLE_SRC` being `$BUNDLE.c`. (I haven't realized this before
aaebb45.)

Also drop redundant `nodist_<target>_SOURCE` lines in units and tunits.

Follow-up to aaebb45 curl#17688
@vszakats vszakats marked this pull request as draft June 21, 2025 00:27
@vszakats vszakats marked this pull request as ready for review June 21, 2025 01:00
@vszakats vszakats closed this in 7d8fa82 Jun 21, 2025
@vszakats vszakats deleted the t-am-bundle-src branch June 21, 2025 08:49
denandz pushed a commit to denandz/curl that referenced this pull request Jun 21, 2025
Derive it from `$BUNDLE` instead. autotools seems to be already relying
on `$BUNDLE_SRC` being equal to `$BUNDLE.c`. (I haven't realized this
before aaebb45.)

Also drop redundant `nodist_<target>_SOURCE` lines in tunits and units.

Follow-up to aaebb45 curl#17688
Follow-up to 2c27a67 curl#17590

Closes curl#17692
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