Re: [swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread Pierre Habouzit via swift-corelibs-dev
> On Aug 19, 2016, at 10:35 PM, William Dillon wrote: > > Ok then. At this point I suppose I'm looking at maintaining a fork of > libdispatch. I can't think of any other solutions that make sense. That sounds like a bit of an extreme answer that won’t get you a lot of sympathy. The problem

Re: [swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread William Dillon via swift-corelibs-dev
Ok then. At this point I suppose I'm looking at maintaining a fork of libdispatch. I can't think of any other solutions that make sense. > On Aug 19, 2016, at 10:31 PM, Pierre Habouzit wrote: > > Dispatch/dispatch.h is a public header. So not really. > > -Pierre on his iPhone > > On Aug 19

Re: [swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread Pierre Habouzit via swift-corelibs-dev
Dispatch/dispatch.h is a public header. So not really. -Pierre on his iPhone > On Aug 19, 2016, at 8:41 PM, William Dillon wrote: > > True enough. In that case, would be acceptable to match by architecture and > skip the import on arm? > >> On Aug 19, 2016, at 5:56 PM, Pierre Habouzit wrot

Re: [swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread William Dillon via swift-corelibs-dev
True enough. In that case, would be acceptable to match by architecture and skip the import on arm? > On Aug 19, 2016, at 5:56 PM, Pierre Habouzit wrote: > > the include was added to dispatch specifically to allow dispatch_io to build > on intel so your patch I think would break Intel. > > I

Re: [swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread Pierre Habouzit via swift-corelibs-dev
the include was added to dispatch specifically to allow dispatch_io to build on intel so your patch I think would break Intel. I think the general problem is likely that glibc is not module friendly today. -Pierre > On Aug 19, 2016, at 11:53 AM, William Dillon via swift-corelibs-dev > wrote:

Re: [swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread Tony Parker via swift-corelibs-dev
One of the challenges we have in removing imports in public headers is that other projects are importing those headers transitively. Someone else may be assuming they get stdio.h by importing CoreFoundation.h or dispatch.h, and using something from that header. - Tony > On Aug 19, 2016, at 11:

[swift-corelibs-dev] Duplicate definition of va_list on Arm

2016-08-19 Thread William Dillon via swift-corelibs-dev
Hi all, In corelibs-foundation project we've been using a patch based on https://github.com/apple/swift-corelibs-foundation/pull/399/files for quite some time (summary: remove #include ). The PR hasn't gotten any where for va

Re: [swift-corelibs-dev] Building with Xcode

2016-08-19 Thread Luke Howard via swift-corelibs-dev
Are thank you, using the workspace fixed it. Sorry for the noise! > On 19 Aug 2016, at 7:27 AM, Brian Gesiak wrote: > > I use the following to build Foundation and XCTest, then run the XCTest tests: > > swift $ utils/build-script --preset corelibs-xctest > > The XCTest.xcworkspace is also cap