Skip to content

Commit

Permalink
Tweaked changeprefix.py to not rename dir component in paths
Browse files Browse the repository at this point in the history
This wasn't implemented correctly anyways, as it would need to recursively
rename directories that may not exist. Things would also get a bit
complicated if only some files in a directory were renamed.

Doable, but not needed for our use case.

For now just ignore any directory components. Though this may be worth
changing if the source directory structure becomes more complicated in
the future (maybe with a -r/--recursive flag?).
  • Loading branch information
geky committed Apr 19, 2023
1 parent ba1c764 commit f0cc1db
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion scripts/changeprefix.py
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,10 @@ def main(from_prefix, to_prefix, paths=[], *,
elif no_renames:
to_path = from_path
else:
to_path, _ = changeprefix(from_prefix, to_prefix, from_path)
to_path = os.path.join(
os.path.dirname(from_path),
changeprefix(from_prefix, to_prefix,
os.path.basename(from_path))[0])

# rename contents
changefile(from_prefix, to_prefix, from_path, to_path,
Expand Down

0 comments on commit f0cc1db

Please sign in to comment.