search

mediamicroservices / mm

MIT License
69 stars 27 forks source link

xml sel fails in linux #205

Open privatezero opened 6 years ago

privatezero commented 6 years ago

It looks like to take advantage of some of the mmfunctions in linux (at least ubuntu 16.04) that a check will have to be added for system as the command xml sel needs to be xmlstarlet sel. If there are no objections I can make this change!

dericed commented 6 years ago

No objection. On Mac homebrew uses one as the alias of another.

retokromer commented 6 years ago

+1

privatezero commented 6 years ago

Oh, if it is an alias on Mac, then maybe it would be simpler to just change all instances of xml sel to xmlstarlet sel rather than do a system check to assign the commands?

privatezero commented 6 years ago

Also adding as a note to self - there needs to be a system check for fonts too. searchfingerprint's visual reporting will fail silently without it.