alex/bcachefs-tools
1
0
Fork 0
mirror of https://github.com/koverstreet/bcachefs-tools.git synced 2025-01-23 00:07:07 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

cmd_mount: Fix test-only mount path

Browse Source 
The comman line help claims that `bcachefs mount <DEV>` without a
mount point will do a dry-run mount - all the steps required to
mount the fs, but without actually doing the final real mount.

Make the code actually do this, rather than complain that you
haven't supplied a mountpoint if you don't provide a mountpoint

Signed-off-by: Christopher James Halse Rogers <raof@ubuntu.com>
This commit is contained in:
Christopher James Halse Rogers 2023-09-01 16:51:32 +10:00 committed by Kent Overstreet
parent e0e3032926
commit 28e6dea653
1 changed files with 17 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
rust-src/src/cmd_mount.rs
View File

@ -145,7 +145,7 @@ struct Cli {
/// Where the filesystem should be mounted. If not set, then the filesystem
/// won't actually be mounted. But all steps preceeding mounting the
/// filesystem (e.g. asking for passphrase) will still be performed.
mountpoint: std::path::PathBuf,
mountpoint: Option<std::path::PathBuf>,
/// Mount options
#[arg(short, default_value = "")]
@ -207,14 +207,23 @@ fn cmd_mount_inner(opt: Cli) -> anyhow::Result<()> {
key::prepare_key(&sbs[0], key)?;
}
info!(
"mounting with params: device: {}, target: {}, options: {}",
devs,
&opt.mountpoint.to_string_lossy(),
&opt.options
);
if let Some(mountpoint) = opt.mountpoint {
info!(
"mounting with params: device: {}, target: {}, options: {}",
devs,
mountpoint.to_string_lossy(),
&opt.options
);
mount(devs, mountpoint, &opt.options)?;
} else {
info!(
"would mount with params: device: {}, options: {}",
devs,
&opt.options
);
}
mount(devs, &opt.mountpoint, &opt.options)?;
Ok(())
}