[Buildroot] [PATCH v2 10/18] erlang-p1_stun: bump to version 0.9.0

[Thomas Petazzoni]

Dear Frank Hunleth,

On Tue,  2 Feb 2016 14:57:36 -0500, Frank Hunleth wrote:
> As part of the version change, the package name changed to use an
> underscore rather than a hyphen. Fortunately, the name change doesn't
> affect the Config.in option, so no legacy handling is needed.
> 
> Signed-off-by: Frank Hunleth <fhunleth at troodon-software.com>

As suggested by Romain in his review, I've dropped the rename.
Generally speaking, when things aren't clear between _ and -, I very
much prefer to see - used in package name (though I admit this is a
personal preference).


>  config BR2_PACKAGE_ERLANG_P1_STUN
> -	bool "erlang-p1-stun"
> -	select BR2_PACKAGE_ERLANG_P1_TLS
> +	bool "erlang-p1_stun"
> +	select BR2_PACKAGE_ERLANG_FAST_TLS

Build testing has shown that BR2_PACKAGE_ERLANG_FAST_TLS was not the
proper dependency, and instead keeping BR2_PACKAGE_ERLANG_P1_TLS was
the right solution. Could you double check why you switched to
fast_tls ?

So, I've applied to next, with the following changes:

    [Thomas:
     - do only the version bump, not the package rename, as suggested by
       Romain Naour
     - erlang-p1-stun continues to depend on erlang-p1-tls, not
       erlang-fast_tls.]

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com