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

Add AS_HELP_STRING to *nix build configure options #3910

Closed
wants to merge 8 commits into from

Conversation

petk
Copy link
Member

@petk petk commented Mar 3, 2019

The Autoconf's default AS_HELP_STRING macro can properly format help strings [1] so watching out if columns are aligned manually won't be needed anymore.

These strings are visible in the configure --help output.

TODO:

  • Format PHP_ARG_*s in configure.ac
  • Fix tabs in pdop_odbc
  • CS style fixes here and there.
  • Update README.SELF-CONTAINED-EXTENSIONS
  • Replace special characters with quadrigraphs

The Autoconf's default AS_HELP_STRING macro can properly format help
strings [1] so watching out if columns are aligned manually is not
anymore.

[1] https://www.gnu.org/software/autoconf/manual/autoconf.html#Pretty-Help-Strings
petk added 4 commits March 3, 2019 22:00
- Refactor remaining options
- Refactor help string for PDO ODBC
- Refactor example in README
@petk petk marked this pull request as ready for review March 3, 2019 23:47
@nikic
Copy link
Member

nikic commented Mar 5, 2019

Is the use of quadrigraphs necessary if our [] are balanced?

@petk
Copy link
Member Author

petk commented Mar 6, 2019

Is the use of quadrigraphs necessary if our [] are balanced?

No, I guess these are really not needed anymore in this particular case... I think that recent Autoconf versions (PHP requires now 2.68) have this fixed a bit, so the next commit omits the quadrigraphs now.

@petk
Copy link
Member Author

petk commented Mar 7, 2019

Applied via 9df6a1e to PHP-7.4+

@petk petk closed this Mar 7, 2019
@petk petk deleted the patch-as-help-string branch March 7, 2019 19:38
@carusogabriel
Copy link
Contributor

@petk Thanks for your amazing work on cleanup some stuff ❤️

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

Successfully merging this pull request may close these issues.

3 participants