mbox series

[0/1] config: Handle dash in library name for AC_LIB_LINKAGEFLAGS_BODY

Message ID 20240707195341.25305-1-abdul.b.ijaz@intel.com
Headers show
Series config: Handle dash in library name for AC_LIB_LINKAGEFLAGS_BODY | expand

Message

Ijaz, Abdul B July 7, 2024, 7:53 p.m. UTC 
From: "Ijaz, Abdul B" <abdul.b.ijaz@intel.com>

Hi All,

This change fixes the handling of dash(-) in the library names like
yaml-cpp for the AC_LIB_LINKAGEFLAGS_BODY script function of the file
"config/lib-link.m4" and used by the autoconf tool.  This is my first
patch for GCC so adding this summary of the change in 0/1 patch and
details of the fix can be seen in the patch 1/1.

Autoconf 2.69 tool execution only adds few new empty lines and some
line numbers are updated in the regenerated configure file after this
change.  There is no significant change in this file because of this
change so have not comitted it.  Since this change only affects the
libraries having having dash in the name.  We need this change eventually
for the GDB source code repository which basically gets the copy of these
scripts from GCC.  Let me know if any testing can be done for this change.
Already mentioned I only tried regenerating the configure file and have
not notice any significant change so it seems nothing need to be tested.
I ran the tests on x86_64 Ubuntu22 setup and there are some unexpeced
failures but nothing seems related to this change.

Lastly want to mention that I do not have the write access.  So I
want to ask if this small change can be accepted without CLA.  The
Signed-off field is already added to the commit to fulfill the DCO.

Waiting for your feedback.

Thanks & Best Regards
Abdul Basit

Ijaz, Abdul B (1):
  config: Handle dash in library name for AC_LIB_LINKAGEFLAGS_BODY

 config/lib-link.m4 | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)