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

colorls with more than zero file gives output with extra newlines #427

Closed
4 tasks
craigemery opened this issue Dec 10, 2020 · 3 comments
Closed
4 tasks

Comments

@craigemery
Copy link

Description

If I run
$ colorls
i.e. with no arguments, all the files are listed with no blank lines.

If I run
$ colorls *.ext
Then the files are all listed with an extra newline between each one.
This feels wrong.

  • Relevant Issues : (none)
  • Relevant PRs : (none)
  • Type of issue :
    • Installation
    • Font-related
    • Feature request
    • [ x] Bug in existing feature
    • Developer mode : Code quality / Tests / Documentation

I see this on macOS and Windows WSL.
colorls --version is 1.4.2

@craigemery craigemery changed the title colorls with more than one file gives output with extra newlines colorls with more than zero file gives output with extra newlines Dec 10, 2020
@craigemery
Copy link
Author

In fact colorls always shows a blank line, but when calling
$ colours
i.e. by itself, there's only one blank line & it's immediately before my prompt appears.
So it's just really noticeable when the blank line appears after every individually specified file.

@avdv
Copy link
Collaborator

avdv commented Dec 22, 2020

Hi, thanks for bringing this up again!

This is basically a known bug which is not easy to fix since the current code structure revolves around the concept of processing each command line argument in separation (especially in regard to the --report option which prints a report after each argument).

What should be done instead is group all arguments into files and directories. Print all file arguments as one group and then print every directory as subsequent groups.

@avdv avdv added the duplicate label Dec 23, 2020
@avdv
Copy link
Collaborator

avdv commented Dec 23, 2020

Duplicate of #233

@avdv avdv marked this as a duplicate of #233 Dec 23, 2020
@avdv avdv closed this as completed Dec 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants