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

irregular "does not look like a valid path or URL" for valid local dir #47

Open
FiXato opened this issue Feb 1, 2022 · 0 comments
Open

Comments

@FiXato
Copy link
Contributor

FiXato commented Feb 1, 2022

Gitty will frequently say a local repo isn't a valid path or URL, even when it is.
Retrying the exact same command sometimes suddenly does make it recognise as valid.

fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty /mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher/
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty /mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher/
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty /mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher/
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty /mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher/
🏠 Repository https://github.com/FiXato/tui_launcher

🐛 4 open issues
4 Labels that auto-update their text based on the output of a command                  10m ◖enhancement◗
3 Feature Requeset: Allow appending multiple shell commands to one command funct…       1y ◖enhancement◗
2 launcher.py cannot locate configs when launched outside of the tui_launcher di…       1y ◖Investigate◗ ◖bug◗ ◖documentation◗
1 Button border drawing issue                                                           1y ◖Investigate◗

📌 No open pull requests

🌳 1 active branch
main     ↑   ↓ Added support for default button markup and renamed 'highlight' t…       1d

🔥 13 commits since creation (a long while ago)
d8bf499 Added support for default button markup and renamed 'highlight' to 'f…          1d
1ee28e9 Added support for dynamic labels and palette names                             10m
e95311c Force display of header / footer if texts set by command-line args             10m
dc4421d Header and footer text can now also be supplied via a command-line ar…         10m
3afabe3 Added chmod +x step                                                             1y
7822341 Typo fix                                                                        1y
75a9014 Fix layout of readme                                                            1y
bb87e4c Added upgrade instructions                                                      1y
f0ca943 Moved usage instructions from launcher's comments to README.                    1y
4b3b635 Major update separating code from config and layout.                            1y

Same thing happens when not specifying a path:

fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
does not look like a valid path or URL
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ gitty
🏠 Repository https://github.com/FiXato/tui_launcher

🐛 4 open issues
4 Labels that auto-update their text based on the output of a command                  10m ◖enhancement◗
3 Feature Requeset: Allow appending multiple shell commands to one command funct…       1y ◖enhancement◗
2 launcher.py cannot locate configs when launched outside of the tui_launcher di…       1y ◖Investigate◗ ◖bug◗ ◖documentation◗
1 Button border drawing issue                                                           1y ◖Investigate◗

📌 No open pull requests

🌳 1 active branch
main     ↑   ↓ Added support for default button markup and renamed 'highlight' t…       1d

🔥 13 commits since creation (a long while ago)
d8bf499 Added support for default button markup and renamed 'highlight' to 'f…          1d
1ee28e9 Added support for dynamic labels and palette names                             10m
e95311c Force display of header / footer if texts set by command-line args             10m
dc4421d Header and footer text can now also be supplied via a command-line ar…         10m
3afabe3 Added chmod +x step                                                             1y
7822341 Typo fix                                                                        1y
75a9014 Fix layout of readme                                                            1y
bb87e4c Added upgrade instructions                                                      1y
f0ca943 Moved usage instructions from launcher's comments to README.                    1y
4b3b635 Major update separating code from config and layout.                            1y

In case timing is relevant, as my laptop is currently under a fairly high load:

fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ time gitty
does not look like a valid path or URL

real    0m2.353s
user    0m0.020s
sys     0m0.029s
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ time gitty
does not look like a valid path or URL

real    0m1.846s
user    0m0.013s
sys     0m0.061s
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ time gitty
does not look like a valid path or URL

real    0m1.280s
user    0m0.010s
sys     0m0.070s
fixato@FiXatop:/mnt/c/Users/FiXato/Desktop/Development/urwid/tui_launcher $ time gitty
🏠 Repository https://github.com/FiXato/tui_launcher

🐛 4 open issues
4 Labels that auto-update their text based on the output of a command                  10m ◖enhancement◗
3 Feature Requeset: Allow appending multiple shell commands to one command funct…       1y ◖enhancement◗
2 launcher.py cannot locate configs when launched outside of the tui_launcher di…       1y ◖Investigate◗ ◖bug◗ ◖documentation◗
1 Button border drawing issue                                                           1y ◖Investigate◗

📌 No open pull requests

🌳 1 active branch
main     ↑   ↓ Added support for default button markup and renamed 'highlight' t…       1d

🔥 13 commits since creation (a long while ago)
d8bf499 Added support for default button markup and renamed 'highlight' to 'f…          1d
1ee28e9 Added support for dynamic labels and palette names                             10m
e95311c Force display of header / footer if texts set by command-line args             10m
dc4421d Header and footer text can now also be supplied via a command-line ar…         10m
3afabe3 Added chmod +x step                                                             1y
7822341 Typo fix                                                                        1y
75a9014 Fix layout of readme                                                            1y
bb87e4c Added upgrade instructions                                                      1y
f0ca943 Moved usage instructions from launcher's comments to README.                    1y
4b3b635 Major update separating code from config and layout.                            1y

real    0m6.203s
user    0m0.319s
sys     0m0.332s

This is on Windows Subsystem for Linux 2 (Kernel version: 5.10.60.1) running Ubuntu 20.04.3 LTS with gitty 0.7.0 (4762bb9) (gitty_0.7.0_linux_x86_64.tar.gz).
The git repo is on the internal Windows drive rather than on a local WSL partition.

(This does not seem like a recent regression, as 0.4.0 has the same issue, though this is the first time I've noticed it.)

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

1 participant