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

Support breeze5-icons 5.55.0 #106

Merged
merged 1 commit into from Jul 25, 2019
Merged

Support breeze5-icons 5.55.0 #106

merged 1 commit into from Jul 25, 2019

Conversation

Vogtinator
Copy link
Contributor

Some files changed location - breaking symlinks.

Some files changed location - breaking symlinks.
@jreidinger
Copy link
Member

in travis build still failing:

/usr/share/icons/breeze-dark/apps/32/pattern-software-management.svg -> /usr/share/icons/breeze-dark/preferences/32/yast-software-group.svg
Add the package providing the target to BuildRequires and Requires
INFO: relinking /usr/share/icons/breeze-dark/apps/32/pattern-cli.svg -> utilities-terminal.svg (was ../../apps/32/utilities-terminal.svg)
INFO: relinking /usr/share/icons/breeze/apps/48/pattern-kde-pim.svg -> kontact.svg (was ../../apps/48/kontact.svg)
INFO: relinking /usr/share/icons/breeze/apps/32/pattern-technical-writing.svg -> kwrite.svg (was ../../apps/32/kwrite.svg)
ERROR: link target doesn't exist (neither in build root nor in installed system):
/usr/share/icons/breeze/apps/32/pattern-software-management.svg -> /usr/share/icons/breeze/preferences/32/yast-software-group.svg

does it build locally with rake osc:build?

@Vogtinator
Copy link
Contributor Author

The situation at the point this PR was created (five months ago!) was a deadlock. breeze5-icons could not get accepted into Tumbleweed as this package here failed to build. Fixing this package can only happen if breeze5-icons is in Tumbleweed though as otherwise travis fails.

As this PR was ignored for a very long time, there was no choice other than adding a workaround in breeze5-icons, so now just retriggering the build should work. Apparently I lack the permissions to do that myself.

@jreidinger
Copy link
Member

@Vogtinator OK, I will retrigger it myself.

@jreidinger
Copy link
Member

ok, now passing, so lets merge

@jreidinger jreidinger merged commit ac131f7 into yast:master Jul 25, 2019
@yast-bot
Copy link
Contributor

✔️ Public Jenkins job #25 successfully finished
✔️ Created OBS submit request #718459

@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #4 successfully finished
✔️ Created IBS submit request #197442

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.

None yet

3 participants