Skip to content

Commit

Permalink
change(version): Update version to 5.1.2
Browse files Browse the repository at this point in the history
  • Loading branch information
AdityaHPatwardhan committed Nov 10, 2023
1 parent 9f2a2db commit 482a8fb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion components/esp_common/include/esp_idf_version.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ extern "C" {
/** Minor version number (x.X.x) */
#define ESP_IDF_VERSION_MINOR 1
/** Patch version number (x.x.X) */
#define ESP_IDF_VERSION_PATCH 1
#define ESP_IDF_VERSION_PATCH 2

/**
* Macro to convert IDF version number into an integer
Expand Down
2 changes: 1 addition & 1 deletion tools/cmake/version.cmake
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
set(IDF_VERSION_MAJOR 5)
set(IDF_VERSION_MINOR 1)
set(IDF_VERSION_PATCH 1)
set(IDF_VERSION_PATCH 2)

set(ENV{IDF_VERSION} "${IDF_VERSION_MAJOR}.${IDF_VERSION_MINOR}.${IDF_VERSION_PATCH}")

5 comments on commit 482a8fb

@stintel
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can 5.1.2 be published to PlatformIO Registry?

@AdityaHPatwardhan
Copy link
Collaborator Author

@AdityaHPatwardhan AdityaHPatwardhan commented on 482a8fb Nov 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @stintel, the support of new releases of esp-idf to the PlatformIO Registry is not managed by Espressif.
More appropriate platform for this discussion might be https://github.com/platformio/platform-espressif32

Just to note here, there were no major changes introduced in our build system structure, so the new version might just work without requiring much effort. You can check if this thread helps you to use the latest version.

@stintel
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @AdityaHPatwardhan, platform-espressif32 is something different. I meant https://registry.platformio.org/tools/platformio/framework-espidf (vs https://registry.platformio.org/platforms/platformio/espressif32). Can you confirm this is handled by PlatformIO?

@AdityaHPatwardhan
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stintel I can only confirm that it is not handled by Espressif. As far as I know, it seems to be managed by PlatformIO team, it would be better for you to reach to them for more details.

@stintel
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I did!

Please sign in to comment.