Skip to content

Some changes#359

Open
Johan-Liebert1 wants to merge 2 commits into
composefs:mainfrom
Johan-Liebert1:some-changes
Open

Some changes#359
Johan-Liebert1 wants to merge 2 commits into
composefs:mainfrom
Johan-Liebert1:some-changes

Conversation

@Johan-Liebert1

@Johan-Liebert1 Johan-Liebert1 commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

clap: Show usage and suggestions

Add features "error-context" and "suggestions" to show suggestions on
incorrect command usage

Before

./target/release/cfsctl --repo sysroot dump-files
error: one or more required arguments were not provided

After

./target/release/cfsctl --repo sysroot dump-files

error: the following required arguments were not provided:
  <IMAGE_NAME>

Usage: cfsctl dump-files <IMAGE_NAME> [FILES]...

For more information, try '--help'.

cfsctl: Make dump_files public

Useful in bootc to get backing path. Also update the signature to take
in a repository and an image name for better UX

Add features "error-context" and "suggestions" to show suggestions on
incorrect command usage

Before

```sh
./target/release/cfsctl --repo sysroot dump-files
error: one or more required arguments were not provided
```

After

```sh
./target/release/cfsctl --repo sysroot dump-files

error: the following required arguments were not provided:
  <IMAGE_NAME>

Usage: cfsctl dump-files <IMAGE_NAME> [FILES]...

For more information, try '--help'.
```

Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
Useful in bootc to get backing path. Also update the signature to take
in a repository and an image name for better UX

Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
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.

1 participant