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

tutorial - obsolete path #3

Closed
mateuszzz88 opened this issue Mar 30, 2017 · 8 comments
Closed

tutorial - obsolete path #3

mateuszzz88 opened this issue Mar 30, 2017 · 8 comments
Assignees

Comments

@mateuszzz88
Copy link

mateuszzz88 commented Mar 30, 2017

this tutorial: https://github.com/GENIVI/capicxx-dbus-tools/wiki/CommonAPI-C---D-Bus-in-10-minutes

has:
<.>/dbus-1.10.10$ patch -p1 < ../common-api-dbus-runtime/src/dbus-patches/capi-dbus-....
it should have:
<.>/dbus-1.10.10$ patch -p1 < ../capicxx-dbus-runtime/src/dbus-patches/capi-dbus-....
or even better should have:
for i in ../capicxx-dbus-runtime/src/dbus-patches/*.patch; do patch -p1 < $i; done

has in CMakeList.txt:
$ENV{RUNTIME_PATH}/capicxx-runtime/
should have:
$ENV{RUNTIME_PATH}/capicxx-core-runtime/

has in HelloWorldClient.cpp and HelloWorldStubImpl.hpp:
#include <v1_0/commonapi/HelloWorldProxy.hpp>
should have:
#include <v1/commonapi/HelloWorldProxy.hpp>
(at least this is what was generated for me)

@gunnarx
Copy link

gunnarx commented May 19, 2017

I agree and have found this error before. @GeniviCommonAPImaintainer did not fix yet, so since I have edit capability I made the proposed change for you. Thanks @mateuszzz88 for your contribution - please keep them coming.

@gunnarx
Copy link

gunnarx commented Jul 13, 2017

I guess this ticket should be closed? I had hoped @GeniviCommonAPImaintainer would check that everything is alright and comment on it.

@ywiyogo
Copy link

ywiyogo commented Nov 29, 2017

No, the missing -core- in the path in CMakeLists.txt and the v1/.. instead of v1_0/.. doesn't fixed yet.

@juliandm
Copy link

juliandm commented Jul 5, 2019

Hi, i can confirm July 2019: this documentation still needs to be fixed. Also i would propose that the following packages are explicitely mentioned as required:

cmake cmake-qt-gui libexpat-dev expat default-jre

@gunnarx
Copy link

gunnarx commented Jul 5, 2019

:-( Send a PR and I will merge it... (I am not the maintainer, but I can merge it for you).

@gunnarx
Copy link

gunnarx commented Jul 5, 2019

Oh sorry, it's in the wiki, so no PRs... OK, I can just edit it.
Hmm, it seems I edited 2 years ago also (but not all of it apparently).
I think I got all of it now @juliandm

@juliandm
Copy link

juliandm commented Jul 7, 2019

Thank you @gunnarx! This small tweak would still be necessary:

<.>/capicxx-dbus-runtime/build$ export PKG_CONFIG_PATH="/dbus-1.8.20"
should be
<.>/capicxx-dbus-runtime/build$ export PKG_CONFIG_PATH="/dbus-1.10.10"

Also i found that the code generators for the 64bit version (x86_64) didn't work for me. If that's also the case for others, maybe it's worth a look. However with a 32bit version of ubuntu they were running fine.

@gunnarx
Copy link

gunnarx commented Jul 7, 2019

OK, fixed.

The runtime error you experienced I imagine could be due to Java version?
See this: COVESA/capicxx-core-tools#15
Here's another ticket on the same
gunnarx/build-common-api-cpp-native#7

(It is mentioned as WARNING in that ticket, but it becomes an ERROR on a later java version)

@gunnarx gunnarx closed this as completed Jul 12, 2019
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

No branches or pull requests

5 participants