-
Notifications
You must be signed in to change notification settings - Fork 36
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 cph list
subcommand
#236
Conversation
Co-authored-by: Ken Odegard <[email protected]>
Co-authored-by: Ken Odegard <[email protected]>
for component in _stream_components(filename, components): | ||
for tar, _ in component: | ||
with redirect_stdout(memfile): | ||
tar.list(verbose=verbose) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, such re-use.
key=lambda line: line.split(None, 5)[-1], | ||
) | ||
else: | ||
lines = sorted(memfile.readlines()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skip the sort?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was trying to achieve output order consistency. In .conda
artifacts, the info/
block will be always put in the end. I've also seen some pkg
components containing stuff under info/
.
Great work, thanks |
Any further comments that need to be addressed? Thanks @dholth |
Description
Reuses
tarfile.Tarfile.list()
across .tar.bz2 and .conda artifacts. The unified interface with the tar generator makes it super easy!This should help with inspecting artifact contents without having to pay for extraction. I picture this in post-conda-build reports in CI.
Checklist - did you ...
news
directory (using the template) for the next release's release notes?