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 the ability to specify the seed data path #230

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

RyuzakiKK
Copy link
Contributor

When using the --seed foo.caibx option, Desync expects to find the data in foo, next to the provided .caibx file.

While this is a valid assumption for files, it requires additional steps when the data is a block device. For example a common suggestion in those cases is to create a symlink next to the seed index.

Instead, now --seed gains the ability to explicitly set a path for the data blob. The syntax becomes:
--seed /path/to/index.caibx:/path/to/data.

If the colon is missing, the old behavior of expecting the data next to the index file will still be used by default.

Fixes: #228

cmd/desync/extract.go Outdated Show resolved Hide resolved
cmd/desync/extract.go Outdated Show resolved Hide resolved
cmd/desync/extract.go Outdated Show resolved Hide resolved
When using the `--seed foo.caibx` option, Desync expects to find the
data in foo, next to the provided .caibx file.

While this is a valid assumption for files, it requires additional steps
when the data is a block device. For example a common suggestion in
those cases is to create a symlink next to the seed index.

Instead, now `--seed` gains the ability to explicitly set a path for the
data blob. The syntax becomes:
`--seed /path/to/index.caibx:/path/to/data`.

If the colon is missing, the old behavior of expecting the data next to
the index file will still be used by default.

Fixes: folbricht#228

Signed-off-by: Ludovico de Nittis <[email protected]>
Copy link
Owner

@folbricht folbricht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@folbricht folbricht merged commit 7562cdd into folbricht:master Dec 7, 2022
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.

Add a --seed-and-data option or something similar
2 participants