beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 15:40:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

Meta: Allow serenity.sh to be used outside of assumed working directory

Browse source 
This change fixes two bugs:

1) If you run `serenity.sh` outside of your serenity git clone, the
   `get_top_dir()` function won't be able to auto detect the serenity
    root dir and we'll error out. By allowing the script to use an
    existing $SERENITY_ROOT vlaue if it exists, we can solve
    this problem.

2) If tried to run a command which ends up calling `build_toolchain()`
   and you were outside of the root of the project, it should just
   fail. Fix this by utilizing `$SERENITY_ROOT`.
This commit is contained in:
Brian Gianforcaro 2021-04-16 18:08:48 -07:00 committed by Linus Groh
parent 127fe38612
commit 253114e3b1
Notes: sideshowbarker 2024-07-18 19:32:15 +09:00 
Author: https://github.com/bgianfo
Commit: https://github.com/SerenityOS/serenity/commit/253114e3b1d
Pull-request: https://github.com/SerenityOS/serenity/pull/6392
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
Meta/serenity.sh
View file

@ -104,8 +104,11 @@ create_build_dir() {
cmd_with_target() {
is_valid_target || ( >&2 echo "Unknown target: $TARGET"; usage )
SERENITY_ROOT="$(get_top_dir)"
export SERENITY_ROOT
if [ ! -d "$SERENITY_ROOT" ]; then
SERENITY_ROOT="$(get_top_dir)"
export SERENITY_ROOT
fi
BUILD_DIR="$SERENITY_ROOT/Build/$TARGET"
if [ "$TARGET" != "lagom" ]; then
export SERENITY_ARCH="$TARGET"
@ -136,7 +139,7 @@ delete_target() {
}
build_toolchain() {
( cd Toolchain && ARCH="$TARGET" ./BuildIt.sh )
( cd "$SERENITY_ROOT/Toolchain" && ARCH="$TARGET" ./BuildIt.sh )
}
ensure_toolchain() {