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

chamtool.py errors while executing CONFIG=? #16

Closed
hsanjuan opened this issue Oct 8, 2016 · 4 comments
Closed

chamtool.py errors while executing CONFIG=? #16

hsanjuan opened this issue Oct 8, 2016 · 4 comments
Assignees
Labels

Comments

@hsanjuan
Copy link
Contributor

hsanjuan commented Oct 8, 2016

$> python3 chamtool.py -p /dev/ttyACM0 -v
[2016-10-08 13:38:47.531460] Opening serial port /dev/ttyACM0 succeeded
[2016-10-08 13:38:47.532812] Executing <VERSION?>: 101:OK WITH TEXT
[2016-10-08 13:38:47.533115] Response: ChameleonMini RevG 161008 using LUFA 151115 compiled with AVR-GCC 4.8.3
[2016-10-08 13:38:47.533382] Executing <CONFIG=?>: Based on the open-source NFC tool ChameleonMini
Traceback (most recent call last):
  File "chamtool.py", line 205, in <module>
    main()
  File "chamtool.py", line 168, in main
    if (chameleon.connect(args.port)):
  File "/multimedia/git/ChameleonMini/Software/Chameleon/Device.py", line 96, in connect
    result = self.getCmdSuggestions(self.COMMAND_CONFIG)
  File "/multimedia/git/ChameleonMini/Software/Chameleon/Device.py", line 168, in getCmdSuggestions
    result = self.getSetCmd(cmd, self.SUGGEST_CHAR)
  File "/multimedia/git/ChameleonMini/Software/Chameleon/Device.py", line 165, in getSetCmd
    return self.writeCmd("{}{}{}".format(cmd, self.SET_CHAR, arg))
  File "/multimedia/git/ChameleonMini/Software/Chameleon/Device.py", line 134, in writeCmd
    statusCode, statusText = status.split(":")
ValueError: need more than 1 value to unpack
@hsanjuan
Copy link
Contributor Author

hsanjuan commented Oct 8, 2016

It's because VERSION? gives a multiline response and only the first line of it is processed

@timokasper
Copy link
Collaborator

timokasper commented Oct 8, 2016

Thanks for this feedback, we have added 3 lines just yesterday so hasn't been tested yet --> we will fix it.

cons0l3 pushed a commit to cons0l3/ChameleonMini that referenced this issue Oct 8, 2016
geo-rg added a commit that referenced this issue Oct 24, 2016
Remove newlines from VERSION command. Fix for #16
@geo-rg geo-rg added verify and removed bug labels Oct 24, 2016
@geo-rg
Copy link
Collaborator

geo-rg commented Oct 24, 2016

Please verify if #25 - which is merged now - solves the problem for you.

@hsanjuan
Copy link
Contributor Author

It does solve the problem. THANKS!

@emsec emsec removed their assignment Feb 15, 2019
maozhenyu123 pushed a commit to maozhenyu123/ChameleonMini that referenced this issue Apr 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants