This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


 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



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]