This is the mail archive of the cygwin@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Serial programming patch


Thanks for your work. Unfortunately, we can't use it for three
reasons:

- The patch is too long to use it without you signing a copyright
  assignment form. Look on http://cygwin.com/contrib.html for details.

- A ChangeLog entry is missing. Please add a plain ChangeLog entry to
  your posting like the ones you can see in the Cygwin ChangeLog file.

- Your code is not conforming to the GNU coding standard.
  Look on http://www.gnu.org/prep/standards_toc.html for details
  (or just compare with the other code in Cygwin).

Then you're adding a member to fhandler_serial to check for the OS
that way:

	osVersion=GetVersion();
	[...]
	if(!(osVersion&0x80000000))

while Cygwin already has a way to check that without the need to
add a member:

	if (os_being_run == winNT)  // or win95, win98, winME

Btw, there's a special mailing list cygwin-patches@cygwin.com to send
and discuss patches to Cygwin. If you want to contribute to the project
it would have some advantages to subscribe to that list and to the list
cygwin-developers@cygwin.com as well.

I hope I didn't discourage you too much. Each contributor (and Chris and
I began as contributors, too) had to go through this.

Corinna

On Tue, May 22, 2001 at 02:27:52PM +0200, Jacek Trzcinski wrote:
> Hi !,
> Here is my patch to serial device. I implemented ioctl function which
> makes possible
> [...]

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.

--
Want to unsubscribe from this list?
Check out: http://cygwin.com/ml/#unsubscribe-simple


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]