Re: [Patch, wwwdocs] Add caveat for AVR port

2016-08-08 Thread Senthil Kumar Selvaraj

Georg-Johann Lay writes:

> On 08.08.2016 10:24, Senthil Kumar Selvaraj wrote:
>> Hi,
>>
>>   This doc patch informs the user that a specific (or higher) version of
>>   binutils is a prerequisite for the fix for a rather vexing bug (PR
>>   71151) that was fixed for 6.2.
>>
>>   I've added it to the Caveats section; is there a better place? If not, ok 
>> to
>>   commit?
>>
>> Regards
>> Senthil
>>
>> Index: changes.html
>> ===
>> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
>> retrieving revision 1.86
>> diff -u -r1.86 changes.html
>> --- changes.html 20 Jun 2016 10:34:00 -  1.86
>> +++ changes.html 8 Aug 2016 08:17:13 -
>> @@ -40,6 +40,10 @@
>>  
>>  
>>
>> +The AVR port requires binutils version 2.26.1 or later for the 
>> fix
>> +for > href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71151;>PR71151
>> +to work.
>> +
>>
>>
>>  
>
>
> Maybe mention that also GCC 6.2 or higher is needed?  After all, it's a bug 
> in 
> the compiler and using respective Binutils without a fixed compiler is 
> pointless...

Ah, I thought the existing page would stay unmodified and my changes would
show up only for 6.2+ :(. I'll say

"With GCC 6.2 and later, the AVR port requires...".

Sounds good to you?

Regards
Senthil


Re: [Patch, wwwdocs] Add caveat for AVR port

2016-08-08 Thread Georg-Johann Lay

On 08.08.2016 10:24, Senthil Kumar Selvaraj wrote:

Hi,

  This doc patch informs the user that a specific (or higher) version of
  binutils is a prerequisite for the fix for a rather vexing bug (PR
  71151) that was fixed for 6.2.

  I've added it to the Caveats section; is there a better place? If not, ok to
  commit?

Regards
Senthil

Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
retrieving revision 1.86
diff -u -r1.86 changes.html
--- changes.html20 Jun 2016 10:34:00 -  1.86
+++ changes.html8 Aug 2016 08:17:13 -
@@ -40,6 +40,10 @@
 
 

+The AVR port requires binutils version 2.26.1 or later for the fix
+for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71151;>PR71151
+to work.
+
   

 



Maybe mention that also GCC 6.2 or higher is needed?  After all, it's a bug in 
the compiler and using respective Binutils without a fixed compiler is pointless...


Johann




Re: [Patch, wwwdocs] Add caveat for AVR port

2016-08-08 Thread Gerald Pfeifer
On Mon, 8 Aug 2016, Senthil Kumar Selvaraj wrote:
>   This doc patch informs the user that a specific (or higher) version of
>   binutils is a prerequisite for the fix for a rather vexing bug (PR
>   71151) that was fixed for 6.2.
> 
>   I've added it to the Caveats section; is there a better place? If not, ok to
>   commit?

Makes sense and looks good (just omit the ... within ...).

Gerald


[Patch, wwwdocs] Add caveat for AVR port

2016-08-08 Thread Senthil Kumar Selvaraj
Hi,

  This doc patch informs the user that a specific (or higher) version of
  binutils is a prerequisite for the fix for a rather vexing bug (PR
  71151) that was fixed for 6.2.

  I've added it to the Caveats section; is there a better place? If not, ok to
  commit?

Regards
Senthil

Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
retrieving revision 1.86
diff -u -r1.86 changes.html
--- changes.html20 Jun 2016 10:34:00 -  1.86
+++ changes.html8 Aug 2016 08:17:13 -
@@ -40,6 +40,10 @@
 
 
 
+The AVR port requires binutils version 2.26.1 or later for the fix
+for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71151;>PR71151
+to work.
+