[PATCH for-4.7] travis: drop broken LLVM repos

[PATCH for-4.7] travis: drop broken LLVM repos

[Doug Goldstein]

LLVM repos are currently down so drop them from being installed so we
can get some testing back.
---
 .travis.yml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index d2e1bec..5bae60d 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -40,7 +40,6 @@ addons:
     apt:
         sources:
             - ubuntu-toolchain-r-test
-            - llvm-toolchain-precise
         packages:
             - zlib1g-dev
             - libncurses5-dev
@@ -70,7 +69,6 @@ addons:
             - gcc-aarch64-linux-gnu
             - gcc-5
             - g++-5
-            - clang-3.8
             - seabios
 # we must set CXX manually instead of using 'language: cpp' due to
 # travis-ci/travis-ci#3871
-- 
2.7.4 (Apple Git-66)


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Ian Jackson]

Doug Goldstein writes ("[PATCH for-4.7] travis: drop broken LLVM repos"):
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.

This is unfortunate churn to be having in our tree.  I'm happy for
this removal commit to go in, but isn't there a way to deal with this
that doesn't involve updating a metadata file in xen.git ?

(This kind of external repo dependency is one reason why osstest likes
to fetch its inputs from a very small number of different places, with
many files and trees copied to xenbits.)

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Wei Liu]

On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.
> ---
>  .travis.yml | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index d2e1bec..5bae60d 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -40,7 +40,6 @@ addons:
>      apt:
>          sources:
>              - ubuntu-toolchain-r-test
> -            - llvm-toolchain-precise
>          packages:
>              - zlib1g-dev
>              - libncurses5-dev
> @@ -70,7 +69,6 @@ addons:
>              - gcc-aarch64-linux-gnu
>              - gcc-5
>              - g++-5
> -            - clang-3.8
>              - seabios
>  # we must set CXX manually instead of using 'language: cpp' due to
>  # travis-ci/travis-ci#3871
> -- 
> 2.7.4 (Apple Git-66)
> 

Can we just wait until llvm sorts itself out?

I don't think the breakage is permanent, right?

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Doug Goldstein]

[-- Attachment #1.1.1: Type: text/plain, Size: 853 bytes --]

On 6/7/16 12:17 PM, Ian Jackson wrote:
> Doug Goldstein writes ("[PATCH for-4.7] travis: drop broken LLVM repos"):
>> LLVM repos are currently down so drop them from being installed so we
>> can get some testing back.
> 
> This is unfortunate churn to be having in our tree.  I'm happy for
> this removal commit to go in, but isn't there a way to deal with this
> that doesn't involve updating a metadata file in xen.git ?
> 
> (This kind of external repo dependency is one reason why osstest likes
> to fetch its inputs from a very small number of different places, with
> many files and trees copied to xenbits.)
> 
> Ian.
> 

You can see it two ways. The tests for that specific version are always
available since they're part of the repo. But if external dependencies
go away then those tests are broken.

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Doug Goldstein]

[-- Attachment #1.1.1: Type: text/plain, Size: 1313 bytes --]

On 6/7/16 12:20 PM, Wei Liu wrote:
> On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
>> LLVM repos are currently down so drop them from being installed so we
>> can get some testing back.
>> ---
>>  .travis.yml | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index d2e1bec..5bae60d 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -40,7 +40,6 @@ addons:
>>      apt:
>>          sources:
>>              - ubuntu-toolchain-r-test
>> -            - llvm-toolchain-precise
>>          packages:
>>              - zlib1g-dev
>>              - libncurses5-dev
>> @@ -70,7 +69,6 @@ addons:
>>              - gcc-aarch64-linux-gnu
>>              - gcc-5
>>              - g++-5
>> -            - clang-3.8
>>              - seabios
>>  # we must set CXX manually instead of using 'language: cpp' due to
>>  # travis-ci/travis-ci#3871
>> -- 
>> 2.7.4 (Apple Git-66)
>>
> 
> Can we just wait until llvm sorts itself out?
> 
> I don't think the breakage is permanent, right?
> 
> Wei.
> 

Its not permanent but it is now unknown when they will return. I figured
I'd toss this patch out there if the community would rather have some
build tests that pass instead of no build tests that run.

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Wei Liu]

On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
> On 6/7/16 12:20 PM, Wei Liu wrote:
> > On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
> >> LLVM repos are currently down so drop them from being installed so we
> >> can get some testing back.
> >> ---
> >>  .travis.yml | 2 --
> >>  1 file changed, 2 deletions(-)
> >>
> >> diff --git a/.travis.yml b/.travis.yml
> >> index d2e1bec..5bae60d 100644
> >> --- a/.travis.yml
> >> +++ b/.travis.yml
> >> @@ -40,7 +40,6 @@ addons:
> >>      apt:
> >>          sources:
> >>              - ubuntu-toolchain-r-test
> >> -            - llvm-toolchain-precise
> >>          packages:
> >>              - zlib1g-dev
> >>              - libncurses5-dev
> >> @@ -70,7 +69,6 @@ addons:
> >>              - gcc-aarch64-linux-gnu
> >>              - gcc-5
> >>              - g++-5
> >> -            - clang-3.8
> >>              - seabios
> >>  # we must set CXX manually instead of using 'language: cpp' due to
> >>  # travis-ci/travis-ci#3871
> >> -- 
> >> 2.7.4 (Apple Git-66)
> >>
> > 
> > Can we just wait until llvm sorts itself out?
> > 
> > I don't think the breakage is permanent, right?
> > 
> > Wei.
> > 
> 
> Its not permanent but it is now unknown when they will return. I figured
> I'd toss this patch out there if the community would rather have some
> build tests that pass instead of no build tests that run.

OK. I'm fine with taking this patch.

Wei.

> 
> -- 
> Doug Goldstein
> 




_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Jan Beulich]

>>> On 07.06.16 at 19:34, <wei.liu2@citrix.com> wrote:
> On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
>> On 6/7/16 12:20 PM, Wei Liu wrote:
>> > On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
>> >> LLVM repos are currently down so drop them from being installed so we
>> >> can get some testing back.
>> >> ---
>> >>  .travis.yml | 2 --
>> >>  1 file changed, 2 deletions(-)
>> >>
>> >> diff --git a/.travis.yml b/.travis.yml
>> >> index d2e1bec..5bae60d 100644
>> >> --- a/.travis.yml
>> >> +++ b/.travis.yml
>> >> @@ -40,7 +40,6 @@ addons:
>> >>      apt:
>> >>          sources:
>> >>              - ubuntu-toolchain-r-test
>> >> -            - llvm-toolchain-precise
>> >>          packages:
>> >>              - zlib1g-dev
>> >>              - libncurses5-dev
>> >> @@ -70,7 +69,6 @@ addons:
>> >>              - gcc-aarch64-linux-gnu
>> >>              - gcc-5
>> >>              - g++-5
>> >> -            - clang-3.8
>> >>              - seabios
>> >>  # we must set CXX manually instead of using 'language: cpp' due to
>> >>  # travis-ci/travis-ci#3871
>> >> -- 
>> >> 2.7.4 (Apple Git-66)
>> >>
>> > 
>> > Can we just wait until llvm sorts itself out?
>> > 
>> > I don't think the breakage is permanent, right?
>> > 
>> > Wei.
>> > 
>> 
>> Its not permanent but it is now unknown when they will return. I figured
>> I'd toss this patch out there if the community would rather have some
>> build tests that pass instead of no build tests that run.
> 
> OK. I'm fine with taking this patch.

Question then is - do we want this on just 4.7, or also unstable?

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Dario Faggioli]

[-- Attachment #1.1: Type: text/plain, Size: 949 bytes --]

On Wed, 2016-06-08 at 03:28 -0600, Jan Beulich wrote:
> >>> On 07.06.16 at 19:34, <wei.liu2@citrix.com> wrote:
> > On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
> 
> >> Its not permanent but it is now unknown when they will return. I
> figured
> >> I'd toss this patch out there if the community would rather have
> some
> >> build tests that pass instead of no build tests that run.
> > 
> > OK. I'm fine with taking this patch.
> 
> Question then is - do we want this on just 4.7, or also unstable?
> 
Both, I think?

So that, AFAIU the issue, people using travis will have both their 4.7
_and_ unstable build staring being tested again?

Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Wei Liu]

On Wed, Jun 08, 2016 at 03:28:56AM -0600, Jan Beulich wrote:
> >>> On 07.06.16 at 19:34, <wei.liu2@citrix.com> wrote:
> > On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
> >> On 6/7/16 12:20 PM, Wei Liu wrote:
> >> > On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
> >> >> LLVM repos are currently down so drop them from being installed so we
> >> >> can get some testing back.
> >> >> ---
> >> >>  .travis.yml | 2 --
> >> >>  1 file changed, 2 deletions(-)
> >> >>
> >> >> diff --git a/.travis.yml b/.travis.yml
> >> >> index d2e1bec..5bae60d 100644
> >> >> --- a/.travis.yml
> >> >> +++ b/.travis.yml
> >> >> @@ -40,7 +40,6 @@ addons:
> >> >>      apt:
> >> >>          sources:
> >> >>              - ubuntu-toolchain-r-test
> >> >> -            - llvm-toolchain-precise
> >> >>          packages:
> >> >>              - zlib1g-dev
> >> >>              - libncurses5-dev
> >> >> @@ -70,7 +69,6 @@ addons:
> >> >>              - gcc-aarch64-linux-gnu
> >> >>              - gcc-5
> >> >>              - g++-5
> >> >> -            - clang-3.8
> >> >>              - seabios
> >> >>  # we must set CXX manually instead of using 'language: cpp' due to
> >> >>  # travis-ci/travis-ci#3871
> >> >> -- 
> >> >> 2.7.4 (Apple Git-66)
> >> >>
> >> > 
> >> > Can we just wait until llvm sorts itself out?
> >> > 
> >> > I don't think the breakage is permanent, right?
> >> > 
> >> > Wei.
> >> > 
> >> 
> >> Its not permanent but it is now unknown when they will return. I figured
> >> I'd toss this patch out there if the community would rather have some
> >> build tests that pass instead of no build tests that run.
> > 
> > OK. I'm fine with taking this patch.
> 
> Question then is - do we want this on just 4.7, or also unstable?
> 

Both IMO.

Wei.

> Jan
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Jan Beulich]

>>> On 07.06.16 at 18:08, <cardoe@cardoe.com> wrote:
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.
> ---

I was about to commit this, but you didn't provide an S-o-b.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH for-4.7] travis: drop broken LLVM repos

[Doug Goldstein]

[-- Attachment #1.1.1: Type: text/plain, Size: 224 bytes --]

On 6/7/16 12:08 PM, Doug Goldstein wrote:
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel