Closed
Bug 686008
Opened 13 years ago
Closed 13 years ago
cfx xpi shouldn't include vim temporary files
Categories
(Add-on SDK Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: david, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
4.26 KB,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
As with bug 565176 but for vim users, I've noticed that vim temporary files are being included in xpis e.g. .panel.js.swp These files then cause warnings on amo, which is frustrating
Comment 1•13 years ago
|
||
Does this patch fix the problem? Is ".swp" the only suffix that vim creates?
Attachment #559572 -
Flags: review?(myk)
Comment 2•13 years ago
|
||
Comment on attachment 559572 [details] [diff] [review] ignore .swp files when building the XPI Code looks good, r=myk. David: anything else we should look out for?
Attachment #559572 -
Flags: review?(myk) → review+
Reporter | ||
Comment 3•13 years ago
|
||
Not that I'm aware of, but the AMO warning is as follows, so might be worth just excluding all hidden files (or speaking to the AMO guys): Hidden files and folders flagged Warning: Hidden files and folders difficult the review process and can contain sensitive information about the system that generated the XPI. Please modify the packaging process so that these files aren't included. resources/jid0-iwogeuibasgv3uv3pial5mthuvq-at-jetpack-tubecheckinfo-data/.panel.js.swp
Comment 4•13 years ago
|
||
Hmm, I wouldn't want to difficult the review process (unnecessarily), so I'm game to exclude hidden files generally.
Comment 5•13 years ago
|
||
by "hidden" I'll assume they mean anything starting with a period. Here's a new patch which hides those too, factors the two filtering routines (XPI generation and data/ directory scanning) into a single utility function, and adds a quick test.
Attachment #559572 -
Attachment is obsolete: true
Attachment #559593 -
Flags: review?(myk)
Comment 6•13 years ago
|
||
Comment on attachment 559593 [details] [diff] [review] ignore all hidden files, refactor utility routine, add tests >+IGNORED_DIRS = [".svn", ".hg", "defaults"] .git?
Attachment #559593 -
Flags: review?(myk) → review+
Comment 7•13 years ago
|
||
Landed in https://github.com/mozilla/addon-sdk/commit/d98f1d0af8bdb97e3c8af2b7f1a80f027fc67126 . I took the liberty of factoring out the filter_dirnames() function too, and included .git in that list. (.git doesn't usually matter because .git is only at the top-level, and the only directories that get copied are under data/ , so filter_dirnames() is really just for revision-control systems that spread their metadata out in each directory like SVN).
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•