[darcs-users] [patch407] hook env vars: add some haddocks (and3 more)

Sittampalam, Ganesh ganesh.sittampalam at credit-suisse.com
Tue Nov 2 14:04:34 UTC 2010


I actually wrote a follow-up (rolling back the bit we don't want) but
forgot to push it. I'll do that this evening.

Guillaume Hoffmann wrote:
> What is the status of these patches?
> Do they still need follow-up patches?
> Are they ok to be pushed to HEAD?
> 
> Florent has this patch that depend on Simon's:
> 
> Sun Oct 17 10:52:26 CEST 2010  Florent Becker
>   <florent.becker at ens-lyon.org> * make progress message in pull less
> silly looking 
> 
> http://bugs.darcs.net/patch416
> 
>> ] hunk ./src/Darcs/Arguments.lhs 1584
>> -  let k = "Defining environment variables"
>> +  let k = "Defining set of chosen patches"
> 
> and it would be nice to have it in HEAD too.
> 
> guillaume
> 
> 
> 2010/10/17 Simon Michael <smichael at fastmail.fm>:
>> On Oct 17, 2010, at 1:40 AM, Ganesh Sittampalam wrote:
>>> 
>>> On Sun, 17 Oct 2010, Eric Kow wrote:
>>>> 
>>>> Do I understand correctly that this is about people wanting to be
>>>> able to send patches mainly for comments (and perhaps being
>>>> surprised if they get applied?)
>>>> 
>>>> Sounds like the usual case is people sending patches
>>>> hoping/expected they get applied, and that the for-comments case
>>>> can be handled with an explicit DO NOT APPLY
>>> 
>>> I think the problem here is that Simon sent a patch that he
>>> considered ready for applying, but when I reviewed it I asked for
>>> some changes, and now he'd prefer to have a clean history. In a
>>> sense 
>>> we explicitly decided against clean histories when we chose to use
>>> screened, but that choice doesn't necessarily have to apply to
>>> every submitter. 
>> 
>> Simple-contributor-feedback: Yes, I intended these patches to be
>> applied, but sometimes I'll get important insights from the review
>> discussion which might make me want to redo or retract a patch. I
>> haven't much experience with the new process but hopefully it won't
>> work against clean history. I like being able to rely on the group's
>> wisdom to keep my stuff out of the repo until it's worthy. But, not a
>> big deal in this case and maybe it won't be in future.
>> 
>> Thanks - Simon
>> _______________________________________________
>> darcs-users mailing list
>> darcs-users at darcs.net
>> http://lists.osuosl.org/mailman/listinfo/darcs-users
>> 
> _______________________________________________
> darcs-users mailing list
> darcs-users at darcs.net
> http://lists.osuosl.org/mailman/listinfo/darcs-users


=============================================================================== 
Please access the attached hyperlink for an important electronic communications disclaimer: 
http://www.credit-suisse.com/legal/en/disclaimer_email_ib.html 
=============================================================================== 



More information about the darcs-users mailing list