Overview

Request 1082923 accepted

Proposal to remove this package. There is nothing left (well, fox16, SR 1082922) that would appear to depend on this by name.


Dominique Leuenberger's avatar
> dependson xorg-x11-libs
xorg-x11-libs :
   OpenSceneGraph
   csound
   eaglemode
   fox16
   ghostscript
   gmic
   gwenhywfar
   libqt5-qtdoc
   mc
   perl-GD
   perl-Prima
   perl-Tk
   perl-Tk-TableMatrix
   python-pygame
   st
   telegram-desktop
   tkimg
   twinkle
   veyon
   wordnet
   workrave
   xsnow

That's a lot for 'nothing'


Jan Engelhardt's avatar

I don't know how OBS gets to that conclusion that (e.g.) st is affected.

$ git clone https://github.com/bmwiedemann/openSUSE && cd openSUSE
$ git grep 'quires.*x11-libs\s*$'
packages/f/fox16/fox16.spec:Requires:       xorg-x11-libs

.

$ osc co openSUSE:Factory/st && cd st && osc build && osc chroot
abuild@a4:~> rpm -e xorg-x11-libs-devel xorg-x11-libs
error: package xorg-x11-libs-devel is not installed
error: package xorg-x11-libs is not installed

Stefan Dirsch's avatar

@jengelh What about xorg-x11-devel subpackage? I could imagine this would still be buildrequired by some packages. Possibly exactly the list Dominique mentioned above?


Jan Engelhardt's avatar

oooh that's a nasty naming scheme.

  • X11:XOrg/xorg-x11: xorg-x11 <s>xorg-x11-devel</s>
  • X11:XOrg/xorg-x11-libs: xorg-x11-libs --xorg-x11-libs-devel-- xorg-x11-devel


Stefan Dirsch's avatar

Ok. I think it's obvious that @jengelh didn't check for xorg-x11-devel Requires/BuildRequires. I can confirm this list depending on xorg-x11-devel BuildRequires:

csound
eaglemode
fox16
ghostscript
gmic
libqt5-qtdoc
mc
perl-GD
perl-Prima
perl-Tk
perl-Tk-TableMatrix
python-pygame
st
telegram-desktop
tkimg
twinkle
veyon
wordnet
workrave
xsnow

Jan Engelhardt's avatar

give at least a reason so we can work on it?


Stefan Dirsch's avatar

Well, you see the list Dominique and I posted, right? The potential issue here is, that by replacing BuildRequires: xorg-x11-devel by pkgconfig(...) lines you may miss some, which silently results in loss of functionality in one of these programs without noticing it. :-( I don't say it wouldn't be worth the effort and I would be happy eventually removing this meta package, but doing is not without a risk ...


Jan Engelhardt's avatar

The list has been operated on, I have made SRs for all the packages. All but 1 are accepted (as usual - a maintainer has seemingly disappeared).

How could I miss some? OBS's answer as to who's utilizing xorg-x11-(devel/libs) should be absolute and transitive, in other words, we should see installation failures in openQA.


Stefan Dirsch's avatar

Ok. So maybe you can do an update on this list and what SRs are still pending then.


Jan Engelhardt's avatar

Certainly.

$ osc whatdependson openSUSE:Factory/xorg-x11-libs standard x86_64
xorg-x11-libs :
      veyon [sr 1087904]

@dimstar can you step in like in the fox16 case?


Stefan Dirsch's avatar
osc maintainer openSUSE:Factory veyon
bugowner of Education/veyon : 
 andythe_great

osc whois andythe_great
andythe_great: "andy great" <andythe_great@pm.me>

Maybe you can contact him directly.


Jan Engelhardt's avatar

SR was finally accepted

Request History
Jan Engelhardt's avatar

jengelh created request

Proposal to remove this package. There is nothing left (well, fox16, SR 1082922) that would appear to depend on this by name.


Saul Goodman's avatar

licensedigger accepted review


Factory Auto's avatar

factory-auto added xorg-x11-libs as a reviewer

submitter not devel maintainer


Factory Auto's avatar

factory-auto accepted review

ok


Jan Engelhardt's avatar

jengelh revoked request


Jan Engelhardt's avatar

jengelh reopened request

all dependent packages have been adjusted or have their SR pending now


Dominique Leuenberger's avatar

dimstar_suse added as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:adi:58"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked "openSUSE:Factory:Staging:adi:58"


Stefan Dirsch's avatar

sndirsch declined review

reject


Stefan Dirsch's avatar

sndirsch declined request

reject


Jan Engelhardt's avatar

jengelh reopened request

i'd love to reopen for reconsideration, all users actually gone now


Stefan Dirsch's avatar

sndirsch accepted review

I fully agree. Please go ahead and remove the package from factory. I may still need it in X11:XOrg though, since I'm building there packages still for older distributions.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:adi:58 got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:adi:58 got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:adi:58 got accepted.

openSUSE Build Service is sponsored by