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

MSI upgrade is deleting some files in lib/ #1714

Open
jay0lee opened this issue Oct 16, 2024 · 6 comments
Open

MSI upgrade is deleting some files in lib/ #1714

jay0lee opened this issue Oct 16, 2024 · 6 comments
Assignees
Labels

Comments

@jay0lee
Copy link
Member

jay0lee commented Oct 16, 2024

Some admins are seeing that when upgrading the Windows MSI for GAM, they are left with an incomplete installation. Files are missing from lib/ and this causes GAM to crash with various missing files/libraries errors.

The issue seems to be that when we build the GAM MSI, heat.exe generates a new Guid ID for each file on each run. So the MSI installer sees them as unique files and for some reason fails to actually replace them.

We need a stable Guid for files across GAM MSI builds.

@jay0lee jay0lee added the bug label Oct 16, 2024
@jay0lee jay0lee self-assigned this Oct 16, 2024
@jay0lee
Copy link
Member Author

jay0lee commented Oct 16, 2024

Workaround:

  1. uninstall GAM7 MSI completely (gam.cfg and project files are NOT deleted).

  2. install latest MSI.

That way we're not actually doing an upgrade and all files are copied.

More permanent fix incoming....

@jay0lee
Copy link
Member Author

jay0lee commented Oct 16, 2024

I've rewritten the tool script at:

https://github.com/GAM-team/GAM/blob/main/src/tools/gen-wix-xml-filelist.py

now after heat.exe runs the above script also runs and sets Guid to a uuid5 based on the source filename. That means that on every GAM MSI build a given filepath will get the same Guid.

This should fix MSI builds going forward but anyone with GAM 7.00.19 or older will need to do the uninstall, reinstall workaround at least once to get this solved.

@taers232c
Copy link
Contributor

taers232c commented Oct 16, 2024 via email

@jay0lee
Copy link
Member Author

jay0lee commented Oct 16, 2024

@taers232c do you know what change caused that? Ideally we would never delete files we didn't install. (and who really cares if c:\gam7 lives past MSI uninstall?)

@taers232c
Copy link
Contributor

taers232c commented Oct 16, 2024 via email

@taers232c
Copy link
Contributor

taers232c commented Oct 16, 2024 via email

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

No branches or pull requests

2 participants