cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] Added AUTH NTLM for SMTP

From: Ben Greear <greearb_at_candelatech.com>
Date: Mon, 08 Aug 2011 16:32:57 -0700

On 08/08/2011 02:39 PM, Steve Holme wrote:
> 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.

In case you are working on Linux, you might find the 'stg' package useful.

It can make creating and editing patch series easier....

Ben

>
> Cheers
>
> Steve
>
> -------------------------------------------------------------------
> List admin: http://cool.haxx.se/list/listinfo/curl-library
> Etiquette: http://curl.haxx.se/mail/etiquette.html

-- 
Ben Greear <greearb_at_candelatech.com>
Candela Technologies Inc  http://www.candelatech.com
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2011-08-09