better way to handle patches

[Tzafrir Cohen]

On Mon, May 15, 2006 at 04:22:41PM -0300, Caio Begotti wrote:
> On Monday 15 May 2006 15:38, Tzafrir Cohen wrote:
> > (that part is quite rewritten in trunk, isn't it?)
> 
> Sorry, trunk from Digium or Pkg-Voip? I didn't get it.

say.c in the Digium trunk.

> 
> > Is "br" the language name to use? It seems that "pt_br" is preffered.
> > You also seem to use "pt-de" in the patch to sayunixtime.c .
> 
> Although the most common for localization stuff would be pt_BR, "br" is the 
> preferred term for Brazilian Portuguese in Asterisk, AFAIK. The "pt-de" 
> things is actually one of some new audio messages required for a proper 
> pronounce in our portuguese variant. That specific audio means "of" or 
> something like that (in english it's not used to say dates or time).

So that patch uses rather non-standard pathes. 

> 
> > As this part is being rewritten, I'm not sure those patches would be
> > welcomed in the main asterisk Mantis. Anyway, unless they are to
> > conflict with other patches, and if they solve real-world problems, I'm
> > for them.
> 
> What you mean by "rewritten"? VoiceMail(), sayunixtime.so and the 'say' core?

say.c

> 
> Celso (the original author) said yesterday to me that he's going to attach it 
> in some Mantis report and wait the commit. But you know, it may takes a lot 
> of time, even it's being really important for portuguese speakers.

(OT: grab trunk and test it there. It's rather different.)

-- 
Tzafrir Cohen      sip:tzafrir at local.xorcom.com
icq#16849755       iax:tzafrir at local.xorcom.com
+972-50-7952406           
tzafrir.cohen at xorcom.com  http://www.xorcom.com