← Back to branch summary

~ubuntu-branches/ubuntu/precise/linux-ti-omap4/precise

« back to all changes in this revision

Viewing changes to Documentation/development-process/6.Followthrough

  • Committer: Bazaar Package Importer
  • Author(s): Paolo Pisati
  • Date: 2011-06-29 15:23:51 UTC
  • mfrom: (26.1.1 natty-proposed)
  • Revision ID: james.westby@ubuntu.com-20110629152351-xs96tm303d95rpbk
Tags: 3.0.0-1200.2
* Rebased against 3.0.0-6.7
* BSP from TI based on 3.0.0

Show diffs side-by-side

added added

removed removed

Lines of Context:
Documentation/development-process/6.Followthrough
66
66
that you don't realize that something is fundamentally wrong or, perhaps,
67
67
you're not even solving the right problem.
68
68
 
 
69
Andrew Morton has suggested that every review comment which does not result
 
70
in a code change should result in an additional code comment instead; that
 
71
can help future reviewers avoid the questions which came up the first time
 
72
around.
 
73
 
69
74
One fatal mistake is to ignore review comments in the hope that they will
70
75
go away.  They will not go away.  If you repost code without having
71
76
responded to the comments you got the time before, you're likely to find
100
105
subsystem to the next; each maintainer has his or her own way of doing
101
106
things.  In particular, there may be more than one tree - one, perhaps,
102
107
dedicated to patches planned for the next merge window, and another for
103
 
longer-term work.  
 
108
longer-term work.
104
109
 
105
110
For patches applying to areas for which there is no obvious subsystem tree
106
111
(memory management patches, for example), the default tree often ends up
109
114
 
110
115
Inclusion into a subsystem tree can bring a higher level of visibility to a
111
116
patch.  Now other developers working with that tree will get the patch by
112
 
default.  Subsystem trees typically feed into -mm and linux-next as well,
113
 
making their contents visible to the development community as a whole.  At
114
 
this point, there's a good chance that you will get more comments from a
115
 
new set of reviewers; these comments need to be answered as in the previous
116
 
round.
 
117
default.  Subsystem trees typically feed linux-next as well, making their
 
118
contents visible to the development community as a whole.  At this point,
 
119
there's a good chance that you will get more comments from a new set of
 
120
reviewers; these comments need to be answered as in the previous round.
117
121
 
118
122
What may also happen at this point, depending on the nature of your patch,
119
123
is that conflicts with work being done by others turn up.  In the worst