<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 14, 2019 at 4:09 PM Joerg Vehlow <<a href="mailto:lkml@jv-coder.de">lkml@jv-coder.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
><br>
>     Defining AC_ARG_WITH within an if to express dependencies does not<br>
>     work.<br>
>     ./configure --with-realtime-testsuite set with_realtime_testsuite=yes,<br>
>     even if with_bash=no or with_python=no. The check is removed<br>
>     completely.<br>
><br>
><br>
> Or, maybe we can make use of AS_IF here? and I noticed there are many <br>
> places that use "if test ..." in the <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> <br>
> <<a href="http://configure.ac" rel="noreferrer" target="_blank">http://configure.ac</a>> file. That looks tangly...<br>
I left a TODO in there so it can be fixed later, maybe together with <br>
other errors. I think this does not block merging this patch first?<br></blockquote><div><br></div><div><span class="gmail_default" style="font-size:small">Well,</span> I just saw adding new #TODO here and hop<span class="gmail_default" style="font-size:small">ing</span> to solve this together, a simple way I was<span class="gmail_default" style="font-size:small"></span> thinking is:<br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">+AS_IF([test "x$with_bash" != xyes || test "x$with_python" != xyes],<br>+  [with_realtime_testsuite=no]<br>+)</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Of course, It's fine to fix that in a separate patch!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
The python requirement for realtime-testsuite is outdated/wrong anyway, <br>
if I remember correctly. A colleague of mine recently<br>
investigated the python requirement and we came to the conclusion, that <br>
it is not needed for running the test.<br>
I think it was used only for manual test result evaluation. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
So there seems to be more work regarding dependencies between configure <br>
options.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Yes, thanks for investigating this. Looking forward to more patches :).</div><div></div><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_default" style="font-size:small"><div class="gmail_default">For this one:</div><div class="gmail_default">    Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></div></div><div class="gmail_default" style="font-size:small"><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>