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

User data for awful.key cannot be shared #1401

Closed
p-himik opened this issue Jan 15, 2017 · 3 comments · Fixed by #1492
Closed

User data for awful.key cannot be shared #1401

p-himik opened this issue Jan 15, 2017 · 3 comments · Fixed by #1492
Assignees

Comments

@p-himik
Copy link

p-himik commented Jan 15, 2017

Output of awesome --version:

awesome v4.0 (Harder, Better, Faster, Stronger)
 • Compiled against Lua 5.2.4 (running with Lua 5.2)
 • D-Bus support: ✔
 • execinfo support: ✔
 • RandR 1.5 support: ✘
 • LGI version: 0.9.0

How to reproduce the issue:
Somewhere in your config where you define keys, replace a local table with description/group with a variable that contains this table. Use the same variable in some other key.
E.g.

local aud = { description = "Raise the volume", group = "audio" }
local globalkeys = awful.util.table.join(
    awful.key({}, "XF86AudioRaiseVolume", apw.up, aud),
    awful.key({ modkey }, "Right", apw.up, aud),
)

Restart with the new config, open hotkeys popup.

Actual result:
image

Expected result:
There should have been two records for different keys with the same description/group. Or at least there should be a documentation notice on awful.key that you can't reuse data argument.

@psychon
Copy link
Member

psychon commented Jan 17, 2017

So, something somewhere is modifying a table that it is not supposed to modify? Or using a table as a key in another table where it shouldn't?

@actionless
Copy link
Member

actionless commented Jan 17, 2017 via email

@psychon
Copy link
Member

psychon commented Jan 17, 2017

Thanks a lot!

actionless added a commit to actionless/awesome that referenced this issue Jan 30, 2017
actionless added a commit to actionless/awesome that referenced this issue Jan 30, 2017
closes awesomeWM#1401

fix(tests: awesomerc): use `awful.key.execute` instead of local helper function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants