Skip to content

xbps-src: document -Q non-recursiveness#55222

Merged
ahesford merged 1 commit intovoid-linux:masterfrom
tornaria:usage
May 6, 2025
Merged

xbps-src: document -Q non-recursiveness#55222
ahesford merged 1 commit intovoid-linux:masterfrom
tornaria:usage

Conversation

@tornaria
Copy link
Contributor

@tornaria tornaria commented May 1, 2025

Document in usage that running check with -Q is not recursive,
and running check with -K is recursive.

This behaviour was implemented in #54633.

Testing the changes

  • I tested the changes in this PR: YES|briefly|NO

@classabbyamp classabbyamp added the xbps-src xbps-src related label May 1, 2025
@classabbyamp
Copy link
Member

recursive might be a bit obtuse, what about specifying it as "and run tests on built dependencies" or something?

@tornaria
Copy link
Contributor Author

tornaria commented May 1, 2025

recursive might be a bit obtuse, what about specifying it as "and run tests on built dependencies" or something?

I'd call that "concise" but point taken. What about:


-Q  Enable running the check stage (for the target package only).

-K  Enable running the check stage with longer tests (and also on built deps).

I'm trying to stick on one 80 char line, spelling "dependencies" brings the line to 86 char.
With two lines:


-Q  Enable running the check stage, for the target package only.

-K  Enable running the check stage with longer tests.
    Unlike -Q, this will also run the check stage on built dependencies.

@ahesford
Copy link
Member

ahesford commented May 5, 2025

I favor the second option. Better to be verbose and break a line.

Document in usage that running check with -Q is not recursive,
and running check with -K is recursive.

This behaviour was implemented in void-linux#54633.
@tornaria
Copy link
Contributor Author

tornaria commented May 6, 2025

A few days later, I now agree verbose is better, and using two lines looks better.

@ahesford ahesford merged commit dab8a31 into void-linux:master May 6, 2025
@tornaria tornaria deleted the usage branch May 6, 2025 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

xbps-src xbps-src related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants