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

[Bug]: Lua script not loading (v1.19.7) #1753

Closed
mtompkins opened this issue Feb 26, 2024 · 5 comments
Closed

[Bug]: Lua script not loading (v1.19.7) #1753

mtompkins opened this issue Feb 26, 2024 · 5 comments
Labels
bug related to incorrect existing implementation of some functionality

Comments

@mtompkins
Copy link

What happened?

Upgrade to 1.19.7 introduced a bug that prevents conky from loading a lua script using load_lua

Note the log indicates llau_load (two L's?)

Also tried $HOME to see if the variable helped instead of ~. Also failed.

Version

1.19.7

Which OS/distro are you seeing the problem on?

Arch Linux

Conky config

...
-- Set the path to your script here.

lua_load = '~/.conky/bargraph_small.lua.1080p-dpw1',
lua_draw_hook_post = 'main_bars',

Stack trace

No response

Relevant log output

conky: desktop window (400007) is subwindow of root window (834)
conky: window type - desktop
conky: drawing to created window (0x3200002)
conky: drawing to double buffer
conky: llua_load: specified script file '~/.conky/bargraph_small.lua.1080p-dpw1' doesn't exist
conky: forked to background, pid is 69678
@mtompkins mtompkins added bug related to incorrect existing implementation of some functionality triage issue that hasn't been verified, categorized or acknowledged yet labels Feb 26, 2024
@plikhari
Copy link

HI

I am not a developer, but I, sometimes answer issues related to LUA.

In this case, Lua cannot find the script - try other ways to state the path. A relative path also works, but perhaps using ~/.conky does not. Try full path like /home/username/.conky/bargraph_small.lua.1080p-dpw1

I am the author of conkywx ( a conky weather app on Arch Linux ) - a conky app that is lua heavy (yes, there is a good sprinkle of Perl as well ) and works without issues.

@mtompkins
Copy link
Author

HI

I am not a developer, but I, sometimes answer issues related to LUA.

In this case, Lua cannot find the script - try other ways to state the path. A relative path also works, but perhaps using ~/.conky does not. Try full path like /home/username/.conky/bargraph_small.lua.1080p-dpw1

I am the author of conkywx ( a conky weather app on Arch Linux ) - a conky app that is lua heavy (yes, there is a good sprinkle of Perl as well ) and works without issues.

Thanks @plikhari ... when I tried $HOME I didn't think to try a fully hardcoded path. Indeed it works.
I do still consider this a bug, however, but appreciate your share!

@plikhari
Copy link

Thanks @plikhari ... when I tried $HOME I didn't think to try a fully hardcoded path. Indeed it works. I do still consider this a bug, however, but appreciate your share!

I do not use the conventional path .conky etc, but conky looks at the root from where it is initiated / started - so I land up using ./lib/which ever script - this never fails.

-------------------------------------------------------------
-- Load lua component of conkywx
-------------------------------------------------------------`
lua_load = './lib/conkywx.lua',
lua_startup_hook = 'startup',
};

conky.text = [[
${lua main -n wxrings -d ./examples/hartley-lua-data.lua }
# ${execpi 3600 ${template0} -c ${template1} -t ${template2}}
]];

`

Here is an example - I have moved away from using conky functions to using pure lua - but you can see the original comment to trigger the non Lua conky.

Here is a link to the latest test conkywx build - I intend updating the AUR soon, but let's see hmmmm

@mtompkins
Copy link
Author

I made this exact change :)

@brndnmtthws brndnmtthws removed the triage issue that hasn't been verified, categorized or acknowledged yet label Mar 9, 2024
@brndnmtthws
Copy link
Owner

This should be fixed with #1779.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug related to incorrect existing implementation of some functionality
Projects
None yet
Development

No branches or pull requests

3 participants