Skip to content

fix(backup): add action to binary#1210

Open
yajo wants to merge 1 commit intoTecnativa:masterfrom
moduon:backup-
Open

fix(backup): add action to binary#1210
yajo wants to merge 1 commit intoTecnativa:masterfrom
moduon:backup-

Conversation

@yajo
Copy link
Copy Markdown
Contributor

@yajo yajo commented Jan 30, 2026

At some point, recent duplicity started yelling this error when running backup:

CommandLineError: Invalid 'file:///mnt/backup/src' action and cannot be implied from the given arguments:

Some time ago, the backup action was implicit, but it must be explicit now.

@moduon MT-13412

Copy link
Copy Markdown
Contributor Author

@yajo yajo left a comment

Choose a reason for hiding this comment

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

Ah sorry, it was a mistake on my side. I was setting in $SRC a value like file:///..., and that seems to be an unsupported protocol. When removing it, the shortcut started working again.

I mean, it's not like this change is gonna hurt, anyway. At least, if you merge it, the error message for the next one will be easier to understand. And this message will be removed when using it:

No valid action found. Will imply 'backup' because a path source was given and target is a url location.

But you can close it if you prefer.

Thanks anyway.

@josep-tecnativa
Copy link
Copy Markdown
Contributor

The fix looks good to me @yajo , but could you review CI please?

@yajo
Copy link
Copy Markdown
Contributor Author

yajo commented Feb 10, 2026

CI failure looks unrelated.

@josep-tecnativa
Copy link
Copy Markdown
Contributor

@yajo , Could you rebase please?

At some point, recent duplicity started yelling this error when running `backup`:

    CommandLineError: Invalid 'file:///mnt/backup/src' action and cannot be implied from the given arguments:

Some time ago, the `backup` action was implicit, but it must be explicit now.

@moduon MT-13412
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.

2 participants