Re: mq_timedsend & mq_timedreceive

[Dmitriy Korovkin <dkorovkin at rambler dot ru>]

Hello Jonathan,
Thank you very much for quick reply. Sorry for inconvenience and thank you 
very much for beautifying my changes. Next time, surely I'll make patches 
according to guidelines you have provided.
Thanks
-- 
Dmitriy

On Mon, 13 Jan 2003, Jonathan Larmour wrote:

> Date: Mon, 13 Jan 2003 05:48:41 +0000
> From: Jonathan Larmour <jifl@eCosCentric.com>
> To: Dmitriy Korovkin <dkorovkin@rambler.ru>
> Cc: ecos-patches@sources.redhat.com
> Subject: Re: mq_timedsend & mq_timedreceive
> 
> Dmitriy Korovkin wrote:
> > Hi,
> > In my work on porting ISaGRAF PRO target to eCos I found that message 
> > queue send and receive with timeout are required. So I've added functions 
> > mq_timedsend and mq_timedreceive defined by POSIX 1003.1d draft. So, 
> > patches to isoinfra, posix and kernel packages a attached. Your coments 
> > are highly appreciated.
> 
> It seems a bit odd to me for POSIX 1003.1d to do this because there's 
> plenty of ways to wake up a blocked function using signals in POSIX. I'm 
> also generally loath to include functions only in a draft, although 
> there's probably no harm as long as people are prepared to accept that the 
> draft may change!
> 
> But no matter. Despite the apparent size of the patch, I don't believe a 
> copyright assignment is required (fortunately!) since almost all is based 
> on the existing mq_send/receive code. However could I ask you in future to 
> a) include ChangeLog entries for each changed package - I have written 
> them for you this time; b) generate patches with "cvs -q diff -u5 -p". For 
> this it may be easiest to add:
> diff -u5 -p
> to your ~/.cvsrc file.
> 
> I made a few tweaks, so I'm including the patch in full again here for the 
> record.
> 
> Thanks for contributing back!
> 
> Jifl
>