Ashok, Excellent. Thanks! Pat From: ASHOKBABU CHANNA [mailto:[email protected]] Sent: Wednesday, June 24, 2015 12:41 AM To: Lankswert, Patrick Cc: iotivity-dev at lists.iotivity.org Subject: Re: RE: New branch and design policies
Pat, Yes, all threading change sets are in available in gerrit. Ashok ------- Original Message ------- Sender : Lankswert, Patrick<patrick.lankswert at intel.com<mailto:patrick.lankswert at intel.com>> Date : Jun 24, 2015 02:13 (GMT+09:00) Title : RE: New branch and design policies Ashok, One additional question: Are all of the threading change sets in gerrit? I want to make sure that he code under review is gerrit is whole and complete if I am going to accept it into master. Pat From: ASHOKBABU CHANNA [mailto:[email protected]] Sent: Tuesday, June 23, 2015 1:57 AM To: Lankswert, Patrick Cc: iotivity-dev at lists.iotivity.org<mailto:iotivity-dev at lists.iotivity.org> Subject: New branch and design policies Dear Pat, Could you clarify on the new branch policy? 1) As far as we know, new features should be committed and reviewed in new branches and only optimizations/bug fixes should go in master branch directly. We are not sure why the optimization/fixes( single thread /multi thread) are moved to new branch and IPV6 modifications such ashttps://gerrit.iotivity.org/gerrit/#/c/1081/ are merged to master mentioning it as high priority. Last 2 months, we resolved merge conflicts and expecting them to be merged . But now all of a sudden Erich is requesting for merge them in a new branch. 2) When we committed source , it?s been requested to make smaller commits but now it?s been asked to make larger commit which is impossible considering all platforms CA supports. And Design Policies: Some of the design decisions are taken internally without any discussion in developer list or mentioning it in dc tg conference calls which happens once in every two weeks. Here are the some comments in gerrit: Previous discussions on this are closed. Discussion was had, removed for good reason. https://gerrit.iotivity.org/gerrit/#/c/1081/42/resource/csdk/connectivity/api/cainterface.h Can we have the discussions open to theall developers instead of only to internal members? Regards, Ashok [cid:image001.gif at 01D0AE63.19275DB0] [http://ext.samsung.net/mailcheck/SeenTimeChecker?do=929864819710bd59be53e73032a63875c8e626278226ae74a1889aa8efbbee4788d6974bd2f79a3cb3b9c254041823979dd130b31b023ef15296970253332b3707805447a154a46fcf878f9a26ce15a0] -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.iotivity.org/pipermail/iotivity-dev/attachments/20150624/a86d8ee1/attachment.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: image001.gif Type: image/gif Size: 13168 bytes Desc: image001.gif URL: <http://lists.iotivity.org/pipermail/iotivity-dev/attachments/20150624/a86d8ee1/attachment.gif>
