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

Try to fix .mv #10076

Merged
merged 8 commits into from
Jul 12, 2023
Merged

Try to fix .mv #10076

merged 8 commits into from
Jul 12, 2023

Conversation

koppor
Copy link
Member

@koppor koppor commented Jul 12, 2023

Working on fixing following:

f$ ./gradlew run

> Configure project :
Project : => 'org.jabref' Java module

> Task :run
2023-07-11 20:37:35 [main] org.jabref.cli.Launcher.main()
ERROR: Unexpected exception: org.h2.mvstore.MVStoreException: The read format 2 is smaller than the supported format 3 [2.2.220/5]
        at com.h2database.mvstore@2.2.220/org.h2.mvstore.datautils.newmvstoreexception(datautils.java:996)

There are also build errors when the update workflow is triggered manually:

https://github.com/JabRef/jabref/actions/runs/5527607325/jobs/10083532466

Starting a Gradle Daemon (subsequent builds will be faster)
/home/runner/work/jabref/jabref/buildSrc/src/copied/java/org/jabref/logic/journals/JournalAbbreviationPreferences.java:5: error: package javafx.beans.property does not exist
import javafx.beans.property.BooleanProperty;
                            ^
/home/runner/work/jabref/jabref/buildSrc/src/copied/java/org/jabref/logic/journals/JournalAbbreviationPreferences.java:6: error: package javafx.beans.property does not exist
import javafx.beans.property.SimpleBooleanProperty;
                            ^

Caused by Observable Preferences S (#9619).

Mandatory checks

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 12, 2023

The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build.

@koppor
Copy link
Member Author

koppor commented Jul 12, 2023

Hotfix, therefore merging

@koppor koppor merged commit 2abcb95 into main Jul 12, 2023
@koppor koppor deleted the fix-mv branch July 12, 2023 08:10
@calixtus
Copy link
Member

Move fast, break things, fix it. Thanks 😅

@ThiloteE
Copy link
Member

Thanks, can confirm: It's fixed :-)

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.

3 participants