-
Notifications
You must be signed in to change notification settings - Fork 679
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
Make android payload buildable on Windows 11 #739
base: master
Are you sure you want to change the base?
Conversation
@smcintyre-r7 do you can review my PR? thanks |
Why this CI does not trigger Metasploit Framework Meterppreter Acceptance like below? like #734 |
It does not run because this is your first contribution, so a committer has to approve the CI run manually. I've approved the run. |
You can check the build failure, but it is:
This is because in our build infrastructure we do not have that version of the SDK. I strongly encourage you to use the Docker image to build like we do. See https://github.com/rapid7/metasploit-payloads/blob/master/docker/Dockerfile |
@bwatters-r7 Could you re-run CI please? |
@soroshsabz re-run'ed
@adfoster-r7 do we use the dockerfile submitted as the build, or do we maintain the build separately from the repo? |
The acceptance tests use the latest published version of the Meterpreter docker image (i.e. rapid7/msf-ubuntu-x64-meterpreter:latest), and not the dockerfile from this PR |
@adfoster-r7 build-tools-23.0.0 has been deprecated and build-tools-23.0.3 replace that how to update rapid7/msf-ubuntu-x64-meterpreter:latest ? for example where is repo? I can make PR to update docker image thanks |
@adfoster-r7 @bwatters-r7 Do you want I downgrade version and make this PR CI green and change docker later? |
any update? |
Sorry for the delay; I think we'd be good to grab the 23.0.3 version bump, does it need Java 1.8 or is it backwards compatible with 1.7? And just to take a step back - it looks like the initial goal of this PR was to add support for compiling the android payloads on a windows 11 host - but it looks like this is currently already possible by building the payloads using Docker. i.e. Since it looks like it's possible to build on windows 11 via docker, would this PR still needed - or is this PR adding something additional that's needed 👀 |
Thanks for reply This PR resolve some problems to build payload directly on Windows that make easy development on Windows (docker file changes only for consistency and try to change CI behavior before that I understand using docker hub image for CI not repo based image)
after this PR merging, I try to improve android payload building and migrate to gradle see #741 for preparing to support more modern SDK and tools |
@adfoster-r7 I hope to answer your questions :) |
ITNOA
Make buildable with JDK 1.8 and Windows 11
see #732