Re: [PATCH] Add support for GnuTLS

On Sat, May 06, 2006 at 02:47:33PM -0400, Bruce Momjian wrote:
> Martijn van Oosterhout wrote:
> > If you like I can split it into two patches, one patch splits the openssl
> > stuff out of the main files and a second which adds gnutls support.
>
> Yes, I understood that, but we now have duplicate files for secure
> connections, meaning we have double maintenance in some cases.

Hmm, I see your point. I guess that's an unavoidable side-effect of the
process :(. However, judging from the CVS logs, these have not been files
with a high change rate. I think it's worth it but I can imagine other
people see that differently.

There is not a lot of code can be shared. What can be already is eg.
prepare_for_client_read and client_read_ended, the names of the files
used, EPIPE handling, etc.

Have a nice day,
--
Martijn van Oosterhout <kleptog(at)svana(dot)org> http://svana.org/kleptog/
> From each according to his ability. To each according to his ability to litigate.