Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "--build-order" to cat (and a couple minor bugfixes) #79

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

tianon
Copy link
Member

@tianon tianon commented Jul 24, 2023

  • fix parsing of empty BASHBREW_ARCH_NAMESPACES
  • better (more contextual) error messages from bashbrew remote arches

- fix parsing of empty `BASHBREW_ARCH_NAMESPACES`
- better (more contextual) error messages from `bashbrew remote arches`
@tianon
Copy link
Member Author

tianon commented Jul 24, 2023

(This is something we've emulated in a few places with bashbrew list --build-order ... | xargs bashbrew cat but that provides a result that's different in kind of subtle ways and I don't see a good reason not to support this here 😅)

@tianon
Copy link
Member Author

tianon commented Jul 24, 2023

To be explicitly clear: this only sorts the top-level arguments to cat in build order - any deeper entry-level sorting would be accomplished in the format string.

@yosifkit yosifkit merged commit 8b801fd into docker-library:master Jul 24, 2023
4 checks passed
@yosifkit yosifkit deleted the cat-build-order branch July 24, 2023 23:54
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