[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: |
Peter Rosin |
Subject: |
Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm |
Date: |
Sat, 27 Apr 2013 19:53:28 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130328 Thunderbird/17.0.5 |
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?
(patch untested)
Cheers,
Peter
break-all-the-way-out-of-the-nm-loop.patch
Description: Text Data
- [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 <=
- Re: [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/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