Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-03 Thread Florian Weimer
* Sebastian Huber:

> On 03.05.24 17:06, Jonathan Wakely wrote:
>>> I think it would be helpful to reference this change in the C
>>> section. This warning to error change causes some issues with
>>> legacy software.
>> I agree it should be mentioned, but I would put it in the caveats
>> section at the top, not as the last item of the C section.
>> How about this? OK for wwwdocs?
>
> This is fine for me as well, thanks.

For me as well.

Thanks,
Florian



Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-03 Thread Sebastian Huber

On 03.05.24 17:06, Jonathan Wakely wrote:
I think it would be helpful to reference this change in the C section. 
This warning to error change causes some issues with legacy software.


I agree it should be mentioned, but I would put it in the caveats
section at the top, not as the last item of the C section.

How about this? OK for wwwdocs?


This is fine for me as well, thanks.

--
embedded brains GmbH & Co. KG
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.hu...@embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/


Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-03 Thread Jakub Jelinek
On Fri, May 03, 2024 at 04:06:28PM +0100, Jonathan Wakely wrote:
> I agree it should be mentioned, but I would put it in the caveats
> section at the top, not as the last item of the C section.
> 
> How about this? OK for wwwdocs?

LGTM.

> commit fe5fd75ea5a7a08eee0831cadbdd05689e9408db
> Author: Jonathan Wakely 
> Date:   Fri May 3 16:04:49 2024 +0100
> 
> Add caveat to GCC 14 release notes about C warnings-as-errors change
> 
> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> index 46a0266d..82906de1 100644
> --- a/htdocs/gcc-14/changes.html
> +++ b/htdocs/gcc-14/changes.html
> @@ -40,6 +40,11 @@ a work-in-progress.
> href="https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wflex-array-member-not-at-end;>-Wflex-array-member-not-at-end
>  to
>identify all such cases in the source code and modify them.
>
> +  C:
> +  Certain warnings about are now errors, see
> +  Porting to GCC 14
> +  for details.
> +  
> href="https://gcc.gnu.org/onlinedocs/gcc/Instrumentation-Options.html;>-fcf-protection=[full|branch|return|none|check]
>is refactored, to override -fcf-protection,
>-fcf-protection=none needs to be added and then

Jakub



Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-03 Thread Jonathan Wakely

On 02/05/24 16:29 +0200, Sebastian Huber wrote:

On 24.04.24 14:28, Sebastian Huber wrote:

- Am 15. Apr 2024 um 8:05 schrieb Sebastian Huber 
sebastian.hu...@embedded-brains.de:


---
v2: Remove listing of options.

htdocs/gcc-14/changes.html | 2 ++
1 file changed, 2 insertions(+)

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 8ac08e9a..665d050a 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -231,6 +231,8 @@ a work-in-progress.
  previous options -std=c2x, -std=gnu2x
  and -Wc11-c2x-compat, which are deprecated but remain
  supported.
+  Some warnings are now errors by default (see also
+  Porting to GCC 14).


C++
--
2.35.3


Ping.


I think it would be helpful to reference this change in the C section. 
This warning to error change causes some issues with legacy software.


I agree it should be mentioned, but I would put it in the caveats
section at the top, not as the last item of the C section.

How about this? OK for wwwdocs?

-- >8 --

commit fe5fd75ea5a7a08eee0831cadbdd05689e9408db
Author: Jonathan Wakely 
Date:   Fri May 3 16:04:49 2024 +0100

Add caveat to GCC 14 release notes about C warnings-as-errors change

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 46a0266d..82906de1 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -40,6 +40,11 @@ a work-in-progress.
   https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wflex-array-member-not-at-end;>-Wflex-array-member-not-at-end
 to
   identify all such cases in the source code and modify them.
   
+  C:
+  Certain warnings about are now errors, see
+  Porting to GCC 14
+  for details.
+  
   https://gcc.gnu.org/onlinedocs/gcc/Instrumentation-Options.html;>-fcf-protection=[full|branch|return|none|check]
   is refactored, to override -fcf-protection,
   -fcf-protection=none needs to be added and then



Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-02 Thread Sebastian Huber

On 24.04.24 14:28, Sebastian Huber wrote:

- Am 15. Apr 2024 um 8:05 schrieb Sebastian Huber 
sebastian.hu...@embedded-brains.de:


---
v2: Remove listing of options.

htdocs/gcc-14/changes.html | 2 ++
1 file changed, 2 insertions(+)

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 8ac08e9a..665d050a 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -231,6 +231,8 @@ a work-in-progress.
   previous options -std=c2x, -std=gnu2x
   and -Wc11-c2x-compat, which are deprecated but remain
   supported.
+  Some warnings are now errors by default (see also
+  Porting to GCC 14).


C++
--
2.35.3


Ping.


I think it would be helpful to reference this change in the C section. 
This warning to error change causes some issues with legacy software.


--
embedded brains GmbH & Co. KG
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.hu...@embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/


Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-04-24 Thread Sebastian Huber
- Am 15. Apr 2024 um 8:05 schrieb Sebastian Huber 
sebastian.hu...@embedded-brains.de:

> ---
> v2: Remove listing of options.
> 
> htdocs/gcc-14/changes.html | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> index 8ac08e9a..665d050a 100644
> --- a/htdocs/gcc-14/changes.html
> +++ b/htdocs/gcc-14/changes.html
> @@ -231,6 +231,8 @@ a work-in-progress.
>   previous options -std=c2x, -std=gnu2x
>   and -Wc11-c2x-compat, which are deprecated but remain
>   supported.
> +  Some warnings are now errors by default (see also
> +  Porting to GCC 14).
> 
> 
> C++
> --
> 2.35.3

Ping.

-- 
embedded brains GmbH & Co. KG
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.hu...@embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/