Re: Submitting patches on behalf of Mellanox

[Anders Broman <anders.broman () ericsson com>]

________________________________
From: wireshark-dev-bounces () wireshark org [mailto:wireshark-dev-bounces () wireshark org] On Behalf Of Slava Koyfman
Sent: den 29 april 2010 15:07
To: wireshark-dev () wireshark org
Subject: [Wireshark-dev] Submitting patches on behalf of Mellanox

Hi,

I work for Mellanox, and we've created several patches to the Infiniband dissector in Wireshark which I'd like to 
submit for inclusion in the mainline. Are there some specific steps I need to follow when submitting patches on behalf 
of a company, or do I just attach them to a bug report as usual?
No special procedure.

Also, the changes include some bugfixes and several feature additions. Currently I have each fix/feature as a separate 
patch. Should I open a separate bug for each of these, attach them all to the same bug report, or combine them and 
submit the changes as one big patch file?
I'd prefere a single bug report or perhaps one with bugfixes and one with features.

Bugfixes could be nice to separate out as they could then be backported to the stable version.

Small patch files are easier to review and can be accepted/rejected on a file per file basis on the other hand applying 
a single file
is easier. So the answer is - it depends :-)
( complexity, number of code lines changed, number of patches...)

Thanks,
Slava.

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe