public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH][pushed] fortran: remove 2 dead links [PR106636]
@ 2022-09-20 12:02 Martin Liška
  2022-09-20 12:17 ` Tobias Burnus
  0 siblings, 1 reply; 4+ messages in thread
From: Martin Liška @ 2022-09-20 12:02 UTC (permalink / raw)
  To: gcc-patches; +Cc: gfortran

	PR fortran/106636

gcc/fortran/ChangeLog:

	* gfortran.texi: Remove 2 dead links.
---
 gcc/fortran/gfortran.texi | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/gcc/fortran/gfortran.texi b/gcc/fortran/gfortran.texi
index 59d673bfc03..25410e6088d 100644
--- a/gcc/fortran/gfortran.texi
+++ b/gcc/fortran/gfortran.texi
@@ -455,9 +455,7 @@ version 2.6, @uref{https://www.openacc.org/}).  See
 The Fortran 95 standard specifies in Part 2 (ISO/IEC 1539-2:2000)
 varying length character strings.  While GNU Fortran currently does not
 support such strings directly, there exist two Fortran implementations
-for them, which work with GNU Fortran.  They can be found at
-@uref{https://www.fortran.com/@/iso_varying_string.f95} and at
-@uref{ftp://ftp.nag.co.uk/@/sc22wg5/@/ISO_VARYING_STRING/}.
+for them, which work with GNU Fortran.
 
 Deferred-length character strings of Fortran 2003 supports part of
 the features of @code{ISO_VARYING_STRING} and should be considered as
-- 
2.37.3


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH][pushed] fortran: remove 2 dead links [PR106636]
  2022-09-20 12:02 [PATCH][pushed] fortran: remove 2 dead links [PR106636] Martin Liška
@ 2022-09-20 12:17 ` Tobias Burnus
  2022-09-20 12:24   ` Martin Liška
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Burnus @ 2022-09-20 12:17 UTC (permalink / raw)
  To: Martin Liška, gcc-patches; +Cc: gfortran

Hi Martin,

On 20.09.22 14:02, Martin Liška wrote:
> diff --git a/gcc/fortran/gfortran.texi b/gcc/fortran/gfortran.texi
> @@ -455,9 +455,7 @@ version 2.6, @uref{https://www.openacc.org/}).  See
>   The Fortran 95 standard specifies in Part 2 (ISO/IEC 1539-2:2000)
>   varying length character strings.  While GNU Fortran currently does not
>   support such strings directly, there exist two Fortran implementations
> -for them, which work with GNU Fortran.  They can be found at
> -@uref{https://www.fortran.com/@/iso_varying_string.f95} and at
> -@uref{ftp://ftp.nag.co.uk/@/sc22wg5/@/ISO_VARYING_STRING/}.
> +for them, which work with GNU Fortran.

Instead of removing the links, can we rather replace it by an updated link?

Richard Townsend has implemented the fortran.com version; webarchive
show that was the 1.3-F version that is still available from his own
homepage at:

http://user.astro.wisc.edu/~townsend/static.php?ref=iso-varying-string
(BTW: I now also added Richard's website to web.archive.org.)

I don't think most users need this module, but still if it is mentioned,
I does not harm to have the link available.

Tobias

-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH][pushed] fortran: remove 2 dead links [PR106636]
  2022-09-20 12:17 ` Tobias Burnus
@ 2022-09-20 12:24   ` Martin Liška
  2022-09-20 13:38     ` Tobias Burnus
  0 siblings, 1 reply; 4+ messages in thread
From: Martin Liška @ 2022-09-20 12:24 UTC (permalink / raw)
  To: Tobias Burnus, gcc-patches; +Cc: gfortran

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

On 9/20/22 14:17, Tobias Burnus wrote:
> Hi Martin,
> 
> On 20.09.22 14:02, Martin Liška wrote:
>> diff --git a/gcc/fortran/gfortran.texi b/gcc/fortran/gfortran.texi
>> @@ -455,9 +455,7 @@ version 2.6, @uref{https://www.openacc.org/}).  See
>>   The Fortran 95 standard specifies in Part 2 (ISO/IEC 1539-2:2000)
>>   varying length character strings.  While GNU Fortran currently does not
>>   support such strings directly, there exist two Fortran implementations
>> -for them, which work with GNU Fortran.  They can be found at
>> -@uref{https://www.fortran.com/@/iso_varying_string.f95} and at
>> -@uref{ftp://ftp.nag.co.uk/@/sc22wg5/@/ISO_VARYING_STRING/}.
>> +for them, which work with GNU Fortran.
> 
> Instead of removing the links, can we rather replace it by an updated link?
> 
> Richard Townsend has implemented the fortran.com version; webarchive
> show that was the 1.3-F version that is still available from his own
> homepage at:
> 
> http://user.astro.wisc.edu/~townsend/static.php?ref=iso-varying-string
> (BTW: I now also added Richard's website to web.archive.org.)
> 
> I don't think most users need this module, but still if it is mentioned,
> I does not harm to have the link available.

Thanks for the archeological work you did.
Sure, what about the suggested patch?

Martin

> 
> Tobias
> 
> -----------------
> Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

[-- Attachment #2: 0001-fortran-add-link-to-ISO_VARYING_STRING-module-PR1066.patch --]
[-- Type: text/x-patch, Size: 1194 bytes --]

From 64376d0f450b03e2bdaad486aaa5df8141b2dde1 Mon Sep 17 00:00:00 2001
From: Martin Liska <mliska@suse.cz>
Date: Tue, 20 Sep 2022 14:23:29 +0200
Subject: [PATCH] fortran: add link to ISO_VARYING_STRING module [PR106636]

	PR fortran/106636

gcc/fortran/ChangeLog:

	* gfortran.texi: Add back link to ISO_VARYING_STRING.
---
 gcc/fortran/gfortran.texi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gcc/fortran/gfortran.texi b/gcc/fortran/gfortran.texi
index 25410e6088d..4ab67700362 100644
--- a/gcc/fortran/gfortran.texi
+++ b/gcc/fortran/gfortran.texi
@@ -455,7 +455,8 @@ version 2.6, @uref{https://www.openacc.org/}).  See
 The Fortran 95 standard specifies in Part 2 (ISO/IEC 1539-2:2000)
 varying length character strings.  While GNU Fortran currently does not
 support such strings directly, there exist two Fortran implementations
-for them, which work with GNU Fortran.
+for them, which work with GNU Fortran. One can be found at
+@uref{http://user.astro.wisc.edu/~townsend/static.php?ref=iso-varying-string}.
 
 Deferred-length character strings of Fortran 2003 supports part of
 the features of @code{ISO_VARYING_STRING} and should be considered as
-- 
2.37.3


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH][pushed] fortran: remove 2 dead links [PR106636]
  2022-09-20 12:24   ` Martin Liška
@ 2022-09-20 13:38     ` Tobias Burnus
  0 siblings, 0 replies; 4+ messages in thread
From: Tobias Burnus @ 2022-09-20 13:38 UTC (permalink / raw)
  To: Martin Liška, gcc-patches; +Cc: gfortran

Hi Martin,

On 20.09.22 14:24, Martin Liška wrote:
> On 9/20/22 14:17, Tobias Burnus wrote:
>> Instead of removing the links, can we rather replace it by an updated link?
>> [...]
> Thanks for the archeological work you did.
> Sure, what about the suggested patch?

LGTM. Thanks!

Tobias

-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-09-20 13:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-20 12:02 [PATCH][pushed] fortran: remove 2 dead links [PR106636] Martin Liška
2022-09-20 12:17 ` Tobias Burnus
2022-09-20 12:24   ` Martin Liška
2022-09-20 13:38     ` Tobias Burnus

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).