On 16 April 2015 at 03:24, David Henningsson <david.henningsson at canonical.com> wrote: > > > On 2015-04-15 17:16, Tanu Kaskinen wrote: >> >> On Tue, 2015-04-14 at 20:32 -0300, Felipe Sateler wrote: >>> >>> From: Arun Raghavan <arun at accosted.net> >>> >>> Currently uses Ubuntu's build dependencies, and runs make check and >>> check-daemon. >>> >>> Based on Arun Raghavan's travis file. Added trusty repositories to get >>> newer libs. >> >> >> If there are failures, where are they reported? >> >>> --- >>> .travis.yml | 21 +++++++++++++++++++++ >>> 1 file changed, 21 insertions(+) >>> create mode 100644 .travis.yml >>> >>> >>> For this to be run automatically someone that controls the pulseaudio >>> name in github must link the repository to travis. >>> >>> diff --git a/.travis.yml b/.travis.yml >>> new file mode 100644 >>> index 0000000..a869372 >>> --- /dev/null >>> +++ b/.travis.yml >>> @@ -0,0 +1,21 @@ >>> +language: c >>> + >>> +compiler: >>> + - gcc >>> + #- clang >>> + >>> +before_install: >>> + - sudo sh -c 'echo "deb http://archive.ubuntu.com/ubuntu/ trusty main >>> universe\ndeb-src http://archive.ubuntu.com/ubuntu/ trusty main universe" >> >>> /etc/apt/sources.list' >>> + - sudo apt-get -qq update >>> +install: >>> + - sudo apt-get -qq build-dep pulseaudio >>> + - sudo apt-get -qq install libcap-dev libjson-c-dev autopoint git-core >>> autoconf automake gcc dh-autoreconf check intltool >> >> >> Why is the second command needed? Shouldn't "apt-get build-dep" already >> get everything needed (well, except git)? > > > Let's go through them: > > * git-core is obviously still needed > > * libcap-dev, dh-autoreconf and intltool - they are already direct build > dependencies and can be removed dh-autoreconf is not really needed. But without the explicit install step they are not upgraded. > > * libjson-c-dev, gcc and auto* - indirect build dependencies and can be > removed; potentially we could replace "gcc" with "build-essential" but I > doubt it's needed > (libjson-c-dev via libjson0-dev, auto* via dh-autoreconf) Same thing about upgrading. Arun said travis base environment's libjson-c was too old. > > * check - needed if we want travis to run "make check". (We could consider > adding a "make check" when we build pulseaudio, it's a trade-off between > increased build times and the chance of discovering something - personally I > think the risk that make check would actually fail is so low that I wonder > if it's worth running it as part of every build) Perhaps automatically running it would encourage adding more tests :) > > * In addition, we could potentially add libwebrtc-audio-processing-dev and > libbluetooth-dev to check build failures for these modules as well. Yes, we should add these. -- Saludos, Felipe Sateler
