<div dir="ltr">My comments weren't a critique of your patch, and probably belong in a separate thread, or even a bug report on <a href="http://dev.openwrt.org">dev.openwrt.org</a>. <div><br></div><div>Apparently the behavior my machine is showing is not the normal or intended behavior. On my machine (Fedora 22, but also had the issue in Fedora 21, if I remember correctly), there is no error message printed when the image generation fails due to being too large. Running V=s shows:</div><div> </div><div>[mktplinkfw] *** error: images are too big</div><div>Makefile:2244: recipe for target 'install' failed</div><div>make[5]: [install] Error 255 (ignored)</div><div><br></div><div>So, the real problem is that for some reason, my system is ignoring the make error. This is unrelated to your patch. My thinking is that it was intended behavior, and would make sense to change it in the same patch. Sorry about the confusion.</div><div><br></div><div><br></div><div>--Jonathan</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 13, 2015 at 10:46 AM Stephan Maka <<a href="mailto:stephan@spaceboyz.net">stephan@spaceboyz.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Jonathan Bennett wrote:<br>
> Tested and seems to work as intended. However, I think the image too big<br>
> messages should be printed at the default verbosity.<br>
<br>
The patch doesn't change any levels.<br>
<br>
> Currently, if make fails to generate the final image, it doesn't give any<br>
> message to acknowledge, and appears to have succeeded, there are just<br>
> "mysteriously" missing files in the output directory.<br>
<br>
All the ERR() outputs are followed by return -1, causing check_options()<br>
and therefore the process and make to fail.<br>
<br>
Sorry if I don't see your point but I'll be happy about follow-up<br>
patches. :)<br>
<br>
<br>
Cheers,<br>
Stephan<br>
</blockquote></div>