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

image command: add cp, ls and tree subcommands #1147

Merged
merged 2 commits into from Aug 13, 2021
Merged

Conversation

francescolavra
Copy link
Member

The TFS filesystem driver now supports reading a filesystem from an image file. Three new subcommands for the ops image command have been added to exploit the new functionality. This makes ops implement most of the features of the dump tool.
Closes #911.

When the path string supplied to mkDirPath() has sequences of
consecutive '/' characters, or is empty (which can happen during
`ops pkg load` command execution), the filesystem code was creating
tuple attributes with emtpy symbols, which are useless.
This commit fixes it.
fs/tfs.go Outdated Show resolved Hide resolved
fs/tfs.go Outdated Show resolved Hide resolved
fs/tfs.go Outdated Show resolved Hide resolved
fs/tfs.go Outdated Show resolved Hide resolved
fs/tfs.go Outdated Show resolved Hide resolved
The TFS filesystem driver now supports reading a filesystem from an
image file. Three new subcommands for the `ops image` command have
been added to exploit this new functionality. This makes ops
implement most of the features of the `dump` tool.
Closes #911.
@francescolavra
Copy link
Member Author

Any errors resulting from parsing numeric values in the TFS file metadata are now caught and reported.

@francescolavra francescolavra merged commit 2810f03 into master Aug 13, 2021
@francescolavra francescolavra deleted the feature/dump branch August 13, 2021 09:37
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.

consider adding native ops dump support as well
2 participants