Ticket #843 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

Thanks application in pippy has python backtrace invalid int

Reported by: thjc Owned by: mchua sdz
Priority: Unspecified by Maintainer Milestone: 0.86
Component: Pippy Version: Unspecified
Severity: Unspecified Keywords:
Cc: jpichon Distribution/OS:
Bug Status: Resolved

Description

From version soas 200905031329

Traceback (most recent call last):

File "/home/liveuser/.sugar/default/org.laptop.Pippy/tmp/pippy_app.py", line 29, in <module>

cols, lines = size()

File "/usr/share/sugar/activities/Pippy.activity/library/pippy/console.py", line 26, in size

cols = int(read_to_delimit('t'))

ValueError: invalid literal for int() with base 10: '21;126'

Attachments

Change History

Changed 4 years ago by RafaelOrtiz

  • milestone changed from soas_fossvt to 0.86

changing milestone

Changed 3 years ago by jpichon

  • cc jpichon added

First, thanks to whoever put in the comment with the "xterm magic" url, that really helped. I attached a patch that should fix this issue, tested on the XO with the G1G1 image (0.82?) and on the latest sugar-jhbuild.

I'm not entirely sure why it works though. According to the xterm link, the output of the command sent to /dev/tty should be '8;<rows>;<columns>t' but on the XO it returns '<rows>;<columns>t" (hence the parsing failing on other systems). Any idea?

Changed 3 years ago by cjb

Changed 3 years ago by sascha_silbe

  • owner changed from lfaraone to mchua sdz
  • distribution changed from SoaS to Unspecified
  • component changed from Pippy to SoaS

Bulk change distribution=SoaS -> component=SoaS

Changed 3 years ago by jpichon

I've pushed the patch to the Pippy mainline. Since the component in the tracker isn't Pippy anymore, I'm not sure what to do with the status field, please update it as needed or let me know if there are additional steps I can help with for this. Thank you!

Changed 3 years ago by sascha_silbe

  • distribution Unspecified deleted
  • component changed from SoaS to Pippy

This seems to have been mistakenly changed to Soas; changing back. Please change status as you feel appropriate, jpichon.

Changed 3 years ago by jpichon

  • status_field changed from Unconfirmed to Resolved

Changed 3 years ago by jpichon

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.