Re: [commit] Last major piece of fork tracing - vfork/exec

Andrew Cagney
2003-08-22T15:32:32Z

>> This patch is the last major piece of fork tracing: support for vfork and
>> exec. I have not enabled the code in infrun which attempts to automatically
>> load the new executable, because I'm not happy with how it works, and
>> because it isn't necessary for this to be useful.
>>
>> I'll move this to the branch in a day or two if there are no problems.
>> I suppose fixing up the expected output in the testsuite is next...
>>

> Please don't. The only things I know of still going down for 6.0 are some testsuite mangling and a re-fix for the red zone.
>
> Anything else can go in 6.1.
>
> My core file stuff, for instance, isn't in 6.0.

Daniel,

I'm really disappointed by the situtation you've created here. You, by
at the last moment trying to add this new functionality to 6.0 branch,
have effectively pushed me into a corner. I've got the following choices:

- yank the existing code
invalidates relevant tests
as the release manager, I get the blame for delaying both the new
feature and the release

- leave the code as is
Again, I get the blame for the feature not being present (after all it
is just wafer thin and won't have any affect on all the test results
that people have been doing up-until now, right?)

- let the change in
invalidating test results
again delay the release ...

Against my better judgment, I've decided to go with option 3. You can
commit this change but the release is delayed an extra week.

I'm now also forced to consider a move away from self regulation and
instead require release manager approval for all branch changes. A
change that would benefit no one :-(

Andrew