[ovs-dev] [PATCH] dpif-netdev: Fix double inclusion of cmap.h

Ilya Maximets i.maximets at samsung.com
Fri Feb 26 05:35:48 UTC 2016


On 26.02.2016 03:11, Joe Stringer wrote:
> On 24 February 2016 at 23:51, Ilya Maximets <i.maximets at samsung.com> wrote:
>> Also, all headers sorted lexicographically.
>>
>> Fixes: 9f361d6b29fb ("dpif-netdev: Use cmap instead of hmap.")
>> Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
> 
> Typically the "Fixes:" tag is used for bugs, to help us determine
> which releases have the problem so we can backport the fix. Generally
> we don't backport trivial patches. Does this cause build issues for
> OVS-2.4?

I don't think that this can cause any build issues. I just wanted to
point out a place where duplication was introduced. You're right, I
should have done it in another way.

> Thanks, I applied this to master.

Thanks.
Best regards, Ilya Maximets.



More information about the dev mailing list