vname path should not start with "/", should incorporate root and corpus translation

kamahen / pykythe

Generate code Python source cross-reference facts in Kythe format

Other

21 stars 3 forks source link

vname path should not start with "/", should incorporate root and corpus translation #24

Open kamahen opened 5 years ago

kamahen commented 5 years ago

After discussion about why underhood didn't work, it turns out that the Kythe API expects path names to not start with "/".

The proper way of doing this is to have mappings from (corpus, root) to a spot in the file system and to pass (corpus,root,path) around wherever current "path" is used.

kamahen commented 4 years ago

Leaving this item open because the fix is a bit of a hack and a better fix should be done.