Uploaded image for project: 'Subversion'
  1. Subversion
  2. SVN-2306

'switch' does not reset the native file system's read-only attribute within a WC

    XMLWordPrintableJSON

    Details

      Description

      If your WC has a file with the svn:needs-lock property set and you then switch
      your WC to a branch where that file does not have that property.  The property
      is removed in the WC but the read-only attribute is not removed from the file.
      
      This causes problems in Subclipse, and somewhat also TortoiseSVN, since we both
      decorate files based on the read-only attribute.  In the case of Subclipse, the
      presence of the read-only attribute causes Eclipse to present the user with the
      Subclipse Lock UI -- a good thing.  However, we are then relying on the svn lock
      command to remove the read-only attribute, which it doesn't in this case.  It
      would be nice as a fail-safe if it could do that.
      
      This should be reproduction recipe.  I did it on Windows, so this may not be
      exactly right.
      
      svn mkdir -m "" url://repos/trunk
      svn co url://repos/trunk test_wc
      cd testwc
      touch foo
      svn add foo
      svn ci -m ""
      svn copy -m "" . url://repos/branch 
      svn propset svn:needs-lock foo
      svn ci -m ""    
      svn switch url://repos/branch
      
      At this point, foo will not have the property, but it will be read-only.
      
      Again, as an additional requuest it would be nice at this point if:
      
      svn lock foo
      
      Would remove the read-only attribute even though the svn:needs-lock property
      does not exist.
      

        Attachments

        1. 5_patch.txt
          3 kB
          Daniel Rall
        2. 4_2306.recipe.sh
          2 kB
          Daniel Rall
        3. 3_stack.txt
          3 kB
          Daniel Rall
        4. 2_patch.txt
          3 kB
          Daniel Rall
        5. 1_patch.txt
          1 kB
          Daniel Rall

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              markphip Mark Phippard
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: