rattler-build icon indicating copy to clipboard operation
rattler-build copied to clipboard

Undefined variables not throwing an error

Open wolfv opened this issue 1 year ago • 1 comments

I think we might want to rethink how we handle undefined variables.

For example, this is not throwing the appropriate Jinja error:

host:
  - ${{ asdasda }}

Instead, it is silently discarded.

wolfv avatar Dec 07 '24 08:12 wolfv

I looked into it and the issue is that minijinja supports only a mode where undefined values always error when we also want them to error when printing. For this reason, we should also set all non-matching platforms and arch's to false instead of relying on Undefined as falsey value.

I have openend a PR on rattler to be able to iterate through all possible values for Platform and Arch: https://github.com/conda/rattler/pull/972

wolfv avatar Dec 09 '24 18:12 wolfv