Correct probable typo.

prevent undefined behaviour at a language level while probing getenv

Backport of a5823872283be23731f1bcde7e19a926c44b31a4

This test in Configure tries to probe for undefined behaviour in
getenv (), but provokes undefined behaviour in C/C++ by falling off
the end of a function with a non-void return type.

Without optimization clang++ generated a ud2 instruction here on
amd64 producing an illegal instruction exception.  With optimization
the test case fell off the end and started re-executing main (),
eventually producing a SIGBUS.

Simply dropping the value of getenv () here and returning NULL wasn't
useful, under -O2 the compiler optimized away the getenv () call,
voiding the whole point of the test.

Merge pull request #75 from khwilliamson/master

Add new probes:  XXX I don't know what to change in the RCS lines

Add probes for ffs, ffsl

XXX RCS changes? Add probe for strxfrm_l

d_endpwent.U: Fix typo in comment

Add check for nl_langinfo_l()

There is already a check for this, but it fails unless the function is
thread safe.  This adds a symbol for when it exists but isn't thread
safe.

Make libpth dist compliant

Merge pull request #70 from khwilliamson/master

Some fixups, and printing to >&4 that needs review

Merge pull request #73 from Perl/ntyni/incpath-backport

Backport: Prevent incpath to spill into libpth

Prevent incpath to spill into libpth

• Use realpath if available
• This might catch more duplicate paths
• Only include real existing paths
• Filter inc paths out of libpth

Before:
incpth: /usr/lib64/gcc/x86_64-suse-linux/7/include /usr/local/include /usr/lib64/gcc/x86_64-suse-linux/7/include-fixed /usr/lib64/gcc/x86_64-suse-linux/7/../../../../x86_64-suse-linux/include /usr/include
libpth: /usr/local/lib /usr/lib64/gcc/x86_64-suse-linux/7/include-fixed /usr/lib64/gcc/x86_64-suse-linux/7/../../../../x86_64-suse-linux/lib /usr/lib /pro/local/lib /lib/../lib64 /usr/lib/../lib64 /lib /lib64 /usr/lib64 /usr/local/lib64

After:
incpth: /usr/lib64/gcc/x86_64-suse-linux/7/include /usr/local/include /usr/lib64/gcc/x86_64-suse-linux/7/include-fixed /usr/x86_64-suse-linux/include /usr/include
libpth: /usr/local/lib /usr/x86_64-suse-linux/lib /usr/lib /pro/local/lib /lib64 /usr/lib64 /lib /usr/local/lib64

(Backported from perl.git d3144c9253d3244cd259a0ae4c0fe81519bf5b53 to
 metaconfig.git by Niko Tyni)

better de-dup metagrep

and skip old working tree

Reword usage messages (thank you Jim)

Backport of 0484bfe840704c501e4b0d8949d48017b336d71e

Fixes backported from Porting/Glossary commits

Test that gcvt/qgcvt are not buggy (Issue 18170)

Backport of cfc95b5630c63927ab68bb4cdc08b1dec23c5569 and previous commits

Consistent indent/whitespace

Whitespace change from 6b6e83fce2074294ac11d547c8a75bc106bebab9

Add Configure probe for getenv() buffer race

Backport of 9d228af78ad17dabb51f9059d215cc88c059a22a

Most implementations do not have a problem with two getenv()'s running
simultaneously in different threads.  But Posix doesn't require such
good behavior.  This adds a simple probe to test the current system.

Prevent non-ASCII in comment

This will cause havoc on EBCDIC systems

Fix build on Haiku beta

Backport of 6b6e83fce2074294ac11d547c8a75bc106bebab9
set correct Haiku usrinc and locinc , add -fno-stack-protector to hints
for amigaos.sh and haiku.sh (and disable accordingly -fstack-protector*
in Configure)

test that gcvt/qgcvt are not buggy (Issue 18170)

Backport of e8b4b5cb56db51c60d23a47fbb2f897e1206c9af
Author: sisyphus <sisyphus@cpan.org>
Date:   Wed Oct 21 10:52:11 2020 +1100

But style adjusted to surrounding code

Prevent errors when run outside metaconfig folder

Add the Configure option to enable strict by default in perl programs.

Do not set strict by default on -e or -E

This is a development only feature to allow us to clean up blead.

./Configure -Dusedevel -des -Dusedefaultstrict

Trap SIGBUS when Configure checks for va_copy (#18148)

On several systems the attempt to determine if we &quot;need va_copy() or
similar&quot; results in a SIGBUS instead of a SIGSEGV because the fault
happens pointing at an io device address.

While reviewing accounting logs during the OpenBSD system build, bluhm@
noticed a core dump and deraadt@ tracked it down.

While researching it was discovered that this is a fairly common case in
other systems as well.

Original diff by deraadt@

Force comments for L_R_TZSET to end in config_h.SH

Merge pull request #71 from richardleach/patch-1

d_localtime_r.U - explicitly free() allocations

d_localtime_r.U - explicitly free() allocations

To keep LeakSanitizer happy. See https://github.com/Perl/perl5/issues/18107 for background.

d_faststdio.U: Use >&4, not >& 4

perl/perlxv.U: Change some echo 2>&1

I don't know why this is redirecting stderr to stdout; why it thinks the
echo could generate output to stderr, but the file descriptor most
things should go to is >&4.

U/perl/Checkcc.U: Use >&4 instead of >&2 for output

compline/ccflags.U: Change some echo 2>&1

I don't know why this is redirecting stderr to stdout; why it thinks the
echo could generate output to stderr, but the file descriptor most
things should go to is >&4.

README: Small fix ups

Merge pull request #69 from khwilliamson/master

Fixes for EBCDIC, AIX, cleanup, some thread cleanup, docs

usethreads.U: Rmv unused variable

Extensions.U: remove unused variable

README: Add instrs for modified units

README: Fix typo

d_longdbl.U: Always echo to &4 file descriptor

Previously only some did; this also removes the space in '>& 4' used in
some places

d_longdbl.U: Mention two symbols in ?C:

These two symbols are defined by this unit, but went unmentioned.

d_union_semun.U: Indent two ?C: lines

These two lines are for describing a data structure.  I've written code
that converts config.h items into pod.  These are the only two lines that
are really problematic, and it is because they should render as-is, but
to indicate that, they shouldn't be at the left margin.

Add d_bitfield.U to see if non-int struct bitfields

The C standard says only int and unsigned int bitfields in structs need
to be supported.  Many compilers also support other-sized ones, such as
short, but not all.  This adds a probe to detect this.

i_db.U: Use new compiler_warning.U

voidflags.U: Use new compiler_warning.U

d_attribut.U: Use new compiler_warning.U

Add compiler_warning.U

This is to see if there were compiler warnings generated.  It turns out
that AIX has very weird syntax for warnings, that don't include that
word, so there is special code for that os.

Also, some compiler's warnings aren't all lowercase.

contains.U: Need a path to the generated script

I tested what would happen with 'contains' if grep is actually broken on
the box, and it failed because it didn't know where the generated
work-around script is.

This commit fixes this, but the fact that no failures have been showing
up indicates that modern greps just aren't broken this way.

contains.U: echo should go to >&4

Copy contains.U from dist to U/modified

This is in preparation for modifying it

d_setlocale.U: Add more info to user messages

d_setlocale.U: Pay attention to hints file

This was ignoring any hints file setting of d_setlocale, and checking if
the system appears to have a sane one.  But there are cases where the
setlocale isn't really sane, even though it appears to be, and the hints
file is used to override this setting.  This commit now pays attention
to that override.

Modernize getverlist

- use strict/warnings
- pack "s>s>s>" to ensure best sorting
- fix small typo in comment

Remove final vestiges of 5.005 threads

Backport more blead Glossary typo fixes

Commit bc2d571f1bf96e369c3d80c4294161b1d3180299 backported the
"patern" typo fix from
https://github.com/Perl/perl5/commit/4357af6ce6bbf62e83c554355bd8063b6fd0880a,
but missed the instances of "usally" on the same lines.

Head.U: Fix Github repository URL

Merge pull request #64 from ntyni/blead-changes

Blead changes

Backport blead Glossary change fixing typos

The Glossary change is part of

 https://github.com/Perl/perl5/commit/4357af6ce6bbf62e83c554355bd8063b6fd0880a

Backport blead Configure change to contact instructions

The Configure change is part of

 https://github.com/Perl/perl5/commit/35e21c5bf73a6039193bb688ec85ff2cc7716443

Minor cleanups

Merge pull request #63 from xenu/xenu/sockaddr-storage

add probe for struct sockaddr_storage

add probe for struct sockaddr_storage

Perl/perl5#17764 needs this

Merge pull request #62 from Perl/fix-repeated-words

Fix repeated-word typos

Fix repeated-word typos

Adapt Configure to GCC version 10

backport of 9f4e6307232229875331a55e44e1245b0b91e219

Partly revert 9c359e8280d731ca92428e10705200e1c7ebf844

No here doc inside case

Sync with minor changes in blead

Add <stdlib.h> for futimes compilation test

Backport f16c94bc75aefb81949b423b399fac4071859f6d

Remove -DUSE_C_BACKTRACE part: from e849841dca2a8b1111999

Patches for meta are never directly from the perl5 tree

Update the reference to the GitHub issue tracker

Declare temporary file 'try' for metalint.

Fix strtoul probe to compile under clang++.

Add the missing stdlib.h header and remove an unnecessary prototype.

Fix strtoll, strotoull, and strtouq probes to compile under clang++.

The test programs were missing the stdlib.h header, and needed
tweaks to the check() prototype to compile under clang++.  These
changes should be fine for C compilers as well, but they have typically
been more forgiving, so shouldn't be affected.

Include <stdio.h> in getpgrp/setpgrp probes for printf prototype.

C compilers typically issue a warning, but c++ compilers may abort with
an error.

Use a compile and run test for clang++.

For glibc, previous reports were that some functions (such as lchown())
are present in libc, but are unimplemented.  That is, they always fail
and set errno=ENOSYS.  Unfortunately, the stub test doesn't work under
clang++.  Thus use a compile and run test.

Merge pull request #61 from xenu/master

add probe for __attribute__(always_inline)

add probe for __attribute__(always_inline)

isless probe in Configure broken #17216

Update README to reflect new location on github

Add check for malloc_usable_size (Richard Leach)

Merge pull request #60 from tonycoz/futimes-exit

c99 requires functions to be declared before they're called

c99 requires functions to be declared before they're called

and exit() is declared in stdlib.h

Reset xxx_convert to empty string if none of 3 utilities is found

Backport 6e404ab585deadc1c32d50513f13b50ae395c00d

This will cause Configure to enter the WHOA block and default to using
sprintf.

Change Gconvert checkit() prototype for [perl #134371].

In the checkit() routine inside Configure, clang++ was taking the
if (strcmp(expect, got)) branch even though the 'expect' and 'got' strings
were identical.  A first step in debugging this was to realize that
the checkit() function never returned a value, so relabel it as void.
With clang version 7.0.1-8 (tags/RELEASE_701/final) (Debian),
this change seems to work around the strcmp issue.

Further Configure gconvert probe cleanups for C++.  As part of debugging
[perl #134375] and [perl #134371], try to eliminate warnings thrown by
clang++.

Merge pull request #58 from tonycoz/sysroot

(perl #134360) --sysroot apparently needs the =

(perl #134360) --sysroot apparently needs the =

See: https://rt.perl.org/Ticket/Display.html?id=134360

Configure: reuse "cached" value of "none" for dir

When value of a directory (i.e. man1dir) is set to "none" because of:
- '-D' option set on cmdline (i.e. -Dman1dir=none) OR
- because value is reused from previous configure run

And when the prefix is different from the previous configure run(*) then
keep the value of "none". predefined/cached value does not contain the
old prefix so there is no harm in keeping it.

(*): due to another bug this also happens when prefix ends with a
     trailing slash

See https://rt.perl.org/Ticket/Display.html?id=134284

Remove trailing '/' from prefix

Change how/when the trailing '/' from a prefix is removed.

The original code had two problems:

1) it only stripped the trailing slash when value was predefined
   (either from -Dprefix=/foo/bar/ or from a previous configure run)
2) when it did strip the value then it also caused 'oldprefix' to
   be set. This in turns causes other predefined (/cached) paths to
   be ignored.

See https://rt.perl.org/Public/Bug/Display.html?id=134284

Remove vestiges of mpeix support (removed in 5.17.x)

backport of e3c1dc81bda4523bfc897fec8afe25b162e29cb6

Fix erroneous references to USE_CBACKTRACE

The #define set in Configure and used in locale.c and util.c is actually
USE_C_BACKTRACE.

backport of 812e2121cb84e726a5a5b5a3a64c6f949426e02f

Allow -Uusedtrace / -Dusedtrace=false / -Dusedtrace=no

Backport perl5 core changes by Andy, correcting Aaron's merge

Assume <stdlib.h> exists when probing for setlocale()

We're on C89, after all :-)

Backport improvements to setlocale() probe

This is from Perl 5 commit 2e3ef32f4acc106de6ae8170099f09a36d4ef490, but
with a couple of changes to appease the metaconfig linter by avoiding
strings of the form /\w+=\S+/ (which it interprets as attempts to set shell
variables).

Backport probes for towlower() and towupper()

From Perl 5 commit 0806cdda2789ca6394976d1ff3e65dd59bcb8d1b (but with a typo
in the glossary description fixed).

Backport detection of <wctype.h>

This corresponds to Perl 5 commit 92271d410d92433b97015908e12c9f262c5953e7

Backport detection improvements for memrchr, strlcat, and strlcpy

This corresponds to Perl commit ba73a4cb8f472480a2d630613d1e9e1172d518d3

Backport ability to include literal text in prototype checks

This corresponds to Perl 5 commit 63c1fa6a98bc60234a21de83dd191cd581a5d073

Backport memmem(3) detection changes from Perl

This corresponds to Perl 5 commits ca152fd8207cf53816b1407d5f54f6ea160a3ef8
and f8d82a1010426d0eb49c33cb903413b882c85c3e.

Fix bug when $exclusions_file is specified in .package file

The actual programs were trying to load the exclusions file before reading
.package (or even, in one case, without trying to read it at all). I failed
to notice this because I had a "-X" option that duplicated the intended use
of the setting in my .package file.

Remove modified d_const.U

Perl no longer needs a d_const.U unit at all, so we needn't keep a modified
version of it.

U/perl/i_db.U: just use C89 "const" without probing