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

[dbus] add lz4 as a dependency #35129

Closed
wants to merge 1 commit into from

Conversation

AnticliMaxtic
Copy link
Contributor

Update dbus port to include lz4 as a dependency

  • Changes comply with the maintainer guide
  • SHA512s are updated for each updated download
  • The "supports" clause reflects platforms that may be fixed by this new version
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

This PR fixes an issue that I have when building the dbus package. Rather than submitting an issue, I am proposing the fix directly. Here is shortend version of the error, full log is attached.

[89/121] : && /usr/bin/clang -fPIC -fPIC -fno-common  -Wall -Warray-bounds -Wcast-align -Wchar-subscripts -Wdeclaration-after-statement -Wdouble-promotion -Wextra -Wfloat-equal -Wformat-nonliteral -Wformat-security -Wformat=2 -Wimplicit-function-declaration -Winit-self -Wmissing-declarations -Wmissing-format-attribute -Wmissing-include-dirs -Wmissing-noreturn -Wmissing-prototypes -Wnested-externs -Wno-error=missing-field-initializers -Wno-error=unused-label -Wno-error=unused-parameter -Wno-missing-field-initializers -Wno-unused-label -Wno-unused-parameter -Wnull-dereference -Wold-style-definition -Wpacked -Wpointer-arith -Wpointer-sign -Wredundant-decls -Wreturn-type -Wshadow -Wsign-compare -Wstrict-aliasing -Wstrict-prototypes -Wswitch-default -Wswitch-enum -Wundef -Wunused-but-set-variable -Wwrite-strings -Wno-error=inline -Wno-error=overloaded-virtual -Wno-error=missing-field-initializers -Wno-error=null-dereference -Wno-error=strict-aliasing -Wno-error=unused-parameter -Wno-unused-parameter -g -D_DEBUG  -Wl,--export-dynamic  -Wl,--version-script=/home/mchristy/.vcpkg/buildtrees/dbus/x64-linux-dbg/dbus/Version -shared -Wl,-soname,libdbus-1.so.3 -o lib/libdbus-1.so.3.38.0 dbus/CMakeFiles/dbus-1.dir/dbus-address.c.o dbus/CMakeFiles/dbus-1.dir/dbus-auth.c.o dbus/CMakeFiles/dbus-1.dir/dbus-bus.c.o dbus/CMakeFiles/dbus-1.dir/dbus-connection.c.o dbus/CMakeFiles/dbus-1.dir/dbus-credentials.c.o dbus/CMakeFiles/dbus-1.dir/dbus-errors.c.o dbus/CMakeFiles/dbus-1.dir/dbus-keyring.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-header.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-byteswap.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-recursive.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-validate.c.o dbus/CMakeFiles/dbus-1.dir/dbus-message.c.o dbus/CMakeFiles/dbus-1.dir/dbus-misc.c.o dbus/CMakeFiles/dbus-1.dir/dbus-nonce.c.o dbus/CMakeFiles/dbus-1.dir/dbus-object-tree.c.o dbus/CMakeFiles/dbus-1.dir/dbus-pending-call.c.o dbus/CMakeFiles/dbus-1.dir/dbus-resources.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server-socket.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server-debug-pipe.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sha.c.o dbus/CMakeFiles/dbus-1.dir/dbus-signature.c.o dbus/CMakeFiles/dbus-1.dir/dbus-syntax.c.o dbus/CMakeFiles/dbus-1.dir/dbus-timeout.c.o dbus/CMakeFiles/dbus-1.dir/dbus-threads.c.o dbus/CMakeFiles/dbus-1.dir/dbus-transport.c.o dbus/CMakeFiles/dbus-1.dir/dbus-transport-socket.c.o dbus/CMakeFiles/dbus-1.dir/dbus-watch.c.o dbus/CMakeFiles/dbus-1.dir/dbus-uuidgen.c.o dbus/CMakeFiles/dbus-1.dir/dbus-transport-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-dataslot.c.o dbus/CMakeFiles/dbus-1.dir/dbus-file.c.o dbus/CMakeFiles/dbus-1.dir/dbus-hash.c.o dbus/CMakeFiles/dbus-1.dir/dbus-internals.c.o dbus/CMakeFiles/dbus-1.dir/dbus-list.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-basic.c.o dbus/CMakeFiles/dbus-1.dir/dbus-memory.c.o dbus/CMakeFiles/dbus-1.dir/dbus-mempool.c.o dbus/CMakeFiles/dbus-1.dir/dbus-string.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sysdeps.c.o dbus/CMakeFiles/dbus-1.dir/dbus-pipe.c.o dbus/CMakeFiles/dbus-1.dir/dbus-file-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-pipe-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sysdeps-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sysdeps-pthread.c.o dbus/CMakeFiles/dbus-1.dir/dbus-userdb.c.o -L/home/mchristy/.vcpkg/buildtrees/dbus/x64-linux-dbg/lib -Wl,-rpath,/home/mchristy/.vcpkg/buildtrees/dbus/x64-linux-dbg/lib:  -lsystemd  -lrt  -lcap  -lcrypt  -llz4d  -llzma  -lzstd  -lmount  -lblkid  /usr/lib/librt.a && :
FAILED: lib/libdbus-1.so.3.38.0 
: && /usr/bin/clang -fPIC -fPIC -fno-common  -Wall -Warray-bounds -Wcast-align -Wchar-subscripts -Wdeclaration-after-statement -Wdouble-promotion -Wextra -Wfloat-equal -Wformat-nonliteral -Wformat-security -Wformat=2 -Wimplicit-function-declaration -Winit-self -Wmissing-declarations -Wmissing-format-attribute -Wmissing-include-dirs -Wmissing-noreturn -Wmissing-prototypes -Wnested-externs -Wno-error=missing-field-initializers -Wno-error=unused-label -Wno-error=unused-parameter -Wno-missing-field-initializers -Wno-unused-label -Wno-unused-parameter -Wnull-dereference -Wold-style-definition -Wpacked -Wpointer-arith -Wpointer-sign -Wredundant-decls -Wreturn-type -Wshadow -Wsign-compare -Wstrict-aliasing -Wstrict-prototypes -Wswitch-default -Wswitch-enum -Wundef -Wunused-but-set-variable -Wwrite-strings -Wno-error=inline -Wno-error=overloaded-virtual -Wno-error=missing-field-initializers -Wno-error=null-dereference -Wno-error=strict-aliasing -Wno-error=unused-parameter -Wno-unused-parameter -g -D_DEBUG  -Wl,--export-dynamic  -Wl,--version-script=/home/mchristy/.vcpkg/buildtrees/dbus/x64-linux-dbg/dbus/Version -shared -Wl,-soname,libdbus-1.so.3 -o lib/libdbus-1.so.3.38.0 dbus/CMakeFiles/dbus-1.dir/dbus-address.c.o dbus/CMakeFiles/dbus-1.dir/dbus-auth.c.o dbus/CMakeFiles/dbus-1.dir/dbus-bus.c.o dbus/CMakeFiles/dbus-1.dir/dbus-connection.c.o dbus/CMakeFiles/dbus-1.dir/dbus-credentials.c.o dbus/CMakeFiles/dbus-1.dir/dbus-errors.c.o dbus/CMakeFiles/dbus-1.dir/dbus-keyring.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-header.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-byteswap.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-recursive.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-validate.c.o dbus/CMakeFiles/dbus-1.dir/dbus-message.c.o dbus/CMakeFiles/dbus-1.dir/dbus-misc.c.o dbus/CMakeFiles/dbus-1.dir/dbus-nonce.c.o dbus/CMakeFiles/dbus-1.dir/dbus-object-tree.c.o dbus/CMakeFiles/dbus-1.dir/dbus-pending-call.c.o dbus/CMakeFiles/dbus-1.dir/dbus-resources.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server-socket.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server-debug-pipe.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sha.c.o dbus/CMakeFiles/dbus-1.dir/dbus-signature.c.o dbus/CMakeFiles/dbus-1.dir/dbus-syntax.c.o dbus/CMakeFiles/dbus-1.dir/dbus-timeout.c.o dbus/CMakeFiles/dbus-1.dir/dbus-threads.c.o dbus/CMakeFiles/dbus-1.dir/dbus-transport.c.o dbus/CMakeFiles/dbus-1.dir/dbus-transport-socket.c.o dbus/CMakeFiles/dbus-1.dir/dbus-watch.c.o dbus/CMakeFiles/dbus-1.dir/dbus-uuidgen.c.o dbus/CMakeFiles/dbus-1.dir/dbus-transport-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-server-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-dataslot.c.o dbus/CMakeFiles/dbus-1.dir/dbus-file.c.o dbus/CMakeFiles/dbus-1.dir/dbus-hash.c.o dbus/CMakeFiles/dbus-1.dir/dbus-internals.c.o dbus/CMakeFiles/dbus-1.dir/dbus-list.c.o dbus/CMakeFiles/dbus-1.dir/dbus-marshal-basic.c.o dbus/CMakeFiles/dbus-1.dir/dbus-memory.c.o dbus/CMakeFiles/dbus-1.dir/dbus-mempool.c.o dbus/CMakeFiles/dbus-1.dir/dbus-string.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sysdeps.c.o dbus/CMakeFiles/dbus-1.dir/dbus-pipe.c.o dbus/CMakeFiles/dbus-1.dir/dbus-file-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-pipe-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sysdeps-unix.c.o dbus/CMakeFiles/dbus-1.dir/dbus-sysdeps-pthread.c.o dbus/CMakeFiles/dbus-1.dir/dbus-userdb.c.o -L/home/mchristy/.vcpkg/buildtrees/dbus/x64-linux-dbg/lib -Wl,-rpath,/home/mchristy/.vcpkg/buildtrees/dbus/x64-linux-dbg/lib:  -lsystemd  -lrt  -lcap  -lcrypt  -llz4d  -llzma  -lzstd  -lmount  -lblkid  /usr/lib/librt.a && :
/usr/bin/ld: cannot find -llz4d: No such file or directory
clang-16: error: linker command failed with exit code 1 (use -v to see invocation)

install-x64-linux-dbg-out.log

@dg0yt
Copy link
Contributor

dg0yt commented Nov 15, 2023

AFAICT dbus doesn't use lz4. The link library is required by libsystemd.

...  -lsystemd  -lrt  -lcap  -lcrypt  -llz4d  -llzma  -lzstd  -lmount  -lblkid  /usr/lib/librt.a && :
/usr/bin/ld: cannot find -llz4d: No such file or directory

The library would be found if the invocation had the -L<vcpkg_root/debug/lib. Not sure where it is lost.

Port libsystemd was added recently, and I expect that there will be a significant number of ports which need to be tested and fixed to interact with it properly. And so dbus really lacks a declared dependency - libsystemd used to come from the syste,

@dg0yt
Copy link
Contributor

dg0yt commented Nov 15, 2023

This can be reproduced when port libsystemd is installed first.
Exactly the type of problem I expected in #31150 (comment).

And I already opened a PR to make dbus depend on libsystemd 🤦 #34333

@AnticliMaxtic
Copy link
Contributor Author

I agree with you @dg0yt the issue is actually with libsystemd and #24333 would fix the problem more appropriately. I will close this PR. Thanks!

@dg0yt
Copy link
Contributor

dg0yt commented Nov 16, 2023

#34333 is ready for testing.

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