Windows missing version metadata when packaged

[bon-ami]

Description

Packaged Windows binaries are missing version information when viewing the details inside Explorer.

Reproduction

1. Package the Windows binary using fyne package.
2. Right-click on the binary and open up the properties viewer.
3. Navigate to the “Details” tab.
4. Notice that the version information is missing.

Device (please complete the following information):

• OS: Windows
• Version: 10
• Go version: 1.18
• Fyne version: 2.1.4

Old description

Describe the bug:

on Windows using MSYS2.
According to #2813, "fyne package" is better than "go build". The latter does not show icon for exe. But the former does not show for fprint(). No version info can be shown for details of the exe (in explorer). So actually, one issue is print and the other is version info. The version problem is probably rooted in Go and all I know is to use "-ldflags -X main.Ver=..." to pass it to the code and cannot show it in Explorer. However fyne package can make it show as 0.0.0. This is puzzling me.
Thank you.
I have FyneApp.toml for Website,Icon, Name, ID.

To Reproduce:

Steps to reproduce the behaviour:

1. fyne package --os windows --appVersion 5.0.0 --appBuild 1 --tags main.Ver=5.0.0,Bld=1
2. run the exe in MINGW command window. it can print into terminal.
3. run the exe in Windows command line. it cannot print into terminal.
4. check detail of exe in Explorer. the file version of it shows 0.0.0.0
5. in my code (package main), Ver & Bld are both "". How to pass a variable?

[andydotxyz]

The version info should appear - this seems like a bug thanks.

The trouble with printing is down to how your OS will launch graphical apps. macOS has the same issue. You see the console when developing but launching a complete graphical app will go to your system console not the terminal you started from.

in my code (package main), Ver & Bld are both "". How to pass a variable?

The usage of tags are not the same as ldflags. Go build tags just control which files will be included, they do not set variables.

[Jacalz]

@bon-ami Can you please stick to one topic per issue in the future? Multiple things going on in one issue makes it a lot harder to read.

I added some more information to the description, which I think should make it a bit easier to track the issue that we need to fix here. Thanks for opening this issue.

[Bluebugs]

I think the fix to this bug is that we currently do not set the version information during fyne build in a way that please the Windows explorer expectation. I tried to do a bit of research on the topic, but kind of failed as the only I could find was rcedit related command. Is there any documentation anywhere pointing toward main.Ver and Bld behing understood by Windows somehow?

[andydotxyz]

Fixed on develop and will release in v2.2.2 as well