Skip to content

Commit

Permalink
Try new env var method
Browse files Browse the repository at this point in the history
  • Loading branch information
yairm210 authored Jul 28, 2024
1 parent c4b659b commit 87bb8c5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:

- name: Build WiX on Windows
run: | # Trying to set env variables for a simple command is like ???, now I truly understand the "Windows Sucks" mentality.
# The space after the version is intentionally ommited, otherwise the env var contains a space. *sigh*
# The space after the version is intentionally omitted, otherwise the env var contains a space. *sigh*
cmd /C "set UNCIV_VERSION=4.12.14&& wix build .\unciv.wxs"
dir
Expand Down
4 changes: 2 additions & 2 deletions unciv.wxs
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
<Shortcut Id="StartMenuShortcut"
Name="$(var.Name)"
Target="[#UncivExe]"
Arguments="--data-dir=[#APPDATAFOLDER]"
Arguments="--data-dir=%APPDATA%\Unciv"
Icon="icon.ico"
Directory="ApplicationProgramsFolder"
WorkingDirectory="INSTALLFOLDER" />
Expand All @@ -52,7 +52,7 @@
</Feature>


<StandardDirectory Id="ProgramFiles64Folder">
<StandardDirectory Id="ProgramFilesFolder">
<Directory Id="INSTALLFOLDER" Name="$(var.InstallFolder)" />
</StandardDirectory>

Expand Down

0 comments on commit 87bb8c5

Please sign in to comment.