Re: [Boost-bugs] [Boost C++ Libraries] #12531: --run_test in Boost 1.62 does not accept test names which contain ':'

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #12531: --run_test in Boost 1.62 does not accept test names which contain ':'
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2016-10-19 20:46:32


#12531: --run_test in Boost 1.62 does not accept test names which contain ':'
----------------------------------------------+------------------------
  Reporter: Igor Akhmetov <igor.akhmetov@…> | Owner: renficiaud
      Type: Bugs | Status: assigned
 Milestone: To Be Determined | Component: test
   Version: Boost 1.62.0 | Severity: Problem
Resolution: | Keywords:
----------------------------------------------+------------------------

Comment (by renficiaud):

> Quoting command line arguments is already required though to correctly
 pass test names which > contain quotes or spaces.

 I am not sure to understand: do you have test names with quotes or spaces?

 Quoting would help in the sense that ':' would not be interpreted as a
 filter separator within quotes, which in turn would remove the need for
 back-slashing the ':'. For your own use case, it would use at much as many
 additional chars, but would have better readability ("ns::X" and ns\:\:X
 have both 2 additional chars, but "ns::X::Y" is more readable and has less
 chars than ns\:\:X\:\:Y).

 You would then have something like

     --run_test="ns::X":"ns::Y":suite1/suite2/"ns2::X","ns2::Y"

 There is indeed one use case needing specifying filters from environment
 variable (and it is on the ticket I mentioned).

 If it were only me, I would rather remove the support for any special
 chars from the test unit names and replace everything by "_" (ns_ _X
 instead of ns::X), that would things much cleaner.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/12531#comment:6>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.

This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:20 UTC