Skip to content

Use cmd.Stdin for input to plz query whatinputs and skip plz-out#336

Closed
goddenrich wants to merge 2 commits intoplease-build:masterfrom
goddenrich:whatinputs-ignore-unknown
Closed

Use cmd.Stdin for input to plz query whatinputs and skip plz-out#336
goddenrich wants to merge 2 commits intoplease-build:masterfrom
goddenrich:whatinputs-ignore-unknown

Conversation

@goddenrich
Copy link
Contributor

Previously we could error because the number of files in a repo were too large. We would get errors like

16:42:24.457 CRITICAL: Failed to load packages: fork/exec /usr/local/bin/plz: argument list too long

We also walk the plz-out directory.
Skip walking the plz-out directory and use cmd.Stdin to pass in the list of files for whatinputs.

@goddenrich goddenrich force-pushed the whatinputs-ignore-unknown branch from 7d10fbd to 5af30a2 Compare February 3, 2026 13:27
@goddenrich
Copy link
Contributor Author

Ive messed this PR up... will split this up and re-raise in separate revisions

@goddenrich goddenrich closed this Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants