[swift-build-dev] Replace PkgConfig.swift with call to pkg-config binary

[Joe DeCapo]

Thanks for the info! I hadn't really looked into it much since the app is working fine otherwise, but the warning still bugged me. I'll file an issue on GitHub with your feedback.

> On Nov 30, 2017, at 11:45 AM, Cory Benfield <cbenfield at apple.com> wrote:
> 
> 
> 
>> On 30 Nov 2017, at 17:12, Joe DeCapo <snoogansbc at gmail.com <mailto:snoogansbc at gmail.com>> wrote:
>> 
>> I've been consistently getting a warning on Heroku when building a Vapor app. I dunno if this is related, or if the issue has to do with Vapor's configuration (I don't have any experience with pkg-config).
>> 
>> warning: error while trying to use pkgConfig flags for CTLS: couldNotFindConfigFile
>> 
>> But a definite +1 if it would fix this warning.
> 
> Assuming ctls is this: https://github.com/vapor/ctls <https://github.com/vapor/ctls>, then this warning is because ctls has a pkgConfig directive that says “ctls”, but there is no ctls.pc file on the system. That’s hardly a surprise, given that ctls is a pure Swift package.
> 
> Step 1 in fixing this would be to get ctls change their .pc file from macos.pc to ctls.pc, but I don’t think that’s enough because the .pc file would still not be on the PKG_CONFIG_PATH. I don’t think the .pc file in ctls works at all, and they should just remove it and have the pkgConfig directive call for “openssl” instead. That will work fine with Homebrew and will achieve the result they’re looking for.
> 
> Cory

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-build-dev/attachments/20171130/24baefa8/attachment.html>