Overview

Request 1111464 superseded

No description set
Loading...

Fabian Vogt's avatar

The

27+- Make %cmake_build insert RELWITHDEBINFO CFLAGS as well to avoid
28+  projects overriding our %optflags with some cmake default.

change is broken. We want -DNDEBUG and others to be set.


Ana Guerrero's avatar

@simotek any comments about this change for cmake? Newer versions are stuck because this.


Jan Engelhardt's avatar

It's not really stuck, is it. There are five or so packages that fell victim, libphonenumber (I think favogt determined the culprit was actually in abseil-cpp), polkit (fails because of rpmlint not cmake),.. maybe we just have to accept that some packages will break. It's not like e.g. gcc/glibc breaking some package far down the line which relied on an implicit #include has stopped gcc/glibc from getting updated (or maybe that was because it was not part of any ring, idk).


Fabian Vogt's avatar

The build failures are just a symptom of the root cause. We want -DNDEBUG for production builds.


Jan Engelhardt's avatar

I don't know what causes rust to fail in Staging:F. Building openSUSE:Factory/rust1.72 with --alternative-project F builds fine, so cmake can't be it.


Ana Guerrero's avatar

@jelgelh There were unrelated packages failing because this staging was shared by other packages waiting (like fmt) but rust failing to build comes from the cmake change. cmake is definitively stuck and I'm going to remove it while you and the other cmake maintainers agree on what to do.


Jan Engelhardt's avatar

I really cannot reproduce the problem; cd openSUSE/Factory/rust1.72; osc build -p /tmp/p (which I've filled with cmake, cmake:mini and cmke:full) produces rust1.72 successfully even with the new cmake. Can we find a new staging with just cmake and rust1.72? Maybe there was something really weird going on in the old F.


Ana Guerrero's avatar

waiting flags discussion


Ana Guerrero's avatar

Unignored: returned to active backlog.

Request History
Simon Lees's avatar

simotek created request


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


Staging Bot's avatar

staging-bot set openSUSE:Factory:Staging:F as a staging project

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


Staging Bot's avatar

staging-bot accepted review

Picked "openSUSE:Factory:Staging:F"


Marcus Rueckert's avatar

darix accepted review

Accepted review for by_group opensuse-review-team request 1111464 from user staging-bot


Ana Guerrero's avatar

anag+factory added factory-staging as a reviewer

Being evaluated by group "factory-staging"


Ana Guerrero's avatar

anag+factory accepted review

Unstaged from project "openSUSE:Factory:Staging:F"


Ana Guerrero's avatar

anag+factory superseded request

new cmake

openSUSE Build Service is sponsored by