[tor-bugs] #25560 [Core Tor/Tor]: test all rust crates for realsies

Tor Bug Tracker & Wiki blackhole at torproject.org
Fri Mar 23 01:21:15 UTC 2018


#25560: test all rust crates for realsies
-----------------------------------------+---------------------------------
 Reporter:  isis                         |          Owner:  (none)
     Type:  defect                       |         Status:  needs_revision
 Priority:  High                         |      Milestone:  Tor:
                                         |  0.3.3.x-final
Component:  Core Tor/Tor                 |        Version:  Tor:
                                         |  0.3.3.3-alpha
 Severity:  Normal                       |     Resolution:
 Keywords:  tor-testing, rust, 033-must  |  Actual Points:
Parent ID:                               |         Points:  .2
 Reviewer:  nickm                        |        Sponsor:  SponsorM-can
-----------------------------------------+---------------------------------

Comment (by isis):

 Okay, quoting directories is actually fixed now, I think.  Surprisingly,
 building in directories with spaces in their names was/is already broken,
 because `src/test/include.am` does this:

 {{{
 test-rust:
         $(TESTS_ENVIRONMENT) $(abs_top_srcdir)/src/test/test_rust.sh
 }}}

 which for me expanded to:

 {{{
 export PYTHON="python"; export SHELL="/bin/bash"; export
 abs_top_srcdir="/home/isis/code/torproject/foo bar/../tor"; export
 abs_top_builddir="/home/isis/code/torproject/foo bar"; export
 builddir="."; export TESTING_TOR_BINARY="./src/or/tor"; export
 CARGO="cargo"; export CARGO_ONLINE=""; /home/isis/code/torproject/foo
 bar/../tor/src/test/test_rust.sh
 }}}

 So I've quoted it in the Makefile as well and everything seems to work
 now.

 Let me know if you want me to squash first.

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


More information about the tor-bugs mailing list