flowintel-cm icon indicating copy to clipboard operation
flowintel-cm copied to clipboard

In some case, missing dependencies lacking "verbosity" on Linux (Ubuntu, experienced on 22.04 LTS)

Open tCR-lux opened this issue 6 months ago • 2 comments

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-dev package pulled with apt on 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 virtualenwrapper module at the end of the requirements.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.

tCR-lux avatar Jun 18 '25 16:06 tCR-lux

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.

tCR-lux avatar Jun 18 '25 18:06 tCR-lux

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!

DavidCruciani avatar Jul 02 '25 06:07 DavidCruciani