/stdhome

To get this branch, use:
bzr branch http://bzr.ed.am/stdhome

« back to all changes in this revision

Viewing changes to lib/stdhome/command/add.py

  • Committer: Tim Marston
  • Date: 2016-12-13 21:40:34 UTC
  • Revision ID: tim@ed.am-20161213214034-nd5t7ztnlrjd627i
fix add command and generic filename expansion/resolution to expend to
homedir-relative filename and absolute filename based on original filename, as
specified, rather than a fully, symlink-resolved filename.  So, e.g., if ~/bob
was a symlink to ~/fred, then ~/bob/a would resolve to the relative filename
bob/a, becuase it is inside the homedir (it would resolve to fred/a otherwise)

Show diffs side-by-side

added added

removed removed

lib/stdhome/command/add.py
24
24
import stdhome.the as the
25
25
from stdhome.deployment import Deployment
26
26
from stdhome.walker.copy_in import CopyInWalker
 
27
from stdhome.walker.walker import Walker
27
28
 
28
29
 
29
30
class AddCommand( Command ):
30
31
 
31
32
 
 
33
        def __init__( self ):
 
34
                self.recursive = False
 
35
 
 
36
 
32
37
        def print_help( self ):
33
38
                print "Usage: " + the.program.name + " add [--repo=REPO] FILE..."
34
39
                print
38
43
                print "Add a named file from the local filesystem, specified relative to the home"
39
44
                print "directory, to the repo, or update an existing file in the repo with your local"
40
45
                print "changes.  Directories can also be added, but note that they are not added"
41
 
                print "recursively (as is common with version control)."
 
46
                print "recursively (as is common with version control) unless this is specifically"
 
47
                print "requested on the commandline."
42
48
                print
43
49
                print "Options:"
44
50
                print "  -r, --repo=REPO  select the repo to check-out or create (defaults to 'home')"
 
51
                print "  -R, --recursive  recursively add directories"
45
52
                print "  -v, --verbose    display information about what is being done"
46
53
                print "      --help       display help and exit"
47
54
                exit( 0 )
49
56
 
50
57
        def parse_command_line( self ):
51
58
                opts, args = getopt.gnu_getopt(
52
 
                        sys.argv[ 1: ], "r:v",
53
 
                        [ "repo=", "verbose", "help" ] )
 
59
                        sys.argv[ 1: ], "r:Rv",
 
60
                        [ "repo=", "recuirsive", "verbose", "help" ] )
54
61
                for opt, optarg in opts:
55
62
                        if opt in [ '--repo', '-r' ]:
56
63
                                if not re.search( '^[-a-zA-z0-9.]+$', optarg ):
57
64
                                        raise the.program.FatalError(
58
65
                                                'invalid repository name: ' + optarg )
59
66
                                the.repo = optarg
 
67
                        elif opt in [ '--recursive', '-R' ]:
 
68
                                self.recursive = True
60
69
                        elif opt in [ '--verbose', '-v' ]:
61
70
                                the.verbose += 1
62
71
                        elif opt == "--help":
79
88
                the.repo.check_dir_exists()
80
89
 
81
90
                # determine files
82
 
                files = self.expand_home_dir_files( self.files )
 
91
                files = self.expand_home_files( self.files, self.recursive )
83
92
 
84
93
                # initialise deployment and ensure it's not ongoing
85
94
                deployment = Deployment()
93
102
                                ( the.repo.name, the.program.name ) )
94
103
 
95
104
                # check status
96
 
                walker = CopyInWalker( files )
 
105
                walker = CopyInWalker( files, True )
97
106
                walker.walk()
98
107
 
99
108
                # make sure all files are added to version control
105
114
                the.repo.vcs.commit()
106
115
 
107
116
 
 
117
 
108
118
        @staticmethod
109
 
        def expand_home_dir_files( files ):
 
119
        def expand_home_files( files, recurse ):
110
120
                """Returns a unique, sorted list of relative filenames, calculated from the list
111
121
                provided, which is made up from individual files and directories
112
 
                relative to the CWD.  Directories are not recursed in to, and leading
113
 
                path components are not returned.  Specified filenames must exist in the
114
 
                home directory (although they may not exist in the repo).
 
122
                relative to the CWD.  As directories are found, they may optionally be
 
123
                recursed in to.  Leading path components of any items are alse returned.
 
124
                Specified filenames must exist in the home directory (although they do
 
125
                not have to exist in the repo).
 
126
 
 
127
                This is a modified version of command.expand_files().
115
128
 
116
129
                """
117
130
 
120
133
 
121
134
                # iterate through file list
122
135
                for file in files:
123
 
                        parts = os.path.split( file )
124
 
                        abs_file = os.path.join(
125
 
                                os.path.realpath( parts[ 0 ] ), parts[ 1 ] )
126
 
 
127
 
                        # check the file is in the home directory
128
 
                        if abs_file[ : len( home_dir_prefix ) ] != home_dir_prefix:
129
 
                                raise the.program.FatalError(
130
 
                                        'not under home directory: %s' % abs_file )
131
 
 
132
 
                        # relative file
133
 
                        rel_file = abs_file[ len( home_dir_prefix ) : ]
134
 
 
135
 
                        # check if file exists in home directory
136
 
                        if not os.path.lexists( abs_file ):
137
 
                                raise the.program.FatalError(
138
 
                                        'not found in home directory: %s' % rel_file )
139
 
 
140
 
                        # # append path parts
141
 
                        # path = rel_file
142
 
                        # while True:
143
 
                        #       path, dummy = os.path.split( path )
144
 
                        #       if len( path ): ret.add( path )
145
 
                        #       else: break
 
136
                        ( rel_file, abs_file ) = Command.resolve_homedir_file( file )
 
137
 
 
138
                        # check that file exists in homedir
 
139
                        if not os.path.exists( abs_file ):
 
140
                                raise the.program.FatalError(
 
141
                                        'not found in home directory: %s' % file )
 
142
 
 
143
                        # ensure parent path parts are included
 
144
                        path = rel_file
 
145
                        while True:
 
146
                                path, dummy = os.path.split( path )
 
147
                                if len( path ): ret.add( path )
 
148
                                else: break
146
149
 
147
150
                        # append the file or directory
148
 
                        ret.add( rel_file )
 
151
                        ret.update( Walker.generate_walk_list(
 
152
                                the.full_home_dir, rel_file, recurse ) )
149
153
 
150
154
                return sorted( ret )