From mboxrd@z Thu Jan 1 00:00:00 1970 From: Li Wang Date: Mon, 18 Nov 2019 14:38:55 +0800 Subject: [LTP] [PATCH] configure.ac: Fix --with* options In-Reply-To: <20191118061634.GA21536@x230> References: <20191113104149.42407-1-lkml@jv-coder.de> <1c4d8fb5-58e4-5483-26c7-7ab38b5fbd9c@jv-coder.de> <20191118061634.GA21536@x230> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it On Mon, Nov 18, 2019 at 2:16 PM Petr Vorel wrote: > Hi, > > Reviewed-by: Petr Vorel > > > Well, I just saw adding new #TODO here and hoping to solve this > together, a > > simple way I was thinking is: > > > +AS_IF([test "x$with_bash" != xyes || test "x$with_python" != xyes], > > + [with_realtime_testsuite=no] > > +) > > > Of course, It's fine to fix that in a separate patch! > > I'm for merging this. I just wonder if it's not better to remove > $with_bash and > $with_python (as part of this patch), if they're not working. > Me too. I think we could fix that in one patch. But as Joerg mentioned they will be working on the #TODO. So maybe it doesn't matter to do right now or later. > > BTW better, than fixing bash/shell would be to actually rewrote tests > which are > using python and bash to C or posix shell. > https://github.com/linux-test-project/ltp/issues/547 > > Kind regards, > Petr > > -- Regards, Li Wang -------------- next part -------------- An HTML attachment was scrubbed... URL: