Re: Solaris /dev/poll support for Squid 3 (how can I contribute)

On 08/10/10 23:13, Peter Payne wrote:
> Dear Mailing List,
>
> I have a contribution to make, a C++ file (and autoconf patches) for
> /dev/poll Solaris support, ported from Squid-2 and in use on the BBC
> (UK) estate managed by Siemens UK with permission from the BBC to
> publish to the Squid open-source project.
>
> I've not contributed to an open source project before and unsure how to
> deliver the patch. We (at the BBC/Siemens) would like to see the patch
> included so that future releases of Squid shall have the Solaris
> /dev/poll support supplied (reducing the need for us to patch each
> following release manually).
>
> If one of you were to e-mail me directly I would be happy to provide the
> following:
> comm_devpoll.cc - /dev/poll support file to add to /src/ subdir
> squid-root.diff - unified diff to update configure.in in the / subdir
> squid-include.diff - unified diff to update autoconf.h.in in the
> /include/ subdir
> squid-src.diff - unified diff to update files Makefile.in and unlinkd.cc
> in the /src/ subdir
>
> The support for /dev/poll will be automatically detected by configure
> and conditionally compiled in.

Greetings Peter,
   Thank you very much for this.

Our procedure for patch inclusion is relatively simple:

  * Post the unified-diff format patches here to the squid-dev list.
  * with a subject starting "[PATCH] "
  * who to credit as author(s), default is the senders name and email.
  * message text describing hat it does for the public commit record and
to assist people understand the code.
  * companies where relevant mention as thanks or sponsors
(these last two are covered by the email you already sent).

  One or more of us will do an audit that it fits on 3.HEAD then reply
with any updates that may be needed.

Most of that you have done already, we just need the patches posted with
author details to start the audit cycle.

On the administrative side. I'm not sure this one is /dev/poll related,
but with the info that BBC is using your patch this reporters address
and build options looks a little suspicious:
http://bugs.squid-cache.org/show_bug.cgi?id=3057
If you can sight it without the patch or can't sight it at all thats
fine, but if introduced it will need fixing.

Amos Jeffries
Squid HTTP Proxy Project
Received on Sat Oct 09 2010 - 01:26:21 MDT