Opened 7 years ago

Last modified 6 years ago

#6226 assigned defect

xo-get activity fails to start on update.1 build 690

Reported by: DanKrejsa Owned by: crazy-chris
Priority: high Milestone:
Component: xo-get Version:
Keywords: rainbow-integration Cc: mstone, ffm
Blocked By: Blocking:
Deployments affected: Action Needed:
Verified: no

Description

The xo-get activity, installed by the xo-get.py command-line tool, downloaded on 26 Jan 2008, fails to start on an update.1 build 690
G1G1 XO. I'll attach the xo-get activity log. (BTW, the log viewer
isn't displaying the log very well... different issue.)

Attachments (1)

tmp8QH6ne.txt (4.4 KB) - added by DanKrejsa 7 years ago.
activity log for xo-get failed startup (org.laptop.xo-get-2.log)

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by DanKrejsa

activity log for xo-get failed startup (org.laptop.xo-get-2.log)

comment:1 follow-up: Changed 7 years ago by mstone

  • Cc mstone added
  • Keywords rainbow-integration added

First - the log looks strange because' it's colored. Read it with 'less -R' and it will be very pretty.

Second - the log indicates that Rainbow's check_bundle_id() assertion is failing. This means that you're passing Rainbow a bundle-name containing characters it doesn't expect. In this case, the '-' is to blame.

This restriction arose because originally, the bundle_ids represented both D-Bus interfaces and D-Bus object paths at the same time. According to the D-Bus spec, '-' characters are not allowed in interface names or method names.

As it stands now, I'm implementing a slightly more restrictive check which requires that bundle_ids consist of period-separated blocks of alphanumerics. I could be convinced that this 'accepted language' isn't right, either because it's too small or because it's not precisely identical with the one specified by the D-Bus spec, but frankly, I'm not sure it's worth it.

Would renaming the 'xo-get' activity to 'XOGet' (or similar) be acceptable?

comment:2 Changed 7 years ago by walter

  • Component changed from distro to xo-get
  • Owner changed from jg to crazy-chris

comment:3 Changed 7 years ago by crazy-chris

  • Status changed from new to assigned

thanks for all the information. i think renaming in xoget or something similar would be the most convenient solution. i'll make a fix for the g1g1 tomorrow, monday.

comment:4 in reply to: ↑ 1 Changed 7 years ago by bert

Replying to mstone:

As it stands now, I'm implementing a slightly more restrictive check which requires that bundle_ids consist of period-separated blocks of alphanumerics.

Do you mean ASCII letters or are all unicode letters supported? (see #6148)

comment:5 follow-up: Changed 7 years ago by crazy-chris

  • Priority changed from normal to high

sorry for the delay -- i was very busy... :p i'll fix it the next days!

comment:6 in reply to: ↑ 5 Changed 7 years ago by ffm

  • Cc ffm added

Replying to crazy-chris:

sorry for the delay -- i was very busy... :p i'll fix it the next days!

any progress?

comment:7 Changed 6 years ago by gregorio

  • Milestone Never Assigned deleted

Milestone Never Assigned deleted

Note: See TracTickets for help on using tickets.