[Buildroot] [PATCH v2] ejabberd: Bump to version 17.11

[Johan Oudinet]

Hi Arnout,

On Thu, Feb 8, 2018 at 6:39 PM, Arnout Vandecappelle <arnout at mind.be> wrote:
>  Originally, this was done with $(shell ....) so it would have worked. Two years
> ago, in commit 7d69a79624b1f (signed off by yours truly) all the $(shell ...)
> constructs were replaced with backticks `` and it stopped working. So it just
> means all those variables would have been empty (the error messages would have
> been visible on stderr but nobody looks at them). And I guess that the libraries
> are just in default locations so it still works even if the lib_dir variables
> are empty.

Thanks for the explanation. Indeed, commit 7d69a79624 changed $(shell
...) to backticks and introduces this bug two years ago :-/
As you said, this was only visible on stderr and I notice it when
bumping ejabberd to 17.11 only.

>  To be honest, I find the solution of copying the script a bit fragile. It would
> be safer to just prepend $TOPDIR to the path so it is absolute.

Great, I was not aware of $TOPDIR. I looked for something like
PACKAGE_DIR but failed to find a solution and finally resigned to copy
this script to $(@D).
I'll send a small patch to remove this unnecessary copy and use $TOPDIR instead.
Thanks.

Best,

-- 
Johan