Re: [patch] output format clash

2011-03-30 Thread Jean-Marc Lasgouttes
Le 30/03/2011 02:42, Julien Rioux a écrit : I've put the simple patch in at r38146. We'll avoid trouble in this cycle and we can work on a more general handling of converters in the next cycle. Very good. The rest of your patches look good too, BTW :) JMarc

Re: [patch] output format clash

2011-03-29 Thread Julien Rioux
On 24/03/2011 5:36 PM, Jean-Marc Lasgouttes wrote: Le 24/03/11 16:54, Julien Rioux a écrit : Can I ago ahead with this or do we attempt perhaps a more general handling of literate documents? I think this is post-2.0 work. In general our converters handling has to be cleaned up to allow removal

Re: [patch] output format clash

2011-03-24 Thread Jean-Marc Lasgouttes
Le 24/03/11 16:54, Julien Rioux a écrit : Can I ago ahead with this or do we attempt perhaps a more general handling of literate documents? I think this is post-2.0 work. In general our converters handling has to be cleaned up to allow removal of many special cases. JMarc

Re: [patch] output format clash

2011-03-24 Thread Julien Rioux
On 23/03/2011 11:23 PM, Julien Rioux wrote: On 23/03/2011 10:19 PM, Richard Heck wrote: On 03/22/2011 05:16 PM, Julien Rioux wrote: You cannot have two modules which both redefine the output format. This is a stupid patch. Something more clever might be done in the c++ code. What happens if t

Re: [patch] output format clash

2011-03-24 Thread Abdelrazak Younes
On 03/24/2011 04:25 AM, Julien Rioux wrote: On 23/03/2011 10:18 PM, Richard Heck wrote: On 03/23/2011 06:27 PM, José Matos wrote: On Wednesday 23 March 2011 21:36:54 Pavel Sanda wrote: others, please note that we have new fellow on the developer's ship so do not forget to give him warm welcome

Re: [patch] output format clash

2011-03-23 Thread Julien Rioux
On 23/03/2011 10:18 PM, Richard Heck wrote: On 03/23/2011 06:27 PM, José Matos wrote: On Wednesday 23 March 2011 21:36:54 Pavel Sanda wrote: others, please note that we have new fellow on the developer's ship so do not forget to give him warm welcome ;) pavel Oh, do you mean that a welcome gr

Re: [patch] output format clash

2011-03-23 Thread Julien Rioux
On 23/03/2011 10:19 PM, Richard Heck wrote: On 03/22/2011 05:16 PM, Julien Rioux wrote: You cannot have two modules which both redefine the output format. This is a stupid patch. Something more clever might be done in the c++ code. What happens if two modules do try to redefine the output form

Re: [patch] output format clash

2011-03-23 Thread Richard Heck
On 03/22/2011 05:16 PM, Julien Rioux wrote: You cannot have two modules which both redefine the output format. This is a stupid patch. Something more clever might be done in the c++ code. What happens if two modules do try to redefine the output format? Should we issue a warning or something,

Re: [patch] output format clash

2011-03-23 Thread Richard Heck
On 03/23/2011 06:27 PM, José Matos wrote: On Wednesday 23 March 2011 21:36:54 Pavel Sanda wrote: others, please note that we have new fellow on the developer's ship so do not forget to give him warm welcome ;) pavel Oh, do you mean that a welcome group hug is in order? ;-) Welcome. :-) And

Re: [patch] output format clash

2011-03-23 Thread José Matos
On Wednesday 23 March 2011 21:36:54 Pavel Sanda wrote: > others, please note that we have new fellow on the developer's ship > so do not forget to give him warm welcome ;) > > pavel Oh, do you mean that a welcome group hug is in order? ;-) Welcome. :-) -- José Abílio

Re: [patch] output format clash

2011-03-23 Thread Pavel Sanda
Julien Rioux wrote: > You cannot have two modules which both redefine the output format. This is > a stupid patch. Something more clever might be done in the c++ code. ok, if there is no better patch yet. others, please note that we have new fellow on the developer's ship so do not forget to giv

[patch] output format clash

2011-03-22 Thread Julien Rioux
You cannot have two modules which both redefine the output format. This is a stupid patch. Something more clever might be done in the c++ code. -- Julien Index: lib/layouts/sweave.module === --- lib/layouts/sweave.module (revision