On Thu, Jan 12, 2017 at 3:55 PM, Khaled Elsayed <khaledieee at gmail.com> wrote:
> > Hi Philippe, > > Thanks for the proposals. We are doing something similar to option 1. > Problem is sometimes original author is bottleneck as he is the only person > authorized to submit a change. > > If we do option 2, would the changes be under the same patch and different > authors will be allowed? > > Only one could be merged because they will conflict, again if it's a shared work put all names in commit message, even if there is one author committing in the end make sure it's the more relevant one and understand what Signed-off: mean. Now, If you are referring to ESP82266 patch mentioned at https://wiki.iotivity.org/hardware Let me give some hints for other pple who want to support new devices. As said on review,I will recommend to try to split in smaller patches, (ie replacing ifdef arduino by ifdef mcu), and rebase on them, to make sure that patch is only about ESP82266 support and the smallest possible as Thiago suggested in this list. Long term benefit is that port to other MCUs will be prepared. Anyway please create a task in jira, and track related changes or efforts. Ask me for help if needed. Kind regards! -- https://wiki.tizen.org/wiki/User:Pcoval -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.iotivity.org/pipermail/iotivity-dev/attachments/20170112/71a08bd1/attachment.html>
