Martchus / syncthingtray

Tray application and Dolphin/Plasma integration for Syncthing
https://martchus.github.io/syncthingtray/
Other
1.68k stars 44 forks source link

Add Fedora 33 builds to OBS #80

Closed Skirmisher closed 3 years ago

Skirmisher commented 3 years ago

Fedora 33 has been GA for about two weeks now. Thanks in advance!

Martchus commented 3 years ago

I've been adding the repo. Let's see whether every package builds. It might take a while until everything is published.

Martchus commented 3 years ago

Unfortunately the packages fail to build. Seems like some rpm macros behave differently under Fedora 33.

Skirmisher commented 3 years ago

Hmm hmm. I grabbed the srpm for c++utilities to take a look. Seems to be pretty simple, just a few changes necessary:

A quick skim of syncthingtray's spec tells me it should work after these changes as well.

Let me know if you want me to test anything else!

Martchus commented 3 years ago

I don't know why this test and others fail under recent Fedora versions:

[  157s] test 1
[  157s]     Start 1: c++utilities_run_tests
[  157s] 
[  157s] 1: Test command: /home/abuild/rpmbuild/BUILD/cpp-utilities-5.8.0/x86_64-redhat-linux-gnu/c++utilities_tests
[  157s] 1: Test timeout computed to be: 10000000
[  157s] 1: /home/abuild/rpmbuild/BUILD/cpp-utilities-5.8.0/testfiles/
[  157s] 1: ./testfiles/
[  157s] 1: Directory used to store working copies:
[  157s] 1: ./testfiles/workingdir/
[  157s] 1: [1mExecuting test cases ...[0m
[  157s] 1: ...............
[  157s] 1/1 Test #1: c++utilities_run_tests ...........Child aborted***Exception:   0.01 sec
[  157s] 
[  157s] 0% tests passed, 1 tests failed out of 1
[  157s] 
[  157s] Total Test time (real) =   0.01 sec
[  157s] 
[  157s] The following tests FAILED:
[  157s]      1 - c++utilities_run_tests (Child aborted)

Under Leap I'm getting Exception: SegFault in tests. Maybe that's related. And apparently my attempts to comment-out the running the tests are not working.

Note that the new macros were not present in Fedora < 33 - at least within my OBS build setup. So I'm only using the new macros for Fedora >= 33. I'm also generally moving to Ninja but it is unresolvable in Fedora so that's openSUSE-only for now.

I'm currently tired of this damn RPM packaging but at least the repo is now initially populated: https://download.opensuse.org/repositories/home:/mkittler/Fedora_33/x86_64

Martchus commented 3 years ago

It should work now, everything got rebuild on 01.12.