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

Build jpackage app-image then other packages after #1319

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TheTollingBell
Copy link
Contributor

Solves #1280.
Another note for maybe the next release would be to fix up the entire package.gradle. Its a huge mess and isn't very idiomatic with Gradle.

// package generic app-image
executeCmd(jPackageFilePath + commonOpts +
// Options for creating the application launcher
" --main-jar ${mainJarName}" +
Copy link
Contributor

@woodser woodser Oct 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting a CI failure for macOS.

I see that these options will not be part of commonOpts for the executeCmd commands below, in case that could have anything to do with it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting a CI failure for macOS.

The CI build error is mv: rename desktop/build/temp-*/binaries/Haveno-*.dmg to /Users/runner/work/haveno/haveno/release/Haveno-1.0.11-mac-installer.dmg: No such file or directory, so looks like the macOS installer isn't built for some reason.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checking it out now

// Copy over app-image for AppImage packaging
copy {
from jpackageAppImage
into new File(jpackageTempDir, "Haveno")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is this hardcoded string "Haveno" used?

Just ensuring it's only temporary in the build process, since the application name / data directory can be customized for different networks, e.g. "Haveno-example" in other places like build.yml and HavenoExecutable.java

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By default, the AppImage building process expects a directory Haveno as a result of making an app-image directory so this is required to prevent changing the AppImage building process.

@woodser
Copy link
Contributor

woodser commented Oct 14, 2024

Getting a CI build error:

Caused by: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
script '/home/runner/work/haveno/haveno/desktop/package/package.gradle': 294: Unexpected input: 'jPackageFilePath + commonOpts + buildingOpts' @ line 294, column 24.
               executeCmd(jPackageFilePath + commonOpts + buildingOpts

@woodser
Copy link
Contributor

woodser commented Oct 26, 2024

Hoping to have an update to get this working :)

@woodser
Copy link
Contributor

woodser commented Oct 28, 2024

Bounty increased to 1 XMR.

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