On Aug 4, 2015, at 1:18 PM, Warren Young <[email protected]> wrote:
> 
> On Aug 4, 2015, at 8:51 AM, Ron W <[email protected]> wrote:
> 
>> Also, if "pop --partial" were also implemented
> 
> I think “pop --partial” only makes sense if it actually modifies the stash to 
> contain only the un-popped partial changes, which seems a bit…excessive for 
> v1 of this feature.

I’ve rethought this.  “fossil stash pop --partial” would be easier to implement 
than apply --partial.

It would be insufficient for apply --partial to just learn to recognize 
already-applied diff chunks as patch(1) does, because the chunks might have 
been modified before being checked in.  Thus, a robust implementation of apply 
--partial would need to have a list of chunks you previously said “yes” to, so 
it can skip them the next time.

Because pop --partial could just rewrite the stash after each run, it wouldn’t 
need that secondary storage area, making it considerably easier to write.
_______________________________________________
fossil-users mailing list
[email protected]
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to