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

Adjust to Autoconf-2.71 #3

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

Conversation

ppisar
Copy link

@ppisar ppisar commented Nov 26, 2021

Autoconf-2.71 miconverts:

AC_CHECK_FUNCS( \
  strcpy \
  strdup \
  strchr \
  strlen \
  strcat \
  strtok \
)

into:

for ac_func in strcpy strdup strchr \ strlen \ strcat \ strtok \
do :
  [...]
done

According to Autoconf documentation, the AC_CHECK_FUNCS supports
new lines as tokenizers.

This patch works around this issue by not escaping the new lines.

https://bugzilla.redhat.com/show_bug.cgi?id=1999491
https://savannah.gnu.org/support/index.php?110571

Autoconf-2.71 fixed handling traling white spaces (marked with an
underscore here):

    AC_CHECK_FUNCS( \
      strcpy \
      strdup \
      strchr \_
      strlen \_
      strcat \_
      strtok \_
    )

and correctly converted them to this shell code:

    for ac_func in strcpy strdup strchr \ strlen \ strcat \ strtok \
    do :
      [...]
    done

This patch removes the trailing new lines.

<https://bugzilla.redhat.com/show_bug.cgi?id=1999491>
<https://savannah.gnu.org/support/index.php?110571>
@ppisar
Copy link
Author

ppisar commented Nov 26, 2021

It's not a bug in Autotools. That were trailing spaces after the backslashes in configure.ac. The new patch fixes it.

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.

1 participant