Skip to content

Commit

Permalink
Do partial revert of previous commit; gmsh-sdk not working in travis
Browse files Browse the repository at this point in the history
  • Loading branch information
@kinnala
kinnala committed Jan 18, 2020
1 parent 53ba03c commit 87363ca
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .travis.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,11 @@ python:
- "3.7"
install:
- sudo apt-get update
- sudo apt-get install gmsh
- sudo apt-get install libsuitesparse-dev
- wget http://gmsh.info/bin/Linux/gmsh-4.3.0-Linux64.tgz
- tar zvxf gmsh-4.3.0-Linux64.tgz
- sudo cp gmsh-4.3.0-Linux64/bin/gmsh /usr/bin
- wget https://repo.continuum.io/miniconda/Miniconda2-latest-Linux-x86_64.sh -O miniconda.sh;
- bash miniconda.sh -b -p $HOME/miniconda
- export PATH="$HOME/miniconda/bin:$PATH"
Expand All @@ -17,7 +21,6 @@ install:
- pip install meshio
- pip install dmsh
- pip install pygmsh
- pip install gmsh-sdk
- pip install scikit-sparse
- pip install pacopy
script:
Expand Down

3 comments on commit 87363ca

@gdmcbain
Copy link
Contributor

@gdmcbain gdmcbain commented on 87363ca Jan 18, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is disappointing. I had been hoping that gmsh-sdk would be a smooth way forward, enabling use.of the Gmsh Python API #180. I can't seem to get.to the Travis CI report from here. I'll try from the desktop tomorrow.

I wonder could it be something like apt-installing gmsh bringing in some dependencies which the binary gmsh installed by gmsh-sdk also requires but doesn't install. I might not notice this on my desktop as I already have those libraries installed for other reasons.

@kinnala
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a side note, gmsh-sdk is licensed under GPL so I'm not willing to import it in the main library.

@kinnala
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's probably not an issue to have a separate GPL-licensed file under skfem.io, though. I need to think about it.

Please sign in to comment.