cURL / Mailing Lists / curl-library / Single Mail

curl-library

RE: [PATCH] Added AUTH NTLM for SMTP

From: Steve Holme <steve_holme_at_hotmail.com>
Date: Mon, 8 Aug 2011 22:39:12 +0100

Hi all,

On Mon, 8 Aug 2011, Daniel Stenberg wrote:

> Not really, most changes people contribute with are much smaller.
>
> I do want to stress that none of us ask you this to be annoying or
> hard to work with. We do this in the interest of getting the changes
> into the project in the best possible way so that we will be able to
> maintain them from then on. We very much appreciate your hard
> work and contributions!

Thanks for clarification guys... I wasn't suggesting for one moment that
anyone was being annoying or difficult. I have worked in a lot of different
development environments, mostly as on a freelance basis, so I fully
understand that every company / team / project has their different ways of
working and this is no different. Not only does the code need reviewing,
making sure it is fit for purpose and checking that it is defect free but it
also need to be integrated with a constantly moving repository - especially
some of which, the http_ntlm module in particular, has been changing quite a
bit recently.

I guess I am a little frustrated because Friday was a nightmare of a day,
whilst I was struggling with GIT and trying to get an all in one patch put
together (as requested) only to asked to then split the code up into three
submissions - some of which I'm still not totally clear on :( Not only that
but I am eager to see my changes pushed to the repository so I can crack on
with the other features and fixes that I have it mind ;-)

I will hopefully have some time tomorrow to prepare the first patch.

Cheers

Steve

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2011-08-08