-
Notifications
You must be signed in to change notification settings - Fork 528
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
config.h: invalid preprocessing directive # HAS_LC_MONETARY_2008 /**/
#21803
Comments
Can you supply the output of:
Thanks. |
|
Also, what -- if anything do you have for:
... in |
|
On a system (Debian Linux 11) which I believe is very similar to yours, I switched locales with:
I then configured perl (blead) normally:
I then called That exhausts my knowledge of these issues. Time to call in the experts: @khwilliamson, @tonycoz, can you take a look? Thanks. |
How about the full output of config.sh and patch Configure with this patch and try again with that |
The problem is with If I call instead just My statement above that config.h contains |
On 1/7/24 08:35, Дилян Палаузов wrote:
The problem is with |nm|. When I execute |./Configure -Dusenm -d|, then
|case "$d_locconv:$d_lc_monetary_2008" in| in |Configure| is substituted
with |case "undef:" in|. Then the patch does help.
If I call instead just |./Configure -d|, then |case
"$d_locconv:$d_lc_monetary_2008"| in |Configure| is replaced with |case
"define:" in| and config.h contains |#define HAS_LC_MONETARY_2008 /**/|.
My statement above that config.h contains |d_lc_monetary_2008='define'|
is not correct. It has this value, when I run |./Configure -d|, but not
when config.h contains |# HAS_LC_MONETARY_2008 /**/|
—
Reply to this email directly, view it on GitHub
<#21803 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA2DH6QH523WAFBDL5LPR3YNK6FDAVCNFSM6AAAAABBPYCJPGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGA4TCOBQHA>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
Please attach config.h and config.sh with and without nm
|
I unpack perl-5.38.2.tar.gz in two directories: perl-5.38.2-with-nm and perl-5.38.2-without-nm In perl-5.38.2-with-nm I do In perl-5.38.2-without-nm I execute |
The files you posted with |
The config.sh/config.h files were without the patch. These are the files with the patch config.sh-with-nm-patched.txt As I mentioned above the patch does help. The major problem is that |
For the record,
|
|
Which linux distribution is this? |
Can the problem be, that |
No, this cannot be the problem, at least not for |
./Configure -Dusenm generates a file libc.list, which contains
The ./Configure cannot find there fprintf in the ways it is searching it, so it sets nm_opt=
Then configure does This change does fix make nm detect correctly log2: --- Configure.orig 2024-01-14 14:16:59.220913131 +0000
+++ Configure 2024-01-14 14:18:17.760706822 +0000
@@ -6674,12 +6674,12 @@
$contains '^fprintf$' libc.list >/dev/null 2>&1; then
eval $xrun
else
- $nm -p $* 2>/dev/null >libc.tmp
+ $nm -p --without-symbol-versions $* 2>/dev/null >libc.tmp
$grep fprintf libc.tmp > libc.ptf
if com="$sed -n -e 's/^.* [ADTSIW] *_[_.]*//p' -e 's/^.* [ADTSIW] //p'";\
eval $xscan; $contains '^fprintf$' libc.list >/dev/null 2>&1
then
- nm_opt='-p'
+ nm_opt='-p --without-symbol-versions'
eval $xrun
else
echo " " After applying this change, I get I am not saying this is an universal fix, but it gives hints where the problem is. I am not going to work on this further, unless I am asked to try particular patches. After applying the first change here with |
@Tux what do you think of dilyan's changes? |
As I spotted, there are currently several I don't think long options are supported at all on older native I also spotter truckloads of |
|
That should be either |
If I recall correctly, you have to apply #21835 and do not pass |
The more I look at this, the more I am convinced that Looking in that area right now |
So for now it is a no-go. You could try to rewrite your patch using |
Yes, but if the plan is to eventually split the logic in ./configure for nm in those supporting |
Then forget that option, as the chance it will be implemented on HP-UX, AIX, Solaris, Irix and all will never happen. Think backwards and add that suggested |
I downloaded perl-5.38.2, unpacked it, ran
./Configure && make
. The latter failed withIndeed, config.h contains
# HAS_LC_MONETARY_2008 /**/
and this is invalid preprocessor input.The text was updated successfully, but these errors were encountered: