ROSE_ORIG_HOST is not written correctly to [template variables] section

[TomekTrzeciak]

Description

ROSE_ORIG_HOST is not written correctly to [template variables] section (missing quotes around the value):

$ cat ~/cylc-run/any_rose_suite/opt/rose-suite-cylc-install.conf
# This file records CLI Options.
# Installed with:
#     * Cylc Rose: 1.7.0
#     * Rose     : 2.6.0
#     * Cylc     : 8.6.0

!opts=

[env]
# ROSE_ORIG_HOST set by cylc install.
ROSE_ORIG_HOST=host.name

[template variables]
# ROSE_ORIG_HOST set by cylc install.
ROSE_ORIG_HOST=host.name

Reproducible Example

Install any suite with rose-suite.conf file.

Expected Behaviour

Needs adding quotes around values in templating sections here.

[oliver-sanders]

[how] is this working now?

[TomekTrzeciak]

[how] is this working now?

Beats me 🤷, but it does work somehow even though it shouldn't. Maybe ROSE_ORIG_HOST gets some special treatment. Rather low priority, since it works, just doesn't look right.

[oliver-sanders]

Or is this some sort of back-compat we added to support explicit strings?

Implementation:

cylc-rose/cylc/rose/jinja2_parser.py

Line 207 in 8a123ad

def literal_eval(self, value):

Not sure if the behaviour is purposeful or not.

[wxtim]

Closed by #398