diff mbox series

rust: Add rust.install-dvi and rust.install-html rules

Message ID 20240404162719.3737066-1-christophe.lyon@linaro.org
State New
Headers show
Series rust: Add rust.install-dvi and rust.install-html rules | expand

Commit Message

Christophe Lyon April 4, 2024, 4:27 p.m. UTC
rust has the (empty) rust.dvi and rust.html rules, but lacks the
(empty) rust.install-dvi and rust.install-html ones.

2024-04-04  Christophe Lyon  <christophe.lyon@linaro.org>

	gcc/rust/
	* Make-lang.in (rust.install-dvi, rust.install-html): New rules.
---
 gcc/rust/Make-lang.in | 2 ++
 1 file changed, 2 insertions(+)

Comments

Thomas Schwinge April 8, 2024, 9:26 p.m. UTC | #1
Hi Christophe!

On 2024-04-04T16:27:19+0000, Christophe Lyon <christophe.lyon@linaro.org> wrote:
> rust has the (empty) rust.dvi and rust.html rules, but lacks the
> (empty) rust.install-dvi and rust.install-html ones.

Thanks, looks good to me.


Grüße
 Thomas


> 2024-04-04  Christophe Lyon  <christophe.lyon@linaro.org>
>
> 	gcc/rust/
> 	* Make-lang.in (rust.install-dvi, rust.install-html): New rules.
> ---
>  gcc/rust/Make-lang.in | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/gcc/rust/Make-lang.in b/gcc/rust/Make-lang.in
> index 4d646018792..4d73412739d 100644
> --- a/gcc/rust/Make-lang.in
> +++ b/gcc/rust/Make-lang.in
> @@ -342,6 +342,8 @@ selftest-rust-valgrind: $(RUST_SELFTEST_DEPS)
>  # should have dependencies on info files that should be installed.
>  rust.install-info:
>  
> +rust.install-dvi:
> +rust.install-html:
>  rust.install-pdf:
>  
>  # Install man pages for the front end. This target should ignore errors.
Arthur Cohen April 29, 2024, 1:24 p.m. UTC | #2
Thanks Christophe!

I've added your patch as part of a documentation pull-request I'm adding 
to our dev repo: https://github.com/Rust-GCC/gccrs/pull/2966

It'll be upstreamed with the next batch of commits we send, as soon as 
trunk reopens fully for 15.1.

Best,

Arthur

On 4/4/24 18:27, Christophe Lyon wrote:
> rust has the (empty) rust.dvi and rust.html rules, but lacks the
> (empty) rust.install-dvi and rust.install-html ones.
> 
> 2024-04-04  Christophe Lyon  <christophe.lyon@linaro.org>
> 
> 	gcc/rust/
> 	* Make-lang.in (rust.install-dvi, rust.install-html): New rules.
> ---
>   gcc/rust/Make-lang.in | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/gcc/rust/Make-lang.in b/gcc/rust/Make-lang.in
> index 4d646018792..4d73412739d 100644
> --- a/gcc/rust/Make-lang.in
> +++ b/gcc/rust/Make-lang.in
> @@ -342,6 +342,8 @@ selftest-rust-valgrind: $(RUST_SELFTEST_DEPS)
>   # should have dependencies on info files that should be installed.
>   rust.install-info:
>   
> +rust.install-dvi:
> +rust.install-html:
>   rust.install-pdf:
>   
>   # Install man pages for the front end. This target should ignore errors.
Christophe Lyon April 29, 2024, 1:30 p.m. UTC | #3
On Mon, 29 Apr 2024 at 15:24, Arthur Cohen <arthur.cohen@embecosm.com> wrote:
>
> Thanks Christophe!
>
> I've added your patch as part of a documentation pull-request I'm adding
> to our dev repo: https://github.com/Rust-GCC/gccrs/pull/2966
>
> It'll be upstreamed with the next batch of commits we send, as soon as
> trunk reopens fully for 15.1.
>

Thanks!

Note that I already pushed the patch as r14-9865-g73fb0a6153f478
so you may have conflicts (easy to fix ;-) )

Christophe

> Best,
>
> Arthur
>
> On 4/4/24 18:27, Christophe Lyon wrote:
> > rust has the (empty) rust.dvi and rust.html rules, but lacks the
> > (empty) rust.install-dvi and rust.install-html ones.
> >
> > 2024-04-04  Christophe Lyon  <christophe.lyon@linaro.org>
> >
> >       gcc/rust/
> >       * Make-lang.in (rust.install-dvi, rust.install-html): New rules.
> > ---
> >   gcc/rust/Make-lang.in | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/gcc/rust/Make-lang.in b/gcc/rust/Make-lang.in
> > index 4d646018792..4d73412739d 100644
> > --- a/gcc/rust/Make-lang.in
> > +++ b/gcc/rust/Make-lang.in
> > @@ -342,6 +342,8 @@ selftest-rust-valgrind: $(RUST_SELFTEST_DEPS)
> >   # should have dependencies on info files that should be installed.
> >   rust.install-info:
> >
> > +rust.install-dvi:
> > +rust.install-html:
> >   rust.install-pdf:
> >
> >   # Install man pages for the front end. This target should ignore errors.
diff mbox series

Patch

diff --git a/gcc/rust/Make-lang.in b/gcc/rust/Make-lang.in
index 4d646018792..4d73412739d 100644
--- a/gcc/rust/Make-lang.in
+++ b/gcc/rust/Make-lang.in
@@ -342,6 +342,8 @@  selftest-rust-valgrind: $(RUST_SELFTEST_DEPS)
 # should have dependencies on info files that should be installed.
 rust.install-info:
 
+rust.install-dvi:
+rust.install-html:
 rust.install-pdf:
 
 # Install man pages for the front end. This target should ignore errors.