Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

exa -G not respected #1198

Closed
Squirreljetpack opened this issue May 28, 2023 · 3 comments
Closed

exa -G not respected #1198

Squirreljetpack opened this issue May 28, 2023 · 3 comments

Comments

@Squirreljetpack
Copy link

When piping exa -G | head -n 4, the output of exa is in one line mode
When running COLUMNS=4 exa -G, the output of exa is in one line mode
This differs from ls ls -C --color=always | head -n 5 which works as expected

EDIT: Seems ls has some trouble with this as well. It can only do 2 columns.
I suppose this has turned into a feature request: can we have an option to limit the number of items shown?

@sersorrel
Copy link

I bisected this to af20828:

$ git bisect bad
af208285e8a0fb383f59372c966bdb77f94006e9 is the first bad commit
commit af208285e8a0fb383f59372c966bdb77f94006e9
Author: Bastien Orivel <eijebong@bananium.fr>
Date:   Fri Dec 10 13:21:13 2021 +0100

    Update term_grid to 0.2

 Cargo.lock                 | 4 ++--
 Cargo.toml                 | 2 +-
 src/output/grid.rs         | 1 +
 src/output/grid_details.rs | 2 ++
 4 files changed, 6 insertions(+), 3 deletions(-)

That commit reverts cleanly today (on c697d06), if anyone wants a quick fix.

I'll try to bisect within term_grid itself if I can.

@sersorrel
Copy link

sersorrel commented Jul 24, 2023

I think this is caused by ogham/rust-term-grid#11, and fixed by ogham/rust-term-grid#12. (I haven't looked at the code at all, but applying the PR makes exa display grids properly again.)

...apparently this was already found two years ago in #990...

@ariasuni
Copy link
Collaborator

ariasuni commented Sep 7, 2023

This is already fixed in master and in the active fork exa.

@ariasuni ariasuni closed this as completed Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants