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

(#2017033) meson: do not fail if rsync is not installed with meson 0.57.2 #233

Merged
merged 1 commit into from Nov 24, 2021

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Nov 23, 2021

mesonbuild/meson#8641

Our CI started to fail. Even if the change is reverted in meson,
we need a quick workaround here.

(cherry picked from commit 7c5fd25)

Related: #2017033

@jamacku jamacku added this to the RHEL-8.6 milestone Nov 23, 2021
@jamacku jamacku requested a review from dtardon November 23, 2021 08:50
@systemd-rhel-bot systemd-rhel-bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Nov 23, 2021
@systemd-rhel-bot systemd-rhel-bot changed the title meson: do not fail if rsync is not installed with meson 0.57.2 (#2017033) meson: do not fail if rsync is not installed with meson 0.57.2 Nov 23, 2021
@jamacku
Copy link
Member Author

jamacku commented Nov 23, 2021

@mrc0mmand Could you please have a look at CI.

@mrc0mmand
Copy link
Member

mrc0mmand commented Nov 23, 2021

I fixed the CentOS CI job. As for GH Actions, the main culprit is test-execute (which is "just" flaky and is missing patches from upstream) and test-seccomp (which strangely fails only on C8, not C8S). Anyway, both fails are unrelated.

Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

LGTM

@systemd-rhel-bot systemd-rhel-bot removed the pr/needs-review Formerly needs-review label Nov 23, 2021
mesonbuild/meson#8641

Our CI started to fail. Even if the change is reverted in meson,
we need a quick workaround here.

(cherry picked from commit 7c5fd25)

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

Successfully merging this pull request may close these issues.

None yet

5 participants