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

[Bug]: libuuid undefined reference to 'uuid_unpack' #2321

Closed
daohu527 opened this issue Jun 27, 2024 · 5 comments
Closed

[Bug]: libuuid undefined reference to 'uuid_unpack' #2321

daohu527 opened this issue Jun 27, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@daohu527
Copy link
Contributor

daohu527 commented Jun 27, 2024

What happened?

When building with bzlmod libuuid the following error occurs.

bazel-out/k8-fastbuild/bin/external/libuuid~/_objs/uuid/gen_uuid.pic.o:gen_uuid.c:function __uuid_generate_random: error: undefined reference to 'uuid_unpack'
bazel-out/k8-fastbuild/bin/external/libuuid~/_objs/uuid/gen_uuid.pic.o:gen_uuid.c:function uuid_generate_md5: error: undefined reference to 'uuid_unpack'
bazel-out/k8-fastbuild/bin/external/libuuid~/_objs/uuid/gen_uuid.pic.o:gen_uuid.c:function uuid_generate_sha1: error: undefined reference to 'uuid_unpack'

Version

Development (host) and target OS/architectures: Linux
Output of bazel --version: 7.2.0

Version of relevant rules from the WORKSPACE or MODULE.bazel file:

bazel_dep(name = "libuuid", version = "2.39.3")

Language(s) and/or frameworks involved:

How to reproduce

No response

Any other information?

I find modules/libuuid/2.39.3/patches/add_build_file.patch missing

+        "libuuid/src/predefined.c",
+        "libuuid/src/unpack.c",
+        "libuuid/src/unparse.c",

and 2.39.3.bcr.1 already added. Should we change to 2.39.3.bcr.1 then

@daohu527 daohu527 added the bug Something isn't working label Jun 27, 2024
@Wyverald
Copy link
Member

@wep21 (module maintainer) for comments

@Wyverald
Copy link
Member

Regarding this part:

I find modules/libuuid/2.39.3/patches/add_build_file.patch missing

+        "libuuid/src/predefined.c",
+        "libuuid/src/unpack.c",
+        "libuuid/src/unparse.c",

and 2.39.3.bcr.1 already added. Should we change to 2.39.3.bcr.1 then

we can add a new version (2.39.bcr.2) with updates to the patch.

@daohu527
Copy link
Contributor Author

I'm not expressing clear. I mean 2.39.3.bcr.1 will work. But the link https://registry.bazel.build/modules/libuuid will also recommend version 2.39.3, so do we need to confirm which version is available and which has bugs and must be upgraded.

Previously, you required that the version must be updated forward

@Wyverald
Copy link
Member

Ah, I see. The fact that registry.bazel.build thinks 2.39.3 is the newer version is a bug, tracked at bazel-contrib/bcr-ui#142.

@daohu527
Copy link
Contributor Author

OK, then close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants