Re: "stg reset --status" doesn't just reset status

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2008-04-01 22:21:24 -0400, Pavel Roskin wrote:

> This time I decided to see what "stg reset --status" actually does,
> and I was unpleasantly surprised that it would do much more that its
> name implies.
>
> It doesn't just reset the status (no idea what it would be, but it
> doesn't sound scary). It removes all local changes. It's essentially
> "git reset --hard". I can easily imagine that some beginner would
> lose valuable changes by following that advice while trying to
> update from the upstream repository.
>
> I would hate to suggest another stg command, as there are too many
> of them already. On the other hand, if "applied" and "unapplied" are
> downgraded to switches for "stg series", we probably could justify
> adding one more command, "stg reset". By the way, the default could
> be to save the changes to a hidden "stash" patch, and the "--hard"
> switch would do a real reset.

(I assume "stg reset --status" is just a typo for "stg status
--reset"?)

I'd be fine with removing status --reset, but since there is currently
no other way to do this in StGit, I expect Catalin would object. (As I
recall, that's precisely what happened when I did try to remove it
some time ago.)

I'm currently (slowly) working on an "stg reset" command that'll be
able to reset the stack to any prior state. It could be made to reset
to the most recent recorded state if no extra argument is given, which
I think would make it do what you want.

-- 
Karl Hasselström, kha@xxxxxxxxxxx
      www.treskal.com/kalle
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux