This is the mail archive of the cygwin-apps mailing list for the Cygwin 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: [patch/rebase] rebase -i and more


On Jun 21 10:34, Charles Wilson wrote:
> On 6/21/2011 10:06 AM, Corinna Vinschen wrote:
> > On Jun 21 09:48, Charles Wilson wrote:
> >> On 6/21/2011 9:27 AM, Corinna Vinschen wrote:
> >>> Boy, what a mess.  In that case, have a look into my patch to
> >>> imagehelper/objectfile.cc, function Win32Path.  The #ifdef stuff
> >>> won't work for MSYS.  Not that I care...
> >>
> >> Of course. :-)
> > 
> > Wouldn't this one cover all cases?
> 
> Yes, it appears so.  Your proposal assumes that !CYGWIN == native
> windows, but -- that's already true throughout.

Would you mid to apply it then?   I can't right now since I'm just
fixing all '-Wall -Werror' compiler warnings in imagehelper.  Boy, don't
you just have to *love* the

  dereferencing type-punned pointer will break strict-aliasing rules

warning?

> You're just not going to get a linux 'rebase' tool to operate on (e.g.)
> your cygwin cross-compiler sysroot, at least not without rewriting a LOT
> of the code. Which ain't happening.

Indeed.  What I think would be more useful is to convert ReBaseImage
to ReBaseImage64.  Or at least to recognize 64 bit images and call
Windows' ReBaseImage64 in that case.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat


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