[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm
From: |
Mike Frysinger |
Subject: |
Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm |
Date: |
Sat, 27 Apr 2013 16:38:38 -0400 |
User-agent: |
KMail/1.13.7 (Linux/3.8.3; KDE/4.6.5; x86_64; ; ) |
On Saturday 27 April 2013 13:53:28 Peter Rosin wrote:
> On 2013-04-27 07:58, Mike Frysinger wrote:
> > The current code tries to locate a compatible nm tool. It starts with
> > a prefixed nm tool (great!) and includes a plain nm too (that's fine).
> > The problem is that the code searches for the prefixed nm before the
> > plain nm (normally fine), but doesn't break once it has found a valid
> > match. It does this so that it if it finds an "OK", but "not great",
> > tool, it'll keep on searching.
>
> I agree this sounds like the wrong this to do, but isn't it better to
> just break all the way out when a "great" nm is found?
for some reason i thought the [n] arg to break wasn't portable. this should
work though.
-mike
signature.asc
Description: This is a digitally signed message part.
- [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Mike Frysinger, 2013/04/27
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Peter Rosin, 2013/04/27
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm,
Mike Frysinger <=
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Peter Rosin, 2013/04/28
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Mike Frysinger, 2013/04/28
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Peter Rosin, 2013/04/29
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Mike Frysinger, 2013/04/29
- Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm, Peter Rosin, 2013/04/29