Skip to contentSkip to conversation

package/lua-cqueues: needs host-m4

PATCH New Thomas Petazzoni 3 participants 3 messages Dec 11, 2018 – Dec 16, 2018

lore
Share & tools
lore

Thomas Petazzoni

author
#0 lore raw

PATCH 1/1 package/lua-cqueues: needs host-m4

The lua-cqueues build system uses m4 during the build:

cd /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src && m4 /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src/errno.c.m4 >| /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src/errno.c.tmp

Therefore, the package should depend on host-m4. This has been detected using per-package directory support, but a minimal defconfig with just lua and lua-cqueues on a system that doesn't have m4 installed system-wide also exhibits the issue.

Signed-off-by Thomas Petazzoni <[email protected]>
---
 package/lua-cqueues/lua-cqueues.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/lua-cqueues/lua-cqueues.mk +1 -1
index 4f4be1e049..4f31c3277f 100644
@@ -8,7 +8,7 @@ LUA_CQUEUES_VERSION = rel-20161215
 LUA_CQUEUES_SITE = $(call github,wahern,cqueues,$(LUA_CQUEUES_VERSION))
 LUA_CQUEUES_LICENSE = MIT
 LUA_CQUEUES_LICENSE_FILES = LICENSE
-LUA_CQUEUES_DEPENDENCIES = luainterpreter openssl
+LUA_CQUEUES_DEPENDENCIES = luainterpreter openssl host-m4
 
 define LUA_CQUEUES_BUILD_CMDS
 	$(TARGET_MAKE_ENV) $(MAKE1) $(TARGET_CONFIGURE_OPTS) -C $(@D) \
Signature · 1 line
2.19.2
Applied
#1 lore raw

PATCH 1/1 package/lua-cqueues: needs host-m4

Quoted 10 lines · Thomas Petazzoni

The lua-cqueues build system uses m4 during the build:

cd /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src && m4 /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src/errno.c.m4 >| /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src/errno.c.tmp

Therefore, the package should depend on host-m4. This has been detected using per-package directory support, but a minimal defconfig with just lua and lua-cqueues on a system that doesn't have m4 installed system-wide also exhibits the issue.

Signed-off-by Thomas Petazzoni <[email protected]>

Applied to master, thanks.

Regards,
Arnout

Quoted 18 lines
---
 package/lua-cqueues/lua-cqueues.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/lua-cqueues/lua-cqueues.mk +1 -1
index 4f4be1e049..4f31c3277f 100644
@@ -8,7 +8,7 @@ LUA_CQUEUES_VERSION = rel-20161215
 LUA_CQUEUES_SITE = $(call github,wahern,cqueues,$(LUA_CQUEUES_VERSION))
 LUA_CQUEUES_LICENSE = MIT
 LUA_CQUEUES_LICENSE_FILES = LICENSE
-LUA_CQUEUES_DEPENDENCIES = luainterpreter openssl
+LUA_CQUEUES_DEPENDENCIES = luainterpreter openssl host-m4
 
 define LUA_CQUEUES_BUILD_CMDS
 	$(TARGET_MAKE_ENV) $(MAKE1) $(TARGET_CONFIGURE_OPTS) -C $(@D) \
#2 lore raw

PATCH 1/1 package/lua-cqueues: needs host-m4

In reply to Thomas Petazzoni
Quoted 11 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Thomas" == Thomas Petazzoni <[email protected]> writes:

The lua-cqueues build system uses m4 during the build:
cd /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src && m4 /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src/errno.c.m4 >| /home/test/autobuild/run/instance-1/output/build/lua-cqueues-rel-20161215/src/errno.c.tmp

Therefore, the package should depend on host-m4. This has been detected using per-package directory support, but a minimal defconfig with just lua and lua-cqueues on a system that doesn't have m4 installed system-wide also exhibits the issue.

Signed-off-by Thomas Petazzoni <[email protected]>

Committed to 2018.11.x, thanks.

Signature · 1 line
Bye, Peter Korsgaard