[Dev] code dumps without proper review
mikko.ylinen at intel.com
Tue Oct 8 06:18:22 GMT 2013
On Mon, Oct 7, 2013 at 2:53 PM, 조영익 <youngik.cho at samsung.com> wrote:
> As I commented on the gerrit request,
> https://review.tizen.org/gerrit/#/c/10439/1, I am the reviewer for the
> code merge. As you already mentioned valuable comments, there is some issue
> with current source code.
> In my opinion, some changes like the current review should be applied on
> the code but it should be separate review request, not the merge commit.
> The merge commit itself has its own meaning of merging some patch sets and
> it should be distinguinished from the changes itself. (If the merge is not
> proper, then each changes should be cherry-picked for the small changes.)
> If you agree with me, then I will ask the developer to submit another
> request for aul git.
I'm not sure I follow the proposal here. Overall, I think the most recent
tizen_2.2 merge is bad
because it deletes files that have been in 'tizen' for a long time.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Dev