extra compiler / linker options (PR #75)


(Drew Crawford) #1

I've noticed that this failed PR:

https://github.com/apple/swift-package-manager/pull/75

Provides either a solution or a workaround for all of these open bugs:

* https://bugs.swift.org/browse/SR-397
* https://bugs.swift.org/browse/SR-415 (possible dupe of SR-397)
* https://bugs.swift.org/browse/SR-235
* https://bugs.swift.org/browse/SR-145
* https://bugs.swift.org/browse/SR-83

Can we resurrect this PR, possibly with a deprecation warning discouraging the feature's use, so as to mitigate fully 20% of the open SwiftPM bugs?

I understand the motivation why not to have this feature, but I think it may provide an effective bandaid while we are focusing on other things (testing).


(Dan Appel) #2

Hi,

In SR-145, Max Howell said he was writing up a proposal to do this on the
clang side of things. It's the right way to do this, but it's going to take
a while. For now, there's basically nothing we can do. I agree that the pr
should be merged temporarily while clang does their thing.

Dan

ยทยทยท

On Sun, Jan 3, 2016 at 6:10 AM Drew Crawford via swift-build-dev < swift-build-dev@swift.org> wrote:

I've noticed that this failed PR:

https://github.com/apple/swift-package-manager/pull/75

Provides either a solution or a workaround for all of these open bugs:

* https://bugs.swift.org/browse/SR-397
* https://bugs.swift.org/browse/SR-415 (possible dupe of SR-397)
* https://bugs.swift.org/browse/SR-235
* https://bugs.swift.org/browse/SR-145
* https://bugs.swift.org/browse/SR-83

Can we resurrect this PR, possibly with a deprecation warning discouraging
the feature's use, so as to mitigate fully 20% of the open SwiftPM bugs?

I understand the motivation why not to have this feature, but I think it
may provide an effective bandaid while we are focusing on other things
(testing).

_______________________________________________
swift-build-dev mailing list
swift-build-dev@swift.org
https://lists.swift.org/mailman/listinfo/swift-build-dev

--
Dan Appel


(Drew Crawford) #3

I agree that the pr should be merged temporarily while clang does their thing.

I have PRed this as #107 <https://github.com/apple/swift-package-manager/pull/107>.