Overview
Request 770034 accepted
- Upgrade to latest snapshot from 2020-01-17
- disable-live-tests.patch: refreshed
- regression.patch: fix a regression in DNS results that contain
both A and AAAA answers.
- Add netcfg as the build requirement and runtime requirement.
ares_getaddrinfo function uses the getservbyport_r function which
requires the /etc/services file to function properly. That config
file is provided by the netcfg package. Unit tests rely on it
too, hence it has to be a build dependency as well.
- Switch to cmake-based build.
Some packages need the cmake build files.
openSUSE:Factory/c-ares@1 -> openSUSE:Leap:15.0/c-ares
the submitted sources are in or accepted for Factory
I think this should not be accepted. c-ares comes as libcares2 from SLE-15 and this is just a rename of that package. To have new package here, it has to go via SLE-15:GA and then the old package can be dropped from Leap.
- Switch to cmake-based build.
seems to cause
Package nghttp2 appears in cycle nghttp2/c-ares Package c-ares appears in cycle nghttp2/c-ares
The issue seems to be with cmake.
A bunch of dependencies of cmake can be built with cmake. When that happens, they generate cmake information files used by other packages. Some packages outright require these files, they won't build at all without it (I am dealing with one such package right now). So if we say that the dependencies of cmake cannot be built using cmake, then this limits the useful of those packages in other cmake-based builds, or outright prevents the packaging of packages that depend on them.
Perhaps there is a way to manually generate the required information files without actually directly using cmake, or by using a multibuild, but I don't know if that is feasible or reliable.
Request History
mrostecki created request
- Upgrade to latest snapshot from 2020-01-17
- disable-live-tests.patch: refreshed
- regression.patch: fix a regression in DNS results that contain
both A and AAAA answers.
- Add netcfg as the build requirement and runtime requirement.
ares_getaddrinfo function uses the getservbyport_r function which
requires the /etc/services file to function properly. That config
file is provided by the netcfg package. Unit tests rely on it
too, hence it has to be a build dependency as well.
- Switch to cmake-based build.
Some packages need the cmake build files.
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
scarabeus_iv accepted review
dimstar_suse set openSUSE:Factory:Staging:L as a staging project
Being evaluated by staging project "openSUSE:Factory:Staging:L"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:L"
dimstar accepted review
okurz-factory accepted review
Staging Project openSUSE:Factory:Staging:L got accepted.
okurz-factory approved review
Staging Project openSUSE:Factory:Staging:L got accepted.
okurz-factory accepted request
Staging Project openSUSE:Factory:Staging:L got accepted.
https://build.opensuse.org/request/show/761679
basically, we have a cmake problem now
yeah i need time to solve it, probably by doing cmake mini flavor
Still a cycle with cmake - we need a solution to that (possibly a cmake-OBS, without curl bindings)
@scarabeus_iv @dimstar_suse: Can we maybe revert the cmake change for now and sort it out in the follow up SR? I urgently need the c-ares update to unlock Cilium and Envoy in Kubic and CaaSP. :/
https://build.opensuse.org/request/show/770285
the cmake fix is already submitted so it should go through with this change
Cool! Thanks for clarification.
Unignored: returned to active backlog.
@dimstar_suse sorry for rushing during the hackweek, but is there anything which prevents this SR from being accepted? Anything which I can fix or help with?