LittleFS is being deleted with update 1.17.375

Wolfieeewolf commented 8 months ago

Somewhere between 1.17.366 and 1.17.375 the LittleFS files are being deleted.

Haven't had a chance to check what version it started from but could be #968 again

Wolfieeewolf commented 8 months ago

So far it appears to be an issue with the BK7231T build. Not having issues with the BK7231N build.

openshwprojects commented 8 months ago

Yes, that's it, growing binary size, we will adjust it later, and maybe also add a forced LittleFS download on OTA, I apologize for the inconvencience

Wolfieeewolf commented 4 months ago

Any progress on this??

I went back and looked at all the changes. Somewhere in 1.17.367 to 1.17.371 lays the problem. I can't pinpoint it more as the commits for 1.17.367 to 1.17.370 are not on Git Hub. I would have to find the source files for those changes and build them. Happy to do so just will take some time to make sure I have the right files.

I have listed the changes below. Somewhere in there is the problem I believe.

OBK_FLAG_DISCOVERY_DONT_MERGE_LIGHTS

Set only first channel by type in VCP tuyaMCU https://github.com/openshwprojects/OpenBK7231T_App/pull/863)

add a sunset/sunrise self test code - it seems it passes correctly

enable sunset events

fix

add sunset/sunrise windows self test

fix ofs

Adding sunrise_sunset to NTP

Wolfieeewolf commented 4 months ago

So just did some test builds

1.17.367 - Keeps LittleFS file 1.17.368 - Error in building file. this could be the issue - changes - add sunset/sunrise windows self test 1.17.369 - Build has the issue of deleting LittleFS - changes - enable sunset events

Build Log of 1.17.368

$ ./b.sh
APP_PATH=apps/OpenBK7231T_App
APP_NAME=OpenBK7231T_App
APP_VERSION=1.0.0
USER_CMD=
Calling original build_app.sh script
cd /cygdrive/c/Users/luppa/downloads/bk7231sdk/apps/OpenBK7231T_App/../../platforms/bk7231t/toolchain/../bk7231t_os && sh build.sh OpenBK7231T_App 1.0.0 bk7231t
From build.sh, variables are:
APP_BIN_NAME=OpenBK7231T_App
APP_VERSION=1.0.0
TARGET_PLATFORM=bk7231t
USER_CMD=
Start Compile
system:CYGWIN_NT-10.0-22631
removing .o files from our folders, and OUR .o and .d files from Debug
find: ‘../../../apps/OpenBK7231T_App/src/hal/*.c’: No such file or directory
find: ‘../../../apps/OpenBK7231T_App/src/libraries/*.c’: No such file or directory
find: ‘../../../apps/OpenBK7231T_App/src/libraries/Arduino-IRremote-mod/*.c’: No such file or directory
find: ‘../../../apps/OpenBK7231T_App/src/libraries/Arduino-IRremote-mod/src/*.c’: No such file or directory
find: ‘../../../apps/OpenBK7231T_App/src/libraries/Arduino-IRremote-mod/src/private/*.c’: No such file or directory
find: ‘../../../apps/OpenBK7231T_App/src/win32/*.c’: No such file or directory
find: ‘../../../apps/OpenBK7231T_App/src/win32/stubs/lwip/apps/*.c’: No such file or directory
not rm -rf ../tuya_common/src/gpio_test.o
not rm -rf ../tuya_common/src/driver/tuya_gpio.o ../tuya_common/src/driver/tuya_key.o ../tuya_common/src/driver/tuya_led.o ../tuya_common/src/driver/tuya_uart.o
find: ‘../../../components’: No such file or directory
not is ci build
make[1]: Entering directory '/cygdrive/c/Users/luppa/downloads/bk7231sdk/platforms/bk7231t/bk7231t_os'
make[2]: Entering directory '/cygdrive/c/Users/luppa/downloads/bk7231sdk/platforms/bk7231t/bk7231t_os'
echo prerequirement0
prerequirement0
===========================================================
Build OpenBK7231T_App
===========================================================
echo prerequirement1
prerequirement1
mkdir -p Debug/obj
mkdir -p Debug/bin
echo prerequirement2
prerequirement2
build ../../../apps/OpenBK7231T_App/src/debug_tuyaMCUsimulator.o
build ../../../apps/OpenBK7231T_App/src/new_cfg.o
build ../../../apps/OpenBK7231T_App/src/new_common.o
build ../../../apps/OpenBK7231T_App/src/new_ping.o
build ../../../apps/OpenBK7231T_App/src/new_pins.o
build ../../../apps/OpenBK7231T_App/src/rgb2hsv.o
build ../../../apps/OpenBK7231T_App/src/tiny_crc8.o
build ../../../apps/OpenBK7231T_App/src/user_main.o
build ../../../apps/OpenBK7231T_App/src/win_main.o
build ../../../apps/OpenBK7231T_App/src/win_main_scriptOnly.o
build ../../../apps/OpenBK7231T_App/src/win_stubs.o
build ../../../apps/OpenBK7231T_App/src/base64/base64.o
build ../../../apps/OpenBK7231T_App/src/bitmessage/bitmessage_read.o
build ../../../apps/OpenBK7231T_App/src/bitmessage/bitmessage_write.o
build ../../../apps/OpenBK7231T_App/src/cJSON/cJSON.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_channels.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_eventHandlers.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_if.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_main.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_newLEDDriver.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_newLEDDriver_colors.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_repeatingEvents.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_script.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_send.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_simulatorOnly.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_tasmota.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_tcp.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_test.o
build ../../../apps/OpenBK7231T_App/src/cmnds/cmd_tokenizer.o
build ../../../apps/OpenBK7231T_App/src/devicegroups/deviceGroups_read.o
build ../../../apps/OpenBK7231T_App/src/devicegroups/deviceGroups_util.o
build ../../../apps/OpenBK7231T_App/src/devicegroups/deviceGroups_write.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_adcButton.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_adcSmoother.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_battery.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_bl0937.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_bl0942.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_bl_shared.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_bmp280.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_bp1658cj.o
../../../apps/OpenBK7231T_App/src/driver/drv_bmp280.c: In function 'BMP280_Init':
../../../apps/OpenBK7231T_App/src/driver/drv_bmp280.c:47:10: warning: unused variable 'buff' [-Wunused-variable]
  uint8_t buff[4];
          ^
../../../apps/OpenBK7231T_App/src/driver/drv_bmp280.c: In function 'BMP280_OnEverySecond':
../../../apps/OpenBK7231T_App/src/driver/drv_bmp280.c:68:25: warning: passing argument 1 of 'BMP280_readTemperature' from incompatible pointer type
  BMP280_readTemperature(&g_temperature);  // read temperature
                         ^
In file included from ../../../apps/OpenBK7231T_App/src/driver/drv_bmp280.c:41:0:
../../../apps/OpenBK7231T_App/src/driver/BMP280.h:263:9: note: expected 'int32_t *' but argument is of type 'int *'
 uint8_t BMP280_readTemperature(int32_t *temp)
         ^
build ../../../apps/OpenBK7231T_App/src/driver/drv_bp5758d.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_bridge_driver.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_chargingLimit.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_cht8305.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_cse7766.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_ddp.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_debouncer.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_dht.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_dht_internal.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_doorSensorWithDeepSleep.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_gn6932.o
../../../apps/OpenBK7231T_App/src/driver/drv_doorSensorWithDeepSleep.c: In function 'DoorDeepSleep_OnEverySecond':
../../../apps/OpenBK7231T_App/src/driver/drv_doorSensorWithDeepSleep.c:65:2: warning: implicit declaration of function 'ota_progress' [-Wimplicit-function-declaration]
  if (ota_progress() >= 0) {
  ^
build ../../../apps/OpenBK7231T_App/src/driver/drv_ht16k33.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_httpButtons.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_hue.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_kp18058.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_kp18068.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_main.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_max72xx_clock.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_max72xx_internal.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_max72xx_single.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_mcp9808.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_ntp.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.o
build ../../../apps/OpenBK7231T_App/src/driver/drv_pt6523.o
../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.c: In function 'NTP_CalculateSunrise':
../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.c:164:2: warning: implicit declaration of function 'dusk2Dawn' [-Wimplicit-function-declaration]
  dusk2Dawn(&sun_data, SUNRISE_FLAG, outHour, outMinute, 0);
  ^
../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.c:164:23: error: 'SUNRISE_FLAG' undeclared (first use in this function)
  dusk2Dawn(&sun_data, SUNRISE_FLAG, outHour, outMinute, 0);
                       ^
../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.c:164:23: note: each undeclared identifier is reported only once for each function it appears in
../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.c: In function 'NTP_CalculateSunset':
../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.c:167:23: error: 'SUNSET_FLAG' undeclared (first use in this function)
  dusk2Dawn(&sun_data, SUNSET_FLAG, outHour, outMinute, 0);
                       ^
make[2]: *** [application.mk:808: ../../../apps/OpenBK7231T_App/src/driver/drv_ntp_events.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/cygdrive/c/Users/luppa/downloads/bk7231sdk/platforms/bk7231t/bk7231t_os'
make[1]: *** [Makefile:5: mp] Error 2
make[1]: Leaving directory '/cygdrive/c/Users/luppa/downloads/bk7231sdk/platforms/bk7231t/bk7231t_os'
make: *** [Makefile:6: all] Error 2

openshwprojects / OpenBK7231T_App

LittleFS is being deleted with update 1.17.375 #1010