Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-19 Thread Martin Furter
On 2016-10-18 17:01 (-0400), Johan Corveleyn wrote: > On Tue, Oct 18, 2016 at 1:35 PM, Daniel Shahaf > wrote: > > Branko Čibej wrote on Tue, Oct 18, 2016 at 13:08:37 +0200: > >> It is. I don't think there's any way we can /not/ make it backwards > >> incompatible, given the current implement

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Johan Corveleyn
On Tue, Oct 18, 2016 at 1:35 PM, Daniel Shahaf wrote: > Branko Čibej wrote on Tue, Oct 18, 2016 at 13:08:37 +0200: >> It is. I don't think there's any way we can /not/ make it backwards >> incompatible, given the current implementation. Which is why I think we >> should just admit that current beh

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Branko Čibej
On 18.10.2016 19:10, Branko Čibej wrote: > On 18.10.2016 18:46, Martin Edgar Furter Rathod wrote: >> On 10/18/16 16:38, Branko Čibej wrote: >>> On 18.10.2016 12:29, Daniel Shahaf wrote: Branko Čibej wrote on Tue, Oct 18, 2016 at 12:06:32 +0200: > If we change the behaviour so that a rule f

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Branko Čibej
On 18.10.2016 18:46, Martin Edgar Furter Rathod wrote: > On 10/18/16 16:38, Branko Čibej wrote: >> On 18.10.2016 12:29, Daniel Shahaf wrote: >>> Branko Čibej wrote on Tue, Oct 18, 2016 at 12:06:32 +0200: If we change the behaviour so that a rule for a (case-insensitive) pattern completely

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Martin Edgar Furter Rathod
On 10/18/16 16:38, Branko Čibej wrote: On 18.10.2016 12:29, Daniel Shahaf wrote: Branko Čibej wrote on Tue, Oct 18, 2016 at 12:06:32 +0200: If we change the behaviour so that a rule for a (case-insensitive) pattern completely overrides a hierarchically previoius rule, we can also introduce the

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Daniel Shahaf
Branko Čibej wrote on Tue, Oct 18, 2016 at 13:08:37 +0200: > It is. I don't think there's any way we can /not/ make it backwards > incompatible, given the current implementation. Which is why I think we > should just admit that current behaviour is a bug. > > Either that, or introduce a new proper

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Branko Čibej
On 18.10.2016 12:29, Daniel Shahaf wrote: > Branko Čibej wrote on Tue, Oct 18, 2016 at 12:06:32 +0200: >> If we change the behaviour so that a rule for a (case-insensitive) >> pattern completely overrides a hierarchically previoius rule, we can >> also introduce the concept of disabling auto-props

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Stefan Hett
On 10/18/2016 12:29 PM, Daniel Shahaf wrote: Branko Čibej wrote on Tue, Oct 18, 2016 at 12:06:32 +0200: On 18.10.2016 09:32, Stefan wrote: On 10/17/2016 15:37, Branko Čibej wrote: On 17.10.2016 14:12, Daniel Shahaf wrote: Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: Omitting

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Daniel Shahaf
Branko Čibej wrote on Tue, Oct 18, 2016 at 12:06:32 +0200: > On 18.10.2016 09:32, Stefan wrote: > > On 10/17/2016 15:37, Branko Čibej wrote: > >> On 17.10.2016 14:12, Daniel Shahaf wrote: > >>> Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: > Omitting the '=' also doesn't cut it

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Stefan Hett
On 10/18/2016 12:06 PM, Branko Čibej wrote: On 18.10.2016 09:32, Stefan wrote: On 10/17/2016 15:37, Branko Čibej wrote: On 17.10.2016 14:12, Daniel Shahaf wrote: Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: Omitting the '=' also doesn't cut it: [[[ C:\autoprops\wc\trunk\dir>

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Branko Čibej
On 18.10.2016 09:32, Stefan wrote: > On 10/17/2016 15:37, Branko Čibej wrote: >> On 17.10.2016 14:12, Daniel Shahaf wrote: >>> Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: Omitting the '=' also doesn't cut it: [[[ C:\autoprops\wc\trunk\dir>svn pg svn:auto-props

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-18 Thread Stefan
On 10/17/2016 15:37, Branko Čibej wrote: > On 17.10.2016 14:12, Daniel Shahaf wrote: >> Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: >>> Omitting the '=' also doesn't cut it: >>> >>> [[[ >>> C:\autoprops\wc\trunk\dir>svn pg svn:auto-props --show-inherited-props >>> C:\autoprops\wc

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-17 Thread Branko Čibej
On 17.10.2016 14:12, Daniel Shahaf wrote: > Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: >> Omitting the '=' also doesn't cut it: >> >> [[[ >> C:\autoprops\wc\trunk\dir>svn pg svn:auto-props --show-inherited-props >> C:\autoprops\wc - *.txt = svn:eol-style=native >> >> C:\autoprops

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-17 Thread Daniel Shahaf
Johan Corveleyn wrote on Mon, Oct 17, 2016 at 10:18:35 +0200: > Omitting the '=' also doesn't cut it: > > [[[ > C:\autoprops\wc\trunk\dir>svn pg svn:auto-props --show-inherited-props > C:\autoprops\wc - *.txt = svn:eol-style=native > > C:\autoprops\wc\trunk - *.txt = > svn:eol-style;svn:mime-type

Re: svn:auto-props: property specified in a parent dir but not specified in a subdir

2016-10-17 Thread Johan Corveleyn
[ dropped users@, added Paul Burba and Julian Foad, who have worked on the inherited-props / repos-dictated-config features in the past, and might have an educated opinion on this :-) ] On Mon, Oct 17, 2016 at 1:45 AM, Daniel Shahaf wrote: > > [ moving to dev@; please reply to dev@ only ] > > Joh