Archy - Archaeology Assistant

490 - Carbonite compatibility improvement

Expected behavior: if you use Carbonite and have its TomTom emulation turned on, Archy will use Carbonite's waypoint system.
In the mean time Archy doesn't actually know anything about Carbonite because of the Carbonite delayed load.

In the file Archy.lua there is a table local CONFIG_UPDATE_FUNCTIONS (line 968)
and there is a tomtom = function(option) ...
at the very begining of the function (about line 1002) please add the folowing line:

private.tomtomExists = (_G.TomTom and _G.TomTom.AddZWaypoint and _G.TomTom.RemoveWaypoint and true or false

so despite the fact that Carbonite loaded later You can still able to enable it in the config section, for the current session.

User When Change
Kaboca Oct 31, 2013 at 17:05 UTC Create

You must login to post a comment. Don't have an account? Register to get one!

  • 1 comment
  • Avatar of myrroddin myrroddin May 03, 2014 at 18:05 UTC - 0 likes

    I suppose adding an Optional Dep to Carbonite in the Archy's ToC would also fix this. Or even to the pkgmeta.

  • 1 comment

Facts

Reported
Oct 31, 2013
Status
New - Issue has not had initial review yet.
Type
Patch - Source code patch for review
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees