[Buildroot] [PATCH v3] luarocks: add buildroot addon

Thomas Petazzoni thomas.petazzoni at bootlin.com
Mon Dec 31 17:06:55 UTC 2018


Hello,

On Wed, 19 Dec 2018 09:17:03 +0100, Francois Perrad wrote:

> ++   if #licenses == 1 then
> ++      f:write(ucname .. '_LICENSE_FILES = $(' .. ucname .. '_SUBDIR)/' .. licenses[1] .. '\n')
> ++   elseif #licenses > 1 then
> ++      f:write(ucname .. '_LICENSE_FILES =')
> ++      for i = 1, #licenses do
> ++         local file = licenses[i]
> ++         f:write(' \\\n    $(' .. ucname .. '_SUBDIR)/' .. file)

This is not good: it indents each line with 4 spaces, while it should
indent with one tab. Due to this, the regenerated .mk files caused
check-package to complain. I fixed the issues manually in:

  https://git.busybox.net/buildroot/commit/?id=42fa5bcbfb48f188c2c180854c0bbdefd98c32c8

But of course the Luarocks addon must be fixed to not re-introduce
similar issues. Could you cook a patch to that effect ?

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the buildroot mailing list