Skip to content

Comments

ci: Add Fedora to the CI environment#347

Merged
ximion merged 2 commits intoximion:masterfrom
Conan-Kudo:add-fedora-ci
Aug 29, 2021
Merged

ci: Add Fedora to the CI environment#347
ximion merged 2 commits intoximion:masterfrom
Conan-Kudo:add-fedora-ci

Conversation

@Conan-Kudo
Copy link
Contributor

As Fedora ships updated GNOME components often earlier than anyone else,
it makes sense to test this to ensure that this continues to work.

@Conan-Kudo Conan-Kudo force-pushed the add-fedora-ci branch 6 times, most recently from 5f64baf to 70f085b Compare August 29, 2021 16:20
@ximion
Copy link
Owner

ximion commented Aug 29, 2021

Do we really need a sanitized build on Fedora as well? I've deliberately put it only on one OS because it's quite slow (and getting faster feedback from the CI is often useful).
Also, Fedora Noob question: How often does rawhide break due to e.g. dependency issues in unrelated packages? If it's rarely/never, then we can leave everything as-is, otherwise we may want to ignore failures for that configuration. I'd also be fine with just testing it out, if there's no definitive answer ^^
Otherwise the PR looks fine, thanks for creating it! :-)

As Fedora ships updated GNOME components often earlier than anyone else,
it makes sense to test this to ensure that this continues to work.
daps is not broadly packaged, so the documentation cannot be built
on most distributions. In particular, it's not available on Fedora
right now, so disable it to make the build succeed.
@Conan-Kudo
Copy link
Contributor Author

Do we really need a sanitized build on Fedora as well? I've deliberately put it only on one OS because it's quite slow (and getting faster feedback from the CI is often useful).

Fedora Rawhide has the literally newest GCC, which I figured would be quite valuable for this.

Also, Fedora Noob question: How often does rawhide break due to e.g. dependency issues in unrelated packages? If it's rarely/never, then we can leave everything as-is, otherwise we may want to ignore failures for that configuration. I'd also be fine with just testing it out, if there's no definitive answer ^^

It's relatively rare these days. The main breakage is around Python rebases usually, but we've streamlined that to avoid that problem as much as possible.

Otherwise the PR looks fine, thanks for creating it! :-)

👍🏾

@ximion
Copy link
Owner

ximion commented Aug 29, 2021

Latest GCC probably includes the dev release sometimes... Which might not be the worst thing to test against, so let's give this a try!

@ximion ximion merged commit 51fa39a into ximion:master Aug 29, 2021
@Conan-Kudo Conan-Kudo deleted the add-fedora-ci branch August 29, 2021 22:54
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