-
Notifications
You must be signed in to change notification settings - Fork 53
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
Get other distros on board #1
Comments
Yeah, actually that was the initial idea. I tried to get @anatol (the Arch Linux maintainer) on board, since the work done in this repo is based on his Arch ruby script, but sadly I didn't get a reply to the mail I send to him. I think if we want to get other distros on board the proper way of doing so would be sending patches to them replacing their build system with this one. Unfortunately, I am very busy currently and thus didn't get around to doing this yet. |
I switched the Void Linux package to this repo void-linux/void-packages@ca265fb, it worked quite flawlessly except some glibc architectures requiring |
Which architectures? Any idea why this is the case? If there is a specfic reason why this is needed we could just add the define to the android-tools/vendor/CMakeLists.txt Lines 36 to 37 in db84fde
|
all 32bit Glibc architectures, check static_assert(sizeof(off_t) == sizeof(int64_t), "Need 64-bit lseek"); |
Fixes build on 32-bit glibc platforms. See: #1 (comment)
Added |
Submitted bug for Fedora to switch since it's super out of date. |
Anatol from Arch Linux project here. Sorry for being late to the party, 2019 was busy time for me and I completely forgotten about this activity. I just got a notification for the last message and (re-)discovered this project. Great work, @nmeum! This build system is definitely an improvement over what we have in Arch. I'll be glad to look into adopting it for Arch. If this happen then more distros are going to be in sync and share testing/development workload. |
Status for Fedora: |
Arch Linux switched its package to https://github.com/nmeum/android-tools Thank you @nmeum and other folks for your work! |
Theory: usage of |
Whoo! 🎉 I also wouldn't mind giving you push access to the repository, if you want it. If more distros end up using this we might as well move it to a dedicated GitHub organization. CI for testing the build on different distros would probably also be useful then.
I think so too, will check later today :) |
Many (maybe not Debian) used Arch as "upstream", I would expect them to follow Arch with the Switch. So far the list should be Alpine, Arch, Exherbo, Void and soon Fedora. |
@vanaf I added a patch fixing that particular build failure (fc2f2d0), it was indeed a missing include. Unfortunately, boringssl (even the current git head from https://github.com/google/boringssl) does not build with gcc 11 currently (likely due to added/improved warnings). I suppose you will also run into this issue with other software using boringssl in Fedora 34. Probably best to report it to boringssl upstream or just compile with |
Chromium builds well with boringssl on gcc-11
|
Lets discuss this further in #16 |
I'll be great to get one. I am not a cmake person, but i'll be glad to help with C/compilation related issues. |
Done, just make sure you use |
Hi, I'd love to make a pull request for Gentoo's repository, but for that this project needs to be licensed. Could you license it please? Thanks. |
That should be fixed with the most recent release (31.0.0), see #21. |
Hey @nmeum, thanks for creating this repository. I've found it from the Alpine aport in testing, and I think it makes a lot of sense to provide a sane build system for
android-tools
.How about asking the maintainers of the packages in the other distributions you mentioned in the
README.md
, if they would like to use this source repository as well (so we have more eyeballs on it, and possibly more people maintaining it)?PS: The link to the Void Linux package is outdated.
The text was updated successfully, but these errors were encountered: