You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by kf...@collab.net on 2005/06/23 19:54:36 UTC

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Dean Thompson <dt...@peakstrategy.net> writes:
> I am fairly certain I have discovered a simple but severe subversion
> bug.  Here is the script of my actions:
> 
> bash-2.05b$ mkdir tmp
> bash-2.05b$ cd tmp
> bash-2.05b$ mkdir svntest
> bash-2.05b$ svn import
> . svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn
> -m "test"
> Adding         svntest
> 
> Committed revision 21.
> bash-2.05b$ rm -rf svntest
> bash-2.05b$ svn co
> svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes
> t
> Checked out revision 21.
> bash-2.05b$ ls
> svntest
> bash-2.05b$ cd svntest
> bash-2.05b$ mkdir a
> bash-2.05b$ mkdir a/b
> bash-2.05b$ svn add a
> A         a
> A         a/b
> bash-2.05b$ svn commit -m "test"
> Adding         a
> Adding         a/b
> 
> Committed revision 22.
> bash-2.05b$ svn move a b
> svn: Working copy 'b/b' is missing or not locked
> bash-2.05b$

Yup, this is a bug.  I've got a more compact reproduction recipe,
based on yours, and I've filed this as

   http://subversion.tigris.org/issues/show_bug.cgi?id=2343

Here's the recipe:

   #!/bin/sh
   
   SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
   SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
   
   URL=file:///`pwd`/repos
   
   rm -rf repos svntest
   
   ${SVNADMIN} create repos
   
   mkdir svntest
   ${SVN} import svntest ${URL}/svntest -m "test"
   rm -rf svntest
   ${SVN} co ${URL}/svntest
   cd svntest
   mkdir a
   mkdir a/b
   ${SVN} add a
   ${SVN} commit -m "test"
   # Even if you insert 'svn up' here, the following move command still
   # results in this error:
   #
   # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
   # svn: Working copy 'b/b' is missing or not locked
   ${SVN} move a b

Thanks for the report!  We'll fix it as soon as we can; obviously, if
you have time/ability to write a patch for it, that would be great.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
On Thursday 23 June 2005 19:31, Philip Martin wrote:
[snip]
> svn_path_is_child looks like it might trigger too often, I think it
> should be more like:
>
>               if (strcmp (this_entry.name, dst_basename) == 0
>                   && strcmp (src, dst_parent) == 0)

I didn't like it either... I'm doing something very similar to the above 
right now.

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by Philip Martin <ph...@codematters.co.uk>.
John Szakmeister <jo...@szakmeister.net> writes:

> --- subversion/libsvn_subr/io.c	(revision 15153)
> +++ subversion/libsvn_subr/io.c	(working copy)
> @@ -641,6 +641,7 @@
>    apr_finfo_t this_entry;
>    apr_int32_t flags = APR_FINFO_TYPE | APR_FINFO_NAME;
>  
> +  printf ("HERE\n");

check

>    /* Make a subpool for recursion */
>    apr_pool_t *subpool = svn_pool_create (pool);
>  
> @@ -714,8 +715,11 @@
>              {
>                /* Prevent infinite recursion by filtering off our
>                   newly created destination path. */
> -              if (strcmp (this_entry.name, dst_basename) == 0)
> -                 continue;
> +              if (svn_path_is_child (src_target, dst_path, subpool)
> +                  && strcmp (this_entry.name, dst_basename) == 0)
> +                {
> +                  continue;
> +                }

svn_path_is_child looks like it might trigger too often, I think it
should be more like:

              if (strcmp (this_entry.name, dst_basename) == 0
                  && strcmp (src, dst_parent) == 0)

-- 
Philip Martin

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
On Thursday 23 June 2005 19:12, John Szakmeister wrote:
> Let me try that again...
>
> On Thursday 23 June 2005 15:54, kfogel@collab.net wrote:
> [snip]
>
> > Thanks for the report!  We'll fix it as soon as we can; obviously, if
> > you have time/ability to write a patch for it, that would be great.
>
> This was bug was introduced in r14401, when trying to prevent infinite
> recursion when doing 'svn cp a a'.
>
> I believe the following patch fixes the problem.  I'm running it
> through the test suite now...

Ignore this patch... it broke the 'svn cp a a' case.  I've got something 
else in the works though.

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by Philip Martin <ph...@codematters.co.uk>.
John Szakmeister <jo...@szakmeister.net> writes:

> --- subversion/libsvn_subr/io.c	(revision 15153)
> +++ subversion/libsvn_subr/io.c	(working copy)
> @@ -641,6 +641,7 @@
>    apr_finfo_t this_entry;
>    apr_int32_t flags = APR_FINFO_TYPE | APR_FINFO_NAME;
>  
> +  printf ("HERE\n");

check

>    /* Make a subpool for recursion */
>    apr_pool_t *subpool = svn_pool_create (pool);
>  
> @@ -714,8 +715,11 @@
>              {
>                /* Prevent infinite recursion by filtering off our
>                   newly created destination path. */
> -              if (strcmp (this_entry.name, dst_basename) == 0)
> -                 continue;
> +              if (svn_path_is_child (src_target, dst_path, subpool)
> +                  && strcmp (this_entry.name, dst_basename) == 0)
> +                {
> +                  continue;
> +                }

svn_path_is_child looks like it might trigger too often, I think it
should be more like:

              if (strcmp (this_entry.name, dst_basename) == 0
                  && strcmp (src, dst_parent) == 0)

-- 
Philip Martin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
On Thursday 23 June 2005 19:12, John Szakmeister wrote:
> Let me try that again...
>
> On Thursday 23 June 2005 15:54, kfogel@collab.net wrote:
> [snip]
>
> > Thanks for the report!  We'll fix it as soon as we can; obviously, if
> > you have time/ability to write a patch for it, that would be great.
>
> This was bug was introduced in r14401, when trying to prevent infinite
> recursion when doing 'svn cp a a'.
>
> I believe the following patch fixes the problem.  I'm running it
> through the test suite now...

Ignore this patch... it broke the 'svn cp a a' case.  I've got something 
else in the works though.

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
Let me try that again...

On Thursday 23 June 2005 15:54, kfogel@collab.net wrote:
[snip]
> Thanks for the report!  We'll fix it as soon as we can; obviously, if
> you have time/ability to write a patch for it, that would be great.

This was bug was introduced in r14401, when trying to prevent infinite 
recursion when doing 'svn cp a a'.

I believe the following patch fixes the problem.  I'm running it through 
the test suite now...

-John

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
On Thursday 23 June 2005 15:54, kfogel@collab.net wrote:
[snip]
> Thanks for the report!  We'll fix it as soon as we can; obviously, if
> you have time/ability to write a patch for it, that would be great.

This was bug was introduced in r14401, when trying to prevent infinite 
recursion when doing 'svn cp a a'.

I believe the following patch fixes the problem.  I'm running it through 
the test suite now...

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
Let me try that again...

On Thursday 23 June 2005 15:54, kfogel@collab.net wrote:
[snip]
> Thanks for the report!  We'll fix it as soon as we can; obviously, if
> you have time/ability to write a patch for it, that would be great.

This was bug was introduced in r14401, when trying to prevent infinite 
recursion when doing 'svn cp a a'.

I believe the following patch fixes the problem.  I'm running it through 
the test suite now...

-John

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
On Friday 24 June 2005 14:44, Dean Thompson wrote:
>  Wow, thanks for the quick and thorough response!  I wish I could
> volunteer a patch, but I only started using subversion this week.  So
> far I love it!

This is fixed on trunk/ in r15157.  Thanks for the report!

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by kf...@collab.net.
Whups -- can you resend as plaintext instead of HTML, please?

See http://subversion.tigris.org/mailing-list-guidelines.html#encodings

Thanks,
-Karl


Dean Thompson <dt...@peakstrategy.net> writes:
> <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
> <html>
> <head>
>   <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
> </head>
> <body bgcolor="#ffffff" text="#000000">
> Wow, thanks for the quick and thorough response!&nbsp; I wish I could
> volunteer a patch, but I only started using subversion this week.&nbsp; So
> far I love it!<br>
> <br>
> Dean<br>
> <br>
> <a class="moz-txt-link-abbreviated" href="mailto:kfogel@collab.net">kfogel@collab.net</a> wrote:
> <blockquote cite="mid85is04x30j.fsf@newton.ch.collab.net" type="cite">
>   <pre wrap="">Dean Thompson <a class="moz-txt-link-rfc2396E" href="mailto:dthompson@peakstrategy.net">&lt;dthompson@peakstrategy.net&gt;</a> writes:
>   </pre>
>   <blockquote type="cite">
>     <pre wrap="">I am fairly certain I have discovered a simple but severe subversion
> bug.  Here is the script of my actions:
> 
> bash-2.05b$ mkdir tmp
> bash-2.05b$ cd tmp
> bash-2.05b$ mkdir svntest
> bash-2.05b$ svn import
> . <a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn</a>
> -m "test"
> Adding         svntest
> 
> Committed revision 21.
> bash-2.05b$ rm -rf svntest
> bash-2.05b$ svn co
> <a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes</a>
> t
> Checked out revision 21.
> bash-2.05b$ ls
> svntest
> bash-2.05b$ cd svntest
> bash-2.05b$ mkdir a
> bash-2.05b$ mkdir a/b
> bash-2.05b$ svn add a
> A         a
> A         a/b
> bash-2.05b$ svn commit -m "test"
> Adding         a
> Adding         a/b
> 
> Committed revision 22.
> bash-2.05b$ svn move a b
> svn: Working copy 'b/b' is missing or not locked
> bash-2.05b$
>     </pre>
>   </blockquote>
>   <pre wrap=""><!---->
> Yup, this is a bug.  I've got a more compact reproduction recipe,
> based on yours, and I've filed this as
> 
>    <a class="moz-txt-link-freetext" href="http://subversion.tigris.org/issues/show_bug.cgi?id=2343">http://subversion.tigris.org/issues/show_bug.cgi?id=2343</a>
> 
> Here's the recipe:
> 
>    #!/bin/sh
>    
>    SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
>    SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
>    
>    URL=<a class="moz-txt-link-freetext" href="file:///">file:///</a>`pwd`/repos
>    
>    rm -rf repos svntest
>    
>    ${SVNADMIN} create repos
>    
>    mkdir svntest
>    ${SVN} import svntest ${URL}/svntest -m "test"
>    rm -rf svntest
>    ${SVN} co ${URL}/svntest
>    cd svntest
>    mkdir a
>    mkdir a/b
>    ${SVN} add a
>    ${SVN} commit -m "test"
>    # Even if you insert 'svn up' here, the following move command still
>    # results in this error:
>    #
>    # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
>    # svn: Working copy 'b/b' is missing or not locked
>    ${SVN} move a b
> 
> Thanks for the report!  We'll fix it as soon as we can; obviously, if
> you have time/ability to write a patch for it, that would be great.
> 
> -Karl
> 
> 
> 
>   </pre>
> </blockquote>
> </body>
> </html>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by kf...@collab.net.
Whups -- can you resend as plaintext instead of HTML, please?

See http://subversion.tigris.org/mailing-list-guidelines.html#encodings

Thanks,
-Karl


Dean Thompson <dt...@peakstrategy.net> writes:
> <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
> <html>
> <head>
>   <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
> </head>
> <body bgcolor="#ffffff" text="#000000">
> Wow, thanks for the quick and thorough response!&nbsp; I wish I could
> volunteer a patch, but I only started using subversion this week.&nbsp; So
> far I love it!<br>
> <br>
> Dean<br>
> <br>
> <a class="moz-txt-link-abbreviated" href="mailto:kfogel@collab.net">kfogel@collab.net</a> wrote:
> <blockquote cite="mid85is04x30j.fsf@newton.ch.collab.net" type="cite">
>   <pre wrap="">Dean Thompson <a class="moz-txt-link-rfc2396E" href="mailto:dthompson@peakstrategy.net">&lt;dthompson@peakstrategy.net&gt;</a> writes:
>   </pre>
>   <blockquote type="cite">
>     <pre wrap="">I am fairly certain I have discovered a simple but severe subversion
> bug.  Here is the script of my actions:
> 
> bash-2.05b$ mkdir tmp
> bash-2.05b$ cd tmp
> bash-2.05b$ mkdir svntest
> bash-2.05b$ svn import
> . <a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn</a>
> -m "test"
> Adding         svntest
> 
> Committed revision 21.
> bash-2.05b$ rm -rf svntest
> bash-2.05b$ svn co
> <a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes</a>
> t
> Checked out revision 21.
> bash-2.05b$ ls
> svntest
> bash-2.05b$ cd svntest
> bash-2.05b$ mkdir a
> bash-2.05b$ mkdir a/b
> bash-2.05b$ svn add a
> A         a
> A         a/b
> bash-2.05b$ svn commit -m "test"
> Adding         a
> Adding         a/b
> 
> Committed revision 22.
> bash-2.05b$ svn move a b
> svn: Working copy 'b/b' is missing or not locked
> bash-2.05b$
>     </pre>
>   </blockquote>
>   <pre wrap=""><!---->
> Yup, this is a bug.  I've got a more compact reproduction recipe,
> based on yours, and I've filed this as
> 
>    <a class="moz-txt-link-freetext" href="http://subversion.tigris.org/issues/show_bug.cgi?id=2343">http://subversion.tigris.org/issues/show_bug.cgi?id=2343</a>
> 
> Here's the recipe:
> 
>    #!/bin/sh
>    
>    SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
>    SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
>    
>    URL=<a class="moz-txt-link-freetext" href="file:///">file:///</a>`pwd`/repos
>    
>    rm -rf repos svntest
>    
>    ${SVNADMIN} create repos
>    
>    mkdir svntest
>    ${SVN} import svntest ${URL}/svntest -m "test"
>    rm -rf svntest
>    ${SVN} co ${URL}/svntest
>    cd svntest
>    mkdir a
>    mkdir a/b
>    ${SVN} add a
>    ${SVN} commit -m "test"
>    # Even if you insert 'svn up' here, the following move command still
>    # results in this error:
>    #
>    # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
>    # svn: Working copy 'b/b' is missing or not locked
>    ${SVN} move a b
> 
> Thanks for the report!  We'll fix it as soon as we can; obviously, if
> you have time/ability to write a patch for it, that would be great.
> 
> -Karl
> 
> 
> 
>   </pre>
> </blockquote>
> </body>
> </html>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by Dean Thompson <dt...@peakstrategy.net>.
Wow, thanks for the quick and thorough response!  I wish I could 
volunteer a patch, but I only started using subversion this week.  So 
far I love it!

>
> Dean
>
> kfogel@collab.net wrote:
>
>>Dean Thompson <dt...@peakstrategy.net> writes:
>>  
>>
>>>I am fairly certain I have discovered a simple but severe subversion
>>>bug.  Here is the script of my actions:
>>>
>>>bash-2.05b$ mkdir tmp
>>>bash-2.05b$ cd tmp
>>>bash-2.05b$ mkdir svntest
>>>bash-2.05b$ svn import
>>>. svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn
>>>-m "test"
>>>Adding         svntest
>>>
>>>Committed revision 21.
>>>bash-2.05b$ rm -rf svntest
>>>bash-2.05b$ svn co
>>>svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes
>>>t
>>>Checked out revision 21.
>>>bash-2.05b$ ls
>>>svntest
>>>bash-2.05b$ cd svntest
>>>bash-2.05b$ mkdir a
>>>bash-2.05b$ mkdir a/b
>>>bash-2.05b$ svn add a
>>>A         a
>>>A         a/b
>>>bash-2.05b$ svn commit -m "test"
>>>Adding         a
>>>Adding         a/b
>>>
>>>Committed revision 22.
>>>bash-2.05b$ svn move a b
>>>svn: Working copy 'b/b' is missing or not locked
>>>bash-2.05b$
>>>    
>>>
>>
>>Yup, this is a bug.  I've got a more compact reproduction recipe,
>>based on yours, and I've filed this as
>>
>>   http://subversion.tigris.org/issues/show_bug.cgi?id=2343
>>
>>Here's the recipe:
>>
>>   #!/bin/sh
>>   
>>   SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
>>   SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
>>   
>>   URL=file:///`pwd`/repos
>>   
>>   rm -rf repos svntest
>>   
>>   ${SVNADMIN} create repos
>>   
>>   mkdir svntest
>>   ${SVN} import svntest ${URL}/svntest -m "test"
>>   rm -rf svntest
>>   ${SVN} co ${URL}/svntest
>>   cd svntest
>>   mkdir a
>>   mkdir a/b
>>   ${SVN} add a
>>   ${SVN} commit -m "test"
>>   # Even if you insert 'svn up' here, the following move command still
>>   # results in this error:
>>   #
>>   # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
>>   # svn: Working copy 'b/b' is missing or not locked
>>   ${SVN} move a b
>>
>>Thanks for the report!  We'll fix it as soon as we can; obviously, if
>>you have time/ability to write a patch for it, that would be great.
>>
>>-Karl
>>
>>
>>
>>  
>>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by Dean Thompson <dt...@peakstrategy.net>.
Wow, thanks for the quick and thorough response!  I wish I could 
volunteer a patch, but I only started using subversion this week.  So 
far I love it!

>
> Dean
>
> kfogel@collab.net wrote:
>
>>Dean Thompson <dt...@peakstrategy.net> writes:
>>  
>>
>>>I am fairly certain I have discovered a simple but severe subversion
>>>bug.  Here is the script of my actions:
>>>
>>>bash-2.05b$ mkdir tmp
>>>bash-2.05b$ cd tmp
>>>bash-2.05b$ mkdir svntest
>>>bash-2.05b$ svn import
>>>. svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn
>>>-m "test"
>>>Adding         svntest
>>>
>>>Committed revision 21.
>>>bash-2.05b$ rm -rf svntest
>>>bash-2.05b$ svn co
>>>svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes
>>>t
>>>Checked out revision 21.
>>>bash-2.05b$ ls
>>>svntest
>>>bash-2.05b$ cd svntest
>>>bash-2.05b$ mkdir a
>>>bash-2.05b$ mkdir a/b
>>>bash-2.05b$ svn add a
>>>A         a
>>>A         a/b
>>>bash-2.05b$ svn commit -m "test"
>>>Adding         a
>>>Adding         a/b
>>>
>>>Committed revision 22.
>>>bash-2.05b$ svn move a b
>>>svn: Working copy 'b/b' is missing or not locked
>>>bash-2.05b$
>>>    
>>>
>>
>>Yup, this is a bug.  I've got a more compact reproduction recipe,
>>based on yours, and I've filed this as
>>
>>   http://subversion.tigris.org/issues/show_bug.cgi?id=2343
>>
>>Here's the recipe:
>>
>>   #!/bin/sh
>>   
>>   SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
>>   SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
>>   
>>   URL=file:///`pwd`/repos
>>   
>>   rm -rf repos svntest
>>   
>>   ${SVNADMIN} create repos
>>   
>>   mkdir svntest
>>   ${SVN} import svntest ${URL}/svntest -m "test"
>>   rm -rf svntest
>>   ${SVN} co ${URL}/svntest
>>   cd svntest
>>   mkdir a
>>   mkdir a/b
>>   ${SVN} add a
>>   ${SVN} commit -m "test"
>>   # Even if you insert 'svn up' here, the following move command still
>>   # results in this error:
>>   #
>>   # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
>>   # svn: Working copy 'b/b' is missing or not locked
>>   ${SVN} move a b
>>
>>Thanks for the report!  We'll fix it as soon as we can; obviously, if
>>you have time/ability to write a patch for it, that would be great.
>>
>>-Karl
>>
>>
>>
>>  
>>


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by Dean Thompson <dt...@peakstrategy.net>.
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Wow, thanks for the quick and thorough response!&nbsp; I wish I could
volunteer a patch, but I only started using subversion this week.&nbsp; So
far I love it!<br>
<br>
Dean<br>
<br>
<a class="moz-txt-link-abbreviated" href="mailto:kfogel@collab.net">kfogel@collab.net</a> wrote:
<blockquote cite="mid85is04x30j.fsf@newton.ch.collab.net" type="cite">
  <pre wrap="">Dean Thompson <a class="moz-txt-link-rfc2396E" href="mailto:dthompson@peakstrategy.net">&lt;dthompson@peakstrategy.net&gt;</a> writes:
  </pre>
  <blockquote type="cite">
    <pre wrap="">I am fairly certain I have discovered a simple but severe subversion
bug.  Here is the script of my actions:

bash-2.05b$ mkdir tmp
bash-2.05b$ cd tmp
bash-2.05b$ mkdir svntest
bash-2.05b$ svn import
. <a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn</a>
-m "test"
Adding         svntest

Committed revision 21.
bash-2.05b$ rm -rf svntest
bash-2.05b$ svn co
<a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes</a>
t
Checked out revision 21.
bash-2.05b$ ls
svntest
bash-2.05b$ cd svntest
bash-2.05b$ mkdir a
bash-2.05b$ mkdir a/b
bash-2.05b$ svn add a
A         a
A         a/b
bash-2.05b$ svn commit -m "test"
Adding         a
Adding         a/b

Committed revision 22.
bash-2.05b$ svn move a b
svn: Working copy 'b/b' is missing or not locked
bash-2.05b$
    </pre>
  </blockquote>
  <pre wrap=""><!---->
Yup, this is a bug.  I've got a more compact reproduction recipe,
based on yours, and I've filed this as

   <a class="moz-txt-link-freetext" href="http://subversion.tigris.org/issues/show_bug.cgi?id=2343">http://subversion.tigris.org/issues/show_bug.cgi?id=2343</a>

Here's the recipe:

   #!/bin/sh
   
   SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
   SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
   
   URL=<a class="moz-txt-link-freetext" href="file:///">file:///</a>`pwd`/repos
   
   rm -rf repos svntest
   
   ${SVNADMIN} create repos
   
   mkdir svntest
   ${SVN} import svntest ${URL}/svntest -m "test"
   rm -rf svntest
   ${SVN} co ${URL}/svntest
   cd svntest
   mkdir a
   mkdir a/b
   ${SVN} add a
   ${SVN} commit -m "test"
   # Even if you insert 'svn up' here, the following move command still
   # results in this error:
   #
   # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
   # svn: Working copy 'b/b' is missing or not locked
   ${SVN} move a b

Thanks for the report!  We'll fix it as soon as we can; obviously, if
you have time/ability to write a patch for it, that would be great.

-Karl



  </pre>
</blockquote>
</body>
</html>


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by John Szakmeister <jo...@szakmeister.net>.
On Thursday 23 June 2005 15:54, kfogel@collab.net wrote:
[snip]
> Thanks for the report!  We'll fix it as soon as we can; obviously, if
> you have time/ability to write a patch for it, that would be great.

This was bug was introduced in r14401, when trying to prevent infinite 
recursion when doing 'svn cp a a'.

I believe the following patch fixes the problem.  I'm running it through 
the test suite now...

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Re: "is missing or not locked" error executing 'svn move dir1 dir2' if dir1 contains its own directory called dir2

Posted by Dean Thompson <dt...@peakstrategy.net>.
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Wow, thanks for the quick and thorough response!&nbsp; I wish I could
volunteer a patch, but I only started using subversion this week.&nbsp; So
far I love it!<br>
<br>
Dean<br>
<br>
<a class="moz-txt-link-abbreviated" href="mailto:kfogel@collab.net">kfogel@collab.net</a> wrote:
<blockquote cite="mid85is04x30j.fsf@newton.ch.collab.net" type="cite">
  <pre wrap="">Dean Thompson <a class="moz-txt-link-rfc2396E" href="mailto:dthompson@peakstrategy.net">&lt;dthompson@peakstrategy.net&gt;</a> writes:
  </pre>
  <blockquote type="cite">
    <pre wrap="">I am fairly certain I have discovered a simple but severe subversion
bug.  Here is the script of my actions:

bash-2.05b$ mkdir tmp
bash-2.05b$ cd tmp
bash-2.05b$ mkdir svntest
bash-2.05b$ svn import
. <a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn</a>
-m "test"
Adding         svntest

Committed revision 21.
bash-2.05b$ rm -rf svntest
bash-2.05b$ svn co
<a class="moz-txt-link-abbreviated" href="mailto:svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes">svn+ssh://dthompso@cvs.peakstrategy.net/usr/local/svn/svntes</a>
t
Checked out revision 21.
bash-2.05b$ ls
svntest
bash-2.05b$ cd svntest
bash-2.05b$ mkdir a
bash-2.05b$ mkdir a/b
bash-2.05b$ svn add a
A         a
A         a/b
bash-2.05b$ svn commit -m "test"
Adding         a
Adding         a/b

Committed revision 22.
bash-2.05b$ svn move a b
svn: Working copy 'b/b' is missing or not locked
bash-2.05b$
    </pre>
  </blockquote>
  <pre wrap=""><!---->
Yup, this is a bug.  I've got a more compact reproduction recipe,
based on yours, and I've filed this as

   <a class="moz-txt-link-freetext" href="http://subversion.tigris.org/issues/show_bug.cgi?id=2343">http://subversion.tigris.org/issues/show_bug.cgi?id=2343</a>

Here's the recipe:

   #!/bin/sh
   
   SVN=${HOME}/src/subversion/subversion/clients/cmdline/svn
   SVNADMIN=${HOME}/src/subversion/subversion/svnadmin/svnadmin
   
   URL=<a class="moz-txt-link-freetext" href="file:///">file:///</a>`pwd`/repos
   
   rm -rf repos svntest
   
   ${SVNADMIN} create repos
   
   mkdir svntest
   ${SVN} import svntest ${URL}/svntest -m "test"
   rm -rf svntest
   ${SVN} co ${URL}/svntest
   cd svntest
   mkdir a
   mkdir a/b
   ${SVN} add a
   ${SVN} commit -m "test"
   # Even if you insert 'svn up' here, the following move command still
   # results in this error:
   #
   # subversion/libsvn_wc/lock.c:677: (apr_err=155005)
   # svn: Working copy 'b/b' is missing or not locked
   ${SVN} move a b

Thanks for the report!  We'll fix it as soon as we can; obviously, if
you have time/ability to write a patch for it, that would be great.

-Karl



  </pre>
</blockquote>
</body>
</html>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org