Ticket #6226 (assigned defect)

Opened 6 years ago

Last modified 6 years ago

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
Action Needed: Verified: no
Deployments affected: Blocked By:
Blocking:

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

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

Change History

Changed 6 years ago by DanKrejsa

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

follow-up: ↓ 4   Changed 6 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?

  Changed 6 years ago by walter

  • owner changed from jg to crazy-chris
  • component changed from distro to xo-get

  Changed 6 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.

in reply to: ↑ 1   Changed 6 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)

follow-up: ↓ 6   Changed 6 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!

in reply to: ↑ 5   Changed 6 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?

  Changed 6 years ago by gregorio

  • milestone deleted

Milestone Never Assigned deleted

Note: See TracTickets for help on using tickets.