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

CI: Fix failure when trying to move Qt x86 zip #260

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

RytoEX
Copy link
Member

@RytoEX RytoEX commented Sep 3, 2024

Description

Due to recent changes to remove unnecessary x86 builds, the Qt x86 artifact no longer exists, so CI fails during release creation. Instead of assuming the file exists for all architectures, use a test and an if statement to check for possible matches.

Motivation and Context

Want to be able to make obs-deps releases.

How Has This Been Tested?

It hasn't.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@RytoEX RytoEX self-assigned this Sep 3, 2024
Due to recent changes to remove unnecessary x86 builds, the Qt x86
artifact no longer exists, so CI fails during release creation. Instead
of assuming the file exists for all architectures, use a test and an if
statement to check for possible matches.
@RytoEX RytoEX merged commit d6c028e into obsproject:master Sep 3, 2024
17 checks passed
@RytoEX RytoEX deleted the fix-ci-release branch September 3, 2024 21:29
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