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

Fail to load /home/runner/work/_actions/lukka/run-cmake/v1/action.yml` #407

Open
lukka opened this issue Apr 3, 2020 · 4 comments
Open
Labels
future Feature work that we haven't prioritized Runner Feature Feature scope to the runner

Comments

@lukka
Copy link

lukka commented Apr 3, 2020

Describe the bug
since runner version '2.168.0' my run-cmake is broken. It is working on Current runner version: '2.165.2'.

Download action repository 'lukka/run-cmake@v1'
##[error](Line: 34, Col: 14):
##[error](Line: 34, Col: 14): Unrecognized named-value: 'env'. Located at position 1 within expression: env.RUNVCPKG_VCPKG_TRIPLET
##[error]Fail to load /home/runner/work/_actions/lukka/run-cmake/v1/action.yml`

My action.yml contains:

vcpkgTriplet:
    default: "${{ env.RUNVCPKG_VCPKG_TRIPLET }}"

To Reproduce
Use lukka/run-cmake@v1 action in your workflow on GitHub.

Expected behavior
vcpkgTriplet should be set to the value of the env var

Runner Version and Platform

Version of your runner?
'2.168.0'

OS of the machine running the runner? OSX/Windows/Linux/...
ubuntu/Linux

What's not working?

The runner fails at parsing action.yml

Job Log Output

lukka/run-cmake#11

@lukka lukka added the bug Something isn't working label Apr 3, 2020
@TingluoHuang
Copy link
Member

@lukka we only support using static well known context for input default value, since default value should always exists, relies on dynamic value doesn't make sense.
https://github.com/actions/runner/blob/master/src/Runner.Worker/action_yaml.json#L88

@lukka
Copy link
Author

lukka commented Apr 4, 2020

It is unfortunate to hear that.

@TingluoHuang
Copy link
Member

@lukka do you mind share how do you use the ${{env.xxx}} in your action's input default value?

@lukka
Copy link
Author

lukka commented Apr 4, 2020

Sure, there is the action.yml. The input parameter is named vcpkgTriplet.

I think i can work around this by setting the default value in the action itself, when core.getInput() return a blank string (afaik it never returns null or undefined).

Anyway it is a nice feature to have to be able to say "the default of this input is (potentially) defined in this environment variable", and imho it would be better to support it rather than not.

@TingluoHuang TingluoHuang added enhancement New feature or request runner Runner Feature Feature scope to the runner and removed bug Something isn't working runner enhancement New feature or request labels Jun 6, 2020
@thboop thboop added the future Feature work that we haven't prioritized label Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
future Feature work that we haven't prioritized Runner Feature Feature scope to the runner
Projects
None yet
Development

No branches or pull requests

3 participants