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

Allow cross builds via construo plugin #289

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

fourlastor
Copy link

Hello! I'm the author of construo, a plugin to cross compile JVM apps. This draft is to give an example on how it could be used to build jewel for different OSes

@rock3r rock3r self-requested a review January 29, 2024 16:23
@rock3r rock3r added documentation Improvements or additions to documentation feature New feature or request labels Jan 29, 2024
@rock3r
Copy link
Collaborator

rock3r commented Jan 29, 2024

Hi Daniele, thanks for the PR! Let me know when the Windows issue is solved :)

@fourlastor
Copy link
Author

roast is no longer considered a virus 🎉 😀 from MS:

Open command prompt as administrator and change directory to c:\Program Files\Windows Defender
Run “MpCmdRun.exe -removedefinitions -dynamicsignatures”
Run "MpCmdRun.exe -SignatureUpdate"

I tested (and had the original reporter testing it as well) and it could run without issues ✨

@rock3r
Copy link
Collaborator

rock3r commented Mar 11, 2024

@fourlastor would you be able to add a docs/distribute.md file with the necessary instructions, so we can actually merge this? We can then reference it from the readme (which I already want to break down in smaller chunks when I have time, like this would be)

@fourlastor
Copy link
Author

Sure, a note though that construo doesn't support signing the executable yet, which is why I opened this as a draft

# Conflicts:
#	gradle/libs.versions.toml
@rock3r
Copy link
Collaborator

rock3r commented Mar 12, 2024

I think it's fine if it's well documented, and you reckon you'll add the signing to it and come back to update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants