<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Richard,</div></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>> I'd be for adding it there, with comment why it's there. We can prevent<br>
>> problems<br>
>> with failing another test in the future. (+ remove it from both tests).<br>
>><br>
><br>
> I'm OK with this fix.<br>
><br>
> @Cyril, @Richard, what do you think? any other thoughts?<br>
<br>
We need to clean up our headers, which is a bigger problem. Most tests<br>
do not need all the stuff in tst_test.h. It is just a load of unecessary<br>
work.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Yes, I agree. It benefits for long-term maintenance of LTP.</div></div><div class="gmail_default" style="font-size:small">So maybe a good start is to be more strict in review new</div><div class="gmail_default" style="font-size:small">patches especially on the includes header check.</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">
<br>
Cleaning up the headers is a big challenge. It would be easier if we<br>
know what will break older distros. So I suggest adding something like:<br>
<br>
#ifdef _X_H<br>
# error "You should include X before Y ..."<br>
#endif<br>
<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Hmm, it's right, but I'm afraid we have to define that in many places</div><div class="gmail_default" style="font-size:small">because include headers are sometimes dispersive or singly.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">I.e.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">We need to define the above hint duplicated for everywhere if includes<br></div><div class="gmail_default" style="font-size:small">the <netinet/in.h>, to avoid include <linux/in.h> before it. This will</div><div class="gmail_default" style="font-size:small">look a bit messy I feel.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">$ git grep netinet/in.h<br>lapi/netinet_in.h:#include <netinet/in.h><br>old/old_safe_net.h:#include <netinet/in.h><br>safe_net_fn.h:#include <netinet/in.h><br>tst_net.h:#include <netinet/in.h><br>tst_safe_net.h:#include <netinet/in.h><br></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">
to one or more headers.<br>
<br>
Otherwise I'm fine with the above solution as a "temporary" fix.<br></blockquote><div> </div><div><br></div></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>