1168436 - STATIC_ROOT is undefined according to the less compiler

Status: RESOLVED INVALID

(MozReview Graveyard :: General, defect, P3)

Description

[Mauro Doglio [:mdoglio]]

When trying to use STATIC_ROOT as part of a less file path to include, ie

@import (reference) "@{STATIC_ROOT}rb/css/defs.less";

the less compiler raise the following error:

>pipeline.exceptions.CompilerError: NameError: variable @STATIC_ROOT is undefined

This happens not only when on mozreview, but also on third party extensions, like rbmotd.

Comment 1

[Mauro Doglio [:mdoglio]]

I looked for help on the reviewboard-dev mailing list, here is the thread:
https://groups.google.com/forum/#!topic/reviewboard-dev/81jTaw4hXkY

Comment 2

[:glob ✱]

@{STATIC_ROOT} appears to work now from within our extension.

Comment 3

[:glob ✱]

(In reply to Byron Jones ‹:glob› from comment #2)
> @{STATIC_ROOT} appears to work now from within our extension.

urgh, it works in our dev/test env, but not on production.

Comment 4

[Mark Côté [:mcote]]

Is this still an issue?  Haven't heard any mention of it in some time.

Comment 5

[:glob ✱]

it's been silent because we've been working around it.

that said i'm pretty sure i found, and fixed, the cause of this issue: the version of lessc installed on the reviewboard admin node was significantly older than what we use in development and test.

i fixed the lessc versioning issue during the reviewboard-fork work, but haven't verified that this issue has been resolved as it's low priority with a functional work around.

cleaning up the css definitions would be nice, let's leave this open.

Comment 6

[BMO Automation]

MozReview is now obsolete. Please use Phabricator instead. Closing this bug.