Skip to content

Add locating vs build tools without full vs installed #2576

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 1 commit into from
Jun 24, 2025

Conversation

martinking71
Copy link
Contributor

Locate buildTools on CI servers where no full VS installed

Describe your changes

Updated 1kiss script to locate visual studio buildTools

Current vswhere call just locates full installs of visual studio.

When using a CI server or different IDE like Jetbrains there is no need for full visual studio install, just build tools so script fails when searching for non existent visual studio

Issue ticket number and link

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

Locate buildTools on CI servers where no full VS installed
@halx99 halx99 changed the title Fix for locating build tools Add locating vs build tools without full vs installed Jun 24, 2025
@halx99 halx99 added the enhancement New feature or request label Jun 24, 2025
@halx99 halx99 added this to the 2.6.2 milestone Jun 24, 2025
@halx99 halx99 merged commit cb97a19 into axmolengine:dev Jun 24, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants