[tor-bugs] #18933 [Core Tor/Tor]: Several problems with test-network Makefile target

Tor Bug Tracker & Wiki blackhole at torproject.org
Fri Apr 29 16:59:00 UTC 2016


#18933: Several problems with test-network Makefile target
-------------------------------+------------------------------------
 Reporter:  isis               |          Owner:  isis
     Type:  defect             |         Status:  needs_review
 Priority:  Medium             |      Milestone:
Component:  Core Tor/Tor       |        Version:  Tor: 0.2.8.2-alpha
 Severity:  Normal             |     Resolution:
 Keywords:  TorCoreTeam201605  |  Actual Points:
Parent ID:                     |         Points:  small
 Reviewer:                     |        Sponsor:  SponsorS
-------------------------------+------------------------------------

Comment (by cypherpunks):

 Replying to [comment:4 isis]:
 > Replying to [comment:3 cypherpunks]:
 > > The second problem is addressed in #18737 but is waiting for #18240 to
 be merged. Depending on whether coverage is enabled, the *-cov binaries
 may or may not be available. After the merge of #18240 i plan on adding a
 fixup patch that adds a variable for tor-gencert (like
 `TESTING_TOR_BINARY`) so the dependencies of the test-network target are
 dynamic. I'm waiting for the merge because the changes are overlapping and
 would lead to merge conflicts.
 >
 > Great! That fixes all of it except for the `$(EXEEXT)` problem for the
 `tor-[cov-]gencert` binary.
 I can think of only one clean way to communicate the `$(EXEEXT)` to the
 script and that is with parameters (similar to Stem's `run-tests.py` (see
 test-stem target)). The added benefit would be that it simplifies test-
 network.sh because it would just use the binaries as is which obsoletes
 the tor-path and coverage options.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/18933#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list