openSUSE / openSUSEway

dotfiles for Sway on openSUSE
MIT License
82 stars 17 forks source link

Update issue sway 1.7 #54

Closed taschenlampe closed 1 year ago

taschenlampe commented 2 years ago

Hi!

I found following message from zypper this morning: (I guess I dont need to translate this into english)

Problem: das zu installierende sway-1.7-1.1.x86_64 erfordert 'sway-branding = 1.7', aber diese Anforderung kann nicht bereitgestellt werden
  Nicht installierbare Anbieter: sway-branding-upstream-1.7-1.1.noarch[download.opensuse.org-oss]
                   sway-branding-upstream-1.7-1.1.noarch[openSUSE-20211025-0]
 Lösung 1: Folgende Aktionen werden ausgeführt:
  Deinstallation von sway-branding-openSUSE-0.14-1.3.noarch
  Deinstallation von openSUSEway-0.14-1.3.noarch
  Deinstallation von patterns-openSUSEway-0.14-1.3.noarch
 Lösung 2: veraltetes sway-1.6.1-3.3.x86_64 beibehalten
 Lösung 3: sway-1.7-1.1.x86_64 durch Ignorieren einiger Abhängigkeiten brechen

How should I proceed as Option 1 would nuke openSWAY

Die folgenden 19 NEUEN Pakete werden installiert:
  libruby3_1-3_1 libSPIRV-Tools-suse24 libwlroots10 ruby3.1
  ruby3.1-rubygem-abstract_method ruby3.1-rubygem-cfa ruby3.1-rubygem-cfa_grub2
  ruby3.1-rubygem-cheetah ruby3.1-rubygem-fast_gettext ruby3.1-rubygem-gem2rpm
  ruby3.1-rubygem-nokogiri ruby3.1-rubygem-ruby-augeas ruby3.1-rubygem-ruby-dbus
  ruby3.1-rubygem-simpleidn ruby3.1-rubygem-unf ruby3.1-rubygem-unf_ext swaybar
  sway-branding-upstream swaynag

Die folgenden 33 Pakete werden GELÖSCHT:
  libruby2_7-2_7 libruby3_0-3_0 libSPIRV-Tools-suse23 libwlroots9 openSUSEway
  patterns-openSUSEway ruby2.7 ruby2.7-rubygem-abstract_method ruby2.7-rubygem-cfa
  ruby2.7-rubygem-cfa_grub2 ruby2.7-rubygem-cheetah ruby2.7-rubygem-fast_gettext
  ruby2.7-rubygem-gem2rpm ruby2.7-rubygem-nokogiri ruby2.7-rubygem-ruby-augeas
  ruby2.7-rubygem-ruby-dbus ruby2.7-rubygem-simpleidn ruby2.7-rubygem-unf
  ruby2.7-rubygem-unf_ext ruby3.0 ruby3.0-rubygem-abstract_method
  ruby3.0-rubygem-cfa ruby3.0-rubygem-cfa_grub2 ruby3.0-rubygem-cheetah
  ruby3.0-rubygem-fast_gettext ruby3.0-rubygem-gem2rpm ruby3.0-rubygem-nokogiri
  ruby3.0-rubygem-ruby-augeas ruby3.0-rubygem-ruby-dbus ruby3.0-rubygem-simpleidn
  ruby3.0-rubygem-unf ruby3.0-rubygem-unf_ext sway-branding-openSUSE

Das folgende Schema wird GELÖSCHT:
  openSUSEway

Of course removing openSUSEway is clearly not an option :-D

jubalh commented 2 years ago

You should choose Lösung 3: sway-1.7-1.1.x86_64 durch Ignorieren einiger Abhängigkeiten brechen.

It's caused by Requires: %{name}-branding = %{version}.

@denisok how should we handle this? Do you want to bump the branding package each time an update is done? Or should we put a <= 1.6 there and update manually each time? Remove the version thing? Nothing sounds perfect :)

denisok commented 2 years ago

@jubalh I will take a look later.

denisok commented 2 years ago

hm, I don't remmeber now how did it work, but I think this was added there: %define sway_version %(rpm -q --queryformat "%%{version}" sway) for that reason.

Maybe we should do Provides: sway-branding = %{sway_version} . But interesting how did it work before. Because there is a same thing for the waybar, and that was updated number of times.

Need to investigate and refresh my memory about branding packages.

mcepl commented 2 years ago

I am not sure whether it is exactly sway’s problem, but with the latest updates I got even greetd broken:

IMG_20220201_222718

Unfortunately, I don’t have other debugging information than that I am using:

denisok commented 2 years ago

@mcepl is that reproducible ? could you change service file and add that RUST_BACKTRACE? I am going to update and see :)

mcepl commented 2 years ago

@mcepl is that reproducible ? could you change service file and add that RUST_BACKTRACE? I am going to update and see :)

Sorry, I am too tired and with my mind too much somewhere else. Where exactly should I put it and what?

denisok commented 2 years ago

@jubalh so the issue is after these changes: https://build.opensuse.org/package/rdiff/X11:Wayland/sway?linkrev=base&rev=109#n50 https://build.opensuse.org/request/show/949422

that actually added dependency on the version. it wasn't there before. I don't know why that change was made

denisok commented 2 years ago

@mcepl is that reproducible ? could you change service file and add that RUST_BACKTRACE? I am going to update and see :)

Sorry, I am too tired and with my mind too much somewhere else. Where exactly should I put it and what?

in /usr/lib/systemd/system/greetd.service in [Service] section

Environment="RUST_BACKTRACE=1"
jubalh commented 2 years ago

@jubalh so the issue is after these changes: https://build.opensuse.org/package/rdiff/X11:Wayland/sway?linkrev=base&rev=109#n50 https://build.opensuse.org/request/show/949422

that actually added dependency on the version. it wasn't there before. I don't know why that change was made

Oh! I even asked the contributor about this whole block in: https://build.opensuse.org/request/show/948519 He mentioned rpmlint suggested this. My bad!

I can remove the version part tomorrow!

denisok commented 2 years ago

@jubalh so the issue is after these changes: https://build.opensuse.org/package/rdiff/X11:Wayland/sway?linkrev=base&rev=109#n50 https://build.opensuse.org/request/show/949422 that actually added dependency on the version. it wasn't there before. I don't know why that change was made

Oh! I even asked the contributor about this whole block in: https://build.opensuse.org/request/show/948519 He mentioned rpmlint suggested this. My bad!

I can remove the version part tomorrow!

https://build.opensuse.org/request/show/950825

done :) I tested on branched repo, update worked fine.

denisok commented 2 years ago

@jubalh so the issue is after these changes: https://build.opensuse.org/package/rdiff/X11:Wayland/sway?linkrev=base&rev=109#n50 https://build.opensuse.org/request/show/949422 that actually added dependency on the version. it wasn't there before. I don't know why that change was made

Oh! I even asked the contributor about this whole block in: https://build.opensuse.org/request/show/948519 He mentioned rpmlint suggested this. My bad!

Interesting.

it might be deprecated, but wiki still says it is not: https://en.opensuse.org/openSUSE:Packaging_Branding

I tried to specify specific version (as 1.7) but it doesn't work. There should be additional magic then. But if it is deprecated - why it works? :) Probably I tried to fix those warning in the past, that why I have sway_version macro, but I failed.

I tried once again now, but it is also fails, even if I specify 1.7 version. It couldn't satisfy the dependency.

Arnavion commented 2 years ago

I have no opinion either way on what the specfile should say. The RPM lint suggested it, and the packages worked for me with its suggestions (I don't use openSUSEway or greetd, just sway and sway-branding-upstream directly), so I didn't know there was a problem. Thanks for fixing it.

denisok commented 2 years ago

I have no opinion either way on what the specfile should say. The RPM lint suggested it, and the packages worked for me with its suggestions (I don't use openSUSEway or greetd, just sway and sway-branding-upstream directly), so I didn't know there was a problem. Thanks for fixing it.

no problem, you did well to try to fix it. Thanks for feedback and approval.

mcepl commented 2 years ago

IMG_20220203_023159

This is probably the best what I can find.

FilippoBonazziSUSE commented 1 year ago

Is this still relevant?

mcepl commented 1 year ago

Let’s kill it for now.

taschenlampe commented 1 year ago

Yes, this can be closed.