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

Correct all version used while building application to ensure Windows… #3638

Merged
merged 2 commits into from
Feb 8, 2023

Conversation

Bluebugs
Copy link
Contributor

@Bluebugs Bluebugs commented Feb 7, 2023

Description:

Compiling for Windows with a short AppVersion, like 1.0, lead to the version not being consistent from Windows perspective and prevent the starting of the application on up to date Windows.

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

Sorry, something went wrong.

Unverified

This user has not yet uploaded their public signing key.
… compatibility.
@Bluebugs Bluebugs requested a review from andydotxyz February 7, 2023 21:29
@Bluebugs
Copy link
Contributor Author

Bluebugs commented Feb 7, 2023

@Jacalz This is why you do not get a working binary on Windows at the moment when fyne-cross use fyne to build. We would want this patch to be in the minor release, I think.

@coveralls
Copy link

coveralls commented Feb 7, 2023

Coverage Status

Coverage: 61.797% (+0.02%) from 61.781% when pulling 8e2636c on Bluebugs:bugs/windows-version into 4b720a7 on fyne-io:develop.

Unverified

This user has not yet uploaded their public signing key.
@andydotxyz
Copy link
Member

Are you sure this is the cause @Bluebugs ? The metadata that is being used does have 3 version digits and a build...
https://github.com/Jacalz/rymdport/blob/main/FyneApp.toml

@Jacalz
Copy link
Member

Jacalz commented Feb 8, 2023

Indeed. It seems like that shouldn't quite impact my use case?

@Bluebugs
Copy link
Contributor Author

Bluebugs commented Feb 8, 2023

@Jacalz indeed, their was another bug in geoffrey that was preventing this also to work. It was a small one in fyne-cross, this one here and one more in geoffrey.

@Bluebugs Bluebugs merged commit bb9acdb into fyne-io:develop Feb 8, 2023
@andydotxyz
Copy link
Member

Agreed, it isn't the root of the problem but is a good improvement anyway

@Bluebugs Bluebugs deleted the bugs/windows-version branch February 8, 2023 16:05
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.

None yet

4 participants