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

Jabref: Update to 5.15 along with packaging fixes #3605

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

A108384
Copy link
Contributor

@A108384 A108384 commented Aug 17, 2024

Summary

  • Update from 5.5 to 5.15 changelogs

  • Add homepage to package.yml

  • Update to openjdk-21

  • Include new MAINTAINERS.md

  • Include monitoring.yml

  • Include appstream metainfo

  • Resolves #3089

  • Resolves #2642

  • Related to #1389

Test Plan

  • Rebuild package and install successfully
  • Run application, creating, modifying, and reloading application

Checklist

  • Package was built and tested against unstable

@A108384 A108384 marked this pull request as ready for review August 21, 2024 10:14
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

This looks like it's including a bundled version of Java in the package. I don't think that's what we want to do.

@A108384
Copy link
Contributor Author

A108384 commented Aug 22, 2024

Yes. I think you are right. I see that now. i'll see what I can do.

packages/j/jabref/package.yml Outdated Show resolved Hide resolved
Copy link
Member

@malfisya malfisya left a comment

Choose a reason for hiding this comment

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

I couldn't get it to build on my system.

Warning] ypkg-build should be run via fakeroot, not as real root user
[Info] Building jabref-5.15
[Build] Building native package
[Source] Extracting source
[Build] Running step: build
[Build] Enabling ccache
[Build] Enabling sccache
+ cd /home/build/YPKG/root/jabref/build/jabref-5.15
+ export 'CFLAGS=-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -Wformat -Wformat-security -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ CFLAGS='-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -Wformat -Wformat-security -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ export 'CXXFLAGS=-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ CXXFLAGS='-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ export 'LDFLAGS=-Wl,--copy-dt-needed-entries -Wl,-O1 -Wl,-z,relro -Wl,-z,now -Wl,-z,max-page-size=0x1000 -Wl,-Bsymbolic-functions -Wl,--sort-common -Wl,-z,pack-relative-relocs'
+ LDFLAGS='-Wl,--copy-dt-needed-entries -Wl,-O1 -Wl,-z,relro -Wl,-z,now -Wl,-z,max-page-size=0x1000 -Wl,-Bsymbolic-functions -Wl,--sort-common -Wl,-z,pack-relative-relocs'
+ export RUSTFLAGS=-Cforce-frame-pointers
+ RUSTFLAGS=-Cforce-frame-pointers
+ export 'FFLAGS=-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ FFLAGS='-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ export 'FCFLAGS=-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ FCFLAGS='-mtune=generic -march=x86-64 -g2 -O2 -pipe -fPIC -fno-plt -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param=ssp-buffer-size=32 -fasynchronous-unwind-tables -ftree-vectorize -feliminate-unused-debug-types -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wno-error -Wp,-D_REENTRANT'
+ export PATH=/usr/lib64/ccache/bin:/usr/bin:/bin:/usr/sbin:/sbin
+ PATH=/usr/lib64/ccache/bin:/usr/bin:/bin:/usr/sbin:/sbin
+ export workdir=/home/build/YPKG/root/jabref/build/jabref-5.15
+ workdir=/home/build/YPKG/root/jabref/build/jabref-5.15
+ export package=jabref
+ package=jabref
+ export release=5
+ release=5
+ export version=5.15
+ version=5.15
+ export sources=/home/build/YPKG/sources
+ sources=/home/build/YPKG/sources
+ export pkgfiles=/home/build/work/files
+ pkgfiles=/home/build/work/files
+ export installdir=/home/build/YPKG/root/jabref/install
+ installdir=/home/build/YPKG/root/jabref/install
+ export PKG_ROOT_DIR=/home/build/YPKG/root/jabref
+ PKG_ROOT_DIR=/home/build/YPKG/root/jabref
+ export PKG_BUILD_DIR=/home/build/YPKG/root/jabref/build
+ PKG_BUILD_DIR=/home/build/YPKG/root/jabref/build
+ export LT_SYS_LIBRARY_PATH=/usr/lib64
+ LT_SYS_LIBRARY_PATH=/usr/lib64
+ export CC=x86_64-solus-linux-gcc
+ CC=x86_64-solus-linux-gcc
+ export CXX=x86_64-solus-linux-g++
+ CXX=x86_64-solus-linux-g++
+ export LD_AS_NEEDED=1
+ LD_AS_NEEDED=1
+ export RUSTC_WRAPPER=/usr/bin/sccache
+ RUSTC_WRAPPER=/usr/bin/sccache
+ export CCACHE_BASEDIR=/home/build/YPKG/root/jabref/build/jabref-5.15
+ CCACHE_BASEDIR=/home/build/YPKG/root/jabref/build/jabref-5.15
+ export SOURCE_DATE_EPOCH=1730246400
+ SOURCE_DATE_EPOCH=1730246400
+ unset DISPLAY SUDO_USER SUDO_GID SUDO_UID SUDO_COMMAND CDPATH
+ export JAVA_HOME=/usr/lib64/openjdk-21
+ JAVA_HOME=/usr/lib64/openjdk-21
+ export PATH=/usr/lib64/openjdk-21/bin:/usr/lib64/ccache/bin:/usr/bin:/bin:/usr/sbin:/sbin
+ PATH=/usr/lib64/openjdk-21/bin:/usr/lib64/ccache/bin:/usr/bin:/bin:/usr/sbin:/sbin
+ export GRADLE_USER_HOME=/home/build/YPKG/root/jabref/build/jabref-5.15
+ GRADLE_USER_HOME=/home/build/YPKG/root/jabref/build/jabref-5.15
+ gradle assemble --no-daemon --no-build-cache -PprojVersionInfo=5.15 -PprojVersion=5.15

Welcome to Gradle 8.10.2!

Here are the highlights of this release:
 - Support for Java 23
 - Faster configuration cache
 - Better configuration cache reports

For more details see https://docs.gradle.org/8.10.2/release-notes.html

To honour the JVM settings for this build a single-use Daemon process will be forked. For more on this, please refer to https://docs.gradle.org/8.10.2/userguide/gradle_daemon.html#sec:disabling_the_daemon in the Gradle documentation.
Daemon will be stopped at the end of the build 

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'JabRef'.
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not resolve com.github.koppor.gradle-modules-plugin:gradle-modules-plugin:jitpack-SNAPSHOT.
     Required by:
         root project : > com.github.koppor:gradle-modules-plugin:jitpack-SNAPSHOT:33f60fd812-1
      > Could not resolve com.github.koppor.gradle-modules-plugin:gradle-modules-plugin:jitpack-SNAPSHOT.
         > Could not get resource 'https://jitpack.io/com/github/koppor/gradle-modules-plugin/gradle-modules-plugin/jitpack-SNAPSHOT/gradle-modules-plugin-jitpack-33f60fd812-1.pom'.
            > Could not GET 'https://jitpack.io/com/github/koppor/gradle-modules-plugin/gradle-modules-plugin/jitpack-SNAPSHOT/gradle-modules-plugin-jitpack-33f60fd812-1.pom'.
               > Read timed out
> There are 6 more failures with identical causes.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 3m 24s
2 actionable tasks: 2 executed
Command '['/bin/bash', '--norc', '--noprofile', '/tmp/ypkg-buildfp46itk8']' returned non-zero exit status 1.
[Build] build failed for jabref
 ✗  Failed to build packages err="Failed to start build of package, reason: exit status 1\n"
task: Failed to run task "build-local": exit status 1

@davidjharder
Copy link
Member

Same here

@A108384
Copy link
Contributor Author

A108384 commented Nov 1, 2024

It looks like that plugin longer is available. This no longer builds for me either. I have reached out to the JabRef team asking for assistance / clarification on this plugin.

--- 2 hours later ---
I did not make any changes but after failing many times the build has been successful several times.
I wonder if the developer adjusted one of the gradle plugins...

I don't know if this is a risky situation and I need to do more research on it.. or if it builds, it builds :)

In the meantime, could you please try again? It is working for me now with no changes and I have cleared the cache and rebuilt successfully a few times.

@davidjharder @malfisya

@A108384 A108384 requested a review from malfisya November 1, 2024 20:51
@davidjharder
Copy link
Member

davidjharder commented Nov 1, 2024

Still getting a timeout on https://jitpack.io/com/github/koppor/gradle-modules-plugin/gradle-modules-plugin/jitpack-SNAPSHOT/gradle-modules-plugin-jitpack-33f60fd812-1.pom

EDIT: Tried a wget and it worked eventually; took 2 minutes. Maybe timeout can be changed to ~5min.

@davidjharder
Copy link
Member

Ok, it built for me. Something is janky with downloading that one file.
I added
-Dorg.gradle.internal.http.socketTimeout=60000 -Dorg.gradle.internal.http.connectionTimeout=60000
to the gradle line

@A108384
Copy link
Contributor Author

A108384 commented Nov 2, 2024

I had more successful rebuilds last night and again this morning with no changes. (US ET)
Should I add the timeout parameters for a better chance of successful builds and resubmit the PR?
It has been working for me, but obviously is not consistent.

@malfisya
Copy link
Member

malfisya commented Nov 2, 2024

It still keeps failing for me. The plugin is still the problem, maybe my location is the cause. So, I cannot verify the functionality of the package. If other staff can verify that jabref working properly, I will leave the review to them.

@malfisya malfisya removed their request for review November 2, 2024 12:21
@malfisya malfisya dismissed their stale review November 2, 2024 12:22

Cannot build this PR

@androidnisse
Copy link
Contributor

Timed out for me as well. In Sweden.

@davidjharder
Copy link
Member

@A108384 add the timeout flags and I'll give it a shot on the build server. Let's try it

**Summary**
- Update from 5.5 to 5.15 [changelogs](https://github.com/JabRef/jabref/releases)
- Add homepage to package.yml
- Update to openjdk-21
- Include new MAINTAINERS.md
- Include monitoring.yml
- Include appstream metainfo

- Resolves (getsolus#3086)
- Resolves (getsolus#2642)
@A108384
Copy link
Contributor Author

A108384 commented Nov 7, 2024

@A108384 add the timeout flags and I'll give it a shot on the build server. Let's try it

I have finally updated the PR. It still failed to build a few times yesterday before it was successful. But then it has built successfully several times since.

I also found this except in the jitpack.io FAQ

How do I resolve Read timed out error in Gradle?

Since version 4.3 Gradle has reduced http timeouts which can cause downloads to time out when JitPack waits for a build to finish. To increase timeouts add these settings to your gradle.properties file:

systemProp.org.gradle.internal.http.connectionTimeout=180000
systemProp.org.gradle.internal.http.socketTimeout=180000

Therefore I used 180000 for the Timeout parameters.

@davidjharder davidjharder merged commit b2d4bb8 into getsolus:main Nov 7, 2024
1 check passed
@davidjharder
Copy link
Member

Successful build!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

jabref does not build
6 participants