Skip to content

Add --reset flag to recipes:install to reset all installed recipes back to their initial state#795

Merged
nicolas-grekas merged 1 commit intomainfrom
force-update
Sep 21, 2021
Merged

Add --reset flag to recipes:install to reset all installed recipes back to their initial state#795
nicolas-grekas merged 1 commit intomainfrom
force-update

Conversation

@nicolas-grekas
Copy link
Copy Markdown
Member

@nicolas-grekas nicolas-grekas commented Sep 6, 2021

Fix #777

Replaces #702, see discussion there.

@nicolas-grekas nicolas-grekas changed the title Add --updated flag to recipes:install, default to reinstalling all recipes Add --reset flag to recipes:install to reset all installed recipes back to their initial state Sep 21, 2021
@nicolas-grekas nicolas-grekas merged commit 78a94b4 into main Sep 21, 2021
@nicolas-grekas nicolas-grekas deleted the force-update branch September 21, 2021 15:30
@fabpot
Copy link
Copy Markdown
Member

fabpot commented Sep 21, 2021

Too bad it has been merged without taking my comment into account. Adding one more random flag name on top of something that is already unclear is not going to help. Just my 2cts.

@nicolas-grekas
Copy link
Copy Markdown
Member Author

I thought replacing the previous --updated (which changed the default behavior) with --reset was what we discussed (to keep the default behavior, which makes sense when updating)...

@nicolas-grekas
Copy link
Copy Markdown
Member Author

(If you mean changing the name of --force for something better, I'd prefer waiting for more work to happen on a future recipes:update before annoying ppl with a deprecation).

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.

Reinstalling recipes works only for updated ones

2 participants