This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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/gfortran] PR18283


Paul Thomas wrote:
> Tobias Schlu"ter wrote:
> 
> 
>>I don't think this patch was approved in the first place. I remember
>>complaining about a few minor details, but I didn't see a fixed 
>>version posted
>>to the list nor anybody else approved it as is. Please have a look at 
>>Kazu's
>>cleanup patch <http://gcc.gnu.org/ml/gcc-patches/2005-05/msg02783.html> --
>>there was definitely something wrong with that patch, and I'm guessing 
>>that
>>some of that wrongness were the spurious whitespace differences I 
>>complained
>>about in the patch you sent.
> 
> 
> 
> Oh nuts! I took your,
> 
> "I think this is ok, but there are a few cases where whitespace-only changes
> have slipped in, and the patch doesn't apply with the weird formatting your
> mailer gives it, so I'm not sure I'm not missing something. Can you somehow
> get a cleaner patch out of your setup?"
> 
> I took your, to be an approval, subject to the last. The fact that I 
> still screwed up on it is really annoying.

You also ignored a number of changes I requested in that mail.

> This time I am doing the preparation of the whole lot under RH9. The 
> mixture of Windows and Cygwin is too much of a hassle to get right. 
> Unfortunately, all my .ssh stuff is on the Cygwin side and I just have 
> not succeed in tranferring it to RH9.

Have you set cygwin to not default to Unix-style line-endings?  Or are using a
Windows editor which gets it wrong?

Cheers,
- Tobi


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