In some case, missing dependencies lacking "verbosity" on Linux (Ubuntu, experienced on 22.04 LTS)
Hello,
Very small dependency issue, but I noticed screen is not always installed on Ubuntu when apt fails with libolm-dev install and more specifically virtualenwrapper seems to be missing from the list in requirements.txt.
As a consequence, the install.sh recipe breaks.
Proposed fix:
- add the
libolm-devpackage pulled withapton a separate file (I did not yet identify if it breaks something later but I suppose without olm dev libs, no bot integration to Matrix somehow), - add
virtualenwrappermodule at the end of therequirements.txt file.
Applying the fix on my side (Ubuntu 22.04 LTS, virtualenvwrapper being installed already system wide in my config btw) allowed me to install somehow and run on localhost.
Regarding libolm-dev, because it seems to be maintained in Universe channel, like the texlive ones, I would simply put the dependecy on the same line of the apt install for texlive stuff. As it does not break the rest of the installation (but I may miss the side effect), I would simply propose to echo some explanation on this line. See https://github.com/flowintel/flowintel/pull/21 from fork.
I don't see any issue with moving libolm-dev to another line, since, as you mentioned, it's only required for the Matrix bot.
I also think it's a good idea to add verbosity to the installation script.
I'll let you finish your PR, and feel free to mark it as ready for review once you're done.
Thanks a lot for catching that!