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

7.88.0: Test error Due to uninitialized value $port #10518

Closed
fundawang opened this issue Feb 15, 2023 · 3 comments
Closed

7.88.0: Test error Due to uninitialized value $port #10518

fundawang opened this issue Feb 15, 2023 · 3 comments
Labels

Comments

@fundawang
Copy link

fundawang commented Feb 15, 2023

I did this

Build curl 7.88.0 from source with OpenAnolis (Fedora rawhide based).

I expected the following

Test pass

curl/libcurl version

7.88.0

operating system

Anolis OS (Fedora rawhide based), with openssl 3.0.7 / perl 5.36.0

Build log here:
https://build.openanolis.cn/taskinfo?taskID=537750 (with build log and build environment included)

test 1434...[HTTP GET resume at exactly the existing file size is fine]
../src/curl --output log/curl1434.out  --include --trace-ascii log/trace1434 --trace-time http://127.0.0.1:34161/1434 -C 100 >log/stdout1434 2>log/stderr1434
CMD (0): ../src/curl --output log/curl1434.out  --include --trace-ascii log/trace1434 --trace-time http://127.0.0.1:34161/1434 -C 100 >log/stdout1434 2>log/stderr1434
--pd---e--- OK (1248 out of 1556, remaining: 00:53, took 0.015s, duration: 03:36)
startnew: perl -I../../tests ../../tests/httpserver.pl --pidfile ".http_unix_server.pid" --logfile "log/http_unix_server.log" --portfile .http_unix_server.port --unix-socket 'http2857587.sock' --srcdir "../../tests/data/.."
RUN: ../src/curl --max-time 13 --output log/http_unix_verify.out --silent --verbose --globoff --unix-socket 'http2857587.sock' "http://127.0.0.1:80/verifiedserver" 2>log/http_unix_verify.log
CMD (0): ../src/curl --max-time 13 --output log/http_unix_verify.out --silent --verbose --globoff --unix-socket 'http2857587.sock' "http://127.0.0.1:80/verifiedserver" 2>log/http_unix_verify.log
Use of uninitialized value $port in concatenation (.) or string at ../../tests/runtests.pl line 1761.
error: Bad exit status from /var/tmp/rpm-tmp.faqYNS (%check)
~~ 
@bagder
Copy link
Member

bagder commented Feb 15, 2023

It would be friendly to include and show the important snippets in the report here.

@bagder bagder added the tests label Feb 15, 2023
@sharewax
Copy link

sharewax commented Feb 15, 2023

I have the same issue while building 7.88.0:

test 1434...[HTTP GET resume at exactly the existing file size is fine]
../libtool --mode=execute /usr/bin/valgrind --tool=memcheck --quiet --leak-check=yes --suppressions=../../tests/valgrind.supp --num-callers=16 --log-file=log/valgrind1434 ../src/curl --output log/curl1434.out  --include --trace-ascii log/trace1434 --trace-time http://127.0.0.1:40351/1434 -C 100 >log/stdout1434 2>log/stderr1434
CMD (0): ../libtool --mode=execute /usr/bin/valgrind --tool=memcheck --quiet --leak-check=yes --suppressions=../../tests/valgrind.supp --num-callers=16 --log-file=log/valgrind1434 ../src/curl --output log/curl1434.out  --include --trace-ascii log/trace1434 --trace-time http://127.0.0.1:40351/1434 -C 100 >log/stdout1434 2>log/stderr1434
--pd---e-v- OK (1243 out of 1549, remaining: 06:46, took 1.293s, duration: 27:32)
startnew: perl -I../../tests ../../tests/httpserver.pl --pidfile ".http_unix_server.pid" --logfile "log/http_unix_server.log" --portfile .http_unix_server.port --unix-socket 'http30512.sock' --srcdir "../../tests/data/.."
RUN: ../src/curl --max-time 13 --output log/http_unix_verify.out --silent --verbose --globoff --unix-socket 'http30512.sock' "http://127.0.0.1:80/verifiedserver" 2>log/http_unix_verify.log
CMD (0): ../src/curl --max-time 13 --output log/http_unix_verify.out --silent --verbose --globoff --unix-socket 'http30512.sock' "http://127.0.0.1:80/verifiedserver" 2>log/http_unix_verify.log
Use of uninitialized value $port in concatenation (.) or string at ../../tests/runtests.pl line 1761.
RPM build errors:

bagder added a commit that referenced this issue Feb 15, 2023
by using a more appropriate variable

Reported-by: fundawang on github
Fixes #10518
@fundawang
Copy link
Author

Thank you, the patch works.

bagder added a commit that referenced this issue Feb 15, 2023
by using a more appropriate variable

Reported-by: fundawang on github
Fixes #10518
Closes #10520
bch pushed a commit to bch/curl that referenced this issue Jul 19, 2023
by using a more appropriate variable

Reported-by: fundawang on github
Fixes curl#10518
Closes curl#10520
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

3 participants