-
Notifications
You must be signed in to change notification settings - Fork 3
Description
Currently, the script sets $XTBPATH and $XTBHOME, assuming that also crest is delivered as part of xtb and thus present in those locations.
However, xtb and crest have been split and are provided as individual tools, with crest relying on the existence of the xtb command. Therefore, when using the precompiled binary of xtb without having crest in $XTBHOME it would still be possible to call runxtb -C crest some.xyz, which would fail. In addition, even with crest installed, the script would fail if run remotely. It would only run locally, if, and we should not assume this, crest is in $PATH.
The location of the crest executable (it is shipped as single binary) could be simply be part of the config file and be exported to $PATH in load_path_settings().