From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Sat, 1 Dec 2018 22:55:10 +0100 Subject: [Buildroot] [PATCH next v3] package/luarocks: rework configuration file for per-package folders In-Reply-To: <20181130103829.32241-1-thomas.petazzoni@bootlin.com> References: <20181130103829.32241-1-thomas.petazzoni@bootlin.com> Message-ID: <20181201225510.43ca2908@windsurf.home> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello, On Fri, 30 Nov 2018 11:38:29 +0100, Thomas Petazzoni wrote: > Currently, luarocks.mk generates a configuration file with hardcoded > STAGING_DIR, TARGET_DIR, TARGET_CC, LUAROCKS_CFLAGS and TARGET_LDFLAGS > values. This is not compatible with per-package folders, where the > value of STAGING_DIR, TARGET_DIR, TARGET_CC and possibly > TARGET_CFLAGS/TARGET_LDFLAGS may be different from one package to the > other. > > Based on input from Fran?ois Perrad, this commit: > > - Changes the Luarocks configuration file to use os_getenv() for the > appropriate variables. Since the contents of this file is not > fixed, it is no longer generated by luarocks.mk using a series of > 'echo' but simply concatenated with the rest of the Luarocks > configuration file. > > - Adjusts LUAROCKS_RUNV_ENV so that the necessary environment > variables are now passed. > > Signed-off-by: Thomas Petazzoni > --- > Changes since v2: > - Do not pass STAGING_DIR/TARGET_DIR in LUAROCKS_RUN_ENV, since they > are already exported globally in the environment. Suggested by > Arnout Vandecappelle. Applied to next, thanks. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com