Overview

Request 836006 accepted

Automatic submission by obs-autosubmit

Loading...

Dominique Leuenberger's avatar

@WernerFink

Package makedepend appears in cycle ncurses/makedepend

Is that really needed? makedepend would need to show up in the ring0 bootstrap cycle, which in plus also requires util-macros and xorgproto in ring0; at least X-stuff in ring 0 sounds way more than what we want to see there.


Dr. Werner Fink's avatar

Ohmm ... it helps to detect dependency problems which might happen with parallel builds (-j <num> with num larger tahn 1)


Dominique Leuenberger's avatar

so the question is then do we need to run this all the time or is it good enough during the dev phase of the package - i.e. in the devel project.

This we could easily achieve with

%bcond_with makedepends
…
%if %{with makedepends}
BuildRequires: makedepend
%endif
…
%build
…
%if %{with makedepends}
make depend
%endif
…

and defining _with_makedepends 1 in prjconf of the devel project.

Then devel would make use of this dep tracker and give you the hints, without costing extra on the actual distro.

This of course is only really viable if issues mostly come from ncurses changes, and not from whatever other package changes might cause


Dr. Werner Fink's avatar

I had seen random errors on builds ... what I do not understand: why does a simple dependency tool cause such an overhead.


Dominique Leuenberger's avatar

Ring0 is the bare minimum I need to copy binaries around and in sycn to get the distro going - for hopefully obvious reasons, I want to keep that as possible, not blowing it up more than is really nescessary

ncurses is part of that ring0 bootstrap cycle already - so new deps it wants to pull in need some thoughts

Request History
Factory Maintainer's avatar

factory-maintainer created request

Automatic submission by obs-autosubmit


Richard Brown's avatar

RBrownSUSE set openSUSE:Factory:Staging:C as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:C"


Richard Brown's avatar

RBrownSUSE accepted review

Picked "openSUSE:Factory:Staging:C"


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Dominique Leuenberger's avatar

dimstar accepted review


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:C got accepted.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:C got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:C got accepted.

openSUSE Build Service is sponsored by