-
Notifications
You must be signed in to change notification settings - Fork 232
Change devtoolset-7 (scl-rh) of CentOS6 #44
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor question and one minor nit but looks good go to me. Thanks @thinker0 ! Will upload the mesos egg as soon as I get a chance to.
) | ||
build_centos6() { | ||
local mesos_version=$1 output_basedir=$2 | ||
local python_outdir=$output_basedir/centos/6/python | ||
local image_name=centos:6 | ||
|
||
pushd "$TMPDIR" | ||
docker run -i -v $(pwd):/mesos-egg $image_name bash <<EOF | ||
docker run -m=2g -i -v $(pwd):/mesos-egg $image_name bash <<EOF |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any particular reason CentOS6 requires the docker container to be limited to 2gb's of memory? Would be worth putting this as a comment if that's the case 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, Removed
set -e -u | ||
|
||
# We need Software Collections for Python 2.7 (we dropped support for 2.6). | ||
# wget must get installed before everything else to fetch the additional repos. | ||
yum -y install centos-release-SCL wget | ||
|
||
# Mesos requires gcc 4.8, which is available in devtools-2 on centos 6. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: We should leave a comment somewhere about why we're installing devtools-7 that way we know where to start looking if things break.
Usage http://mirror.centos.org/centos/6/sclo/x86_64/rh/devtoolset-7/