2011/8/25 Daniel Stenberg wrote:

> I think adding it among the bits is better than having it as a completely
> separate option.

I also prefer that one.

> But also, as Steve brought up, we're in feature freeze right now and even if
> it isn't a rule we can ignore at times I think we first need to consider it
> carefully first if it really needs to go in now.

Yes of course. That is your decision.

> We do have these feature
> freeze periods in order to "settle" and iron out bugs.

Yes certainly. I've attached chronology that shows that feature freeze
period has not been ignored. Everything pushed after Aug-16 have been
fixes for stuff already in master repo.

> Of course we can
> still work on patches and discuss how to do things for inclusion after the
> next release.

Patch requested today should show that there's little left. But
functionality this patch would provide already existed in first
patches provided Jul-29 and Aug-02 by Steve.

It is not my decision weather this should go in before next release.
It is yours.

What I'm asking here is you postpone this decision until Steve
provides requested patch and we can see if it needs further polishing
and better evaluate risks associated with pushing it in feature freeze
period.

Steve, please, I'll test your patches locally here. Don't let the ball
drop now ;-)
-- 
-=[Yang]=-
Chronology of NTLM SMTP fixes...

Jul-23 Steve joined the mailing list
Jul-29 Steve provides first patch
Aug-02 Steve provides followup patch
Aug-04 TODO-RELEASE item #303 added
Aug-04 Help with TODO-RELEASE items requested
Aug-05 I ask Steve for a consolidated patch of Jul-29 and Aug-02
Aug-05 Steve provides consolidated patch
Aug-08 I ask steve to break the consolidated patch into pieces
Aug-09 Feature freeze announced for Aug-16
Aug-09 Steve provides 
0001-Tidied-up-http_ntlm-prior-to-splitting-the-ntlm-spec.patch
Aug-09 I push patch along with TAB and lengthfixes
Aug-10 Steve provides 
0001-Moved-ntlm-specific-code-into-separate-curl_ntlm-mod.patch
Aug-10 I require some modifications in that patch
Aug-10 Steve provides 0001-Tidied-up-more-inconsistent-spacing.patch
Aug-11 I pushed patch with a couple of edits
Aug-11 Steve provides 
0001-Moved-ntlm-specific-code-into-separate_curl_ntlm_mod_2.patch
Aug-11 I require some modifications in that patch
Aug-12 Steve provides 
0001-Moved-ntlm-specific-code-into-separate-curl_ntlm-mod_3.patch
Aug-12 I push part of Aug-12 patch
Aug-13 I push minor edits
Aug-14 I push some adjustments
Aug-15 I push some adjustments
Aug-16 FEATURE FREEZE
Aug-22 I push remaining bits from Aug-12 patch
Aug-22 I ask steve another patch for the SMTP authentication orderingstuff
Aug-22 Steve provides 
0001-Modified-the-order-of-the-preferred-authentication-m.patch
Aug-24 I push Curl_base64 interface fixes
Aug-24 I push Aug-22 patch
Aug-24 Steve provides 
0001-Added-the-ability-to-specify-whether-authenticated-c.patch
Aug-25 I push NTLM refactoring followup
Aug-25 I Ask Steve to change Aug-24 patch to better fit libcurl API

Here we are.
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to