Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update gnulib to stable-202301 #211

Open
wants to merge 1 commit into
base: 3_5
Choose a base branch
from

Conversation

sidsbrmnn
Copy link
Member

@sidsbrmnn sidsbrmnn commented Mar 6, 2023

This pull request is to update Gnulib to its latest stable upstream, stable-202301. The currently used snapshot of Gnulib is as old as February 9, 2016 and gnulib has undergone plenty of improvements and security fixes on vulnerabilities. Amanda maintains a snapshot of Gnulib with its code to maintain portability and consistency across Unix-like platforms where the GNU C Library is not the default C Library.

Changes Made

  • config/gnulib: updated *.m4 macros for Gnulib (written by gnulib-tool command)
  • gnulib: updated Gnulib (written by gnulib-tool command)
  • gnulib/regenerate/regenerate: updated script with the commit hash of the Gnulib snapshot used and cleaned up the script for readability
  • config/amanda/socklen_t_equiv.m4: updated checks for sys/socket.h header file using the newer gl_SYS_SOCKET_H macro
  • configure.ac: added default value to GNULIBHEADERS_OVERRIDE_WINT_T as it is left undefined due to a macro override by gettext

Testing Done

I've currently tested the builds on CentOS 7, Rocky Linux 9 and Fedora 37.

@chandraravipati
Copy link
Member

@sidsbrmnn please add more detailed description explaining the need for these changes and the changes.

@sidsbrmnn
Copy link
Member Author

@sidsbrmnn please add more detailed description explaining the need for these changes and the changes.

I've updated the description with the details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants