On 08/19/2014 09:38 AM, Martin Kosek wrote:
On 08/18/2014 05:17 PM, thierry bordaz wrote:
On 08/18/2014 05:10 PM, Petr Viktorin wrote:
On 08/18/2014 05:03 PM, thierry bordaz wrote:
...
Simply reply to this mail with the revised patch attached.

As for attaching patches to the tickets, I've never done it and no one ever
complained, so it's not necessary. But attach it if you like :)
Good to know :). Thanks
It is not required. It is only stated as "optional" step in
http://www.freeipa.org/page/Contribute/Code#Submit_a_patch

I personally attach at least the first version to help pairing the ticket with
the patch number and better searching in the list. The job is done by my patch
submission script, so it is no additional work for me anyway :-)

Ok. Thanks for the explanation.
In fact with multi patch ticket, adding all patches (original+reviewed) seems too noisy. I will keep the process to add in the ticket the first version of each patch.

thanks
thierry

Martin

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to