phev-remote / phevctl

42 stars 16 forks source link

can't build #15

Open profigriller opened 2 years ago

profigriller commented 2 years ago

I need to build it as I anned to run on a reapsberry using ARM

I get build error CMake Error: The following variables are used in this project, but they are set to NOTFOUND. Please set them or make sure they are set and tested correctly in the CMake files: ARGP_LIB linked by target "phevctl" in directory /home/pi/test/phevbuild/external/phevctl

I have no clue on how to proceed

martingjohn commented 2 years ago

I get the same thing.

I guess it's something to do with the .gitmodules

[submodule "externals/CMake-argp"]
        path = externals/CMake-argp
        url = git@github.com:alehaa/CMake-argp.git

I can't seem to clone that directly

# git clone git@github.com:alehaa/CMake-argp.git
Cloning into 'CMake-argp'...
git@github.com: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Tried changing that to the https version https://github.com/alehaa/CMake-argp.gitwhich I can clone ok but didn't seem to help.

For the moment, I've rolled back to prior to the argp addition

git checkout 486d021fb9afb953f06229bbdb2334304868a7b8
bphermansson commented 2 years ago

I use the same solution as @martingjohn, shall see if I can find a better solution.

matti commented 2 years ago

same here

bphermansson commented 2 years ago

I tried to just comment out the find_library line in CMakeLists.txt and it seems to work:

@@ -12,7 +12,7 @@ endif()

-find_library(ARGP_LIB argp) +#find_library(ARGP_LIB argp)

gmta commented 2 years ago

Looking at: https://github.com/phev-remote/phevctl/pull/13

Pulling in another external library, using /usr/local/include, not sure how this could have ever worked for anyone else but the author of that PR... I'd roll those changes back!

bphermansson commented 2 years ago

Looking at: #13

Pulling in another external library, using /usr/local/include, not sure how this could have ever worked for anyone else but the author of that PR... I'd roll those changes back!

The problem still exists?