Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install pkg-config files also for non-CMake builds #950

Merged
merged 1 commit into from
Sep 18, 2018

Conversation

mhei
Copy link
Contributor

@mhei mhei commented Sep 4, 2018

At the moment, pkg-config hint files are only installed when CMake is
used as build system. However, it is very convenient for programs using
libmosquitto to have these files always in place, so let's add it
here, too.

Signed-off-by: Michael Heimpold michael.heimpold@i2se.com


From my point of view, this is a fix - YMMV, so I could rebase to develop if desired...

At the moment, pkg-config hint files are only installed when CMake is
used as build system. However, it is very convenient for programs using
libmosquitto to have these files always in place, so let's add it
here, too.

Signed-off-by: Michael Heimpold <michael.heimpold@i2se.com>
karlp pushed a commit to openwrt/packages that referenced this pull request Sep 4, 2018
Install the .pc files to staging directory to help other packages
to find the libraries.

Since the build does not use CMake, we need to manually install the
files and replace two variables using sed.

Filed upstream as eclipse/mosquitto#950

Signed-off-by: Michael Heimpold <michael.heimpold@i2se.com>
Tested-by: Karl Palsson <karlp@etactica.com>
@karlp
Copy link
Contributor

karlp commented Sep 4, 2018

travis failure is about ssl cert validation failures, seems invalid for this PR?
ssl.SSLError: [Errno 1] _ssl.c:510: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed

@ralight
Copy link
Contributor

ralight commented Sep 18, 2018

Looks good to me, I'd agree it is a fix.

The ssl errors are due to expired test certificates, I've regenerated them.

@ralight ralight merged commit a459e93 into eclipse:fixes Sep 18, 2018
@ralight ralight added this to the 1.5.2 milestone Sep 18, 2018
@mhei mhei deleted the make-pkg-config branch September 18, 2018 19:04
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants