Only add warnings and std flags if supported by compiler (autoconf) #145

Merged
plicease merged 2 commits from solaris-ac into master 5 years ago
plicease commented 5 years ago (Migrated from github.com)
Owner

This adds autoconf detection for the -Wall and -std= flags. I am proposing as a separate PR as it adds a developer dependency on autoconf-archive. If that is undesirable I can probably rework it.

This adds autoconf detection for the `-Wall` and `-std=` flags. I am proposing as a separate PR as it adds a developer dependency on `autoconf-archive`. If that is undesirable I can probably rework it.
Owner

Would it be possible to include the autoconf-archive macros somehow?

Would it be possible to include the autoconf-archive macros somehow?
plicease commented 5 years ago (Migrated from github.com)
Owner

Are you asking if the macro can be bundled with pkgconf?

Are you asking if the macro can be bundled with pkgconf?
Owner

yes, such as in acinclude.m4

yes, such as in `acinclude.m4`
plicease commented 5 years ago (Migrated from github.com)
Owner

Rebased.

I bundled using AC_CONFIG_MACRO_DIRS as per this advice: https://www.gnu.org/software/automake/manual/html_node/Local-Macros.html
( if this is bad advice, please let me know :) )

I noticed that the bundled macro file includes license reference to GPL3 :( There is an "autoconf exception" for the license referred to here: https://www.gnu.org/software/autoconf-archive/ax_check_compile_flag.html . I am not a lawyer, so I can't say with certainty if that would force a license change, which (seems to me) would be unacceptable. My read is "autoconf exception" is fine for distributing the generated configure, but I am not sure about distributing just the m4 file.

Rebased. I bundled using AC_CONFIG_MACRO_DIRS as per this advice: https://www.gnu.org/software/automake/manual/html_node/Local-Macros.html ( if this is bad advice, please let me know :) ) I noticed that the bundled macro file includes license reference to GPL3 :( There is an "autoconf exception" for the license referred to here: https://www.gnu.org/software/autoconf-archive/ax_check_compile_flag.html . I am not a lawyer, so I can't say with certainty if that would force a license change, which (seems to me) would be unacceptable. My read is "autoconf exception" is fine for distributing the generated configure, but I am not sure about distributing just the m4 file.
The pull request has been merged as a8a65c7f6c.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ariadne/pkgconf#145
Loading…
There is no content yet.