This is the mail archive of the
cygwin-developers
mailing list for the Cygwin project.
Re: Seeking developer to assist with adding OpenAFS Reparse Tag Support to Cygwin
On Mar 10 15:19, Christopher Faylor wrote:
> On Sun, Mar 10, 2013 at 09:09:42AM -0400, Jeffrey Altman wrote:
> >On 3/10/2013 1:50 AM, Christopher Faylor wrote:
> >>
> >>> Do not hesitate to ask if you have any questions.
> >>
> >> Could you reformat that without the MS-DOS line endings.
> >
> >Here is the same file after running 'dos2unix' on it.
>
> Sorry to nickel and dime you, but convention is to use unified diff
> format when submitting patches. I should have mentioned that before.
>
> >From what I can see, your changes don't follow the indentation
> convention of the code around it, i.e., the changes in path.h
> seem to use four spaces for indentation.
A ChangeLog entry would be helpful, too.
This is all nicely described on http://cygwin.com/contrib.html
under the "When you have finalized your changes" section.
Thanks,
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Maintainer cygwin AT cygwin DOT com
Red Hat