[wwwdocs] Add caveat for AVR port

Submitted by Senthil Kumar Selvaraj on Aug. 8, 2016, 8:24 a.m.

Details

Message ID 87r39zhdn8.fsf@atmel.com
State New
Headers show

Commit Message

Senthil Kumar Selvaraj Aug. 8, 2016, 8:24 a.m.
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

Comments

Gerald Pfeifer Aug. 8, 2016, 9:01 a.m.
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 <p>...</p> within <li>...</li>).

Gerald
Georg-Johann Lay Aug. 8, 2016, 10:25 a.m.
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 -0000	1.86
> +++ changes.html	8 Aug 2016 08:17:13 -0000
> @@ -40,6 +40,10 @@
>      </ul>
>      </li>
>
> +    <li><p>The AVR port requires binutils version 2.26.1 or later for the fix
> +    for <a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71151">PR71151</a>
> +    to work.</p></li>
> +
>    </ul>
>
>  <!-- .................................................................. -->


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
Senthil Kumar Selvaraj Aug. 8, 2016, 10:38 a.m.
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 -0000	1.86
>> +++ changes.html	8 Aug 2016 08:17:13 -0000
>> @@ -40,6 +40,10 @@
>>      </ul>
>>      </li>
>>
>> +    <li><p>The AVR port requires binutils version 2.26.1 or later for the fix
>> +    for <a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71151">PR71151</a>
>> +    to work.</p></li>
>> +
>>    </ul>
>>
>>  <!-- .................................................................. -->
>
>
> 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

Patch hide | download patch | download mbox

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 -0000	1.86
+++ changes.html	8 Aug 2016 08:17:13 -0000
@@ -40,6 +40,10 @@ 
     </ul>
     </li>
 
+    <li><p>The AVR port requires binutils version 2.26.1 or later for the fix
+    for <a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71151">PR71151</a>
+    to work.</p></li>
+
   </ul>
 
 <!-- .................................................................. -->