Remove old ToS generation code, once new ToS have rolled out and old links are obsolete

RESOLVED FIXED in Firefox 40

Status

defect
P2
normal
Rank:
25
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
mozilla39
Points:
2
Dependency tree / graph
Bug Flags:
firefox-backlog +
qe-verify -

Firefox Tracking Flags

(firefox40 fixed)

Details

(Whiteboard: [tech-debt])

Attachments

(1 attachment)

Assignee

Description

5 years ago
Bug 1105488 is switching the ToS url away from the loop-client server.

This means we should be able to clean up our code at some stage. I suspect we need:

- File a bug on ops to make the existing ToS url a redirect (for FF 34 users)
- Remove the old generation code (not sure if we want to keep Grunt for some of the perf improvements we want to do).
backlog: --- → Fx38?

Updated

5 years ago
Assignee: nobody → standard8
backlog: Fx38? → tech-debt
Priority: -- → P2
Assignee

Updated

5 years ago
Depends on: 1120856
Assignee

Comment 1

4 years ago
Ops has asked me to make at least part of this happen as it helps with some new automation they are creating. The initial change I did to test for them wasn't too difficult, so lets just see if we can knock this off before the next release.
Iteration: --- → 40.1 - 13 Apr
Points: --- → 2
Rank: 25
Flags: qe-verify-
Flags: firefox-backlog+
Assignee

Updated

4 years ago
OS: Mac OS X → All
Hardware: x86 → All
Assignee

Comment 2

4 years ago
This is basically a backout of bug 1044411 but keeping some bits intact as they seemed good to keep.

Ted: Could you review the .hgignore parts? (the Makefile isn't part of the m-c build system atm).

Dan: Ops want this for the next loop-client release at it makes their process simpler and easier to automate. We can always add grunt back in later if we need it for the minify/concatinate stuff, but for now, lets just drop it and simplify things.
Attachment #8585397 - Flags: review?(ted)
Attachment #8585397 - Flags: review?(dmose)
Attachment #8585397 - Flags: review?(ted) → review+
Comment on attachment 8585397 [details] [diff] [review]
Remove old ToS generation code from Loop Standalone. NPOTB DONTBUILD

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

Looks great; thanks for doing this.  I don't think we're going to want to go back to grunt.  The lesson I learned with this stuff is that debugging grunt is a giant pain.  We might consider gulp one day, but perhaps Make will continue to be sufficient.
Attachment #8585397 - Flags: review?(dmose) → review+
Assignee

Comment 4

4 years ago
https://hg.mozilla.org/integration/fx-team/rev/d7e29ec6a512
Target Milestone: --- → mozilla39
https://hg.mozilla.org/mozilla-central/rev/d7e29ec6a512
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.