public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] download_prerequisites: add --only-gettext
@ 2023-12-02 15:47 Arsen Arsenović
  2023-12-04 13:12 ` Richard Biener
  0 siblings, 1 reply; 2+ messages in thread
From: Arsen Arsenović @ 2023-12-02 15:47 UTC (permalink / raw)
  To: gcc-patches; +Cc: Arsen Arsenović

contrib/ChangeLog:

	* download_prerequisites
	<arg parse>: Parse --only-gettext.
	(echo_archives): Check only_gettext and stop early if true.
	(helptext): Document --only-gettext.
---
Afternoon,

This patch adds a --only-gettext option to download_prerequisites for
when the only useful dependency to download is gettext (which will
restore a gcc source tree to a similar 'intlness' as before the
externalization of gettext-runtime).

For context, see
https://inbox.sourceware.org/CAFiYyc2-JxH358GUcZfR4iBMq5qj6Nf4W=7LYoQyw6b-u8Dfqw@mail.gmail.com/

OK for trunk?

TIA, have a lovely day!

 contrib/download_prerequisites | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/contrib/download_prerequisites b/contrib/download_prerequisites
index 9568091c0dba..30ff0cc9491a 100755
--- a/contrib/download_prerequisites
+++ b/contrib/download_prerequisites
@@ -36,16 +36,18 @@ gettext='gettext-0.22.tar.gz'
 base_url='http://gcc.gnu.org/pub/gcc/infrastructure/'
 
 echo_archives() {
+    echo "${gettext}"
+    if "${only_gettext}"; then return; fi
     echo "${gmp}"
     echo "${mpfr}"
     echo "${mpc}"
-    echo "${gettext}"
     if [ ${graphite} -gt 0 ]; then echo "${isl}"; fi
 }
 
 graphite=1
 verify=1
 force=0
+only_gettext=false
 OS=$(uname)
 
 if type wget > /dev/null ; then
@@ -74,6 +76,7 @@ The following options are available:
  --no-verify      don't verify package integrity
  --sha512         use SHA512 checksum to verify package integrity (default)
  --md5            use MD5 checksum to verify package integrity
+ --only-gettext   inhibit downloading any package but gettext
  --help           show this text and exit
  --version        show version information and exit
 "
@@ -159,6 +162,9 @@ do
                 chksum_extension='md5'
                 verify=1
                 ;;
+            --only-gettext)
+                only_gettext=true
+                ;;
             -*)
                 die "unknown option: ${arg}"
                 ;;
-- 
2.43.0


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

* Re: [PATCH] download_prerequisites: add --only-gettext
  2023-12-02 15:47 [PATCH] download_prerequisites: add --only-gettext Arsen Arsenović
@ 2023-12-04 13:12 ` Richard Biener
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2023-12-04 13:12 UTC (permalink / raw)
  To: Arsen Arsenović; +Cc: gcc-patches

On Sat, Dec 2, 2023 at 4:53 PM Arsen Arsenović <arsen@aarsen.me> wrote:
>
> contrib/ChangeLog:
>
>         * download_prerequisites
>         <arg parse>: Parse --only-gettext.
>         (echo_archives): Check only_gettext and stop early if true.
>         (helptext): Document --only-gettext.
> ---
> Afternoon,
>
> This patch adds a --only-gettext option to download_prerequisites for
> when the only useful dependency to download is gettext (which will
> restore a gcc source tree to a similar 'intlness' as before the
> externalization of gettext-runtime).
>
> For context, see
> https://inbox.sourceware.org/CAFiYyc2-JxH358GUcZfR4iBMq5qj6Nf4W=7LYoQyw6b-u8Dfqw@mail.gmail.com/
>
> OK for trunk?

OK

> TIA, have a lovely day!
>
>  contrib/download_prerequisites | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/contrib/download_prerequisites b/contrib/download_prerequisites
> index 9568091c0dba..30ff0cc9491a 100755
> --- a/contrib/download_prerequisites
> +++ b/contrib/download_prerequisites
> @@ -36,16 +36,18 @@ gettext='gettext-0.22.tar.gz'
>  base_url='http://gcc.gnu.org/pub/gcc/infrastructure/'
>
>  echo_archives() {
> +    echo "${gettext}"
> +    if "${only_gettext}"; then return; fi
>      echo "${gmp}"
>      echo "${mpfr}"
>      echo "${mpc}"
> -    echo "${gettext}"
>      if [ ${graphite} -gt 0 ]; then echo "${isl}"; fi
>  }
>
>  graphite=1
>  verify=1
>  force=0
> +only_gettext=false
>  OS=$(uname)
>
>  if type wget > /dev/null ; then
> @@ -74,6 +76,7 @@ The following options are available:
>   --no-verify      don't verify package integrity
>   --sha512         use SHA512 checksum to verify package integrity (default)
>   --md5            use MD5 checksum to verify package integrity
> + --only-gettext   inhibit downloading any package but gettext
>   --help           show this text and exit
>   --version        show version information and exit
>  "
> @@ -159,6 +162,9 @@ do
>                  chksum_extension='md5'
>                  verify=1
>                  ;;
> +            --only-gettext)
> +                only_gettext=true
> +                ;;
>              -*)
>                  die "unknown option: ${arg}"
>                  ;;
> --
> 2.43.0
>

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

end of thread, other threads:[~2023-12-04 13:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-02 15:47 [PATCH] download_prerequisites: add --only-gettext Arsen Arsenović
2023-12-04 13:12 ` Richard Biener

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).