From mboxrd@z Thu Jan  1 00:00:00 1970
From: Romain Naour <romain.naour@openwide.fr>
Date: Sat, 02 Nov 2013 16:46:18 +0100
Subject: [Buildroot] [PATCH v2 1/1] ngrep: fix static linking issue with
 libpcre
In-Reply-To: <20131101175957.689162a5@skate>
References: <5255D08E.3000307@openwide.fr> <20131101175957.689162a5@skate>
Message-ID: <52751E4A.1070808@openwide.fr>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

Le 01/11/2013 17:59, Thomas Petazzoni a ?crit :
> Dear Romain Naour,
>
> On Wed, 09 Oct 2013 23:54:22 +0200, Romain Naour wrote:
>> Adding -lpcre in LDFLAGS place the library before object files:
>> gcc ... -lpcre -L .../sysroot/usr/lib -s -o ngrep ngrep.o -lpcap
>>
>> Makefile.in is patched to use @LIBS@ for adding -lpcre in the right place.
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/684/684a3bed28d13ef5e5156257aade3b9aff32f180
>>
>> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> Thanks, but this patch no longer applies on top of master, and more
> importantly, the ngrep-1.45-Makefile.in.patch doesn't apply to the
> ngrep sources. Could you rename the ngrep patches to use numbers, i.e
> ngrep-01-<foo>.patch, and use a more descriptive name than
> ngrep-1.45-Makefile.in.patch?
>
> (Should be done in two patches: one renaming the existing patches, one
> adding the pcre static link fix).
>
> Thanks a lot!
>
> Thomas
Hi Thomas,

Ok, I'll send two patches to fix this issue.

Best regards,
Romain