Skip to content

Commit a4633fc

Browse files
authored
chore: Update dependencies and remove submodules (#10)
* chore: Update dependencies and remove submodules * fix ci and sa * Update to use 5.3 instead of 5.4 because of i2c driver conflict with esp32-camera * update to use latest esp32-camera with support for legacy i2c driver even on esp-idf 5.4 * fix actions to ensure submodules are checked out
1 parent 2578d92 commit a4633fc

File tree

12 files changed

+56
-27
lines changed

12 files changed

+56
-27
lines changed

.github/workflows/build.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,11 @@ jobs:
1111
- name: Checkout repo
1212
uses: actions/checkout@v4
1313
with:
14-
submodules: 'recursive'
14+
submodules: true
1515

1616
- name: Build Main
1717
uses: espressif/esp-idf-ci-action@v1
1818
with:
19-
esp_idf_version: v5.2.2
19+
esp_idf_version: release-v5.4
2020
target: esp32
2121
path: '.'

.github/workflows/package_main.yml

Lines changed: 26 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,43 +5,64 @@ on:
55
branches: [main]
66
release:
77
types: [published]
8+
workflow_dispatch:
89

910
jobs:
1011
build:
1112

1213
runs-on: ubuntu-latest
1314
continue-on-error: false
1415

16+
outputs:
17+
zipfile-id: ${{ steps.zip_step.outputs.artifact-id }}
18+
1519
steps:
1620
- name: Checkout repo
1721
uses: actions/checkout@v4
1822
with:
19-
submodules: 'recursive'
23+
submodules: true
2024

2125
- name: Build Main Code
2226
uses: espressif/esp-idf-ci-action@v1
2327
with:
24-
esp_idf_version: v5.2.2
28+
esp_idf_version: release-v5.4
2529
target: esp32
2630
path: '.'
2731

2832
- name: Upload Build Outputs
29-
uses: actions/upload-artifact@v3
33+
uses: actions/upload-artifact@v4
34+
id: zip_step
3035
with:
3136
name: build-artifacts
3237
path: |
38+
build/*.bin
39+
build/*.elf
3340
build/bootloader/bootloader.bin
3441
build/partition_table/partition-table.bin
35-
build/*.bin
42+
build/flasher_args.json
3643
build/flash_args
3744
3845
- name: Attach files to release
39-
uses: softprops/action-gh-release@v1
46+
uses: softprops/action-gh-release@v2
4047
if: ${{ github.event.release && github.event.action == 'published' }}
4148
with:
4249
files: |
4350
build/*.bin
51+
build/*.elf
4452
build/bootloader/bootloader.bin
4553
build/partition_table/partition-table.bin
54+
build/flasher_args.json
4655
build/flash_args
4756
57+
package:
58+
name: Package the binaries into an executables for Windows, MacOS, and Linux (Ubuntu)
59+
needs: build
60+
strategy:
61+
matrix:
62+
os: [windows-latest, macos-latest, ubuntu-latest]
63+
runs-on: ${{ matrix.os }}
64+
steps:
65+
- uses: esp-cpp/esp-packaged-programmer-action@v1.0.5
66+
with:
67+
zipfile-id: ${{ needs.build.outputs.zipfile-id }}
68+
programmer-name: 'camera-streamer_programmer'

.github/workflows/static_analysis.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,17 +9,15 @@ jobs:
99
steps:
1010
- name: Checkout repo
1111
uses: actions/checkout@v2
12-
with:
13-
submodules: 'recursive'
1412

1513
- name: Run static analysis
1614
uses: esp-cpp/StaticAnalysis@master
1715
with:
1816
# Do not build the project and do not use cmake to generate compile_commands.json
1917
use_cmake: false
2018

21-
# Use the 5.2 release version since it's what we build with
22-
esp_idf_version: release/v5.2
19+
# Use the 5.4 release version since it's what we build with
20+
esp_idf_version: release/v5.4
2321

2422
# (Optional) cppcheck args
25-
cppcheck_args: -i$GITHUB_WORKSPACE/components/espp -i$GITHUB_WORKSPACE/components/esp32-camera -i$GITHUB_WORKSPACE/components/esp-protocols --force --enable=all --inline-suppr --inconclusive --platform=mips32 --std=c++17 --suppressions-list=$GITHUB_WORKSPACE/suppressions.txt
23+
cppcheck_args: --force --enable=all --inline-suppr --inconclusive --platform=mips32 --std=c++17 --suppressions-list=$GITHUB_WORKSPACE/suppressions.txt

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,3 +35,4 @@ sdkconfig
3535
sdkconfig.old
3636
.DS_Store
3737
dependencies.lock
38+
managed_components/

.gitmodules

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,3 @@
1-
[submodule "components/espp"]
2-
path = components/espp
3-
url = git@github.com:esp-cpp/espp
41
[submodule "components/esp32-camera"]
52
path = components/esp32-camera
6-
url = git@github.com:espressif/esp32-camera
7-
[submodule "components/esp-protocols"]
8-
path = components/esp-protocols
9-
url = git@github.com:espressif/esp-protocols
3+
url = git@github.com:esp-cpp/esp32-camera

CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
# The following lines of boilerplate have to be in your project's CMakeLists
22
# in this exact order for cmake to work correctly
3-
cmake_minimum_required(VERSION 3.5)
3+
cmake_minimum_required(VERSION 3.20)
44

55
include($ENV{IDF_PATH}/tools/cmake/project.cmake)
66

77
# add the component directories that we want to use
88
set(EXTRA_COMPONENT_DIRS
9-
"components/espp/components"
10-
"components/esp-protocols/components/mdns"
119
)
1210

1311
add_compile_definitions(BOARD_HAS_PSRAM)

components/esp-protocols

Lines changed: 0 additions & 1 deletion
This file was deleted.

components/espp

Lines changed: 0 additions & 1 deletion
This file was deleted.

main/idf_component.yml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
## IDF Component Manager Manifest File
2+
dependencies:
3+
## Required IDF version
4+
idf:
5+
version: '>=5.0'
6+
espressif/mdns: '>=1.8'
7+
# NOTE: we cannot use the idf component because for anything ESP-IDF >= 5.4 it
8+
# automatically uses the new I2C driver, which espp doesn't support yet.
9+
#
10+
# espressif/esp32-camera: '>=2.0'
11+
espp/rtsp: '>=1.0'
12+
espp/wifi: '>=1.0'
13+
espp/monitor: '>=1.0'
14+
espp/socket: '>=1.0'
15+
espp/nvs: '>=1.0'
16+
espp/task: '>=1.0'
17+
espp/esp32-timer-cam: '>=1.0'

0 commit comments

Comments
 (0)