All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [git commit] package/Config.in: explain that lua package names should start with lua
@ 2017-03-06 20:20 Thomas Petazzoni
  0 siblings, 0 replies; only message in thread
From: Thomas Petazzoni @ 2017-03-06 20:20 UTC (permalink / raw)
  To: buildroot

commit: https://git.buildroot.net/buildroot/commit/?id=687a313179b07865a12e9a35bc4c506fb371d2d0
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

We have decided that lua packages should have a name that starts with
lua (like is the case for python and perl). However, we're not going to
rename all the existing lua packages that don't start with lua. This
makes it unclear for people adding packages how they should name the
package, so add a comment to package/Config.in to explain it.

It's rather terse but it gets the message across.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/Config.in b/package/Config.in
index 78acc53..cfe7fc6 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -525,6 +525,10 @@ endif
 if BR2_PACKAGE_HAS_LUAINTERPRETER && !BR2_STATIC_LIBS
 # lua modules are dynamically loaded, so not available on static builds
 menu "Lua libraries/modules"
+# When adding lua packages, make sure the name start with "lua". If the
+# upstream package is "luafoo", call it "luafoo" in Buildroot; if the upstream
+# package is "foo", call it "lua-foo". Many packages currently don't follow
+# this convention, but that's historical accident.
 	source "package/argparse/Config.in"
 	source "package/cgilua/Config.in"
 	source "package/copas/Config.in"

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2017-03-06 20:20 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-06 20:20 [Buildroot] [git commit] package/Config.in: explain that lua package names should start with lua Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.