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

Setting appVersion for QML from Version: in spec-file #66

Merged
merged 5 commits into from
Feb 6, 2024

Conversation

Olf0
Copy link
Collaborator

@Olf0 Olf0 commented Feb 5, 2024

No description provided.

@Olf0 Olf0 added the enhancement Enhances something label Feb 5, 2024
@Olf0 Olf0 self-assigned this Feb 5, 2024
Copy link
Collaborator Author

@Olf0 Olf0 left a comment

Choose a reason for hiding this comment

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

This should cover all necessary changes, see https://github.com/search?q=repo%3Asailfishos-applications%2Ffilecase%20appVersion&type=code

P.S.: @dcaliste, I hope to have adopted your FlowPlayer PR #32 correctly and would appreciate a quick review of this PR very much.

Copy link

@dcaliste dcaliste left a comment

Choose a reason for hiding this comment

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

This looks good to me. Didn't compile and try, but I cannot spot any misspell or wrongness.

@Olf0 Olf0 merged commit 5f178fd into devel Feb 6, 2024
1 check passed
@Olf0 Olf0 deleted the auto-set-version branch February 6, 2024 19:25
@Olf0
Copy link
Collaborator Author

Olf0 commented Feb 6, 2024

This looks good to me.

Thank you very much!

Didn't compile …

This aspect is taken care of by CI (GitHub-workflow for each commit to devel- and master-branch): It does compile.

… and try …

Neither did I, but I simply define that as a job for the rc-testers. 😉

@dcaliste
Copy link

dcaliste commented Feb 7, 2024

Good, I didn't pay attention that the CI was checking the build. I did build for completeness myself a bit later after sending the feedback anyway ;-)

@Olf0
Copy link
Collaborator Author

Olf0 commented Feb 7, 2024

I did build for completeness myself …

Would it be helpful for you, if I let CI also build for aarch64@SFOS4.3.0 (runs on SFOS 4.4.0 & 4.5.0, too) for each commit to devel- and master-branch? Or armv7hl@SFOS4.3.0, if that is appropriate for your testing device?

I currently only build for i486@SFOS3.0.2 as a compile test, because that downloads a much smaller SFOS-SDK Docker-image (~ 800 MB vs. 3 GB) and compiles faster than for armv7hl or aarch64 as target architecture. Hence the current state is the quickest way to see, if the source code still compiles after a commit.

Only when setting a git-tag, the source code is compiled for all three architectures with multiple SFOS-SDK releases.

The results (i.e. RPM(s) and logs) of all these CI runs can be easily accessed by the Actions tab in GitHub's web-frontend.

@dcaliste
Copy link

dcaliste commented Feb 8, 2024

Thanks for proposing. I think, it's currently fine as you set it up. I compile on the SDK, locally anyway for every change I make. And quickly test on device. Of course, I do this for the 4.5 target. So building with the simplest 3.0 for the CI, I think already provide a good hint on what could happen.

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

Successfully merging this pull request may close these issues.

2 participants