install-build-deps script #4505
Conversation
0cjs
force-pushed
the
dev/cjs/24g05/install-build-deps-script
branch
from
c6ae4a0
to
58842df
Compare
0cjs
force-pushed
the
dev/cjs/24g05/install-build-deps-script
branch
2 times, most recently
from
7e2073a
to
e5ea4e4
Compare
|
This has now been tested on a Mac and fixes applied to make that work properly. (I didn't look closely enough at the Mac code when I was testing on Debian.) It's also been rebased on to the top of the current |
Baltoli
requested review from
dwightguth,
ehildenb,
rv-jenkins,
F-WRunTime and
Baltoli
as code owners
|
@0cjs you will need to rebase this PR on the |
This removes the `apt-get install` and `brew install` instructions and lists of packages (and, for Debian, Haskell Stack as well) from the README.md file and moves them to a script that will determine which set to install and do the install. (This can easily be expanded to support other systems if necessary.) The improvement to developer convenience should be obvious. This also tweaks the bits of Markdown we touched to move the referenced URLs to the end of the file, to make it easier to read the plaintext. The script has been tested on Debian 12, Bash 3.2.48(3) on a Debian system and on MacOS (using some version of Bash 3).
0cjs
force-pushed
the
dev/cjs/24g05/install-build-deps-script
branch
from
a66e079
to
a566905
Compare
|
It's now rebased. BTW, I see no harm in you guys just doing this directly (since you're already changing the tracking ("base") branch of the developer's branch anyway. That would help speed things up. (It's reasonable to bail on it if there's any issues doing that, of course, and ask the original developer to fix conflicts or whatever.) |
Add an
install-build-depsscript to replace the manual commands in the README. See commit message(s) for details.WARNING: This commit is based on the commit in #4054; if you merge this it will merge that one too. (The script includes the
libunwind-devpackage added in that commit.)You'll probably want to do a quick check that this works on MacOS and update the commit message appropriately.