Dmitry Eremin-Solenikov(lumag) replied on github web page:

shippable.yml
line 52
@@ -0,0 +1,56 @@
+language: c
+
+compiler:
+  - gcc
+  - clang
+
+env:
+  matrix:
+    - CONF=""
+    # - CONF="--disable-abi-compat"
+    # - CONF="--enable-schedule-sp"
+    # - CONF="--enable-schedule-iquery"
+    # - CONF="--enable-dpdk-zero-copy"
+    - CROSS_ARCH="arm64"
+    # - CROSS_ARCH="armhf" CFLAGS="-march=armv7-a"
+    # - CROSS_ARCH="powerpc"
+    # - CROSS_ARCH="i386"
+  global:
+    - PLACEHOLDER=""
+
+build:
+  pre_ci:
+    # use Dockerfile to install additional CI dependencies
+    - docker build -t=odp/dev ./scripts
+
+  # use image built in 'pre_ci' for CI job 
+  pre_ci_boot:
+    image_name: odp/dev
+    image_tag: latest
+    pull: false
+    options: 
+
+  ci:
+    - echo 1000 | sudo tee /proc/sys/vm/nr_hugepages
+    - sudo mkdir -p /mnt/huge
+    - sudo mount -t hugetlbfs nodev /mnt/huge
+    - mkdir -p /dev/shm/odp
+    - |
+      libtoolize --force --copy
+      aclocal
+      autoheader
+      automake --add-missing --include-deps --copy
+      autoconf
+      ./configure --prefix=$HOME/cunit-install --enable-debug 
--enable-automated --enable-basic --enable-console --enable-examples 
--enable-test || cat config.log
+      make
+      sudo make install
+    - ./bootstrap
+    - export 
PKG_CONFIG_PATH="$HOME/cunit-install/lib/pkgconfig:${PKG_CONFIG_PATH}"
+    - export LD_LIBRARY_PATH="$HOME/cunit-install/lib:$LD_LIBRARY_PATH"
+    - ./configure --disable-test-perf
+    - make
+    - sudo LD_LIBRARY_PATH="$HOME/cunit-install/lib:$LD_LIBRARY_PATH" 
ODP_SHM_DIR=/dev/shm/odp make check


Comment:
Replace with `sudo env ODP_SHM_DIR=/dev/shm/odp make check`

> Dmitry Eremin-Solenikov(lumag) wrote:
> Also you can drop these two lines. Again, CUnit handling


>> Dmitry Eremin-Solenikov(lumag) wrote:
>> ```diff
>> + libcunit1-dev \
>> ```


>>> Dmitry Eremin-Solenikov(lumag) wrote:
>>> This whole block (lines 38--46) is a part of CUnit handling and so should 
>>> be dropped


>>>> Dmitry Eremin-Solenikov(lumag) wrote:
>>>> This should not be necessary. Ubuntu 16.04 already provides CUnit 2.1-3. 
>>>> So it is better to use system one.


>>>>> Dmitry Eremin-Solenikov(lumag) wrote:
>>>>> Please use command from updated .travis.yml


>>>>>> muvarov wrote
>>>>>> Coverity I also thing should go away. This script does not send results 
>>>>>> to Coverity.


>>>>>>> muvarov wrote
>>>>>>> this needs to be alphabetic sorted.


https://github.com/Linaro/odp/pull/231#discussion_r145571088
updated_at 2017-10-18 23:43:05

Reply via email to