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

Incorrect version information applied to Windows releases #7441

Closed
Wedge009 opened this issue Mar 8, 2023 · 11 comments
Closed

Incorrect version information applied to Windows releases #7441

Wedge009 opened this issue Mar 8, 2023 · 11 comments
Assignees
Labels
Bug Issues involving unexpected behavior. Low Priority Issues that will cause no meaningful problems if left unaddressed. Packaging Packaging toolchain and configuration-related issues. Stable 1.16 Issues specific to the Wesnoth 1.16 maintenance branch. Windows OS-specific issues that apply to Microsoft Windows

Comments

@Wedge009
Copy link
Member

Wedge009 commented Mar 8, 2023

Game and System Information

  • Version: At least 1.16.x and 1.17.x
  • Downloaded from: sourceforge.net and Steam
  • Build info: official release builds
  • OS: Windows

Description of the bug

1.16.8 release downloaded via Steam:
image

1.17.13 release downloaded from sourceforge.net:
image

I suppose this is low priority, but it causes confusion when encountering problems like in #7438.

Steps to reproduce the behavior

Check the version information in Properties -> Details (or simply wait for tool-tip pop-up with the same information in Windows Explorer).

Expected behavior

Version information should match the actual release version.

Additional context

No response

@Wedge009 Wedge009 added Bug Issues involving unexpected behavior. Windows OS-specific issues that apply to Microsoft Windows Packaging Packaging toolchain and configuration-related issues. Low Priority Issues that will cause no meaningful problems if left unaddressed. labels Mar 8, 2023
@loonycyborg loonycyborg self-assigned this Mar 8, 2023
loonycyborg added a commit that referenced this issue Mar 8, 2023
Previous way of addressing compiled .rc files led to stale intermediate
targets leading to issue #7441
loonycyborg added a commit that referenced this issue Mar 9, 2023
Previous way of addressing compiled .rc files led to stale intermediate
targets leading to issue #7441
@Wedge009
Copy link
Member Author

Wedge009 commented Mar 19, 2023

1.17.14 Windows executables are showing the correct information in the version now. Thanks. Awaiting the next 1.16 release before closing.

Edit: File version is 1.17.14.0, but Product version is 1.17.13+dev. The former is what shows in the on-hover tool-tip so it's probably fine - and I've never understood why Microsoft requires two different versions - but I wonder if the latter should be 1.17.14.0 as well.

@Wedge009
Copy link
Member Author

1.16.9 release now reports 1.16.9.0 for file version, 1.16.8+dev for product version.

@Wedge009
Copy link
Member Author

I'm re-opening this. 1.17.18 release, downloaded from sourceforge.net, has 'File version' reported as 1.17.15.0 and 'Product version' reported as 1.17.15. Why does this happen?

As an aside, the original report shows 'Copyright' as 2003-2018 and 1.17.18 release has 'Copyright' as 2003-2022. Still not quite right, although an improvement over the older releases.

@Wedge009 Wedge009 reopened this Jun 21, 2023
@soliton-
Copy link
Member

@loonycyborg looks like 9df0a59 was not enough.

The copyright can be adjusted in packaging/windows/wesnoth.rc.

@loonycyborg
Copy link
Member

loonycyborg commented Jun 21, 2023

That happens mostly because that file is in packaging/windows, I tried making all builds out-of-tree but it seems something still makes file wesnoth.o in original dir and new builds will prefer original tree one despite it being old one.

@loonycyborg
Copy link
Member

In fact it's all because of confusion caused by packaging/windows symlink, I need to think how to reorganize this build step to avoid this.

loonycyborg added a commit that referenced this issue Jun 23, 2023
they were created there via symlink which confused subsequent builds
leading to issue #7441
@Wedge009
Copy link
Member Author

Windows 1.17.19 release:

  • File version: 1.17.19.0
  • Product version: 1.17.19
  • Copyright: 2003-2023

Seems good for now, so closing. Thanks.

@Wedge009
Copy link
Member Author

Wedge009 commented Sep 2, 2023

Because 1.17 was working (and I just confirmed 1.17.20 follows the same - correct - pattern as 1.17.19), I'd neglected to check 1.16 releases. I'm not sure if the change for master also needs to be applied for 1.16.

Despite what the details say, this is the properties listing for Wesnoth 1.16.10 official Windows release (note the date modified), 1.16.9 was released in April 2023.

image

@Wedge009 Wedge009 reopened this Sep 2, 2023
@Wedge009 Wedge009 added the Stable 1.16 Issues specific to the Wesnoth 1.16 maintenance branch. label Sep 2, 2023
loonycyborg added a commit that referenced this issue Sep 6, 2023
@soliton-
Copy link
Member

Is this fixed now?

@Wedge009
Copy link
Member Author

Wedge009 commented Oct 27, 2023

Need to wait for 1.16.11 release.

@Wedge009
Copy link
Member Author

Wedge009 commented Jan 9, 2024

Looks good for at least the Steam release executable. Thanks.

@Wedge009 Wedge009 closed this as completed Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues involving unexpected behavior. Low Priority Issues that will cause no meaningful problems if left unaddressed. Packaging Packaging toolchain and configuration-related issues. Stable 1.16 Issues specific to the Wesnoth 1.16 maintenance branch. Windows OS-specific issues that apply to Microsoft Windows
Projects
None yet
Development

No branches or pull requests

3 participants