Enhancement request to switch off command long_desc wrap formatter. #100

bdiz · 2012-07-08T21:07:40Z

I've got my own text wrap algorithm based on stty size. But then description gets re-wrapped.

The text was updated successfully, but these errors were encountered:

davetron5000 · 2012-07-09T14:15:57Z

Is the wrap not working? It should use the COLUMNS environment variable, or stty size if that is not available. What shell are you using?

bdiz · 2012-07-09T14:55:02Z

It appeared, just from resizing/re-running, that the wrapping algorithm is fixed. I'm using 2.0.0.rc2 with tcsh. There is no $COLUMNS but stty size works.

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:16 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc wrap formatter. (#100)

Is the wrap not working? It should use the COLUMNS environment variable, or stty size if that is not available. What shell are you using?

Reply to this email directly or view it on GitHub:
#100 (comment)

davetron5000 · 2012-07-09T14:57:30Z

Cool. I cribbed the algorithm from another project, but it's intended to
work in most reasonable shells (which includes tcsh :).

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog
Fork me on Github: http://davetron5000.github.com

On Mon, Jul 9, 2012 at 10:55 AM, bdiz <
reply@reply.github.com

wrote:

It appeared, just from resizing/re-running, that the wrapping algorithm is
fixed. I'm using 2.0.0.rc2 with tcsh. There is no $COLUMNS but stty size
works.

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:16 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Is the wrap not working? It should use the COLUMNS environment
variable, or stty size if that is not available. What shell are you
using?

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

bdiz · 2012-07-09T15:01:39Z

I'm sorry. By "fixed" I meant "width is unchanging". :)

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:58 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc wrap formatter. (#100)

Cool. I cribbed the algorithm from another project, but it's intended to work in most reasonable shells (which includes tcsh :).

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog Fork me on Github: http://davetron5000.github.com

On Mon, Jul 9, 2012 at 10:55 AM, bdiz <
reply@reply.github.com

wrote:

It appeared, just from resizing/re-running, that the wrapping
algorithm is fixed. I'm using 2.0.0.rc2 with tcsh. There is no
$COLUMNS but stty size works.

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:16 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Is the wrap not working? It should use the COLUMNS environment
variable, or stty size if that is not available. What shell are you
using?

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

davetron5000 · 2012-07-09T15:11:11Z

Ah, I see. When I run a gli app under tcsh, it seems to wrap things fine.

It's possible that GLI doesn't think the stty command exists in your
environment. Here's how it determines that:

ENV['PATH'].split(File::PATH_SEPARATOR).any? {|dir| File.exists?
File.join(dir, command) }

Can you try that in IRB (or as a debug statement in your app) and see what
it does? For me, on OS X, it's /bin/stty.

I'm wondering if it's only working as a shell built-in for you, it doesn't
get detected…

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog
Fork me on Github: http://davetron5000.github.com

On Mon, Jul 9, 2012 at 11:01 AM, bdiz <
reply@reply.github.com

wrote:

I'm sorry. By "fixed" I meant "width is unchanging". :)

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:58 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Cool. I cribbed the algorithm from another project, but it's intended to
work in most reasonable shells (which includes tcsh :).

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog Fork me on Github:
http://davetron5000.github.com

On Mon, Jul 9, 2012 at 10:55 AM, bdiz <
reply@reply.github.com

wrote:

It appeared, just from resizing/re-running, that the wrapping
algorithm is fixed. I'm using 2.0.0.rc2 with tcsh. There is no
$COLUMNS but stty size works.

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:16 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Is the wrap not working? It should use the COLUMNS environment
variable, or stty size if that is not available. What shell are you
using?

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

bdiz · 2012-07-10T04:20:28Z

irb(main):004:0> ENV['PATH'].split(File::PATH_SEPARATOR).any? {|dir| File.exists? File.join(dir, 'stty') }
=> true

Seems ok.
-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 8:11 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc wrap formatter. (#100)

Ah, I see. When I run a gli app under tcsh, it seems to wrap things fine.

It's possible that GLI doesn't think the stty command exists in your environment. Here's how it determines that:

ENV['PATH'].split(File::PATH_SEPARATOR).any? {|dir| File.exists?
File.join(dir, command) }

Can you try that in IRB (or as a debug statement in your app) and see what it does? For me, on OS X, it's /bin/stty.

I'm wondering if it's only working as a shell built-in for you, it doesn't get detected…

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog Fork me on Github: http://davetron5000.github.com

On Mon, Jul 9, 2012 at 11:01 AM, bdiz <
reply@reply.github.com

wrote:

I'm sorry. By "fixed" I meant "width is unchanging". :)

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:58 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Cool. I cribbed the algorithm from another project, but it's intended
to work in most reasonable shells (which includes tcsh :).

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog Fork me on Github:
http://davetron5000.github.com

On Mon, Jul 9, 2012 at 10:55 AM, bdiz < reply@reply.github.com

wrote:

It appeared, just from resizing/re-running, that the wrapping
algorithm is fixed. I'm using 2.0.0.rc2 with tcsh. There is no
$COLUMNS but stty size works.

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:16 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Is the wrap not working? It should use the COLUMNS environment
variable, or stty size if that is not available. What shell are you
using?

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

davetron5000 · 2012-07-11T14:54:04Z

That is very strange. Any chance you are the STDIN? The code checks if
STDIN is a tty and, if not, will not do any word-wrapping. That might not
be correct behavior, now that I think about it, but it could explain what
you are seeing…

Dave

On Tue, Jul 10, 2012 at 12:20 AM, bdiz <
reply@reply.github.com

wrote:

irb(main):004:0> ENV['PATH'].split(File::PATH_SEPARATOR).any? {|dir|
File.exists? File.join(dir, 'stty') }
=> true

Seems ok.
-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 8:11 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Ah, I see. When I run a gli app under tcsh, it seems to wrap things fine.

It's possible that GLI doesn't think the stty command exists in your
environment. Here's how it determines that:
ENV['PATH'].split(File::PATH_SEPARATOR).any? {|dir| File.exists?
File.join(dir, command) }
Can you try that in IRB (or as a debug statement in your app) and see what
it does? For me, on OS X, it's /bin/stty.

I'm wondering if it's only working as a shell built-in for you, it doesn't
get detected…

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog Fork me on Github:
http://davetron5000.github.com

On Mon, Jul 9, 2012 at 11:01 AM, bdiz <
reply@reply.github.com

wrote:

I'm sorry. By "fixed" I meant "width is unchanging". :)

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:58 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Cool. I cribbed the algorithm from another project, but it's intended
to work in most reasonable shells (which includes tcsh :).

Dave

Buy My Book: http://www.awesomecommandlineapps.com
My Blog: http://www.naildrivin5.com/blog Fork me on Github:
http://davetron5000.github.com

On Mon, Jul 9, 2012 at 10:55 AM, bdiz < reply@reply.github.com

wrote:

It appeared, just from resizing/re-running, that the wrapping
algorithm is fixed. I'm using 2.0.0.rc2 with tcsh. There is no
$COLUMNS but stty size works.

-----Original Message-----
From: David Copeland [mailto:reply@reply.github.com]
Sent: Monday, July 09, 2012 7:16 AM
To: Delsol, Ben
Subject: Re: [gli] Enhancement request to switch off command long_desc
wrap formatter. (#100)

Is the wrap not working? It should use the COLUMNS environment
variable, or stty size if that is not available. What shell are you
using?

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

Reply to this email directly or view it on GitHub:
#100 (comment)

davetron5000 closed this as completed in 63c0823 Sep 15, 2012

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Enhancement request to switch off command long_desc wrap formatter. #100

Enhancement request to switch off command long_desc wrap formatter. #100

bdiz commented Jul 8, 2012

davetron5000 commented Jul 9, 2012

bdiz commented Jul 9, 2012

davetron5000 commented Jul 9, 2012

bdiz commented Jul 9, 2012

davetron5000 commented Jul 9, 2012

bdiz commented Jul 10, 2012

davetron5000 commented Jul 11, 2012

Enhancement request to switch off command long_desc wrap formatter. #100

Enhancement request to switch off command long_desc wrap formatter. #100

Comments

bdiz commented Jul 8, 2012

davetron5000 commented Jul 9, 2012

bdiz commented Jul 9, 2012

davetron5000 commented Jul 9, 2012

bdiz commented Jul 9, 2012

davetron5000 commented Jul 9, 2012

bdiz commented Jul 10, 2012

davetron5000 commented Jul 11, 2012