From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Thu, 23 Jul 2015 23:11:20 +0200 Subject: [Buildroot] [PATCH V2] c-icap: fix config script by adding $(STAGING_DIR) prefix to link path. It fixes c-icap-modules build. In-Reply-To: <1437641147-6894-1-git-send-email-guillaume.gardet@oliseo.fr> References: <20150722155117.46a0e983@free-electrons.com> <1437641147-6894-1-git-send-email-guillaume.gardet@oliseo.fr> Message-ID: <20150723231120.75b681e8@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Dear Guillaume GARDET, On Thu, 23 Jul 2015 10:45:47 +0200, Guillaume GARDET wrote: > Reported-by: Thomas Petazzoni > Signed-off-by: Guillaume GARDET > Signed-off-by: Thomas Petazzoni I applied, but quite a few things were still wrong with the patch: * The commit title is too long. It should be less than 80 characters. The commit title should only be a summary of the commit log itself. * The commit title is wrong: you're no longer adding $(STAGING_DIR) to the link path * The commit log was missing a reference to the autobuilder failure that this patch is fixing. I've fixed those problems and applied. See: http://git.buildroot.net/buildroot/commit/?id=6c73215493335f1ede26cad2d3898ef543d871d6 Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com