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

Cannot change language for duckencoder #1

Closed
ludoz opened this issue Mar 16, 2017 · 4 comments
Closed

Cannot change language for duckencoder #1

ludoz opened this issue Mar 16, 2017 · 4 comments
Assignees

Comments

@ludoz
Copy link

ludoz commented Mar 16, 2017

Hello,

I tried to change the keyboard layout. So i change lang="us" by lang="fr" in the payload1.txt with no success.

So i commented the lang variable in payload1.txt and add it to setup.cfg. No more success ...

Nothing happen when i plug the Pi, when the lang is set to "us" it works !

So i decided to look deeper and when i start P4wnP1 with the script startup_p4wnp1.sh and the lang defined to "us" , it's ok. The same test but with lang defined to "fr" give me that :

Traceback (most recent call last): File "/home/pi/P4wnP1/duckencoder/duckencoder.py", line 494, in <module> main(sys.argv[1:]) File "/home/pi/P4wnP1/duckencoder/duckencoder.py", line 470, in main language=readResource(script_dir + "/resources/" + lang + ".properties") File "/home/pi/P4wnP1/duckencoder/duckencoder.py", line 23, in readResource val = splitted[1].strip() IndexError: list index out of range

Can you see where is the problem ?

@mame82
Copy link
Collaborator

mame82 commented Mar 16, 2017

Yes...Problem resides here The equal signs for the ISO chars are missing. I'm going to fix this as soon I have time to (next week). Thanks for testing "fr"

@mame82 mame82 self-assigned this Mar 16, 2017
@mame82 mame82 added the bug label Mar 16, 2017
@mame82
Copy link
Collaborator

mame82 commented Mar 17, 2017

@ludoz
The issue should be fixed now. Could you please update the duckencoder submodule and run a test?

@ludoz
Copy link
Author

ludoz commented Mar 18, 2017

Thanks, it works for FR. I think you'll have the same problem with others language. When FR was not working i tested other azerty layout like de or es and they were not working. I say that for your information.

Thanks for this great platform. Lot of fun.

@ludoz ludoz closed this as completed Mar 18, 2017
@mame82
Copy link
Collaborator

mame82 commented Mar 18, 2017

I'll check the other language property files for similar bugs. Unfortunately I'm not able to test all layouts to be working as intended on my own. You already helped with FR, thanks againg. DE and US have been tested by myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants