Re: [libav-devel] [PATCH] configure: cosmetics: Wrap a configure error string to the paragraph length

2017-09-23 Thread Vittorio Giovara 
On Sat, Sep 23, 2017 at 11:14 AM, Diego Biurrun  wrote:

> On Thu, Sep 21, 2017 at 07:08:00PM +0200, Vittorio Giovara wrote:
> > On Thu, Sep 21, 2017 at 6:34 PM, Diego Biurrun  wrote:
> > > On Thu, Sep 21, 2017 at 01:37:03PM +0200, Vittorio Giovara wrote:
> > > > --- a/configure
> > > > +++ b/configure
> > > > @@ -401,8 +401,8 @@ include the log this produces with your report.
> > > >  EOF
> > > >  else
> > > >  cat < > > > -Include the log file "$logfile" produced by configure as this will
> help
> > > > -solving the problem.
> > > > +Include the log file "$logfile" produced by configure as this
> > > > +will help solving the problem.
> > > >  EOF
> > >
> > > What paragraph length are you wrapping this to? It currently matches
> the
> > > length of the previous output lines quite well ...
> >
> > I'm talking about the output lines on a 80chars console
> > This is how I see it: https://www.dropbox.com/s/
> qfpgk7i9qs3htwk/Screenshot%
> > 202017-09-21%2019.05.15.png?dl=0
>
> OK, noted. I'll queue it up with more cosmetics that might be coming, OK?
>

ok with me
-- 
Vittorio
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] configure: cosmetics: Wrap a configure error string to the paragraph length

2017-09-23 Thread Diego Biurrun 
On Thu, Sep 21, 2017 at 07:08:00PM +0200, Vittorio Giovara wrote:
> On Thu, Sep 21, 2017 at 6:34 PM, Diego Biurrun  wrote:
> > On Thu, Sep 21, 2017 at 01:37:03PM +0200, Vittorio Giovara wrote:
> > > --- a/configure
> > > +++ b/configure
> > > @@ -401,8 +401,8 @@ include the log this produces with your report.
> > >  EOF
> > >  else
> > >  cat < > > -Include the log file "$logfile" produced by configure as this will help
> > > -solving the problem.
> > > +Include the log file "$logfile" produced by configure as this
> > > +will help solving the problem.
> > >  EOF
> >
> > What paragraph length are you wrapping this to? It currently matches the
> > length of the previous output lines quite well ...
> 
> I'm talking about the output lines on a 80chars console
> This is how I see it: https://www.dropbox.com/s/qfpgk7i9qs3htwk/Screenshot%
> 202017-09-21%2019.05.15.png?dl=0

OK, noted. I'll queue it up with more cosmetics that might be coming, OK?

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] configure: cosmetics: Wrap a configure error string to the paragraph length

2017-09-21 Thread Vittorio Giovara 
On Thu, Sep 21, 2017 at 6:34 PM, Diego Biurrun  wrote:

> On Thu, Sep 21, 2017 at 01:37:03PM +0200, Vittorio Giovara wrote:
> > --- a/configure
> > +++ b/configure
> > @@ -401,8 +401,8 @@ include the log this produces with your report.
> >  EOF
> >  else
> >  cat < > -Include the log file "$logfile" produced by configure as this will help
> > -solving the problem.
> > +Include the log file "$logfile" produced by configure as this
> > +will help solving the problem.
> >  EOF
>
> What paragraph length are you wrapping this to? It currently matches the
> length of the previous output lines quite well ...
>

I'm talking about the output lines on a 80chars console
This is how I see it: https://www.dropbox.com/s/qfpgk7i9qs3htwk/Screenshot%
202017-09-21%2019.05.15.png?dl=0
-- 
Vittorio
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] configure: cosmetics: Wrap a configure error string to the paragraph length

2017-09-21 Thread Diego Biurrun 
On Thu, Sep 21, 2017 at 01:37:03PM +0200, Vittorio Giovara wrote:
> --- a/configure
> +++ b/configure
> @@ -401,8 +401,8 @@ include the log this produces with your report.
>  EOF
>  else
>  cat < -Include the log file "$logfile" produced by configure as this will help
> -solving the problem.
> +Include the log file "$logfile" produced by configure as this
> +will help solving the problem.
>  EOF

What paragraph length are you wrapping this to? It currently matches the
length of the previous output lines quite well ...

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] configure: cosmetics: Wrap a configure error string to the paragraph length

2017-09-21 Thread Vittorio Giovara 
---
 configure | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index a3cfe37680..8e588ac104 100755
--- a/configure
+++ b/configure
@@ -401,8 +401,8 @@ include the log this produces with your report.
 EOF
 else
 cat