download_prerequisites: add --only-gettext

Message ID 20231202155249.1334525-1-arsen@aarsen.me
State Committed
Commit eb6c2bcb673c1de622395f0b4c30b65b776ad55d
Headers
Series download_prerequisites: add --only-gettext |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_gcc_check--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_gcc_build--master-arm success Testing passed
linaro-tcwg-bot/tcwg_gcc_check--master-arm success Testing passed

Commit Message

Arsen Arsenović Dec. 2, 2023, 3:47 p.m. UTC
  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(-)
  

Comments

Richard Biener Dec. 4, 2023, 1:12 p.m. UTC | #1
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
>
  

Patch

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}"
                 ;;