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

made vzlogger compile with libcurl4-gnutls-dev instead of libcurl4-openssl-dev #97

Merged
merged 1 commit into from
Jan 2, 2015

Conversation

mbehr1
Copy link
Contributor

@mbehr1 mbehr1 commented Jan 2, 2015

  • replaced FindCurl by FindCURL from cmake (FindCurl was doing a static link where as the libs were configured for dynamic linking)
  • MySmartgrid has dependencies to openssl. added to cmake. So we actually have to link against gnutls and openssl. We should change MySmartgrid to work with gnutls as well.

On Linux (both RPI and Ubuntu normal and Travis-CI) this works. I have no clue about Win32...

@andig
Copy link
Contributor

andig commented Jan 2, 2015

You've removed FindCurl but not added FindCURL- is this already part of cmake? Don't care about win32...

Do we need wiki updates for the vzlogger installation?

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jan 2, 2015

Yes, part of cmake

Am 02.01.2015 um 13:39 schrieb andig notifications@github.com:

You've removed FindCurl but not added FindCURL- is this already part of cmake? Don't care about win32...


Reply to this email directly or view it on GitHub #97 (comment).

Gruß

Matthias Behr

andig added a commit that referenced this pull request Jan 2, 2015
made vzlogger compile with libcurl4-gnutls-dev instead of libcurl4-openssl-dev
@andig andig merged commit dc42b69 into volkszaehler:master Jan 2, 2015
@mbehr1
Copy link
Contributor Author

mbehr1 commented Jan 2, 2015

btw: in case of problems after such a cmake config change a :
rm CMakeCache.txt

inside vzlogger directory can help.

Am 02.01.2015 um 17:49 schrieb andig notifications@github.com:

Merged #97 #97.


Reply to this email directly or view it on GitHub #97 (comment).

Gruß
Matthias=

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants