diff options
author | hisham <hisham@9ca3f7c1-7366-0410-b1a3-b5c78f85698c> | 2009-10-05 14:45:21 +0000 |
---|---|---|
committer | hisham <hisham@9ca3f7c1-7366-0410-b1a3-b5c78f85698c> | 2009-10-05 14:45:21 +0000 |
commit | d43d6e738d618fc8e7ddcb905b29f97e2ee0f9a4 (patch) | |
tree | 8cf713363f0ec7046978213d47b0be688cbb0634 | |
parent | 230da1deaefff2404e0ea4aa380c2e7cda306733 (diff) | |
download | luarocks-d43d6e738d618fc8e7ddcb905b29f97e2ee0f9a4.tar.gz luarocks-d43d6e738d618fc8e7ddcb905b29f97e2ee0f9a4.tar.bz2 luarocks-d43d6e738d618fc8e7ddcb905b29f97e2ee0f9a4.zip |
code cleanup
git-svn-id: http://luarocks.org/svn/luarocks/trunk@64 9ca3f7c1-7366-0410-b1a3-b5c78f85698c
-rw-r--r-- | src/luarocks/rep.lua | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/luarocks/rep.lua b/src/luarocks/rep.lua index 8d3ecfb2..eaedb991 100644 --- a/src/luarocks/rep.lua +++ b/src/luarocks/rep.lua | |||
@@ -210,11 +210,9 @@ local function resolve_conflict(name, version, target) | |||
210 | return nil, cversion | 210 | return nil, cversion |
211 | end | 211 | end |
212 | if name ~= cname or deps.compare_versions(version, cversion) then | 212 | if name ~= cname or deps.compare_versions(version, cversion) then |
213 | print("MOVE EXISTING, MAKE WAY FOR NEW") | ||
214 | fs.move(target, path.versioned_name(target, cname, cversion)) | 213 | fs.move(target, path.versioned_name(target, cname, cversion)) |
215 | return target | 214 | return target |
216 | else | 215 | else |
217 | print("INSTALL NEW WITH DIFFERENT NAME") | ||
218 | return path.versioned_name(target, name, version) | 216 | return path.versioned_name(target, name, version) |
219 | end | 217 | end |
220 | end | 218 | end |
@@ -236,17 +234,17 @@ function deploy_files(name, version) | |||
236 | return nil, "Could not create "..deploy_dir | 234 | return nil, "Could not create "..deploy_dir |
237 | end | 235 | end |
238 | for file, sub in pairs(file_tree) do | 236 | for file, sub in pairs(file_tree) do |
237 | local source = dir.path(source_dir, file) | ||
239 | local target = dir.path(deploy_dir, file) | 238 | local target = dir.path(deploy_dir, file) |
240 | if type(sub) == "table" then | 239 | if type(sub) == "table" then |
241 | ok, err = deploy_file_tree(sub, dir.path(source_dir, file), dir.path(deploy_dir, file)) | 240 | ok, err = deploy_file_tree(sub, source, target) |
242 | if not ok then return nil, err end | 241 | if not ok then return nil, err end |
243 | fs.remove_dir_if_empty(target) | 242 | fs.remove_dir_if_empty(source) |
244 | else | 243 | else |
245 | if fs.exists(target) then | 244 | if fs.exists(target) then |
246 | target, err = resolve_conflict(name, version, target) | 245 | target, err = resolve_conflict(name, version, target) |
247 | if err then return nil, err.." Cannot install new version." end | 246 | if err then return nil, err.." Cannot install new version." end |
248 | end | 247 | end |
249 | local source = dir.path(source_dir, file) | ||
250 | ok, err = move_fn(source, target) | 248 | ok, err = move_fn(source, target) |
251 | if not ok then return nil, err end | 249 | if not ok then return nil, err end |
252 | end | 250 | end |
@@ -281,7 +279,7 @@ function delete_version(name, version) | |||
281 | for file, sub in pairs(file_tree) do | 279 | for file, sub in pairs(file_tree) do |
282 | local target = dir.path(deploy_dir, file) | 280 | local target = dir.path(deploy_dir, file) |
283 | if type(sub) == "table" then | 281 | if type(sub) == "table" then |
284 | local ok, err = delete_deployed_file_tree(sub, dir.path(deploy_dir, file)) | 282 | local ok, err = delete_deployed_file_tree(sub, target) |
285 | fs.remove_dir_if_empty(target) | 283 | fs.remove_dir_if_empty(target) |
286 | else | 284 | else |
287 | local versioned = path.versioned_name(target, name, version) | 285 | local versioned = path.versioned_name(target, name, version) |