Opened 14 years ago

Closed 14 years ago

Last modified 10 years ago

#181 closed defect (fixed)

replace deprecated os.popen by subprocess

Reported by: alsroot Owned by: tomeu
Priority: normal Milestone:
Component: Sugar Version: Git as of bugdate
Severity: Keywords: r+
Cc: Distribution/OS:
Bug Status:

Description

in python2.6 os.popen is marked as deprecated
I have no idea why it breaks datastore (since thats only warning) but that is fact on Mandriva(cooker) - after calling replaced code (see patch) datastore connections are failed (dbus timeout error).
After replacing os.popen2 by subprocess everything is ok

Attachments (1)

sugar-datastore-popen.patch​ (648 bytes) - added by alsroot 14 years ago.

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by alsroot

comment:1 Changed 14 years ago by alsroot

  • Keywords r? added

comment:2 Changed 14 years ago by tomeu

  • Keywords r+ added; r? removed
  • Resolution set to fixed
  • Status changed from new to closed
  • Version changed from unspecified to Git as of bugdate

Pushed, thanks!

comment:3 Changed 10 years ago by dnarvaez

  • Component changed from sugar-datastore to Sugar
Note: See TracTickets for help on using tickets.