Skip to content

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?

@Thaodan
Copy link
Contributor

Thaodan commented Oct 18, 2025

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.

If it would also have an option to force that would be good. Something like git's hard and mixed reset would be nice. I.e. for mixed reset to only reset but not commit.

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.

5 participants