Skip to content

Conversation

@munix9
Copy link
Contributor

@munix9 munix9 commented Apr 30, 2025

openSUSE Tumbleweed uses now system fmt and libusb

openSUSE Tumbleweed uses now system fmt and libusb
@Biswa96
Copy link
Collaborator

Biswa96 commented Apr 30, 2025

Could you explain why CI fails in tumbleweed? Like, do the bundle ones need any extra patch or compiler options etc.?

@munix9
Copy link
Contributor Author

munix9 commented Apr 30, 2025

I suspect an incompatibility of the bundled fmtlib with the standard clang-20 system package - but I can't say anything about the reason at the moment.

It works with clang-19, that would be an alternative:
master...munix9:android-tools:test2
https://github.com/munix9/android-tools/actions/runs/14761500418

With openSUSE, we always use the packages offered by the system if possible, as long as they are available and up-to-date.

@munix9
Copy link
Contributor Author

munix9 commented Apr 30, 2025

I guess it's a bug in fmt < 11.0.3 in interaction with clang-20.
Probably the fix was not backported in the fmt-10 branch:
fmtlib/fmt#4247
fmtlib/fmt#4177

@munix9
Copy link
Contributor Author

munix9 commented May 5, 2025

FYI:
The build with fedora:rawhide and clang (20) does not work either: https://github.com/munix9/android-tools/actions/runs/14830772511

In the meantime, version 36.0.0 of the platform-tools has been released (https://developer.android.com/tools/releases/platform-tools#revisions), but there is still no tag for it in the repository.

The fmtlib has been updated to a patched 11.0.2 version: https://android.googlesource.com/platform/external/fmtlib.git

If we can assume that version 36.0.0 uses this patched fmtlib version (fmtlib @ 97ceae9), then the build with clang 20 works again without errors: https://github.com/munix9/android-tools/actions/runs/14830990188

Now we can only wait until the tag has been created and the assumptions made above are correct.

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