[ci] added job for swig compilation on Linux with artifacts #2125
Conversation
| @@ -41,6 +41,26 @@ RUN curl -sL https://cmake.org/files/v3.14/cmake-3.14.1-Linux-x86_64.sh -o cmake | |||
| && ./cmake.sh --prefix=/usr/local --exclude-subdir \ | |||
| && rm cmake.sh | |||
|
|
|||
| # Install Java | |||
| RUN add-apt-repository ppa:openjdk-r/ppa -y \ | |||
There was a problem hiding this comment.
OpenJDK-8 is available from default ppa only since Ubuntu 14.10, but we use 14.04.
https://askubuntu.com/a/944260
| ENV JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64/ | ||
|
|
||
| # Install SWIG | ||
| RUN curl -sL https://downloads.sourceforge.net/project/swig/swig/swig-3.0.12/swig-3.0.12.tar.gz -o swig.tar.gz \ |
There was a problem hiding this comment.
apt-get installs SWIG 2.0 on Ubuntu 14.04.
There was a problem hiding this comment.
I noticed that too, thanks for fixing
| RUN curl -sL https://downloads.sourceforge.net/project/swig/swig/swig-3.0.12/swig-3.0.12.tar.gz -o swig.tar.gz \ | ||
| && tar -xzf swig.tar.gz \ | ||
| && cd swig-3.0.12 \ | ||
| && ./configure --prefix=/usr/local --without-pcre \ |
There was a problem hiding this comment.
@imatiach-msft Is it OK to build SWIG without regex? Do you need this functionality in SWIG?
There was a problem hiding this comment.
Actually not sure. Let's start with this for now and validate the binaries work.
| make -j4 || exit -1 | ||
| if [[ $COMPILER == "gcc" ]]; then | ||
| objdump -T $BUILD_DIRECTORY/lib_lightgbm.so > $BUILD_DIRECTORY/objdump.log || exit -1 | ||
| objdump -T $BUILD_DIRECTORY/lib_lightgbm_swig.so >> $BUILD_DIRECTORY/objdump.log || exit -1 |
There was a problem hiding this comment.
@imatiach-msft Are there any other dependencies which we may want to check, except those that are already mentioned here?
There was a problem hiding this comment.
nope, those are the only two. you have a script to validate glibc and gomp version? That's very cool.
|
@imatiach-msft Please review |
| @@ -25,6 +25,8 @@ jobs: | |||
| bdist: | |||
| TASK: bdist | |||
| PYTHON_VERSION: 3.6 | |||
| swig: | |||
| TASK: swig | |||
There was a problem hiding this comment.
ah, so this is how you turn that 'task' flag on, I see
| @@ -15,7 +15,7 @@ jobs: | |||
| vmImage: 'ubuntu-16.04' | |||
| container: ubuntu1404 | |||
| strategy: | |||
| maxParallel: 6 | |||
| maxParallel: 7 | |||
There was a problem hiding this comment.
what does maxParallel do?
There was a problem hiding this comment.
|
@imatiach-msft Now You can download it from Azure Pipelines page. Navigate here: https://lightgbm-ci.visualstudio.com/lightgbm-ci/_build?definitionId=1, select commit, and click to
For the latest artifacts from
|
|
@guolinke Please merge it. Since I cannot trigger main docker update from any branch other than |
|
@guolinke Also please re-run CI for merged commit when docker will be ready. I suppose CI is faster than docker and will use outdated docker image for the first time, which will result in failing CI and artifacts' unavailability. |
Closed #1945, closed #2124.
Compilation of SWIG wrapper with Clang was tested too: 3ee90d2
