-
Notifications
You must be signed in to change notification settings - Fork 155
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
Android build fixes #571
base: main
Are you sure you want to change the base?
Android build fixes #571
Conversation
219f610
to
36ef62e
Compare
36ef62e
to
f49b254
Compare
@martelletto can you re-run the last build? I think the Windows build failure is just a fluke "The hosted runner: GitHub Actions 13 lost communication with the server. Anything in your workflow that terminates the runner process, starves it for CPU/Memory, or blocks its network access can cause this error." |
Thank you for the PR! To merge it, we need to:
I unfortunately won't be able to look into it in the near future, but am happy to provide feedback if anyone is interested. |
@martelletto isn't it possible to simply merge this first batch of Android build fixes, considering it is truly meant to make it build, not really make it work yet? In my case I only wish to build libfido2 for Android in order to build Win32-OpenSSH for Android, simply because it depends on it. I don't even know if it's functional, but at least I can build Win32-OpenSSH now, and it should be functional even without FIDO2 support. I haven't tried building for iOS yet, but I was thinking of doing the same, at least for libssh because iOS doesn't let you build and run executables easily. |
@awakecoding, unless it is an absolute blocker for you, our preference would be to wait and test before merging. |
@martelletto it is not an absolute blocker for me, but this is as far as my current contribution will go. If you don't merge it, it will rot on a branch for years until somebody else comes along to decide and do the work all the way and helps you set up automated testing for Android at the same time. My downstream patches are currently based off the Microsoft libfido2 fork which already modifies the build system, I am trying to reduce my reliance on custom patches such that I could easily fork from the original upstream repository (this one) in the future. The current pull request is not meant to close the "Android support" feature request, as it only makes it build for now. I don't have the knowledge or experience to push it any further than that, but someone else eventually will. When this happens, we don't want duplicated efforts to just get it to build initially. The primary value for me, however, is that even if not officially supported, I can correctly build Win32-OpenSSH for Android, despite not truly supporting FIDO2 at this point. |
@awakecoding, I understand your point, and truly hope you understand ours too. Given it's not the functionality you are after, would you agree this is something that would ideally be tackled in Win32-OpenSSH? Upstream OpenSSH can be built without libfido2 (--disable-security-key). I suppose something similar could be implemented in Win32-OpenSSH. Happy to give it a shot next week if you think it is a good idea. |
Fix various issues to get libfido2 to build on Android. I have no idea if it works, but getting it to build is the first step.
I prebuilt and inject all dependencies as static libraries in my build, and I have a few more patches on my side to make it easier to inject non-default libraries. However, this would require larger changes that should probably come in a second pull request.