Skip to content

Miscellaneous build script improvements #142

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

Merged
merged 4 commits into from
Feb 25, 2023
Merged

Conversation

ethomson
Copy link
Member

  • Document the arm64 parameter that already exists
  • Complain if no architecture is specified (instead of just nooping)
  • Don't specify Visual Studio version; let cmake choose the version
  • Don't build the tests or the CLI (so that they remain out of the resulting nuget package)

Apparently not specifying a platform means "don't do anything" which is
counterintuitive. Tell people we're not going to do anything.
@ethomson ethomson merged commit ed246f2 into master Feb 25, 2023
@ethomson ethomson deleted the ethomson/script_fixes branch February 25, 2023 14:15
@@ -11,6 +11,8 @@
If set, the x64 version will be built.
.PARAMETER arm64
If set, the arm64 version will be built.
.PARAMETER arm64
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ethomson This is a duplicate entry. It was already there on line 12

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.

None yet

2 participants