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

Fix some minor build system annoyances #1027

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

rzumer
Copy link
Contributor

@rzumer rzumer commented Sep 13, 2021

I found it mildly annoying to have to run some steps manually before and after the main build, so this PR automates these steps for non-ARM builds (on ARM, MSBuild pre-/post-build steps don't seem to be supported).

It also fixes an issue in the dependency build script where the environment is modified for the whole session, which can cause it to fail due to vcvarsall trying to make (I assume) PATH too long when called multiple times.

This solves potential issues with vsvarsall
generating oversized environment variables with
multiple runs.
This removes a manual step for first-time debugging and
allows quick testing of theme changes.

It is only supported for x86/x64 builds.
This runs only if libcurl_a.lib is not found in its expected location.

It is only compatible with x86/x64 builds.
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.

1 participant