[Buildroot] [PATCH v3 1/1] package/at: updated version from 3.1.13 to 3.1.16

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sun Sep 20 13:08:38 UTC 2015


Dear Francesco Nwokeka,

On Tue, 08 Sep 2015 14:13:59 +0200, Francesco Nwokeka wrote:
> Version 3 of the patch to update the "at" package.

You received some comments from Arnout Vandecappelle on September 8th
on this patch. Could you take them into account and send an updated
version of the patch?

I was going to apply by fixing the problems pointed by Arnout, but one
requires a bit more work:

"""
> * Merged previous Makefile.in patches into one.  

 Ick, that's not what I intended! We want a separate patch file for each
separate change. The two patches that you squashed into one do two
 distinct changes:

 - change IROOT into DESTDIR
 - don't try to install as root

So it's good to keep them separate.

 My proposal was to drop the first patch and just use IROOT. In
 buildroot, we can easily use IROOT instead of DESTDIR by setting

AT_INSTALL_TARGET_OPTS = IROOT=$(TARGET_DIR) install
"""

Could you fix that and send a new version?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list