[OE-core] [PATCH] libpcap: Fix build when PACKAGECONFIG ipv6 is not enable

[Otavio Salvador]

On Thu, Nov 17, 2016 at 2:24 PM, Christopher Larson <clarson at kergoth.com> wrote:
>
> On Thu, Nov 17, 2016 at 9:21 AM, Fabio Berton
> <fabio.berton at ossystems.com.br> wrote:
>>
>> No, I created a patch, git format-patch and then edit generated files with
>> Upstream-Status tag and added to recipe. Is this wrong?
>
>
> As I indicated in my first reply, it’s best to put the tag outside the
> generated patch (above it, or below the —-), as it isn’t part of the commit,
> only part of the patch file. It’s minor, and you don’t need to re-submit,
> but in general the tag is not part of the commit message. For example, if
> your patch was applied to a git repository with git-am, it’d be in the
> commit message, which should not be the case.

To be honest, there a many files which does as Fabio did and I also
personally do as well. I don't like it outside of the commit log as it
looks out of context.

If someone is getting the patch file, it is expected they will review
it and not blindly apply it on upstream repository.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750