[Serdev] Re: [Serusers] serweb / Make a patch file

Jan Janak jan at iptel.org
Wed Jul 23 19:19:05 UTC 2003


We use diff -Naur <orig_src_tree> <modified_src_tree>.

It is good to always check that the patch really contains your changes,
because from time to time it may contain also some temporary files,
differencies in CVS control directories and so on.

It is always better to submit many small patches containing "unrelated"
changes than just a big one.

The patch can be then applied using patch -p0 < patchfile in the parent
directory of the source tree.
 
 Jan.

On 23-07 11:59, Marcelo Schmidt wrote:
> Agree.  In short you use preferably diff -u or diff -c and you apply the
> diffs with 'patch'.
> I don't have any samples right here but I am sure someone has some in
> the list.
> 
> -m
> 
> -----Original Message-----
> From: Greg Fausak [mailto:greg at august.net] 
> Sent: Wednesday, July 23, 2003 11:50 AM
> To: Marcelo Schmidt; 'Jan Janak'
> Cc: 'Jiri Kuthan'; 'serusers'
> Subject: RE: [Serusers] serweb / Make a patch file
> 
> 
> Hi guys,
> 
> I've noticed that lately the list here seems to be
> getting a little short tempered. The main guys work very
> hard on this software.  Often they get the same questions
> over and over again.  Let's not forget that the authors
> of this software have worked thousands of hours for no compensation to
> write this.  They don't owe us anything, and we should really appreciate
> the donation!
> 
> There is some effort that must be expended to identify a
> bug, fix it, then provide a patch to the authors so that they can easily
> administrate the checkin to the source tree. Simply pointing out a bug
> to one of the authors is nice, but if you are capable it would be polite
> to go the extra few steps.
> 
> I've been writing software for 25 years, and I am not clear how to make
> a patch file.  Can someone provide an example :-)
> 
> Just my two cents,
> 
> ---greg
> Greg Fausak
> 
> 
> > 
> > 
> > You wrote this couple of weeks before: I have no crystal ball and I am
> 
> > not going to reply to this thread anymore... With the same info I sent
> 
> > you all, someone else "Jiri" found the problem
> > at that specific reply you wrote above.  Which it tells me you don't
> > know it all and you read email better.
> > If this is the way you try to provide any help, please don't help me.
> > 
> > -m
> > 
> > -----Original Message-----
> > From: Jan Janak [mailto:jan at iptel.org]
> > Sent: Wednesday, July 23, 2003 10:50 AM
> > To: Marcelo Schmidt
> > Cc: Jiri Kuthan; serusers
> > Subject: Re: [Serusers] serweb
> > 
> > 
> > On 23-07 10:45, Marcelo Schmidt wrote:
> > > Well, right besides the method allow_call_time_pass_reference is
> > > deprecated in php4 but the latest version of serweb uses it anyway, 
> > > doh :)
> > 
> >   So what ? Fix it instead of complaining..
> > 
> > > And following the error log to what really matters:
> > > [Wed Jul 23 10:15:34 2003] [error] PHP Fatal error:  Call
> > to undefined
> > > function:  mysql_pconnect() in
> > > /usr/pkg/share/httpd/htdocs/iptel/phplib/db_mysql.inc on line 73
> > 
> >   install php mysql support.
> > 
> >     Jan.
> > 
> > _______________________________________________
> > Serusers mailing list
> > serusers at lists.iptel.org http://lists.iptel.org/mailman/listinfo/serusers
> > 
> 




More information about the Serdev mailing list