Bug 1943112

Summary: libstatgrab: FTBFS with upcoming autoconf-2.71
Product: [Fedora] Fedora Reporter: Ondrej Dubaj <odubaj>
Component: libstatgrabAssignee: Tim Orling <TicoTimo>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: i, jima, ofalk, TicoTimo, tim
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-08-05 00:14:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1942967    

Description Ondrej Dubaj 2021-03-25 12:54:25 UTC
Your package fails to build with the newest upcoming autoconf-2.71, which is part of a wide Fedora change. Please see the attached copr: https://copr.fedorainfracloud.org/coprs/odubaj/autoconf-2.70/packages/. More information about testing your package when building with autoconf available here: https://fedoraproject.org/wiki/Changes/Autoconf_271#How_To_Test

Comment 1 Oliver Falk 2021-03-31 14:37:10 UTC
Upstream issue: https://github.com/libstatgrab/libstatgrab/issues/129 just created.

Comment 2 Tim Orling 2021-04-10 19:32:35 UTC
Upstream has a fix: https://github.com/libstatgrab/libstatgrab/pull/128

Asking on IRC if we can get a new release... otherwise we can just pull in the one patch for rawhide.

Comment 3 Ondrej Dubaj 2021-04-12 07:20:15 UTC
Gentle ping.

Comment 4 Ondrej Dubaj 2021-05-04 08:34:30 UTC
Hi, any updates here please ?

Thanks.

Comment 5 Oliver Falk 2021-05-04 08:44:32 UTC
https://github.com/libstatgrab/libstatgrab/pull/128 has been merged AFAICS.

However, no new release has been built (since 2019 actually). I asked in issue 129 if they intend to build a new release soon or not.

IMHO, @TicoTimo could update source/SPEC now with the single patch; Given it has been accepted upstream.

Comment 6 Ondrej Dubaj 2021-05-04 08:47:18 UTC
That would be great. Thanks for you effort!

Comment 7 Oliver Falk 2021-05-04 08:53:06 UTC
I only pinged on the issue... No too much effort so far :-)

I can also pull the single patch into our (Fedora) repo, if Tim doesn't find the time; Or @jima can maybe do it, if Tim allows - I think he is co-maintainer if I remember correctly.

Comment 8 Ondrej Dubaj 2021-05-04 09:00:57 UTC
If new release will come in ~2 months, there is no need to do double work and backport a single patch :) If we do not know when the new release will come, I would prefer backporting the patch to Fedora and mark this issue as resolved.

Comment 9 Oliver Falk 2021-05-04 14:50:25 UTC
A few minutes ago on the GH issue:

"""I want to fix FBSD12 vmmeter issue before ... thank to new laptop I can install appropriate VM soon."""

So maybe worth waiting a bit. Backporting is still an option then. We just shouldn't loose sight on it.

Comment 10 Tim Orling 2021-05-19 05:17:02 UTC
FWIW, I pinged upstream again on irc for a new release. There are several commits since 0.92 and I don’t want to be the human that decides which to include in Fedora. If you feel strongly otherwise, comment and we can add the tech debt. Or you too can join freenode #libstatgrab and become a voice for change.

Comment 11 Oliver Falk 2021-05-28 07:31:49 UTC
Seems like there is still the vmmeter struct open which they want to fix before a new release.

Comment 12 Ondrej Dubaj 2021-07-01 10:39:19 UTC
We have still some time to wait, thank you for information.

Comment 13 Tim Bishop 2021-07-28 19:24:15 UTC
libstatgrab 0.92.1 is now available.

Comment 15 Tim Orling 2021-07-29 19:15:02 UTC
https://copr.fedorainfracloud.org/coprs/odubaj/autoconf-2.70/build/2352954/

Seems like we can close this @ofalk @odubaj ?

Comment 16 Oliver Falk 2021-07-29 19:31:38 UTC
Looks good from my POV!

Comment 17 Tim Orling 2021-08-05 00:14:15 UTC
Based on clean build and Oliver Falk LGTM, closing.