Closed Bug 482733 Opened 11 years ago Closed 11 years ago should support copying directories recursively


(Firefox Build System :: General, defect)

Not set


(Not tracked)



(Reporter: ted, Assigned: ted)



(Keywords: fixed1.9.1)


(3 files, 1 obsolete file)

Apparently the in-tree nsinstall.c gained the ability to recursively copy directories sometime back in 2001. does not have this ability, but it should.
This just refactors into a main nsinstall() method, and adds some simple unit tests. I'll actually fix the bug in a followup patch.
Attachment #366869 - Flags: review?(l10n)
Added a unittest for -m (not on windows), and got rid of the Windows line endings. (ew)
Attachment #366869 - Attachment is obsolete: true
Attachment #366917 - Flags: review?(l10n)
Attachment #366869 - Flags: review?(l10n)
Ok, actual bugfix, with a new unit test to test it.
Attachment #366919 - Flags: review?(l10n)
Comment on attachment 366917 [details] [diff] [review]
refactor + unittets, rev 2

I know that hg is weak on that, but is a hg diff -w or -b easier to review than this one?
Attachment #366917 - Flags: review?(l10n) → review+
Comment on attachment 366919 [details] [diff] [review]
fix to copy directories recursively

r+ with nits. Can you rename files to entries? copy_all_files feels strange when you're copying dirs, too.

Same for the variable names inside the loops?
Attachment #366919 - Flags: review?(l10n) → review+
Pushed to m-c:
Closed: 11 years ago
Resolution: --- → FIXED
And of course I forgot to fix the copy in js/src:
Blocks: 483800
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.