Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docker: refactor manylinux Dockerfile #137

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

radusuciu
Copy link
Contributor

Submitting as draft because it's not currently building, though I think it's related to the build server being down.

Can we, like we did for the other Dockerfile, instead install more of the packages from yum?

@radusuciu radusuciu marked this pull request as draft February 2, 2024 17:05
python-devel \
qt5-qtsvg \
libtool \
cmake3
# RUN yum install -y libsvm-devel glpk-devel libzip-devel zlib-devel xerces-c-devel bzip2-devel sqlite-devel hdf5-devel
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, apparently we tried that before.
Until recently CoinOR (fixed) and Wildmagick (removed) were problems.
We can try CoinOR from yum if we find it.

Comment on lines +26 to +29
# TODO: do we need sqlite-devel?
# TODO: can we install boost from yum?
# TODO: is coin-or-CoinUtils-devel useful?
# TODO: do we need KissFFT? It's not installed in the regular contrib Dockerfile
Copy link
Contributor

Choose a reason for hiding this comment

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

@jpfeuffer I think we don't need KissFFT right?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think not

RUN cmake -DBUILD_TYPE=ALL ../contrib && rm -rf archives src

WORKDIR ${CONTRIB_BUILD_DIR}
RUN cmake -DBUILD_TYPE="BOOST;COINOR;KISSFFT;OPENMP" ../contrib && rm -rf archives src
Copy link
Contributor

Choose a reason for hiding this comment

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

this currently doesn't work because our IT is down (again)

Copy link
Contributor

Choose a reason for hiding this comment

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

On Linux you probably don't need contrib.
I guess even centos7 has everything we need.

Copy link
Contributor

Choose a reason for hiding this comment

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

that would simplify things a lot

Copy link
Contributor

Choose a reason for hiding this comment

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

I was too quick. Should have known that CoinOR is basically available nowhere..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, IIRC this was one of the blockers of using yum for everything

qt5-qtsvg-devel \
eigen3-devel \
python-devel
# TODO: do we need sqlite-devel?
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes. It is probably a dependency of something

eigen3-devel \
python-devel
# TODO: do we need sqlite-devel?
# TODO: can we install boost from yum?
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes probably

python-devel
# TODO: do we need sqlite-devel?
# TODO: can we install boost from yum?
# TODO: is coin-or-CoinUtils-devel useful?
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes but probably a dependency of other CoinOR libs

cmake3 \
# OpenMS and PyOpenMS build dependencies
libsvm-devel \
glpk-devel \
Copy link
Contributor

Choose a reason for hiding this comment

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

Avoid, if it works with coinor

@timosachsenberg
Copy link
Contributor

@radusuciu I hope I am not asking for too much, but can you try if we can get away with just the yum packages?

@radusuciu
Copy link
Contributor Author

@radusuciu I hope I am not asking for too much, but can you try if we can get away with just the yum packages?

Not asking for too much, that was my goal! I had some issues though but will need to take another pass after taking a look at your comments. I might not have time to do this right away but will comment back when I do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants