Skip to content
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.

Sign up for GitHub

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

Jump to bottom

#warning "**Y2038** time_t is only 32-bits in this build configuration stops the build for esp32 due to -Werror=cpp #2784

Closed
xeonqq opened this issue Jun 1, 2024 · 5 comments
Closed

#warning "**Y2038** time_t is only 32-bits in this build configuration stops the build for esp32 due to -Werror=cpp #2784

xeonqq opened this issue Jun 1, 2024 · 5 comments

Comments

@xeonqq
Copy link
Contributor

xeonqq commented Jun 1, 2024

When using the lastest sming in docker and try to build esp32
I have the following error:
Screenshot from 2024-06-01 21-11-29

Have to delete that line from DataTime.cpp to make it work, I don't know where this -Werror=cpp is defined. Or if this warning is necessary.
@mikee47
Copy link
Contributor

mikee47 commented Jun 1, 2024

See issue #2758.

  1. Build with STRICT=1 to allow compilation to continue
  2. Fix the problem. i.e. you're using an old IDF version (4.3 or 4.4)

@xeonqq
Copy link
Contributor Author

xeonqq commented Jun 1, 2024

I just rebuild the docker image with the Dockerfile, cuz I want the new websocket fix. Shouldn't it pull the latest IDF version automatically for me?

@mikee47
Copy link
Contributor

mikee47 commented Jun 1, 2024

Ah, I see. The default IDF installed is version 4.4, we should probably update that to 5.2.

@mikee47
Copy link
Contributor

mikee47 commented Jun 1, 2024

For now, you can modify the dockerfile as per https://sming.readthedocs.io/en/latest/_inc/Sming/Arch/Esp32/README.html#idf-versions:

RUN INSTALL_IDF_VER=5.2 Tools/install.sh $INSTALL_ARGS

This was referenced Jun 1, 2024
Closed
Merged
slaff pushed a commit that referenced this issue Jun 3, 2024
@mikee47
Upgrade toolchains (#2787)
4753ef4 
This PR updates the embedded toolchains to more recent versions, setting the new default installed version for Esp32 to IDF 5.2. The library testing github actions script has been updated to include missing esp32c2 architecture and test against IDF 4.4 and 5.2.

**Esp32 IDF SDK**

The installation scripts now install IDF version 5.2 by default (see issue #2784 regarding docker build failure because of Y2038 warning with IDF < 5).
Deprecation warnings printed for IDF 4.3 and 4.4 after final link step. See #2732 for discussion regarding old versions.
Windows CI tests for IDF 4.3, 5.0 removed.

**Extend library tests to include missing esp32 variants**

CI testing when developing libraries is much more efficient than running the main Sming CI tests.
Looks like these haven't been updated for a while so have added the `esp32s3` and `esp32c2` variants
and testing against IDF 5.2 for all esp32 variants. Previously was only 4.4 (old default).

**Esp8266 Toolchain**

The installer has been updated to use the latest toolchain (Feb 23), gcc 10.3.

**RP2040 Toolchain**

The installer has been updated to use the latest toolchain (Oct 23), gcc 13.2.
@mikee47
Copy link
Contributor

mikee47 commented Jul 3, 2024

Warning now removed.

@mikee47 mikee47 closed this as completed Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mikee47 @xeonqq