search

robotology / human-dynamics-estimation

Software repository for estimating human dynamics
BSD 3-Clause "New" or "Revised" License
83 stars 28 forks source link

Fix CI on MacOS #214

Closed lrapetti closed 4 years ago

lrapetti commented 4 years ago

Currently is failing (see https://travis-ci.org/github/robotology/human-dynamics-estimation/builds/733972242).

It seems to be failing when brew install eigen, at the moment I don't have many clues regarding the reason, the error I see is:

Error: eigen 3.3.7 is already installed
lrapetti commented 4 years ago

Maybe @traversaro experienced something similar?

traversaro commented 4 years ago

Maybe @traversaro experienced something similar?

Apparently brew prints an error if a port request to be installed is already installed but it is not up to date, see https://stackoverflow.com/questions/43619480/upgrade-or-install-a-homebrew-formula for more info. A quick solution may be just to remove eigen from the installed formulas.

traversaro commented 4 years ago

Back in time on Travis I remember that we had a brewInstallOrUpgrade script exactly for this, but on GitHub Action I never had hte need for something like that (probably because no formulas are pre-installed).

lrapetti commented 4 years ago

Thanks @traversaro, I will try to adopt brewInstallOrUpgrade being currently using Travis for this repo. Still, I don't know why eigen appears to be already installed.

traversaro commented 4 years ago

Still, I don't know why eigen appears to be already installed.

The Travis macOS image can change from time to time, and it is possible that they decided to pre-install Eigen now.

claudia-lat commented 4 years ago

The related PR has been merged. Closing issue