Skip to content

tests: remove debug requirement on 38 tests - #15114

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/tests-output-dir
Closed

tests: remove debug requirement on 38 tests#15114
bagder wants to merge 2 commits into
masterfrom
bagder/tests-output-dir

Conversation

@bagder

@bagder bagder commented Oct 1, 2024

Copy link
Copy Markdown
Member

For all tests using -O that were previously relying on a debug build and the CURL_TESTDIR environment variable, use the plain --output-dir option instead so that they can run proper in non-debug builds.

For all tests using -O that were previously relying on a debug build and
the CURL_TESTDIR environment variable, use the plain --output-dir option
instead so that they can run proper in non-debug builds.
@bagder bagder added the tests label Oct 1, 2024
@dfandrich

This comment was marked as outdated.

@bagder

bagder commented Oct 2, 2024

Copy link
Copy Markdown
Member Author

vcpkg flakiness

@bagder bagder closed this in 39697de Oct 2, 2024
@bagder
bagder deleted the bagder/tests-output-dir branch October 2, 2024 06:14
vszakats added a commit to vszakats/curl that referenced this pull request Jan 1, 2025
`--output-dir` option can be used as an alternative in all builds.

Follow-up to 39697de curl#15114
vszakats added a commit that referenced this pull request Jan 2, 2025
No longer used by curl tests.

`--output-dir` option can be used as an alternative in all builds.

Follow-up to 39697de #15114
Closes #15893
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
For all tests using -O that were previously relying on a debug build and
the CURL_TESTDIR environment variable, use the plain --output-dir option
instead so that they can run proper in non-debug builds.

Closes curl#15114
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
No longer used by curl tests.

`--output-dir` option can be used as an alternative in all builds.

Follow-up to 39697de curl#15114
Closes curl#15893
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.

2 participants