Skip to content

Making a patch

6 messages · Andrew Robinson, Duncan Murdoch, Gregor Gorjanc +1 more

#
Dear R developers,

is there a preferred format or strategy for making a patch to
contribute to a package that is maintained by R-core?  Berwin Turlach
and I have written a very minor extension to lmeControl to allow it to
pass an argument to nlminb for the maximum number of evaluations of
the objective function.  I've edited the nlme/R/lme.R and
nlme/man/lmeControl.Rd files.  I can diff the files, but the options
seem numerous.  I haven't been able to find any hinst on the preferred
approach.  Any advice would be welcomed.

Cheers

Andrew
#
Hi,

Andrew Robinson <A.Robinson <at> ms.unimelb.edu.au> writes:
look at 

http://article.gmane.org/gmane.comp.lang.r.devel/8858

perhaps this (with modifications if necesarry) could be added to developer page 

http://developer.r-project.org/

Afterall, there is a good guide on posting mails to r-mailing lists and it would
be good to have the same for pacthes.

Gregor
#
Thanks Gregor,

I will go ahead with that unless anyone has any other advice or preferences.

Cheers

Andrew
On Sun, Jul 23, 2006 at 11:37:42PM +0000, Gregor Gorjanc wrote:

  
    
#
On 7/23/2006 6:04 PM, Andrew Robinson wrote:
Gregor pointed out a description of the suggested format.  But the 
strategy is also important:  in order for your patch to be incorporated, 
you need to convince someone to accept it.  To do that, you should post 
the following sort of information along with your patch:

  - what the patch does
  - what problem it solves
  - what problems it may cause

You should also make sure your patch is complete, e.g. if it changes 
lmeControl, then you should also include a patch to the documentation 
for lmeControl.

Duncan Murdoch
1 day later
#
Gregor Gorjanc <gregor.gorjanc <at> bfro.uni-lj.si> writes:
...
Is there also any strategy in accepting patches in plain text or "somehow
compressed" format? The later due to line wrapping in some mailers, etc.

Thanks!

Gregor
#
I've tried to summarize the information in this thread into a Wiki page: 
http://wiki.r-project.org/rwiki/doku.php?id=misc:rpatch

Please make an additions or alterations as appropriate!

-- Tony Plate
Duncan Murdoch wrote: