cmd/server/commands: use relative path as id instead of last part

[erizocosmico]

Fixes #681

Still can have collisions, but reduces the probability of having them.

Signed-off-by: Miguel Molina miguel@erizocosmi.co

[kuba--]

The only thing what I can think right now is if we can internally keep paths in a unix format, so always convert \ to / and if you have a windows HDD prefix, e.g.: C:\ just convert it to: /c/
WDYT?

[erizocosmico]

@kuba-- why would you want to store all paths in unix format?

[kuba--]

@erizocosmico - just to have it internally consistent (like you start gitbase on some OS and query on another). Like you have inside a zip file - all entries have unix separator(/) even if you use windows or pass \

[erizocosmico]

Sounds good to me

…

On 17 May 2019, 17:27 +0200, Kuba Podgórski ***@***.***>, wrote: @erizocosmico - just to have it internally consistent (like you start gitbase on some OS and query on another). Like you have inside a zip file - all entries have unix separator(/) even if you use windows or pass \ — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[juanjux]

Windows 10 will accept / and \ as path separators, don't know about previous versions, anyway unifying doesn't hurt.