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.

@Districh-ru Districh-ru linked an issue Jan 9, 2025 that may be closed by this pull request
2 tasks
@Districh-ru
Copy link
Collaborator

Districh-ru commented Jan 9, 2025

Hi, @lovenemesis, this PR should fix the issue you have encountered on PS Vita.
Could you please test it on your hardware?
fheroes2_psv.zip

@oleg-derevenetz oleg-derevenetz added the PlayStation Vita PlayStation Vita related stuff label Jan 9, 2025
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) PlayStation Vita PlayStation Vita related stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error 0x80101104 with 1.1.5 Release on PSV
3 participants