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

Set the PROJECT_VERSION for PSV in XX.YY format #9423

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

oleg-derevenetz
Copy link
Collaborator

@oleg-derevenetz oleg-derevenetz commented Jan 6, 2025

Related to #9418 (comment)

Still can not detect some type of errors - e.g. the absence of version.txt itself (will set the version to 00.00 in this case), because GNU Make runs the shell in sh mode, which doesn't support the bashextensions - e.g. the -o pipefail option, but it's still better than no error handling at all.

@oleg-derevenetz oleg-derevenetz added bug Something doesn't work deployment Deployment stuff (distributions, installers, etc) labels Jan 6, 2025
@oleg-derevenetz oleg-derevenetz added this to the 1.1.6 milestone Jan 6, 2025
@oleg-derevenetz oleg-derevenetz marked this pull request as ready for review January 7, 2025 08:52
@Districh-ru
Copy link
Collaborator

Hi, @oleg-derevenetz, this PR works good on emulator:
изображение

I've asked Pønytail to test this build on his hardware.
If everything is OK, I'll approve this PR.

@oleg-derevenetz
Copy link
Collaborator Author

this PR works good on emulator

As far as I understand, on emulator everything works fine even without this PR? :)

@Districh-ru
Copy link
Collaborator

As far as I understand, on emulator everything works fine even without this PR? :)

Yes, the emulator reads the version string and displays it (as a string) when installing and then in the applications list:
изображение

The real hardware, I think, does some version analysis which needs XX.YY format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something doesn't work deployment Deployment stuff (distributions, installers, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants