You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Noorul Islam K M <no...@collab.net> on 2011/07/13 09:37:44 UTC

Recursive deletion of property throws unnecessary warning

Initial mail from Geoff Rowell is here. 
http://svn.haxx.se/dev/archive-2009-07/0191.shtml

He said:

- Subversion 1.6.3 on Windows 
  
When recursively deleting a property from a working copy, an unnecessary 
warning is thrown because the property isn't defined on the root working 
copy folder. 
  
svn pd my:property /my/wc -R 
property ' my:property ' deleted from '/my/wc/subfolder' 
. 
svn: Attempting to delete nonexistent property 'my:property' 
---

I just checked with 1.7 version on linux and I get the following.

noorul@noorul:/tmp/wc/repos1$ tree
.
`-- A
    `-- B
        `-- lambda

2 directories, 1 file
noorul@noorul:/tmp/wc/repos1$ ~/projects/subversion/builds/trunk/bin/svn pd my:property -R
Attempting to delete nonexistent property 'my:property' on '.'
Attempting to delete nonexistent property 'my:property' on 'A'
Attempting to delete nonexistent property 'my:property' on 'A/B'
Attempting to delete nonexistent property 'my:property' on 'A/B/lambda'

I think this output is noisy. Posting this again since the initial
thread did not get any attention.

Thanks and Regards
Noorul


Re: Recursive deletion of property throws unnecessary warning

Posted by Noorul Islam K M <no...@collab.net>.
"Bert Huijben" <be...@qqmail.nl> writes:

>> -----Original Message-----
>> From: Noorul Islam K M [mailto:noorul@collab.net]
>> Sent: woensdag 13 juli 2011 9:38
>> To: Subversion
>> Subject: Recursive deletion of property throws unnecessary warning
>> 
>> 
>> Initial mail from Geoff Rowell is here.
>> http://svn.haxx.se/dev/archive-2009-07/0191.shtml
>> 
>> He said:
>> 
>> - Subversion 1.6.3 on Windows
>> 
>> When recursively deleting a property from a working copy, an unnecessary
>> warning is thrown because the property isn't defined on the root working
>> copy folder.
>> 
>> svn pd my:property /my/wc -R
>> property ' my:property ' deleted from '/my/wc/subfolder'
>> .
>> svn: Attempting to delete nonexistent property 'my:property'
>> ---
>> 
>> I just checked with 1.7 version on linux and I get the following.
>> 
>> noorul@noorul:/tmp/wc/repos1$ tree
>> .
>> `-- A
>>     `-- B
>>         `-- lambda
>> 
>> 2 directories, 1 file
>> noorul@noorul:/tmp/wc/repos1$
>> ~/projects/subversion/builds/trunk/bin/svn pd my:property -R
>> Attempting to delete nonexistent property 'my:property' on '.'
>> Attempting to delete nonexistent property 'my:property' on 'A'
>> Attempting to delete nonexistent property 'my:property' on 'A/B'
>> Attempting to delete nonexistent property 'my:property' on 'A/B/lambda'
>> 
>> I think this output is noisy. Posting this again since the initial
>> thread did not get any attention.
>
> Another interesting bit is that it doesn't notify that the property is
> deleted from the place where it did exist?
> (Can you confirm that bit or did it not exist in your test?)
>
> If that also applies to the api level, I would call that a regression as
> there is no other way to retrieve which nodes were changed by a propset or
> propdel then to capture the notifications.

I did not cover that in my test case. See here, it works.

noorul@noorul:/tmp/wc/repos1$ ~/projects/subversion/builds/trunk/bin/svn ps 'foo:bar' foobar -R A/B/lambda 
property 'foo:bar' set on 'A/B/lambda'
noorul@noorul:/tmp/wc/repos1$ ~/projects/subversion/builds/trunk/bin/svn pd foo:bar -R
Attempting to delete nonexistent property 'foo:bar' on '.'
Attempting to delete nonexistent property 'foo:bar' on 'A'
Attempting to delete nonexistent property 'foo:bar' on 'A/B'
property 'foo:bar' deleted from 'A/B/lambda'.

Thanks and Regards 
Noorul

RE: Recursive deletion of property throws unnecessary warning

Posted by Bert Huijben <be...@qqmail.nl>.

> -----Original Message-----
> From: Noorul Islam K M [mailto:noorul@collab.net]
> Sent: woensdag 13 juli 2011 9:38
> To: Subversion
> Subject: Recursive deletion of property throws unnecessary warning
> 
> 
> Initial mail from Geoff Rowell is here.
> http://svn.haxx.se/dev/archive-2009-07/0191.shtml
> 
> He said:
> 
> - Subversion 1.6.3 on Windows
> 
> When recursively deleting a property from a working copy, an unnecessary
> warning is thrown because the property isn't defined on the root working
> copy folder.
> 
> svn pd my:property /my/wc -R
> property ' my:property ' deleted from '/my/wc/subfolder'
> .
> svn: Attempting to delete nonexistent property 'my:property'
> ---
> 
> I just checked with 1.7 version on linux and I get the following.
> 
> noorul@noorul:/tmp/wc/repos1$ tree
> .
> `-- A
>     `-- B
>         `-- lambda
> 
> 2 directories, 1 file
> noorul@noorul:/tmp/wc/repos1$
> ~/projects/subversion/builds/trunk/bin/svn pd my:property -R
> Attempting to delete nonexistent property 'my:property' on '.'
> Attempting to delete nonexistent property 'my:property' on 'A'
> Attempting to delete nonexistent property 'my:property' on 'A/B'
> Attempting to delete nonexistent property 'my:property' on 'A/B/lambda'
> 
> I think this output is noisy. Posting this again since the initial
> thread did not get any attention.

Another interesting bit is that it doesn't notify that the property is
deleted from the place where it did exist?
(Can you confirm that bit or did it not exist in your test?)

If that also applies to the api level, I would call that a regression as
there is no other way to retrieve which nodes were changed by a propset or
propdel then to capture the notifications.

	Bert

> 
> Thanks and Regards
> Noorul