[RFA/win32] move win32_xfer_shared_library to (new) win32-tdep...
Christopher Faylor
cgf-use-the-mailinglist-please@sourceware.org
Sun Jan 11 04:24:00 GMT 2009
On Sun, Jan 11, 2009 at 08:18:48AM +0400, Joel Brobecker wrote:
>> >This patch moves the win32_xfer_shared_library function to a file
>> >that is not specific to the target CPU. I introduced win32-tdep,
>> >to follow the naming scheme used by win32-nat, even though these
>> >file can also work on 64bit windows.
>>
>> If this file can work on 64 bit windows then it seems like it should be
>> renamed to something more generic rather than potentially duplicating
>> the same code in two places.
>>
>> Should win32-nat.c just be renamed to windows-nat.c?
>
>I think this would be a good idea. Same for win32-tdep.
>
>I suggest I commit the two remaining patches as is, and then do
>the move as a separate step. That way, I don't have to rework
>the patches after the move is done.
That's fine.
>In terms of the move itself, how do we want it to be done.
>CVS doesn't support moves per se. Usually, in my own little CVS
>repos where I don't use branches, I do a "mv" inside the CVS
>repository. But I know that this brings its own problems.
>Should we use "cvs remove + cvs add"?
I don't know of any way to do a "mv" in CVS. If no one objects, I'll do
some surgery on the CVS repository to make a copy of windows-nat.c
and windows-tdep.c to make it look like they always existed and then
I'll cvs delete the old files and change Makefile.in.
cgf
More information about the Gdb-patches
mailing list