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

Support "!=" in built-in template processor #358

Open
xenoterracide opened this issue Jul 17, 2021 · 5 comments
Open

Support "!=" in built-in template processor #358

xenoterracide opened this issue Jul 17, 2021 · 5 comments

Comments

@xenoterracide
Copy link

xenoterracide commented Jul 17, 2021

maybe a bug, maybe a PEBCAK (I apologize if it is), I have this code, but for some reason the output of yadm alt doesn't include the contents of the developer.sh

{% if yadm.user != "root" %}
  {% include "../extra/developer.sh" %}
{% endif %}

https://github.com/xenoterracide/dotfiles/blob/2002b498c5bfad7c93b959dcbbf7e75fdb69d4a2/.config/yadm/alt/.zshrc%23%23template%2Ce.zsh#L34

@erijo
Copy link
Contributor

erijo commented Jul 17, 2021

Does it work if you instead use == and put the include in an else clause?

@xenoterracide
Copy link
Author

yes and no, this doesn't, it treats the include as just text, but if I remove the indentation it works. changing that may be another ticket... though the way I'd like it to behave is include that code prepended with that indentation. I don't think I should need the else for this though.

{% if yadm.user == "root" %}
{% else %}
  {% include "../extra/developer.zsh" %}
{% endif %}

@erijo
Copy link
Contributor

erijo commented Jul 18, 2021

Yes, {% must be first on the line and only == is supported. For more advanced templates you could perhaps use ninja instead?

@xenoterracide
Copy link
Author

maybe, but only having == is unintuitive. If that's all that can be supported perhaps it's possible to throw an error? I honestly haven't considered my config to be particularly complicated, honestly this is the only conditional I have.

@TheLocehiliosan TheLocehiliosan changed the title include inside of if block not evaluated Support "!=" in built-in template processor Nov 29, 2021
@github-actions
Copy link

This issue has been labeled as stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants