« Return to Thread: Re: svn commit: r1293766 - /subversion/branches/1.7.x/STATUS

Re: svn commit: r1293766 - /subversion/branches/1.7.x/STATUS

by Julian Foad :: Rate this Message:

| View in Thread

steveking@... wrote on Sun, Feb 26, 2012 at 07:47:02 -0000:

>  + * r1235831, r1236099
>  +   Tweak the parsing of the ssl-authority-files config option to make it
>  +   easier for users to configure.
>  +   Justification:
>  +     Low impact change to help users.
>  +   Notes:
>  +     Both r1235831 and r1236099 conflict and will need a branch.
>  +   Votes:
>  +     +1: gstein, rhuijben, steveking

It looks like a lovely change for 1.8, but am I the only one who thinks we shouldn't back-port this because it isn't
forward-compatible?  That is, Subversion 1.7.0 isn't compatible with
configurations created by a user of this patch.

For reference, r1235831's log message is:

  > Improve the handling of the ssl-authority-files config option:
  > allow leading and trailing whitespace on the filenames to make
  > it easier for the user to format the list of files.

Are we willing to break our compatibility guarantees for the sake of this convenience?  I've added my veto to the STATUS file pending the outcome of this discussion.

- Julian

 « Return to Thread: Re: svn commit: r1293766 - /subversion/branches/1.7.x/STATUS