<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Petr,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><div dir="ltr" class="gmail_attr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail_default" style="font-size:small">...</span><br>
> I guess that we can even drop the TCID extern definition from the tst_test.h<br>
> header if the weak attribute works fine.<br>
<br>
I'd be for adding this patch (both version works), Cyril, will you send it<br>
to ML / push it?<br>
<br>
I'd also be for adding <span class="gmail_default" style="font-size:small"></span>-fno-common to Travis after this fix.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">After building LTP with CFLAGS="-fno-common" locally, I just find more places that need to do the same improvement.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">    ltp/testcases/kernel/controllers/libcontrollers/libcontrollers.h:56: multiple definition of `num_line'</div><div class="gmail_default" style="font-size:small">    ltp/lib/../include/tst_hugepage.h:16: multiple definition of `Hopt'</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">Sorry for involving the definition in tst_hugetlb.h without 'extern' issue many hours ago. It'd be appreciated if you can fix them together in your coming patch.</div></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Hope this is a timely reminder in case you formatting patch V2 :).</div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>