asterisk: update AST_EXT_TOOL_CHECK for OpenWrt 673/head
authorSebastian Kemper <sebastian_ml@gmx.net>
Fri, 20 Aug 2021 20:59:49 +0000 (22:59 +0200)
committerSebastian Kemper <sebastian_ml@gmx.net>
Fri, 20 Aug 2021 21:29:31 +0000 (23:29 +0200)
commit79a0acfe56256748615159f6e08687570ac2cf88
tree6f3b636549e190c57e2245f9dee982738d13df3b
parent2feac1069fbaf58a110c741bcba4f9e092ec2704
asterisk: update AST_EXT_TOOL_CHECK for OpenWrt

Issue #672 shows that the current macro has some pitfalls, at least when
running within OpenWrt's buildroot.

The macro is used to detect these tools:

  1. xml2-config
  2. mysql_config
  3. neon-config
  4. net-snmp-config
  5. sdl-config (we're not using SDL)

The macro also adds configure arguments that allow to specify a path
like so:

  --with-mysqlclient=PATH

Macro problems:

  1. If no PATH is specified, it adds "/bin" to the search path.
     Obviously this is a bad thing when cross-compiling.
  2. If a PATH is specified, it adds it to the start of every include
     and library path it encounters. These paths are then broken.

This commit makes problem 1 go away by making the addition of
"${$1_DIR}/bin" to the PATH dependent on there being a PATH handed to
the macro. An alternative would be to remove the PATH addition
altogether, but then the macro wouldn't behave as expected.

It also removes the path filtering of the include and lib dirs to
address problem 2.

Closes #672

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>
net/asterisk/patches/160-AST_EXT_TOOL_CHECK.patch [new file with mode: 0644]