`setup.py install` should run make

NEW
Unassigned

Status

6 years ago
6 years ago

People

(Reporter: erik, Unassigned)

Tracking

Trunk
x86
macOS
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
Running setup.py install should build the Clang indexing plugin so it gets installed with the built package. This will make the built package actually work for indexing. At the moment, you have to run `make` manually and *then* do setup.py install, which is surprising.

Here are some ideas on how to implement this: https://github.com/mozilla/dxr/pull/110#issuecomment-19125266
(Reporter)

Comment 1

6 years ago
This is blocked on getting DXR deployed on a box that has LLVM 3.2 and can actually build modern things (not our old RHEL 5.5 box).
(Reporter)

Comment 2

6 years ago
When this is done, also remove the 2>/dev/null on the run of setup.py install in deploy.py.
You need to log in before you can comment on or make changes to this bug.