[PATCH 1/1] gcc-14: document P1689R5 scanning output support

[PATCH 1/1] gcc-14: document P1689R5 scanning output support

[Ben Boeckel]

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

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 7278f753..b506eeb1 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -112,6 +112,17 @@ a work-in-progress.</p>
       </li>
     </ul>
   </li>
+  <li>C++ module scanning for named modules is now available:
+    <ul>
+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
+      describing dependencies of source files.
+      </li>
+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
+      <code>-fdeps=target=</code> flags may be used to generate P1689 output
+      (the <code>p1689r5</code> format is the only available format today).
+      </li>
+    </ul>
+  </li>
 </ul>
 
 <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
-- 
2.42.0

Re: [PATCH 1/1] gcc-14: document P1689R5 scanning output support

[Ben Boeckel]

On Mon, Nov 20, 2023 at 11:22:56 -0500, Ben Boeckel wrote:
> ---
>  htdocs/gcc-14/changes.html | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> index 7278f753..b506eeb1 100644
> --- a/htdocs/gcc-14/changes.html
> +++ b/htdocs/gcc-14/changes.html
> @@ -112,6 +112,17 @@ a work-in-progress.</p>
>        </li>
>      </ul>
>    </li>
> +  <li>C++ module scanning for named modules is now available:
> +    <ul>
> +      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
> +      describing dependencies of source files.
> +      </li>
> +      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> +      <code>-fdeps=target=</code> flags may be used to generate P1689 output
> +      (the <code>p1689r5</code> format is the only available format today).
> +      </li>
> +    </ul>
> +  </li>
>  </ul>
>  
>  <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
> -- 
> 2.42.0

Ping? Is this the right place to submit this patch?

Thanks,

--Ben

Re: [PATCH wwwdocs 1/1] gcc-14: document P1689R5 scanning output support

[Arsen Arsenović]

[-- Attachment #1: Type: text/plain, Size: 409 bytes --]

Hi Ben,

Ben Boeckel <ben.boeckel@kitware.com> writes:

> Ping? Is this the right place to submit this patch?

Yes, this is the correct list, though it is usually recommended to use
--subject-prefix='PATCH wwwdocs' or such, to catch the right eyes.  See:
https://gcc.gnu.org/contribute.html#webchanges

I've added it to my subject, hopefully that works.

Have a lovely day!
--
Arsen Arsenović

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 251 bytes --]

Re: [PATCH wwwdocs 1/1] gcc-14: document P1689R5 scanning output support

[Ben Boeckel]

On Sat, Jan 06, 2024 at 14:17:14 +0100, Arsen Arsenović wrote:
> Hi Ben,
> 
> Ben Boeckel <ben.boeckel@kitware.com> writes:
> 
> > Ping? Is this the right place to submit this patch?
> 
> Yes, this is the correct list, though it is usually recommended to use
> --subject-prefix='PATCH wwwdocs' or such, to catch the right eyes.  See:
> https://gcc.gnu.org/contribute.html#webchanges
> 
> I've added it to my subject, hopefully that works.

No bites yet… Anyone willing to review this patch so that it gets
mentioned on the website?

Thanks,

--Ben

Re: [PATCH 1/1] gcc-14: document P1689R5 scanning output support

[Jonathan Wakely]

On 20/11/23 11:22 -0500, Ben Boeckel wrote:
>---
> htdocs/gcc-14/changes.html | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
>diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
>index 7278f753..b506eeb1 100644
>--- a/htdocs/gcc-14/changes.html
>+++ b/htdocs/gcc-14/changes.html
>@@ -112,6 +112,17 @@ a work-in-progress.</p>
>       </li>
>     </ul>
>   </li>
>+  <li>C++ module scanning for named modules is now available:
>+    <ul>
>+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
>+      describing dependencies of source files.
>+      </li>
>+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
>+      <code>-fdeps=target=</code> flags may be used to generate P1689 output

This should be -fdeps-target= not -fdeps=target=.


>+      (the <code>p1689r5</code> format is the only available format today).

I wish the option was more descriptive than "p1689r5", which nobody is
going to remember (but I assume we don't actually need to specify it
explicitly since it's the only supported format).

>+      </li>
>+    </ul>

Do we need a list for this item? It seems a bit weird that the first
list item is just the paper  How about just a single paragraph?

C++ module scanning for named modules is now available, based on the
format described in <a href="https://wg21.link/P1689R5">P1689R5</a>,
Format for describing dependencies of source files. The
<code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
<code>-fdeps-target=</code> flags may be used to generate dependency
information. In GCC 14 <code>p1689r5</code> is the only valid argument
for <code>-fdeps-format=</code>.


>+  </li>
> </ul>
>
> <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
>-- 
>2.42.0
>

Re: [PATCHv2 wwwdocs 1/1] gcc-14: document P1689R5 scanning output support

[Ben Boeckel]

[-- Attachment #1: Type: text/plain, Size: 2087 bytes --]

On Tue, Apr 30, 2024 at 10:24:44 +0100, Jonathan Wakely wrote:
> On 20/11/23 11:22 -0500, Ben Boeckel wrote:
> >---
> > htdocs/gcc-14/changes.html | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> >diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> >index 7278f753..b506eeb1 100644
> >--- a/htdocs/gcc-14/changes.html
> >+++ b/htdocs/gcc-14/changes.html
> >@@ -112,6 +112,17 @@ a work-in-progress.</p>
> >       </li>
> >     </ul>
> >   </li>
> >+  <li>C++ module scanning for named modules is now available:
> >+    <ul>
> >+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
> >+      describing dependencies of source files.
> >+      </li>
> >+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> >+      <code>-fdeps=target=</code> flags may be used to generate P1689 output
> 
> This should be -fdeps-target= not -fdeps=target=.

Whoops, yep.

> >+      (the <code>p1689r5</code> format is the only available format today).
> 
> I wish the option was more descriptive than "p1689r5", which nobody is
> going to remember (but I assume we don't actually need to specify it
> explicitly since it's the only supported format).

All users of the flag should be having it in the build system itself;
hand-coded makefiles can use it, but will need considerable `jq`
gymnastics to turn the output into properly understood make syntax
snippets for their build.

> >+      </li>
> >+    </ul>
> 
> Do we need a list for this item? It seems a bit weird that the first
> list item is just the paper  How about just a single paragraph?
> 
> C++ module scanning for named modules is now available, based on the
> format described in <a href="https://wg21.link/P1689R5">P1689R5</a>,
> Format for describing dependencies of source files. The
> <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> <code>-fdeps-target=</code> flags may be used to generate dependency
> information. In GCC 14 <code>p1689r5</code> is the only valid argument
> for <code>-fdeps-format=</code>.

Sounds good. New patch attached.

--Ben

[-- Attachment #2: 0001-gcc-14-document-P1689R5-scanning-output-support.patch --]
[-- Type: text/plain, Size: 1223 bytes --]

From d973efa9689db7d46211721e7c00feea7e6445a6 Mon Sep 17 00:00:00 2001
From: Ben Boeckel <ben.boeckel@kitware.com>
Date: Thu, 2 May 2024 14:00:01 -0400
Subject: [PATCH 1/1] gcc-14: document P1689R5 scanning output support

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

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 8dfbf7dc..8998e6c0 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -513,6 +513,12 @@ a work-in-progress.</p>
   <li>GCC supports a new pragma <code>&#x23;pragma GCC novector</code> to
       indicate to the vectorizer not to vectorize the loop annotated with the
       pragma.</li>
+  <li>C++ module scanning for named modules is now available, based on the
+    format described in<a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
+    describing dependencies of source files. The <code>-fdeps-format=</code>,
+    <code>-fdeps-file=</code>, and <code>-fdeps-target=</code> flags may be
+    used to generate dependency information. In GCC 14 <code>p1689r5</code> is
+    the only valid argument for <code>-fdeps-format=</code>.</li>
 </ul>
 
 <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
-- 
2.44.0

Re: [PATCHv2 wwwdocs 1/1] gcc-14: document P1689R5 scanning output support

[Jonathan Wakely]

On Thu, 2 May 2024 at 19:03, Ben Boeckel <ben.boeckel@kitware.com> wrote:
>
> On Tue, Apr 30, 2024 at 10:24:44 +0100, Jonathan Wakely wrote:
> > On 20/11/23 11:22 -0500, Ben Boeckel wrote:
> > >---
> > > htdocs/gcc-14/changes.html | 11 +++++++++++
> > > 1 file changed, 11 insertions(+)
> > >
> > >diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> > >index 7278f753..b506eeb1 100644
> > >--- a/htdocs/gcc-14/changes.html
> > >+++ b/htdocs/gcc-14/changes.html
> > >@@ -112,6 +112,17 @@ a work-in-progress.</p>
> > >       </li>
> > >     </ul>
> > >   </li>
> > >+  <li>C++ module scanning for named modules is now available:
> > >+    <ul>
> > >+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
> > >+      describing dependencies of source files.
> > >+      </li>
> > >+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> > >+      <code>-fdeps=target=</code> flags may be used to generate P1689 output
> >
> > This should be -fdeps-target= not -fdeps=target=.
>
> Whoops, yep.
>
> > >+      (the <code>p1689r5</code> format is the only available format today).
> >
> > I wish the option was more descriptive than "p1689r5", which nobody is
> > going to remember (but I assume we don't actually need to specify it
> > explicitly since it's the only supported format).
>
> All users of the flag should be having it in the build system itself;
> hand-coded makefiles can use it, but will need considerable `jq`
> gymnastics to turn the output into properly understood make syntax
> snippets for their build.
>
> > >+      </li>
> > >+    </ul>
> >
> > Do we need a list for this item? It seems a bit weird that the first
> > list item is just the paper  How about just a single paragraph?
> >
> > C++ module scanning for named modules is now available, based on the
> > format described in <a href="https://wg21.link/P1689R5">P1689R5</a>,
> > Format for describing dependencies of source files. The
> > <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> > <code>-fdeps-target=</code> flags may be used to generate dependency
> > information. In GCC 14 <code>p1689r5</code> is the only valid argument
> > for <code>-fdeps-format=</code>.
>
> Sounds good. New patch attached.

Thanks, Ben. I reformatted the whitespace so all the lines were under
80 columns and pushed it to wwwdocs.