Summary: | apxs gives "use of uninitialized value in concatenation" error | ||
---|---|---|---|
Product: | Apache httpd-2 | Reporter: | Reshma V Kumar <reskumar> |
Component: | All | Assignee: | Apache HTTPD Bugs Mailing List <bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | FixedInTrunk, PatchAvailable |
Priority: | P2 | ||
Version: | 2.4.38 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | AIX | ||
Attachments: |
do not remove parentheses, otherwise variables are not evaluated
avoid printing uninitialized variables |
Description
Reshma V Kumar
2019-04-01 08:57:27 UTC
I am getting this error due to SH_LDFLAGS. If SH_LDFLAGS is empty, then I dont get this error. (In reply to Reshma V Kumar from comment #1) > I am getting this error due to SH_LDFLAGS. If SH_LDFLAGS is empty, then I > dont get this error. Hi, I was unable to replicate your problem by building 2.4.38 I tried using EXTRA_LDFLAGS=-Wl,-brtl or SH_LDFLAG=-Wl,-brtl Could you provide a more detailed replication procedure to help pinpoint the problem ? Once the build is successful and the package is installed, executing the following command gives the warning # /opt/freeware/bin/apxs -q libexecdir Use of uninitialized value in concatenation (.) or string at /opt/freeware/bin/apxs line 222. /opt/freeware/lib/httpd/modules Following are the values from /var/www/build/config_vars.mk file LDFLAGS = -L/opt/freeware/lib -Wl,-blibpath:/opt/freeware/lib:/usr/lib:/lib -Wl,-bmaxdata:0x80000000 SH_LDFLAGS = $(EXTRA_LDFLAGS) $(EXTRA_LIBS) EXTRA_LDFLAGS = -Wl,-brtl EXTRA_LIBS = The warning is removed if SH_LDFLAGS in /var/www/build/config_vars.mk is empty SH_LDFLAGS = # /opt/freeware/bin/apxs -q libexecdir /opt/freeware/lib/httpd/modules Created attachment 36976 [details]
do not remove parentheses, otherwise variables are not evaluated
I cannot find any reason why parentheses are removed from the variable.
This patch worked for me. Will this patch be commited to httpd source code? I would commit it smoothly if someone could explain to me why parenthesis are removed on current implementation. https://github.com/apache/httpd/commit/8e67a36d9ba392a693c463afe8de8a9181d1f7e4#diff-1874f6a844636ac4ceb88701b5e8f023 This change has been introduced in the source code through the above commit. @dougm has committed this change. Created attachment 37091 [details]
avoid printing uninitialized variables
Diff that doesn't remove parenthesis but still works.
This patch resolved the error Fixed in r1875534. |