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

Add openJDK 20 compatibility #6758

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-latest, macOS-latest, windows-latest]
java: [ '11', '11.0.3', '15', '15.0.5']
java: [ '11.0.19', '15.0.10', '17.0.7', '20.0.1']
name: Test Java ${{ matrix.Java }}, ${{ matrix.os }}
steps:
- uses: actions/checkout@v3.5.3
Expand All @@ -23,7 +23,8 @@ jobs:
uses: actions/setup-java@v3.11.0
with:
java-version: ${{ matrix.java }}
distribution: 'zulu'
java-package: jdk+fx
distribution: zulu

- name: Build with Gradle
uses: gradle/gradle-build-action@v2.6.0
Expand Down
4 changes: 2 additions & 2 deletions build-logic/gradle-tasks/build.gradle
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
plugins {
id 'org.jetbrains.kotlin.jvm' version '1.7.10'
id 'org.gradle.kotlin.kotlin-dsl' version '2.4.1'
id 'org.jetbrains.kotlin.jvm' version '1.8.20'
id 'org.gradle.kotlin.kotlin-dsl' version '4.0.14'
}

repositories {
Expand Down
4 changes: 2 additions & 2 deletions build-logic/tor-binary/build.gradle
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
plugins {
id 'org.jetbrains.kotlin.jvm' version '1.7.10'
id 'org.gradle.kotlin.kotlin-dsl' version '2.4.1'
id 'org.jetbrains.kotlin.jvm' version '1.8.20'
id 'org.gradle.kotlin.kotlin-dsl' version '4.0.14'
}

repositories {
Expand Down
10 changes: 7 additions & 3 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ buildscript {
maven { url "https://plugins.gradle.org/m2/" }
}
dependencies {
classpath 'com.github.jengelman.gradle.plugins:shadow:5.2.0'
classpath 'org.openjfx:javafx-plugin:0.0.10'
classpath 'com.github.johnrengelman:shadow:8.1.1'
classpath 'org.openjfx:javafx-plugin:0.0.14'
}
}

Expand All @@ -22,7 +22,7 @@ configure(rootProject) {

configure(subprojects) {
ext { // in alphabetical order
javafxVersion = '16'
javafxVersion = '20.0.1'
}
}

Expand Down Expand Up @@ -77,6 +77,10 @@ configure([project(':cli'),
script.text = script.text.replace(
'DEFAULT_JVM_OPTS=""', 'DEFAULT_JVM_OPTS="-XX:MaxRAM=8g -Xss1280k -XX:+UseG1GC ' +
'-XX:MaxHeapFreeRatio=10 -XX:MinHeapFreeRatio=5 -XX:+UseStringDeduplication ' +
'--add-opens=javafx.controls/com.sun.javafx.scene.control.behavior=ALL-UNNAMED ' +
'--add-opens=javafx.controls/com.sun.javafx.scene.control=ALL-UNNAMED ' +
'--add-opens=javafx.graphics/com.sun.javafx.scene=ALL-UNNAMED ' +
'--add-opens=java.base/java.lang.reflect=ALL-UNNAMED ' +
'-Djava.net.preferIPv4Stack=true"')
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@
import bisq.core.dao.state.model.governance.DaoPhase;
import bisq.core.locale.Res;

import com.jfoenix.controls.JFXProgressBar;

import javafx.scene.control.Label;
import javafx.scene.control.ProgressBar;
import javafx.scene.layout.HBox;
Expand All @@ -44,7 +42,7 @@
@Slf4j
public class SeparatedPhaseBars extends VBox {
// Last day for creating github compensation request issue, as decided by general consensus
private static final double LAST_COMP_REQ_GH_ISSUE = (double) 18 / 25;
// private static final double LAST_COMP_REQ_GH_ISSUE = (double) 18 / 25;
private double labelMinWidth = 150;
private double breakMinWidth = 20;
private int totalDuration;
Expand All @@ -70,14 +68,14 @@ public SeparatedPhaseBars(List<SeparatedPhaseBarsItem> items) {
item.setTitleLabel(titleLabel);
titlesBars.getChildren().addAll(titleLabel);

JFXProgressBar progressBar = new JFXProgressBar();
ProgressBar progressBar = new ProgressBar();
progressBar.setMinHeight(9);
progressBar.setMaxHeight(9);
progressBar.progressProperty().bind(item.progressProperty);
progressBar.setOpacity(item.isShowBlocks() ? 1 : 0.25);
if (item.phase.name().startsWith("PROPOSAL")) {
progressBar.setSecondaryProgress(LAST_COMP_REQ_GH_ISSUE);
}
// if (item.phase.name().startsWith("PROPOSAL")) {
// progressBar.setSecondaryProgress(LAST_COMP_REQ_GH_ISSUE);
// }
progressBars.getChildren().add(progressBar);
item.setProgressBar(progressBar);
});
Expand Down
5 changes: 2 additions & 3 deletions desktop/src/main/java/bisq/desktop/main/MainView.java
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@

import com.jfoenix.controls.JFXBadge;
import com.jfoenix.controls.JFXComboBox;
import com.jfoenix.controls.JFXProgressBar;

import javafx.animation.Animation;
import javafx.animation.KeyFrame;
Expand Down Expand Up @@ -573,7 +572,7 @@ private VBox createSplashScreen() {
};
model.getWalletServiceErrorMsg().addListener(walletServiceErrorMsgListener);

btcSyncIndicator = new JFXProgressBar();
btcSyncIndicator = new ProgressBar();
btcSyncIndicator.setPrefWidth(305);
btcSyncIndicator.progressProperty().bind(model.getCombinedSyncProgress());

Expand Down Expand Up @@ -825,7 +824,7 @@ private AnchorPane createFooter() {
p2pNetworkProgressBar.setProgress(0);
});

p2pNetworkProgressBar = new JFXProgressBar(-1);
p2pNetworkProgressBar = new ProgressBar(-1);
p2pNetworkProgressBar.setMaxHeight(2);
p2pNetworkProgressBar.prefWidthProperty().bind(p2PNetworkLabel.widthProperty());

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,6 @@

import de.jensd.fx.fontawesome.AwesomeIcon;

import com.jfoenix.controls.JFXProgressBar;

import javafx.stage.Stage;

import javafx.scene.control.Label;
Expand Down Expand Up @@ -157,7 +155,7 @@ public void initialize() {
addConfidenceColumn();
addTxTypeColumn();

chainSyncIndicator = new JFXProgressBar();
chainSyncIndicator = new ProgressBar();
chainSyncIndicator.setPrefWidth(120);
chainSyncIndicator.setProgress(-1);
chainSyncIndicator.setPadding(new Insets(-6, 0, -10, 5));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,9 @@

import bisq.common.UserThread;

import com.jfoenix.controls.JFXProgressBar;

import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.control.ProgressBar;
import javafx.scene.control.Separator;
import javafx.scene.layout.GridPane;
import javafx.scene.layout.HBox;
Expand Down Expand Up @@ -138,7 +137,7 @@ private void addContent() {
addWizardsToGridPane(step3);
addRegionToGridPane();

JFXProgressBar progressBar = new JFXProgressBar();
ProgressBar progressBar = new ProgressBar();
progressBar.setMinHeight(19);
progressBar.setMaxHeight(19);
progressBar.setPrefWidth(9305);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,14 +29,11 @@
import bisq.core.user.CookieKey;
import bisq.core.user.User;

import bisq.common.UserThread;
import bisq.common.config.Config;
import bisq.common.util.Utilities;

import com.google.common.base.Joiner;

import com.jfoenix.controls.JFXProgressBar;

import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
Expand Down Expand Up @@ -150,7 +147,7 @@ private void addContent() {
downloadingFileLabel.setOpacity(0.2);
GridPane.setHalignment(downloadingFileLabel, HPos.LEFT);

progressBar = new JFXProgressBar(0L);
progressBar = new ProgressBar(0L);
progressBar.setMaxHeight(4);
progressBar.managedProperty().bind(progressBar.visibleProperty());

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,6 @@
import de.jensd.fx.fontawesome.AwesomeDude;
import de.jensd.fx.fontawesome.AwesomeIcon;

import com.jfoenix.controls.JFXProgressBar;

import javafx.scene.control.Label;
import javafx.scene.control.ProgressBar;
import javafx.scene.control.ScrollPane;
Expand Down Expand Up @@ -350,7 +348,7 @@ protected void addTradeInfoBlock() {
timeLeftTextField = labelTextFieldVBoxTuple3.second;
timeLeftTextField.setMinWidth(400);

timeLeftProgressBar = new JFXProgressBar(0);
timeLeftProgressBar = new ProgressBar(0);
timeLeftProgressBar.setOpacity(0.7);
timeLeftProgressBar.setMinHeight(9);
timeLeftProgressBar.setMaxHeight(9);
Expand Down
4 changes: 2 additions & 2 deletions gradle/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ Visit the [Gradle website](https://gradle.org/releases/) and decide the:

Adjust the following command with tha arguments above and execute it twice:

./gradlew wrapper --gradle-version 6.6.1 \
./gradlew wrapper --gradle-version 8.2 \
--distribution-type all \
--gradle-distribution-sha256-sum 11657af6356b7587bfb37287b5992e94a9686d5c8a0a1b60b87b9928a2decde5
--gradle-distribution-sha256-sum 5022b0b25fe182b0e50867e77f484501dba44feeea88f5c1f13b6b4660463640

The first execution should automatically update:

Expand Down
2 changes: 1 addition & 1 deletion gradle/libs.versions.toml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ junit-jupiter = { strictly = '5.7.0' }
kotlin = { strictly = '1.3.41' }
logback = { strictly = '1.1.11' }
lombok = { strictly = '1.18.22' }
mockito = { strictly = '4.0.0' }
mockito = { strictly = '5.4.0' }
natpryce-make-it-easy = { strictly = '4.0.1' }
netlayer = { strictly = '0.7.6' }
protobuf = { strictly = '3.19.1' }
Expand Down
Loading