Closed Bug 1229199 Opened 6 years ago Closed 5 years ago

"mach eslint --setup" bails at eslint installation, even when eslint is already installed (in a system directory, where I lack write privileges)

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox45 affected)

RESOLVED WORKSFORME
Tracking Status
firefox45 --- affected

People

(Reporter: dholbert, Unassigned)

Details

STR:
 1. Follow STR from bug 1229194.

 2. When "./mach eslint --setup" fails, note the npm command that it's trying to run...
    /usr/bin/npm install eslint -g
 ...and run that command using "sudo", e.g.:
    sudo /usr/bin/npm install eslint -g

 3. Try to run ./mach eslint --setup again.

ACTUAL RESULTS:
Same failure as in bug 1229194 -- namely, the output starts with:
{
Installing eslint using "/usr/bin/npm install eslint -g"...
npm ERR! error rolling back Error: EACCES, unlink '/usr/local/bin/eslint'
[...]
}
...even though I already have eslint installed.

EXPECTED RESULTS:
Since I already installed eslint (using "sudo" as required), we should be able to skip past that part of the setup step, to whatever's supposed to happen next (e.g. downloading eslint-plugin-mozilla).
Summary: "mach eslint --setup" bails at eslint installation, even if eslint is already installed → "mach eslint --setup" bails at eslint installation, if eslint is already installed in a system directory (where I lack write privileges)
Summary: "mach eslint --setup" bails at eslint installation, if eslint is already installed in a system directory (where I lack write privileges) → "mach eslint --setup" bails at eslint installation, even when eslint is already installed (in a system directory, where I lack write privileges)
I believe this isn't an issue now that bug 1229194 is fixed. Running ./mach eslint --setup multiple times works for me.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.