search

maximkulkin / esp32-homekit-camera

Firmware for esp32-camera module to act as Apple Homekit IP camera
MIT License
394 stars 78 forks source link

Error while compiling #39

Closed Electric-90 closed 4 years ago

Electric-90 commented 4 years ago

I am using mingw32 on Windows as my development environment for esp-idf. I was able to successfully compile the LED example for esp32 from esp-homekit-demo but I am getting the following error while trying to compile esp32-homekit-camera.

I did use the --recursive flag while cloning esp-idf & esp32-homekit-camera repository. I also performed git submodule update --init --recursive after the repositories were cloned.

compile error

kenthinson commented 4 years ago

I believe I am having the same issue.

CC examples/esp32-homekit-camera/build/x264/x264-snapshot-20181221-2245-stable/common/base.o cc1: error: -Werror=format-truncation=: no option -Wformat-truncation= make[1]: *** [x264-snapshot-20181221-2245-stable/common/base.o] Error 1

GermanSheepDog commented 4 years ago

I got the same problems, please look here: https://github.com/maximkulkin/esp32-homekit-camera/issues/38 I commented out the line.

Electric-90 commented 4 years ago

@GermanSheepDog I tried commenting out #CFLAGS += -Wno-error=format-truncation= line and the x264 packages compiled successfully but then I got stuck at the following error:

image

EDIT :- It turns out that the esp-idf version I am currently using doesn't have esp_sntp.h file. I tried using esp-idf = v3.3, v3.1.6 but even those don't work .

HomeACcessoryKid commented 4 years ago

I got the same error when using the ESP-IDF = v3.3

when using v4.1-dev (which is what Max seems to be using) I got a whole pile of errors related to spi_xxx

I think that for settling these issues we need to agree on a common version of esp_idf.

My personal favour would be v3.3 since it is LTS but Max has the ultimate say in this.

I also would like to know his macOS, his Xcode and xtensa version preferences

Any choice is valid, as long as people know what to aim for…

And please include the sdkconfig in the repository (get rid of the wifi-password though)

my 2 cents, HacK

On 13 Oct 2019, at 20:29, Electric-90 notifications@github.com wrote:

@GermanSheepDog https://github.com/GermanSheepDog I tried commenting out #CFLAGS += -Wno-error=format-truncation= line and the x264 packages compiled successfully but then I got stuck at the following error:

https://user-images.githubusercontent.com/47690999/66720063-74a1de80-ee15-11e9-8519-c3798093becb.png — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/maximkulkin/esp32-homekit-camera/issues/39?email_source=notifications&email_token=AFOQOQR72IN7G3FYBI5EMM3QONSIVA5CNFSM4JAF2JA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBC4R2Y#issuecomment-541444331, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFOQOQU7RBXS6OOROE6SQEDQONSIVANCNFSM4JAF2JAQ.

maximkulkin commented 4 years ago

@HomeACcessoryKid A few people (including you) reported those problems with spi_xxx but surprisingly I have never seen those. Could you be more specific on what those errors are, which version of ESP-IDF you use and how do you build the project.

It feels that a lot of issues originate in different project settings, but I would prefer not to commit the whole sdkconfig as it would cement out settings that were never important and distract from what really important settings are. The way I do that (and recommend to anybody) is when you update an ESP-IDF, or esp32-homekit-camera, or any other component - you should start fresh: make clean ; rm sdkconfig After that, set values as described in README and it should work. I've done it with different ESP-IDF versions and different modules and it always worked for me.

As of supporting v3.3 (as being their LTS release) - it feels that v3.3 is the last in 3.x series and not surprisingly they promised to keep it for some time, but do you know what LTS in this even mean?

Electric-90 commented 4 years ago

Hi, @maximkulkin Which esp-idf version are you using ? I have tried v3.3 and v3.1.6 and in both I get the esp_sntp.h error mentioned above. I also tried make clean but still no success.
There is a lot of inconsistency of packages included in different versions of esp-idf. Thus, we should agree on a common version of esp-idf for further development.

EDIT :- Just tried this commit :- https://github.com/maximkulkin/esp32-homekit-camera/commit/4f66382baf2bc70398d2574ddd08eefd3d3ec3eb, it compiled successfully on esp-idf-v3.3. 👍