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.

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

[23.05] openwisp-monitoring: align PKG_SOURCE_VERSION with PKG_VERSION #25420

Conversation

pandafy
Copy link
Contributor

@pandafy pandafy commented Nov 21, 2024

Maintainer: @nemesifier
Compile tested: ramips
Run tested: ramips

Description:
Previously, PKG_SOURCE_VERSION in the Makefile was incorrectly set to 0.1.1 instead of tracking PKG_VERSION. This mismatch caused compilation issues for the package.

This fix ensures PKG_SOURCE_VERSION dynamically aligns with PKG_VERSION to prevent future discrepancies.

@pandafy
Copy link
Contributor Author

pandafy commented Nov 21, 2024

@1715173329 can you help me understand why the CI is failing here?

@pandafy pandafy changed the title openwisp-monitoring: align PKG_SOURCE_VERSION with PKG_VERSION [23.openwisp-monitoring: align PKG_SOURCE_VERSION with PKG_VERSION Nov 21, 2024
@pandafy pandafy changed the title [23.openwisp-monitoring: align PKG_SOURCE_VERSION with PKG_VERSION [23.05] openwisp-monitoring: align PKG_SOURCE_VERSION with PKG_VERSION Nov 21, 2024
@1715173329
Copy link
Member

@1715173329 can you help me understand why the CI is failing here?

It caused by recent container breakage, and CI on 23.05 is not updated.

@1715173329
Copy link
Member

Could you please rebase upon 23.05 HEAD? It should be fixed by 7ee4fe7.

@pandafy pandafy force-pushed the openwisp-monitoring-openwrt-23.05-fix branch 4 times, most recently from afdce9a to 4212dba Compare November 22, 2024 13:18
Previously, `PKG_SOURCE_VERSION` in the Makefile was incorrectly
set to `0.1.1` instead of tracking `PKG_VERSION`. This mismatch
caused compilation issues for the package.

This fix ensures `PKG_SOURCE_VERSION` dynamically aligns with
`PKG_VERSION` to prevent future discrepancies.

Signed-off-by: Gagan Deep <pandafy.dev@gmail.com>
@pandafy pandafy force-pushed the openwisp-monitoring-openwrt-23.05-fix branch from 4212dba to 8940bef Compare November 22, 2024 13:24
@pandafy
Copy link
Contributor Author

pandafy commented Nov 22, 2024

@1715173329 the CI passes now.

@1715173329 1715173329 merged commit 714aeeb into openwrt:openwrt-23.05 Nov 22, 2024
12 checks passed
@1715173329
Copy link
Member

@1715173329 the CI passes now.

Merged, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants