From: Martin C. <mc...@re...> - 2025-05-16 20:29:59
|
On Thu 2025-05-15 12:34 , Cyril Hrubis wrote: > Hi! > > LTP_COLORIZE_OUTPUT allows more complicated setup (y/1 always, n/0: never), > > but any value here enables the feature (e.g. LTP_COLORIZE_OUTPUT=0). > > This would be nice to unify, but I'm ok to ignore it as it's before release. > > We also have LTP_ENABLE_DEBUG that has the same values so let's keep the > API the same. I've send v3 with this and the docs changes. > > > > + > > > assert_test_fn(); > > > > > TCID = tid = get_tid(argv); > > > > Adding docs (+ sort variables while at it). > > I've only added the docs for LTP_REPRODUCIBLE_OUTPUT in v3, the sorting > should go in a separate patch. We also miss at least TST_ENABLE_DEBUG in > the output of -h switch as well. Thank-you, guys. I've added a preparation to the valgrind-ltp tester: https://sourceware.org/git/?p=valgrind.git;a=blob;f=auxprogs/ltp-tester.sh;h=036f196ce21ae6cc692de205ab49774638cfb925;hb=5894abc5fa9de30c6b4dde453bff3ac1034aa330#l19 Once the new version of LTP ships, we can try dropping our workarounds. Cheers, Martin |