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] v1.0.3 Release binary: unable to parse config files with multiple lines #5

Open
fsworld009 opened this issue Aug 10, 2024 · 1 comment

Comments

@fsworld009
Copy link

Environment: Bazzite OS on GPD Win 4 2023
Binary: the release binary in https://github.com/pelrun/pyWinControls/releases/tag/v1.0.3

Reproducable steps:

$ ./gpdconfig -d config.conf
$ ./gpdconfig -s config.conf
RuntimeError: Invalid key 'UP
'. Must be one of ['NONE', 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z', '1', '2', '3', '4', '5', '6', '7', '8', '9', '0', 'ENTER', 'ESC', 'BACKSPACE', 'TAB', 'SPACE', 'MINUS', 'EQUAL', 'LEFTBRACE', 'RIGHTBRACE', 'BACKSLASH', 'HASHTILDE', 'SEMICOLON', 'APOSTROPHE', 'GRAVE', 'COMMA', 'DOT', 'SLASH', 'CAPSLOCK', 'F1', 'F2', 'F3', 'F4', 'F5', 'F6', 'F7', 'F8', 'F9', 'F10', 'F11', 'F12', 'SYSRQ', 'SCROLLLOCK', 'PAUSE', 'INSERT', 'HOME', 'PAGEUP', 'DELETE', 'END', 'PAGEDOWN', 'RIGHT', 'LEFT', 'DOWN', 'UP', 'NUMLOCK', 'KPSLASH', 'KPASTERISK', 'KPMINUS', 'KPPLUS', 'KPENTER', 'KP1', 'KP2', 'KP3', 'KP4', 'KP5', 'KP6', 'KP7', 'KP8', 'KP9', 'KP0', 'KPDOT', '102ND', 'COMPOSE', 'POWER', 'KPEQUAL', 'F13', 'F14', 'F15', 'F16', 'F17', 'F18', 'F19', 'F20', 'F21', 'F22', 'F23', 'F24', 'OPEN', 'HELP', 'PROPS', 'FRONT', 'STOP', 'AGAIN', 'UNDO', 'CUT', 'COPY', 'PASTE', 'FIND', 'MUTE', 'VOLUMEUP', 'VOLUMEDOWN', 'KPCOMMA', 'RO', 'KATAKANAHIRAGANA', 'YEN', 'HENKAN', 'MUHENKAN', 'KPJPCOMMA', 'HANGEUL', 'HANJA', 'KATAKANA', 'HIRAGANA', 'ZENKAKUHANKAKU', 'KPLEFTPAREN', 'KPRIGHTPAREN', 'LEFTCTRL', 'LEFTSHIFT', 'LEFTALT', 'LEFTMETA', 'RIGHTCTRL', 'RIGHTSHIFT', 'RIGHTALT', 'RIGHTMETA', 'MOUSE_WHEELUP', 'MOUSE_WHEELDOWN', 'MOUSE_LEFT', 'MOUSE_RIGHT', 'MOUSE_MIDDLE', 'MOUSE_FAST']

It works if I manually make a config file with only one line.

Based on error message, it looks like there's an issue on parsing multiline text, and the parsed key for the lu becomes UP\n instead of UP

@pelrun
Copy link
Owner

pelrun commented Aug 12, 2024

Yeah. That's actually already fixed by #4, I just hadn't packaged it up into a new release. Will look at doing that asap.

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

No branches or pull requests

2 participants