desktop-packages team mailing list archive

[James Ferguson <james.ferguson@xxxxxxxxxxxxx>]

** Description changed:

  Ubuntu 11.10
  Jockey 0.9.4-0ubuntu10
  
- I'm chasing another problem, but some logic looks problematic:
+ Some logic looks problematic and is causing jockey to fail to find
+ drivers on our on-disk repositories.
  
  The code in OSLib::has_repositories assumes URIs - "://" from apt-cache
  policy.  If the archive is file based it will be "file:/foo/bar/..." and
  so will not satisfy the test.
  
  Local on-disk archives are used a lot in OEM on customer's first boot.
+ 
+ ----
+ 
+ There is also a similar problem in the detection.py code -
+ DriverDB::_cache_id(self), which also assumes a "://".  When I altered
+ it to this (along with fixing the above) jockey worked as I expected:
+ 
+     m = re.match(".*::/{1,2}(.*)")
+     if m:
+          u = m.group(1)
+ 
+ But I'm not sure if that's sufficiently tight?  Perhaps
+ ".*(file:/|://)(.*)", and then group(2)?

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to jockey in Ubuntu.
https://bugs.launchpad.net/bugs/913927

Title:
  Jockey will fail against local disk archive

Status in “jockey” package in Ubuntu:
  New

Bug description:
  Ubuntu 11.10
  Jockey 0.9.4-0ubuntu10

  Some logic looks problematic and is causing jockey to fail to find
  drivers on our on-disk repositories.

  The code in OSLib::has_repositories assumes URIs - "://" from apt-
  cache policy.  If the archive is file based it will be
  "file:/foo/bar/..." and so will not satisfy the test.

  Local on-disk archives are used a lot in OEM on customer's first boot.

  ----

  There is also a similar problem in the detection.py code -
  DriverDB::_cache_id(self), which also assumes a "://".  When I altered
  it to this (along with fixing the above) jockey worked as I expected:

      m = re.match(".*::/{1,2}(.*)")
      if m:
           u = m.group(1)

  But I'm not sure if that's sufficiently tight?  Perhaps
  ".*(file:/|://)(.*)", and then group(2)?

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/jockey/+bug/913927/+subscriptions