Commit 51ce337
authored
Rearrange JS dependencies so we can build parseq-tracevis with Python 3+ (#337)
Pinning Python to 2.7 is not working anymore since GitHub dropped support for
Python 2.7. We originally pinned it to avoid an issue where `node-gyp` fails
when combined with Python 3.x during `npm install`. The reasons for this failure
are:
- `d3` depends either directly or transitively on `contextify`, a _really_ old
library that needs native code from V8 to install.
- `contextify` builds native code via `node-gyp`.
- The version of `node-gyp` used does not play well with Python 3.x.
A proper path forward would be to update and modernize the entire JavaScript
stack used in `parseq-tracevis` -- which is a non-trivial amount of work.
As a workaround, we can use a pre-built version of `d3`. A local copy of `d3`
v3.x has been added to the local repository, and the dependency on `d3` has been
removed from `package.json`. Doing this avoids the need for building native
code, and thus the need for `node-gyp`. This allows us to build using Python 3.x
because now there is no conflict.1 parent 3e6e50b commit 51ce337
6 files changed
Lines changed: 9 additions & 11 deletions
File tree
- .github/workflows
- subprojects/parseq-tracevis
- js
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
22 | | - | |
23 | | - | |
24 | 21 | | |
25 | 22 | | |
26 | 23 | | |
27 | 24 | | |
28 | 25 | | |
29 | | - | |
| 26 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
26 | | - | |
27 | | - | |
28 | 25 | | |
29 | 26 | | |
30 | 27 | | |
31 | | - | |
| 28 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
136 | 136 | | |
137 | 137 | | |
138 | 138 | | |
139 | | - | |
| 139 | + | |
140 | 140 | | |
141 | 141 | | |
142 | 142 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
17 | 16 | | |
18 | 17 | | |
19 | 18 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
86 | 86 | | |
87 | 87 | | |
88 | 88 | | |
89 | | - | |
| 89 | + | |
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
| |||
0 commit comments