-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge upstream #4593
base: main
Are you sure you want to change the base?
Merge upstream #4593
Changes from all commits
fd6b12c
9192d29
db6b1b2
2a0634d
d5148ac
5feace1
22f5d62
8738ac1
15d8f99
31d3600
c2ebdb7
dba71a4
e7ee31c
815cc34
f9234b3
2285ed9
233e272
56680d6
f1e0ece
4adf2fa
5983ec1
07acf63
59f6994
21e277d
7642b2d
73db7ba
33c8ce6
4323624
a68133f
5c769b3
6cecd22
5e3d67e
8123869
64641d7
27bcd8e
5adc477
6a053ec
378f87d
9c7ba88
149062f
19ae28c
fde87ba
c111b71
52c6c11
8157035
b32ca19
907b099
a645526
90b7cc6
bca4cc0
62e6543
9559ca1
dfdb6a5
525bc7f
6dd5f45
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,3 +15,4 @@ | |
node_modules | ||
.idea | ||
.direnv | ||
.vscode/extensions.json | ||
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -12,7 +12,7 @@ | |||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = travis_esp8266, travis_esp32 | ||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||
# Release binaries | ||||||||||||||||||||||||||||||||||||||||||||||||
default_envs = nodemcuv2, esp8266_2m, esp01_1m_full, esp32dev, esp32_eth, esp32s2_saola, esp32c3 | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = nodemcuv2, esp8266_2m, esp01_1m_full, esp32dev, esp32_eth | ||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||
# Build everything | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = esp32dev, esp8285_4CH_MagicHome, codm-controller-0.6-rev2, codm-controller-0.6, esp32s2_saola, d1_mini_5CH_Shojo_PCB, d1_mini, sp501e, travis_esp8266, travis_esp32, nodemcuv2, esp32_eth, anavi_miracle_controller, esp07, esp01_1m_full, m5atom, h803wf, d1_mini_ota, heltec_wifi_kit_8, esp8285_H801, d1_mini_debug, wemos_shield_esp32, elekstube_ips | ||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -28,7 +28,7 @@ default_envs = nodemcuv2, esp8266_2m, esp01_1m_full, esp32dev, esp32_eth, esp32s | |||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = h803wf | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = d1_mini_debug | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = d1_mini_ota | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = esp32dev | ||||||||||||||||||||||||||||||||||||||||||||||||
default_envs = esp32dev | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = esp8285_4CH_MagicHome | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = esp8285_H801 | ||||||||||||||||||||||||||||||||||||||||||||||||
; default_envs = d1_mini_5CH_Shojo_PCB | ||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -115,8 +115,11 @@ build_flags = | |||||||||||||||||||||||||||||||||||||||||||||||
-D DECODE_LG=true | ||||||||||||||||||||||||||||||||||||||||||||||||
-DWLED_USE_MY_CONFIG | ||||||||||||||||||||||||||||||||||||||||||||||||
; -D USERMOD_SENSORSTOMQTT | ||||||||||||||||||||||||||||||||||||||||||||||||
-fexceptions | ||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||
build_unflags = | ||||||||||||||||||||||||||||||||||||||||||||||||
-fno-exceptions | ||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+118
to
+122
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Contradictory exception handling flags You've added both This configuration may lead to unexpected behavior during builds. PlatformIO applies build_unflags before build_flags, so the final result would be enabling exceptions. If that's your intention, you should remove one of these flags to avoid confusion: build_flags =
-DMQTT_MAX_PACKET_SIZE=1024
-DSECURE_CLIENT=SECURE_CLIENT_BEARSSL
-DBEARSSL_SSL_BASIC
-D CORE_DEBUG_LEVEL=0
-D NDEBUG
#build_flags for the IRremoteESP8266 library (enabled decoders have to appear here)
-D _IR_ENABLE_DEFAULT_=false
-D DECODE_HASH=true
-D DECODE_NEC=true
-D DECODE_SONY=true
-D DECODE_SAMSUNG=true
-D DECODE_LG=true
-DWLED_USE_MY_CONFIG
; -D USERMOD_SENSORSTOMQTT
-fexceptions
build_unflags =
- -fno-exceptions 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||
# enables all features for travis CI | ||||||||||||||||||||||||||||||||||||||||||||||||
build_flags_all_features = | ||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -148,9 +151,9 @@ framework = arduino | |||||||||||||||||||||||||||||||||||||||||||||||
board_build.flash_mode = dout | ||||||||||||||||||||||||||||||||||||||||||||||||
monitor_speed = 115200 | ||||||||||||||||||||||||||||||||||||||||||||||||
# slow upload speed (comment this out with a ';' when building for development use) | ||||||||||||||||||||||||||||||||||||||||||||||||
upload_speed = 115200 | ||||||||||||||||||||||||||||||||||||||||||||||||
; upload_speed = 115200 | ||||||||||||||||||||||||||||||||||||||||||||||||
# fast upload speed (remove ';' when building for development use) | ||||||||||||||||||||||||||||||||||||||||||||||||
; upload_speed = 921600 | ||||||||||||||||||||||||||||||||||||||||||||||||
upload_speed = 921600 | ||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||
# ------------------------------------------------------------------------------ | ||||||||||||||||||||||||||||||||||||||||||||||||
# LIBRARIES: required dependencies | ||||||||||||||||||||||||||||||||||||||||||||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Conflicting ignore settings for extensions.json
The newly added line ignores
.vscode/extensions.json
, but line 6 has!.vscode/extensions.json
which explicitly includes this file. These directives contradict each other, causing ambiguous behavior.Apply this diff to resolve the conflict:
Alternatively, if the intent is to include extensions.json in version control, remove line 18 instead.