migrate loop webclient standalone Makefile infrastructure to Grunt

RESOLVED INVALID

Status

RESOLVED INVALID
4 years ago
4 years ago

People

(Reporter: dmose, Unassigned)

Tracking

unspecified
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [tech-debt])

(Reporter)

Description

4 years ago
In bug 1044411, grunt and bower were introduced to build localization stuff in the .  We get significant network effect benefits (being able to easily use and update packages and tasks in npm and bower.io).  For now, we still have the Makefile infrastructure around, which leaves us with more complexity and maintainability than we'd like.  We should port the stuff currently done by the Makefile to Grunt and eventually kill the Makefile.
(Reporter)

Updated

4 years ago
Depends on: 1044411
Shouldn't we first move the standalone client to live outside of m-c to open a whole range of possibilities?

Else we should wait on having NodeJS available on the build infra.
(Reporter)

Comment 2

4 years ago
I should have mentioned here what I added as a comment to the Makefile itself during review: this Makefile doesn't get used by the Mozilla invoked by the build infrastructure at all.  It's only used for development and deployment at the moment.  You're quite right that the full benefits don't accrue until later, but I think that's somewhat orthogonal to the maintainability improvements that this cleanup offers.
Whiteboard: [tech-debt]

Updated

4 years ago
backlog: --- → Fx36+

Updated

4 years ago
backlog: Fx36+ → Fx37+

Updated

4 years ago
backlog: Fx37+ → backlog
(Reporter)

Comment 3

4 years ago
Enough experience with Grunt suggests that debugging it is so painful that it offsets the benefits.  We discussed this at retrospective a while back, and there's no plan to invest further in Grunt.  We should probably remove it once we stop depending it on for the ToS stuff.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.