Kamesh Jayachandran wrote:
>
>
> Hyrum K. Wright wrote:
>> arfrever_at_tigris.org wrote:
>> > * Apply patch at,
>>> http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=137828
>>> Fixes compiler warning, which has been introduced while resolving the
>>> conflict from merge of r30633 from trunk.
>>> Votes: - +1 kameshj
>>> + +1 kameshj, arfrever
>> Hmmm. Instead of voting to apply the patch at that address, perhaps we
>> should create a branch, apply the patch to the branch, and then vote on
>> the branch? It seems a bit easier for people to review, and a bit more
>> inline with our process. Of course, if we want to change the process,
>> that's fine, too. :)
>
> Why should we have short-lived-branch to track one trivial-item specific
> for 1.5.x alone and merge later, why not rather directly commit it after
> approval.
You're right, it's trivial and probably doesn't matter a whole lot.
But...something just doesn't seem right about committing directly to a
stabilization branch, particularly during the RC soak period. I agree
that the problem should be fixed, and that this patch looks like the
right way to do it, I'm just questioning whether or not we should bend
(or modify) our typical, albeit nebulous, process in this instance.
Branches are cheap, merging is easy, so why *not* create a short-lived
branch to track this patch, just like we do with conflicting merges from
trunk?
(That being said, do what you will. I'm not going to veto this
particular change, but I did want to express my concerns.)
-Hyrum
Received on 2008-04-28 17:23:58 CEST