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

Build failure: baresip, migrate to cmake #220762

Closed
ehmry opened this issue Mar 12, 2023 · 3 comments · Fixed by #218525
Closed

Build failure: baresip, migrate to cmake #220762

ehmry opened this issue Mar 12, 2023 · 3 comments · Fixed by #218525

Comments

@ehmry
Copy link
Contributor

ehmry commented Mar 12, 2023

Baresip fails to build after 5a65a41 because the baresip ./configure no longer finds librem through pkg-config.

Building baresip with Cmake resolves this, but then there is a new build failure:

CMake Error at cmake_install.cmake:75 (file):
  file RPATH_CHANGE could not write new RPATH:

    

  to the file:

    /nix/store/r3ycf5gqivja9krbrypbvh915vw6h3qg-baresip-2.9.0/bin/baresip

  The current RUNPATH is:

    /nix/store/r3ycf5gqivja9krbrypbvh915vw6h3qg-baresip-2.9.0/lib64:/nix/store/r3ycf5gqivja9krbrypbvh915vw6h3qg-baresip-2.9.0/lib:/nix/store/mg9l7phyhvi16p9g8g3g8fbyj4mr79gq-zlib-1.2.13/lib:/nix/store/5438qhyypn1kbdnm4312v8db4mkapwhl-openssl-3.0.8/lib:/nix/store/fnvyx1xa7hkah730bps4f9qm097wjms5-libre-2.9.0/lib:/nix/store/dkax2c0d6zjjkm4bwv52lsi8cgxg5n29-librem-2.10.0/lib:/nix/store/76l4v99sk83ylfwkz8wmwrm4s8h73rhd-glibc-2.35-224/lib:/nix/store/2w4k8nvdyiggz717ygbbxchpnxrqc6y9-gcc-12.2.0-lib/lib

  which does not contain:

    /var/empty/local/lib:/build/source/build:

  as was expected.

Steps To Reproduce

Steps to reproduce the behavior:

  1. build baresip

Notify maintainers

@elohmeier @raskin

@ehmry
Copy link
Contributor Author

ehmry commented Mar 12, 2023

@amjoseph-nixpkgs

@ghost
Copy link

ghost commented Mar 16, 2023

Sorry!

I think #218525 fixes this. I developed and tested them together but wanted to take more time to be sure baresip was finding all of its dependencies.

I didn't realize that merging one without the other would break things.

We can either merge #218525 or else revert #218518 and 218520

@ghost ghost self-assigned this Mar 16, 2023
@ehmry
Copy link
Contributor Author

ehmry commented Mar 19, 2023

Waiting for #218525 is fine by me.

@ghost ghost closed this as completed in #218525 Mar 21, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant