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 revert command to revert a package to a rev #823

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lethliel
Copy link
Member

@lethliel lethliel commented Jul 7, 2020

This adds a revert command to restore a specific revision
of a package.

Basically this is just a osc copypac -r <rev> <proj> <pac> <proj>

implements #808

This adds a revert command to restore a specific revision
of a package.

Basically this is just a `osc copypac -r <rev> <proj> <pac> <proj>`
@marcus-h
Copy link
Member

marcus-h commented Jul 7, 2020 via email

@lethliel
Copy link
Member Author

lethliel commented Jul 7, 2020

@mcepl Perhaps you can clarify if this is the expected behavior.
At least it is what I understood from your script.

@mcepl
Copy link
Contributor

mcepl commented Jul 7, 2020

On 2020-07-07 01:23:56 -0700, Marco Strigl wrote:
If the command is executed in a package wc, should we also reset the local wc state (see [2])?

I would be completely OK if the command complained that the working directory is in “dirty” state (i.e., there are non-commited changes) and refused to continue. I believe some other subcommands (sr?) do exactly that.

@dmach dmach marked this pull request as draft August 12, 2022 08:31
@dmach
Copy link
Contributor

dmach commented Mar 9, 2023

This pull request has been in a draft state for quite some time.
Could you either finish it, close it or add a comment that you're still planning to work on it?

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.

4 participants