Re: [swift-dev] Swift CI PR builder dispatch linux failure

2016-09-26 Thread Michael Gottesman via swift-dev
Well in that case, isn't the obvious answer to move the libdispatch linux overlay into the swift? Michael > On Sep 26, 2016, at 1:23 PM, Jordan Rose via swift-dev > wrote: > > The problem with moving the Darwin Dispatch overlay there is that other > overlays depend on Dispatch, and we’re not

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 - Long Test (swift 3.0) #24

2016-09-26 Thread mishal_shah via swift-dev
Updated all of the jobs to use: (cd "${WORKSPACE}/swift-corelibs-libdispatch"; git submodule foreach --recursive git clean -fdx; git submodule init; git submodule update) > On Sep 26, 2016, at 1:00 PM, Tony Parker wrote: > > Mishal, can you help us get this fixed on the build server? > >>>

Re: [swift-dev] Swift CI PR builder dispatch linux failure

2016-09-26 Thread Jordan Rose via swift-dev
The problem with moving the Darwin Dispatch overlay there is that other overlays depend on Dispatch, and we’re not ready to move those out somewhere else. That would compound this cross-repo dependency problem. Jordan > On Sep 26, 2016, at 12:32, Daniel A. Steffen wrote: > > this may be an u

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 - Long Test (swift 3.0) #24

2016-09-26 Thread Tony Parker via swift-dev
Mishal, can you help us get this fixed on the build server? >> I think the fix for this is a one-time forced update (or removal to get a >> clean checkout) of libdispatch on the build machines. >> Thanks, - Tony > On Sep 26, 2016, at 12:19 PM, David P Grove wrote: > > Sorry, > I think the fi

Re: [swift-dev] Swift CI PR builder dispatch linux failure

2016-09-26 Thread Daniel A. Steffen via swift-dev
this may be an unintended consequence of us putting the Linux overlay into the library repo (as opposed to the compiler repo). I still think that is the right place for the overlay to live though, and that we should move the Darwin overlay there as well medium term (and work out any resulting b

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 - Long Test (swift 3.0) #24

2016-09-26 Thread David P Grove via swift-dev
Sorry, I think the fix for this is a one-time forced update (or removal to get a clean checkout) of libdispatch on the build machines. The problem is that libpwq has config.h.in checked into git. That file really shouldn't be in source control because it is generated by autotools

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 15.10 (master) #7860

2016-09-26 Thread Tony Parker via swift-dev
Thanks Pushkar. I only had one comment on that PR. Looks like we should be able to merge it soon. - Tony > On Sep 26, 2016, at 11:30 AM, Pushkar N Kulkarni > wrote: > > Only the download task tests are disabled. And this is the second time we are > seeing this data task test timing out. >

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 - Long Test (swift 3.0) #24

2016-09-26 Thread Tony Parker via swift-dev
Dave, is there something that autoconf is doing here that needs to be adjusted to fit in with the build script? Submodule path 'libkqueue': checked out '1873b1864195ff00e82bec17fa650a89c4575d41' error: Your local changes to the following files would be overwritten by checkout: config.h.

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 15.10 (master) #7860

2016-09-26 Thread Pushkar N Kulkarni via swift-dev
Only the download task tests are disabled. And this is the second time we are seeing this data task test timing out. I'd be glad to get more reviews and suggestions on: https://github.com/apple/swift-corelibs-foundation/pull/613 to have loopback tests replace these tests with external URL. Thanks.P

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 15.10 (master) #7860

2016-09-26 Thread Tony Parker via swift-dev
Hm, I thought the URLSession tests were still disabled? - Tony > On Sep 26, 2016, at 10:41 AM, mishal_shah wrote: > > Hi, > > This failure does not look like released to this commit. > > Foundation test failed due to timeout. > > Test Case 'TestURLSession.test_dataTaskWithURL' started at

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 15.10 (master) #7860

2016-09-26 Thread mishal_shah via swift-dev
Hi, This failure does not look like released to this commit. Foundation test failed due to timeout. Test Case 'TestURLSession.test_dataTaskWithURL' started at 11:00:53.813 TestFoundation/TestNSURLSession.swift:40: error: TestURLSession.test_dataTaskWithURL : Asynchronous wait failed - Excee

[swift-dev] TWISt-shout Newsletter 2016-09-26

2016-09-26 Thread Kenny Leung via swift-dev
Here is your TWISt-shout Newsletter for the week of 2016-09-19 to 2016-09-25 https://github.com/pepperdog/TWISt-shout/blob/master/2016/TWISt-shout-2016-09-26.md Enjoy! -Kenny ___ swift-dev mailing list swift-dev@swift.org https://lists.swift.org/mail

Re: [swift-dev] Swift CI PR builder dispatch linux failure

2016-09-26 Thread Brian Croom via swift-dev
When I was working this summer on getting SourceKit to build on Linux, one challenge I encountered with getting the build to work cleanly was that SourceKit is in the Swift repo, but it has a dependency on libdispatch. Moving the dispatch overlay code such that libdispatch could always be built bef

Re: [swift-dev] Swift CI PR builder dispatch linux failure

2016-09-26 Thread David P Grove via swift-dev
We think that the libdispatch incremental rebuilds didn't know that if swiftc changed it needed to invalidate the compilation of the swift dispatch overlay files (missing dependency in the make rule). A localized change that could be enough to fix the problem is libdispatch PR #178. If that isn'

Re: [swift-dev] Swift CI PR builder dispatch linux failure

2016-09-26 Thread Jordan Rose via swift-dev
Oh, I didn’t realize we had a separate copy of the overlay code (almost certainly the right thing to do at this point). But in that case, why are we seeing any of these errors? Jordan > On Sep 25, 2016, at 11:38, David P Grove wrote: > > The order may need to vary by platform. On Linux, the D