Skip to content

Commit

Permalink
ESP32: drop submodule and add script to get it as a clone and/or subm…
Browse files Browse the repository at this point in the history
…odule

works with either a submodule setup or a 'clone' for either type of dev
  • Loading branch information
davidbuzz authored and tridge committed Nov 2, 2021
1 parent 8f3dd40 commit 854266d
Show file tree
Hide file tree
Showing 6 changed files with 60 additions and 5 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -145,3 +145,4 @@ persistent.dat
dumpstack_*out
build.tmp.binaries/
tasklist.json
modules/esp_idf
3 changes: 0 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,3 @@
[submodule "modules/CrashDebug"]
path = modules/CrashDebug
url = https://github.com/adamgreen/CrashDebug
[submodule "modules/esp_idf"]
path = modules/esp_idf
url = https://github.com/espressif/esp-idf.git
2 changes: 1 addition & 1 deletion Tools/ardupilotwaf/esp32.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def bldpath(path):
env.DEFAULT_PARAMETERS = cfg.srcnode.abspath()+"/libraries/AP_HAL_ESP32/boards/defaults.parm"
print("USING DEFAULT_PARAMETERS:"+str(env.DEFAULT_PARAMETERS))

env.append_value('GIT_SUBMODULES', 'esp_idf')
#env.append_value('GIT_SUBMODULES', 'esp_idf')


def pre_build(self):
Expand Down
55 changes: 55 additions & 0 deletions Tools/scripts/esp32_get_idf.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
#!/bin/bash
# if you have modules/esp_idf setup as a submodule, then leave it as a submodule and switch branches
if [ ! -d modules ]; then
echo "this script needs to be run from the root of your repo, sorry, giving up."
exit 1
fi
echo `ls modules`
cd modules

if [ ! -d esp_idf ]; then
echo 'did not find modules/esp_idf folder, making it.' ;
mkdir -p -v esp_idf
else
echo 'found modules/esp_idf folder' ;
fi
echo "looking for submodule or repo..."
if [ `git submodule | grep esp_idf | wc | cut -c1-7` == '1' ]; then
echo "found real submodule, syncing"
../Tools/gittools/submodule-sync.sh >/dev/null
else
echo "esp_idf is NOT a submodule"

if [ ! `ls esp_idf/install.sh 2>/dev/null` ]; then
echo "found empty IDF, cloning"
# add esp_idf as almost submodule, depths uses less space
#git clone -b v4.2 --single-branch --depth 10 https://github.com/espressif/esp-idf.git esp_idf
git clone -b 'release/v4.2' https://github.com/espressif/esp-idf.git esp_idf
# check if we've got v4.2 checked out, only this version of esp_idf is tested and works?

fi
fi

echo "inspecting possible IDF... "
cd esp_idf
echo `git rev-parse HEAD`
# these are a selection of possible specific commit/s that represent v4.2 branch of the esp_idf
if [ `git rev-parse HEAD` == 'f370d5089f61ac39f183109b6b2908700cfe3b0a' ]; then
echo "IDF version 'release/4.2' found OK, great.";
elif [ `git rev-parse HEAD` == 'c40f2590bf759ff60ef122afa79b4ec04e7633d2' ]; then
echo "IDF version 'v4.2' found OK, great.";
else
echo "looks like an idf, but not v4.2 branch, trying to switch branch and reflect upstream";
../../Tools/gittools/submodule-sync.sh >/dev/null
git fetch ; git checkout -f release/v4.2

# retry same as above
echo `git rev-parse HEAD`
if [ `git rev-parse HEAD` == 'f370d5089f61ac39f183109b6b2908700cfe3b0a' ]; then
echo "IDF version 'release/4.2' found OK, great.";
elif [ `git rev-parse HEAD` == 'c40f2590bf759ff60ef122afa79b4ec04e7633d2' ]; then
echo "IDF version 'v4.2' found OK, great.";
fi
fi
cd ..

3 changes: 3 additions & 0 deletions libraries/AP_HAL_ESP32/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,10 @@ sudo pacman -S --needed gcc git make flex bison gperf python-pip cmake ninja cca
cd ardupilot
git submodule update --init --recursive
# ensure the idf exists in modules/esp_idf (supports pre-existing submodules and/or cloning it for you)
./Tools/scripts/esp32_get_idf.sh
cd modules/esp_idf
# use it:
./install.sh
unset IDF_PATH
source ./export.sh
Expand Down
1 change: 0 additions & 1 deletion modules/esp_idf
Submodule esp_idf deleted from 559504

0 comments on commit 854266d

Please sign in to comment.