VisualStudioVersion is not set

First off – thanks for the hard work and contribution!

I’m working on a custom pipeline – the build step of my project uses a tool which must be run in the context of a VS dev prompt and relies on the variable VisualStudioVersion.

Here’s a stripped down version of my pipeline:

jobs:
  build:
    runs-on: windows-latest
    steps:
    # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
    - uses: actions/checkout@v2
    - uses: ilammy/msvc-dev-cmd@v1.0.1
    - name: Build ament source
      run: colcon build --merge-install --cmake-args -A x64
      working-directory: ./dev/ament
      shell: cmd

The colcon build command reports VisualStudioVersion is not set, please run within a Visual Studio Command Prompt.

I’m really new to GitHub actions so this very well could be user error. I’ve read some things about needing to explicity export environment variables to pipelines or does your extension take care of this?

[ilammy]

I don’t quite understand why the build tool (I did not write it) requires that variable, but it is working fine for me to just set that variable in my actions workflow to ‘fake’ the tool.

Maybe the tool’s authors thought that since you’re building for Windows, all this stuff will be already set and they can piggyback on these variables. It also may be that they look only at VisualStudioVersion and then infer other settings based on that.

So, the PR still stands related to this issue, but I don’t technically need it anymore. I’ll leave it up to you to determine if it’s good to have or not 😃

I’ll merge it anyway since maybe it will be useful to someone else too.

Nevertheless, I’m happy that your issue got resolved one way or another. And that you’ve got some experience with GitHub Actions 😉

Thanks for the contribution!

[ilammy]

I’ve pushed v1.1.0 which contains this fix.

Any of the following specs will pull the updated version:

- uses: ilammy/msvc-dev-cmd@v1.1.0
- uses: ilammy/msvc-dev-cmd@v1       # preferred