Closed Bug 730803 Opened 8 years ago Closed 8 years ago

Don't ship unnecessary files with test pilot

Categories

(Thunderbird :: General, defect)

defect
Not set

Tracking

(thunderbird12 fixed)

RESOLVED FIXED
Thunderbird 13.0
Tracking Status
thunderbird12 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

- bootstrap.js is something for working with jetpack, which we don't do yet.
- content/debug.html is just debug, and we don't need it in the release version of the xpi.

These will help with passing AMO review until we get these files to a better standard.
Attached patch The fixSplinter Review
I've also dropped status_mobile.html as we don't need that at the moment and it gives extra validation warnings that we don't need.
Attachment #601555 - Flags: review?(irving)
Comment on attachment 601555 [details] [diff] [review]
The fix

Review of attachment 601555 [details] [diff] [review]:
-----------------------------------------------------------------

Looks fine.
Attachment #601555 - Flags: review?(irving) → review+
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/bdeddfcc3da3
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Comment on attachment 601555 [details] [diff] [review]
The fix

[Triage Comment]
Taking to beta as I want to get the version of test pilot we're distributing, synced up with the version on AMO.
Attachment #601555 - Flags: approval-comm-beta+
Target Milestone: --- → Thunderbird 13.0
You need to log in before you can comment on or make changes to this bug.