On Sep 10, 2007, at 5:25 PM, mabshoff wrote:

> On Sep 10, 4:17 pm, "Joel B. Mohler" <[EMAIL PROTECTED]> wrote:
>> On Monday 10 September 2007 18:21, Pablo De Napoli wrote:
>>
>>> I could not read your modification to my patch since you've uploaded
>>> that as an hg bundle. I think that for trac the best would be to
>>> upload patches in plain text. That way, trac knows how to format  
>>> it so
>>> that it can be easy read on the web
>>> (it would be nice to have something like trac but based on  
>>> mercurial,
>>> rather than in subversion, but I doubt that such a program exists)
>>
>> Yes, I did put it up as an hg bundle.  I agree these are annoying,  
>> but they do
>> one thing nicely -- in that bundle are two patches which cleanly  
>> separates
>> the patch you wrote and the modifications I made to it.  That  
>> separation is
>> not such a big deal here, but in general I think it is nice which  
>> is why I
>> did it this way.  Of course, you can apply this to a vanilla  
>> 2.8.4.1 with 'hg
>> apply' (or whatever the hg_sage.<method> way is).  For your own  
>> plain text
>> pleasure, my changes are attached.
>>
>> I'd be interested to hear what other people have to say about  
>> proper trac
>> etiquette with bundles and patches building on one another.
>>
>
> I would say that bundles are better than a number of patches that have
> to be applied manually one after the other. It is more painful to
> learn how to use bundles (I can tell because I still am not 100% on
> the finer points of merucrial like branching and so on), but in the
> end the workflow ends up much better. There was a video tutorial by
> William for SD4 I believe and I think it is only somewhere at
> sagemath.org, so use that as a starting point.

Patches are much easier to read, but bundles are so much easier to  
apply (especially if there are several of them, or there are non- 
trivial dependancies). Ideally, one should be able to "open up" a  
bundle and view it as a sequence of diffs online--does the mercurial  
plugin for trac have such an ability?  If not, it might be worth  
adding (as both trac and mercurial are both python and open source).

- Robert

--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to sage-devel@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/sage-devel
URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/
-~----------~----~----~----~------~----~------~--~---

Reply via email to