Skip to content

Build: Increase build dependencies for 3.1.0.#4142

Open
clumens wants to merge 1 commit into
ClusterLabs:mainfrom
clumens:T1021
Open

Build: Increase build dependencies for 3.1.0.#4142
clumens wants to merge 1 commit into
ClusterLabs:mainfrom
clumens:T1021

Conversation

@clumens

@clumens clumens commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Fixes T1021

@clumens
clumens requested a review from nrwahl2 July 1, 2026 13:59
@clumens

clumens commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

@nrwahl2 I think you'll like this one.

@nrwahl2

nrwahl2 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

@clumens Some, possibly all, of the CI failures are due to what we agreed we're okay with: the older OS releases in CI don't have the required package versions. We'd need to work with Fabio and Chrissie on that.

Comment thread configure.ac Outdated
The bump in the gettext requirement in particular silences a lot of
warnings I've been seeing for years when running ./autogen.sh.

Fixes T1021
@clumens

clumens commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

@clumens Some, possibly all, of the CI failures are due to what we agreed we're okay with: the older OS releases in CI don't have the required package versions. We'd need to work with Fabio and Chrissie on that.

Yep, I figured that would happen. I wanted to get an ACK on doing this first, and then I'll talk to them about what needs to be done (I assume on Monday or Tuesday, given the holidays), and then once CI is ready I'll merge it.

@nrwahl2 nrwahl2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This basically looks good to me. A few comments.

Comment thread rpm/pacemaker.spec.in
Requires: %{pkgname_bzip2_devel}%{?_isa}
Requires: corosync-devel >= 2.0.0
Requires: corosync-devel >= 3.1.9
Requires: glib2-devel%{?_isa}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version checking is inconsistent -- which is not a new issue. I don't know why we don't have version requirements here for some of the packages, like glib2-devel. Similarly, we check for some things (e.g., the corosync version) in the spec file but not in configure.ac.

Comment thread INSTALL.md
| 1.13 or later | automake | automake | automake |
| 2.64 or later | autoconf | autoconf | autoconf |
| 1.16 or later | automake | automake | automake |
| 2.71 or later | autoconf | autoconf | autoconf |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we can and should enforce this somewhere (for autoconf)

Comment thread INSTALL.md
| documentation | | python3-sphinx | python3-sphinx | python3-sphinx |
| documentation (PDF) | | latexmk texlive texlive-capt-of texlive-collection-xetex texlive-fncychap texlive-framed texlive-multirow texlive-needspace texlive-tabulary texlive-titlesec texlive-threeparttable texlive-upquote texlive-wrapfig texlive-xetex | texlive texlive-latex | texlive texlive-latex-extra |
| annotated source code as HTML via "make global" | | global | global | global |
| RPM packages via "make rpm" | 4.19.1 or later | rpm | rpm | (n/a) |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't appear to enforce this one anywhere either

@nrwahl2 nrwahl2 added the waiting for author Review has been provided, and we're waiting for a response from the author of the pull request label Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for author Review has been provided, and we're waiting for a response from the author of the pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants