flux-core icon indicating copy to clipboard operation
flux-core copied to clipboard

replace or remove current Lua bindings

Open grondo opened this issue 6 years ago • 3 comments

Current Lua bindings were written for an older Flux API and have to either be updated or removed.

It isn't clear if there is really a use case for the Lua bindings. Outside of the prototype execution system, the Lua bindings went largely unused, and most if not all Flux users will be much more comfortable using the Python bindings going forward.

The only uses cases for Lua bindings I can come up would be for embedding in services (e.g. a reduction handle could take a blob of Lua), and those would probably be custom bindings for the use case, not generic 1-to-1 bindings for the Flux API.

#814 is also slightly related.

grondo avatar Feb 05 '19 18:02 grondo

@grondo, is this still something you want to consider? happened to pop up in a search today

trws avatar Sep 08 '25 17:09 trws

Good question. We should probably do this at some point. The shell initrc is the main Lua use case. Also flux-cron but this could easily be rewritten (there's also an issue somewhere to consider removing the cron module in Flux)

grondo avatar Sep 08 '25 17:09 grondo

I would need to look back at the shell initrc stuff to remember how much of the bindings it uses. My guess is not very much, and what is needed could just be moved into src/shell/rc.c

grondo avatar Sep 08 '25 17:09 grondo