Oops, I had pressed reply instead reply all. The proposal <https://github.com/aciidb0mb3r/swift-evolution/blob/custom-targets-layout/proposals/NNNN-package-manager-custom-targets-layout.md> is updated with this change.
On Mon, Mar 27, 2017 at 12:32 PM, Ankit Aggarwal <[email protected]> wrote: > Hi Brent, > > I agree, this sounds like a reasonable thing to do. We can probably emit a > warning when its set to an empty array. > > Thanks! > > On Mon, Mar 27, 2017 at 10:57 AM, Brent Royal-Gordon via swift-build-dev < > [email protected]> wrote: > >> On Mar 24, 2017, at 1:26 PM, Ankit Aggarwal via swift-evolution < >> [email protected]> wrote: >> >> sources: This property defines the source files to be included in the >> target, relative to the target path. The default value of this property >> will be an empty array, which means all valid source files found in the >> target's path will be included. >> >> >> Stupid little niggle: could this be an optional array with a default >> value of `nil`? It seems more natural to treat the absence of a `sources` >> list as meaning "auto-detect" than to treat an empty `sources` list that >> way. >> >> -- >> Brent Royal-Gordon >> Architechies >> >> >> _______________________________________________ >> swift-build-dev mailing list >> [email protected] >> https://lists.swift.org/mailman/listinfo/swift-build-dev >> >> >
_______________________________________________ swift-evolution mailing list [email protected] https://lists.swift.org/mailman/listinfo/swift-evolution
