[PATCH]: Add flock syscall emulation
Sun Nov 30 17:57:00 GMT 2003
> On Thu, Nov 27, 2003 at 02:51:10PM -0500, Nicholas Wourms wrote:
>>Here is a patch to add the flock() syscall to Cygwin. I've noticed that some
> Applied with changes.
> I've run indent on flock.c since its formatting was non-GNU.
I can understand why you did it in this case (the tabs were out of
control), but can we make an exception for bsd/isc-derived code? I
think that enforcing this rule strictly on written-from-scratch source
is ok, but doing it on derived source reduces the overall transparency
of changes against the upstream version. Indent in GNU mode will
practically rewrite every line in just about any BSD source file out
there, since their formatting style is clearly different. I'd argue,
however, that the BSD style is just as clear and readable as the GNU
style. For instance, because DJ didn't run localtime.cc through indent,
resolving the changes in the latest tzdata sources is like 50x easier.
> I've removed the _DEFUN, since it's nowhere else used in Cygwin. I've
> added a change message to include/sys/file.h according to the
> licensing terms in include/sys/copying.dj.
The _DEFUN was leftover from when I had been debating about whether to
add it to newlib or cygwin, but clearly cygwin was the proper choice.
Thanks for catching that DJ licensing thing, though.
> I removed the _flock from cygwin.din. This should in future really only be
> used if newlib expects a new syscall.
As I said, it was mostly an afterthought.
>>As a side note, I noticed that sys/file.h was from DJGPP. Will DJ allow us to
>>use code from DJGPP in Newlib/Cygwin? If so, I noticed some code in the DJGPP
>>libc which I could port and use in future contributions.
> This is a question you should ask DJ. AFAIK, DJGPP is GPLd. This
> would mean that its code isn't suitable for use in Cygwin itself.
Ok, I'll ask him, though I find it curious that there is DJGPP libc code
in Cygwin despite this. I'll also see if he's interested in the changes
Thanks for cleaning it up.
More information about the Cygwin-patches