On 02/10/2016 05:03 PM, Russell Bryant wrote:
> A recent change changed this constructor to "init_this_module". This
> breaks the build on windows according to AppVeyor. Include the module
> name in the constructor name to work around the issue.
>
> Fixes: 922fed065e65 ("vlog: Make the most common module reference more
> direct.")
> Signed-off-by: Russell Bryant <[email protected]>
> ---
> include/openvswitch/vlog.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/openvswitch/vlog.h b/include/openvswitch/vlog.h
> index feade72..de64cbd 100644
> --- a/include/openvswitch/vlog.h
> +++ b/include/openvswitch/vlog.h
> @@ -181,7 +181,7 @@ void vlog_rate_limit(const struct vlog_module *, enum
> vlog_level,
> VLL_INFO, /* min_level */ \
> true /* honor_rate_limits */ \
> }; \
> - OVS_CONSTRUCTOR(init_this_module) { \
> + OVS_CONSTRUCTOR(init_this_module_##MODULE) { \
> vlog_insert_module(&this_module.list); \
> } \
> \
>
I'm headed out for the day. If you agree with this, feel free to push
it since it's a build fix.
--
Russell Bryant
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
