Skip to content

runtests: drop orphaned, no-op -k option#22100

Closed
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:keepoutfiles
Closed

runtests: drop orphaned, no-op -k option#22100
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:keepoutfiles

Conversation

@vszakats

@vszakats vszakats commented Jun 19, 2026

Copy link
Copy Markdown
Member

And the corresponding internal variable.

This option became the always-enabled default earlier, via #4035.

Reported-by: Zartaj Majeed
Ref: #22098
Follow-up to 6617db6 #4035

And the corresponding internal variable.

Reported-by: Zartaj Majeed
Ref: curl#22098
Follow-up to 6617db6 curl#4035
@vszakats vszakats closed this in 04305a3 Jun 19, 2026
@vszakats vszakats deleted the keepoutfiles branch June 19, 2026 12:47
@thesamesam

Copy link
Copy Markdown
Contributor

After this, -k seems to be interpreted as something else, rather than the script erroring out. I only noticed by chance because the tests completed very quickly.

gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Jun 24, 2026
Dropped `-k` for tests as it was removed upstream in
04305a3e40989d3731e97bd0ef41bbd55c680a3f, though it seemed to manifest
in a strange way with the test runner exiting 0 but not running (m?)any
tests. I've mentioned it on the relevant PR.

Bug: curl/curl#22100
Bug: https://bugs.gentoo.org/978006
Signed-off-by: Sam James <sam@gentoo.org>
vszakats added a commit that referenced this pull request Jun 25, 2026
Restore processing this option to avoid falling it through and
misinterpreted as something else, which in turn disables tests.

Exit with an error instead. We delete completely in December 2026.

Reported-by: Sam James
Bug: #22100 (comment)
Follow-up to 04305a3 #22100
Follow-up to 6617db6 #4035

Closes #22157
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.

2 participants