Re: The pgperltidy diffs in HEAD - Mailing list pgsql-hackers

From Dagfinn Ilmari Mannsåker
Subject Re: The pgperltidy diffs in HEAD
Date
Msg-id 87a5096oc6.fsf@wibble.ilmari.org
Whole thread Raw
In response to Re: The pgperltidy diffs in HEAD  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: The pgperltidy diffs in HEAD
List pgsql-hackers
Daniel Gustafsson <daniel@yesql.se> writes:

> [2. text/x-diff; v2_perltidyversion.diff]
> diff --git a/doc/src/sgml/func/func-string.sgml b/doc/src/sgml/func/func-string.sgml
> index 7ad1436e5f8..646b5d6d8c4 100644
> --- a/doc/src/sgml/func/func-string.sgml
> +++ b/doc/src/sgml/func/func-string.sgml
> @@ -173,7 +173,7 @@

This seems unrelated to the rest of the patch patch.

> diff --git a/src/tools/pgindent/pgperltidy b/src/tools/pgindent/pgperltidy
> index 6af27d21d55..6fac758665a 100755
> --- a/src/tools/pgindent/pgperltidy
> +++ b/src/tools/pgindent/pgperltidy
> @@ -7,6 +7,12 @@ set -e
>  # set this to override default perltidy program:
>  PERLTIDY=${PERLTIDY:-perltidy}
> 
> +PERLTIDY_VERSION=20230309
> +if ! $PERLTIDY -v | grep -q $PERLTIDY_VERSION; then
> +    echo "error: pgperltidy requires perltidy v$PERLTIDY_VERSION"

I just realised, this message should really go to stderr, i.e. have >&2
on the end.

> +    exit 1
> +fi
> +
>  . src/tools/perlcheck/find_perl_files
> 
>  find_perl_files "$@" | xargs $PERLTIDY --profile=src/tools/pgindent/perltidyrc

- ilmari



pgsql-hackers by date:

Previous
From: Hannu Krosing
Date:
Subject: Re: Patch: VACUUM should ignore (CREATE |RE)INDEX CONCURRENTLY for xmin horizon calculations
Next
From: Daniel Gustafsson
Date:
Subject: Re: The pgperltidy diffs in HEAD