> On 20 Jun 2023, at 17:14, Roman Arutyunyan <a...@nginx.com> wrote: > > Hi, > > On Tue, May 23, 2023 at 08:29:03PM +0400, Sergey Kandaurov wrote: >> >>> On 23 May 2023, at 19:48, Roman Arutyunyan <a...@nginx.com> wrote: >>> >>> # HG changeset patch >>> # User Roman Arutyunyan <a...@nginx.com> >>> # Date 1684856745 -14400 >>> # Tue May 23 19:45:45 2023 +0400 >>> # Branch quic >>> # Node ID bd25277d0ed77a90766bc2f4153538c86621cdfc >>> # Parent a83071314a22652d4a552e72e13d18aa4c01d79e >>> Configure: dump unsupported repository error and exit. >>> >>> diff --git a/auto/configure b/auto/configure >>> --- a/auto/configure >>> +++ b/auto/configure >>> @@ -4,6 +4,13 @@ >>> # Copyright (C) Nginx, Inc. >>> >>> >>> +cat << END >>> + >>> +$0: this repository is no longer supported, see README >>> + >>> +END >>> +exit 1 >>> + >>> LC_ALL=C >>> export LC_ALL >>> >> >> Something similar to what we use for invalid options might be enough. >> Also, using "error" might help emphasize this is an error: >> >> diff --git a/auto/configure b/auto/configure >> --- a/auto/configure >> +++ b/auto/configure >> @@ -4,6 +4,9 @@ >> # Copyright (C) Nginx, Inc. >> >> >> +echo "$0: error: this repository is no longer supported, see README" >> +exit 1 >> + >> LC_ALL=C >> export LC_ALL > > Usually two empty lines are added as well (see auto/lib/perl/conf for > example): > > diff --git a/auto/configure b/auto/configure > --- a/auto/configure > +++ b/auto/configure > @@ -4,6 +4,12 @@ > # Copyright (C) Nginx, Inc. > > > +echo > +echo "$0: error: this repository is no longer supported, see README" > +echo > + > +exit 1 > + > LC_ALL=C > export LC_ALL >
Agree, I feels better. > >> >> >> Personally, I think this makes sense, in order to help break pipelines. >> -- Sergey Kandaurov _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org https://mailman.nginx.org/mailman/listinfo/nginx-devel
