quarto-ext / shinylive

Quarto extension to embed Shinylive for Python applications
https://quarto-ext.github.io/shinylive/
MIT License
144 stars 8 forks source link

Using extension on environment where quarto is not on PATH #50

Closed cderv closed 4 months ago

cderv commented 5 months ago

Currently the extension is calling quarto from LUA and assumes it will be on PATH https://github.com/quarto-ext/shinylive/blob/213a995285e5f8c46293532f6cd9ac3e0d050b7a/_extensions/quarto-ext/shinylive/shinylive.lua#L416-L421

This may not be true (e.g in Posit Connect environment cc @aronatkins @colearendt).

For solving issue this could be done to point to the quarto binary

pandoc.path.join({os.getenv('QUARTO_BIN_PATH'), 'quarto'})

though it needs to be adapted for Windows (which requires .exe or .cmd depending if released or dev version.

We may need to add a help in our Quarto LUA API to get easily the PATH to quarto binary used from the lua filter in an extension cc @cscheid

cderv commented 5 months ago

Once this is available

but then it will mean that new version of shinylive will require quarto 1.5 unless we decide to backport #9572 in 1.4. But usually we backport only regression