<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 16, 2015 at 5:27 PM, Kevin Darbyshire-Bryant <span dir="ltr"><<a href="mailto:kevin@darbyshire-bryant.me.uk" target="_blank">kevin@darbyshire-bryant.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 16/10/15 11:24, Loganaden Velvindron wrote:<br>
><br>
><br>
> On Fri, Oct 16, 2015 at 1:39 PM, Loganaden Velvindron<br>
</span><span class="">> <<a href="mailto:loganaden@gmail.com">loganaden@gmail.com</a> <mailto:<a href="mailto:loganaden@gmail.com">loganaden@gmail.com</a>>> wrote:<br>
><br>
><br>
><br>
>     Building the latest tc-adv causes issues with gcc.<br>
><br>
><br>
>     gcc -Wall -Wstrict-prototypes  -Wmissing-prototypes<br>
>     -Wmissing-declarations -Wold-style-definition -Wformat=2 -O2<br>
>     -I../include -DRESOLVE_HOSTNAMES -DLIBDIR=\"/usr/lib\"<br>
>     -DCONFDIR=\"/etc/iproute2\" -D_GNU_SOURCE  -DHAVE_SETNS -DHAVE_ELF<br>
>     -DCONFIG_GACT -DCONFIG_GACT_PROB<br>
>     -DIPT_LIB_DIR=\"/usr/lib/iptables\" -DYY_NO_INPUT   -c -o<br>
>     q_cake0.o q_cake0.c<br>
>     In file included from q_cake.c:48:0:<br>
>     q_cake.c: In function<br>
>     ‘cake_print_xstats’:../include/utils.h:177:55: error: expected<br>
>     expression before ‘)’ token # define offsetof(type, member) (<br>
>     (size_t) & ( (type *) 0 )->member )<br>
>                             ^q_cake.c:500:31: note: in expansion of<br>
>     macro ‘offsetof’<br>
>          && RTA_PAYLOAD(xstats) >= offsetof(*stnc, capacity_estimate))<br>
><br>
>     GCC version:<br>
>     gcc version 5.2.0 (GCC)<br>
><br>
><br>
> The commit causing the issue is this one:<br>
><br>
> <a href="https://github.com/dtaht/tc-adv/commit/9b2dc094b2b1526ec4222efc1b0a1d02812def5a" rel="noreferrer" target="_blank">https://github.com/dtaht/tc-adv/commit/9b2dc094b2b1526ec4222efc1b0a1d02812def5a</a><br>
><br>
> I'm not sure about the fix. here is it:<br>
><br>
> <a href="https://github.com/dtaht/tc-adv/pull/7" rel="noreferrer" target="_blank">https://github.com/dtaht/tc-adv/pull/7</a><br>
><br>
><br>
><br>
</span>Hi Logan,<br>
<br>
Thanks for the report.  It breaks here too :-( I'm annoyed with myself<br>
for merging the latest, latest, latest commits (around 02:00 my time)<br>
early this morning without at least compile testing them.  I think it<br>
was about 05:00 whilst doing an overnight shift (Aussie MotoGP) and the<br>
box I did it on is a little out of date and slow (would require a<br>
toolchain rebuild which is about 4 hours!)  Normally I at least compile<br>
test.  Oh well, move on...<br>
<br>
I'm sure Jonathan will pick this up shortly - In fact as I type this<br>
after just compile testing & installing your fix I notice that he has :-)<br>
<br>
Logan, thanks very much for your fix, it's appreciated.  I will however<br>
close your pull request and merge in the (identical!) fix from upstream<br>
to keep the git histories a little closer.<br>
<br></blockquote><div><br></div><div>Thanks for updating the master tree. I was working on another issue, when I encountered this problem with the latest sync :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cheers,<br>
<br>
Kevin<br>
<br>
<br>
<br>
</blockquote></div><br></div></div>