Opened 6 years ago

Last modified 6 years ago

#8396 new defect

copy-to-journal does not work from console

Reported by: bert Owned by: marco
Priority: high Milestone: 8.2.0 (was Update.2)
Component: sugar Version: not specified
Keywords: relnote r+ Cc: Phil, mstone
Blocked By: #7811 Blocking:
Deployments affected: Action Needed: test in release
Verified: no

Description

Apparently it does not connect to the user's DBus session.

Attachments (2)

copy-journal.patch (784 bytes) - added by bert 6 years ago.
0001-Make-copy-from-journal-and-copy-to-journal-work-from.patch (1.2 KB) - added by bert 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by mstone

  • Action Needed changed from never set to diagnose
  • Blocked By 7811 added
  • Cc Phil added
  • Keywords relnote added
  • Milestone changed from Not Triaged to 8.2.0 (was Update.2)
  • Priority changed from normal to high

Bert, Phil -- I'd be interested in patches which fix this but I am not presently willing to block the release on it. Thanks for testing it, though!

comment:2 Changed 6 years ago by bert

  • Cc mstone added

Simple fix - add this:

if os.path.exists("/tmp/olpc-session-bus"):
    os.environ["DBUS_SESSION_BUS_ADDRESS"] = "unix:path=/tmp/olpc-session-bus"

before importing the datastore. Attaching patch.

Changed 6 years ago by bert

comment:3 Changed 6 years ago by marco

  • Action Needed changed from diagnose to review
  • Keywords r+ added

We should really figure out how to make dbus autolaunch work on the XO... but it's OK as a temporary workaround. Thanks.

Please check in on master and sucrose-0.82

comment:4 Changed 6 years ago by marco

  • Action Needed changed from review to package

Pushed.

comment:5 Changed 6 years ago by marco

  • Action Needed changed from package to test in build

comment:6 Changed 6 years ago by marco

  • Action Needed changed from test in build to approve for release

comment:7 Changed 6 years ago by marco

sugar-datastore-0.82.1-1.olpc3

Approved.

comment:8 Changed 6 years ago by marco

  • Action Needed changed from approve for release to add to release

comment:9 Changed 6 years ago by marco

|Testcase|

Go to a text console. Add add a file to the journal using the copy-to-journal script. Check that you get no errors on the command line. Go back to sugar and check that the file has been added.

comment:10 Changed 6 years ago by cscott

  • Action Needed changed from add to release to test in release

sugar-datastore-0.82.1-1.olpc3 added to stable repository:

http://mock.laptop.org/gitweb/gitweb.cgi?p=repos;a=commitdiff;h=62bb06d5174062dc1641f9d8e384acca21f22f68

Should be in stable build 760 and later. Please test.

Note: See TracTickets for help on using tickets.