search

andresgongora / synth-shell

Boost your terminal, script by script
GNU General Public License v3.0
992 stars 241 forks source link

better-ls not listing subdirectory #240

Closed croMicLi closed 1 year ago

croMicLi commented 1 year ago

Describe the bug better-ls does not list a subdirectory

To Reproduce Steps to reproduce the behavior: With synth-shell installed:

  1. Go to your home directory
  2. mkdir test1
  3. cd test1
  4. mkdir test2
  5. cd test2
  6. mkdir test3
  7. cd ~
  8. mv test1/ Downloads
  9. cd Downloads
  10. cd test1
  11. cd test2
  12. ls The "test3" directory will not be shown, but ls -a does show it. Removing better-ls from the .bashrc fixes this, adding it again caused the issue to appear once more. Did this several times to confirm that it was connected to better-ls.

Tested on Ubuntu 22.04 and on Kubuntu 22.10.

Expected behavior The "test3" subdirectory should be listed.

andresgongora commented 1 year ago

Hmmm yup, I can confirm that I can replicate this issue. I guess it has something to do with the amount of recursiveness in the script. I'll have a go at it

andresgongora commented 1 year ago

This should be fixed now in bc2b3af. It was a typo where better-ls did not properly check if the current directory had files/folders that needed to be displayed. Feel free to reopen the issue if the problem persists, and thanks for catching the bug :)