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

Building 2.0.0 fails with unresolved references #8

Closed
sertys3 opened this issue Nov 4, 2024 · 2 comments · Fixed by #9
Closed

Building 2.0.0 fails with unresolved references #8

sertys3 opened this issue Nov 4, 2024 · 2 comments · Fixed by #9

Comments

@sertys3
Copy link

sertys3 commented Nov 4, 2024

Trying to build on ubuntu with cmake fails miserably on two steps. The documented cmake command is invalid at first. It should be :
cmake -S . -B build -DCONFIG_USE_SYSTEM_HEAP=y

Yet building with : cmake --build build produces an undefined reference with :

[ 76%] Built target storage
[ 90%] Built target crypto
[ 96%] Built target milenage
[ 98%] Linking C executable softsim
/usr/bin/ld: CMakeFiles/softsim.dir/main.c.o: in function `vpcd_rx':
main.c:(.text+0x72): undefined reference to `ss_logp'
/usr/bin/ld: main.c:(.text+0xcf): undefined reference to `ss_logp'
/usr/bin/ld: main.c:(.text+0x122): undefined reference to `ss_logp'
/usr/bin/ld: main.c:(.text+0x174): undefined reference to `ss_logp'

I can only guess this is related to how dependencies are described with the cmake lists.

Please advise,
Daniel

@roni1234321
Copy link

roni1234321 commented Nov 10, 2024

try this
cmake -S . -B build -DCONFIG_USE_SYSTEM_HEAP=ON -DCONFIG_USE_LOGS=ON -DBUILD_TESTING=ON

@benjaminbruun benjaminbruun linked a pull request Nov 12, 2024 that will close this issue
@benjaminbruun
Copy link
Collaborator

Thank you for posting the observation here. I have updated the initial instructions of the readme file to include a config that actually builds.
A todo might be to do a base config that does not require the command line inputs. But at least you now know what is being chosen without anything happening automatically behind the scene.

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 a pull request may close this issue.

3 participants