Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#2462 closed defect (fixed)

make test fails at spawn_background_fail

Reported by: qbi Owned by:
Priority: Low Milestone: Tor: 0.2.3.x-final
Component: Core Tor/Tor Version: Tor: unspecified
Severity: Keywords: test tor-client
Cc: kubieziel@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

At the latest git HEAD I get the following error when running make test:

# make test
…
util/spawn_background_fail: 
  FAIL test_util.c:1372: assert(pos == strlen(expected_out)): 47 vs 60
  [spawn_background_fail FAILED]

Child Tickets

Change History (7)

comment:1 Changed 8 years ago by nickm

Milestone: Tor: 0.2.3.x-final

What OS?

comment:2 in reply to:  1 Changed 8 years ago by qbi

Replying to nickm:

What OS?

I tested it on GNU/Linux systems (Ubuntu 10.04 and Debian squeeze).

comment:3 Changed 8 years ago by keb

i got same output on ubuntu 11.04

[23:23] nickm> [...] please comment out the line in src/test/test_util.c that says " tt_int_op(pos, ==, strlen(expected_out));". I think it's line 1372.

ok with libevent-2.0.11 and
"./configure --with-libevent-dir=/usr/local/lib --disable-largefile --enable-dependency-tracking --enable-bufferevents --enable-gcc-warnings-advisory --enable-instrument-downloads"
output is now:

util/spawn_background_fail:

FAIL test_util.c:1373: assert(stdout_buf == expected_out): <ERR: Failed to spawn background process - code > vs <ERR: Failed to spawn background process - code 9/2

[spawn_background_fail FAILED]

comment:4 Changed 8 years ago by nickm

Status: newneeds_review

Pussible fix in branch bug2462 in my public repository.

comment:5 Changed 8 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

keb confirms the fix. hooray, merging.

comment:6 Changed 7 years ago by nickm

Keywords: tor-client added

comment:7 Changed 7 years ago by nickm

Component: Tor ClientTor
Note: See TracTickets for help on using tickets.