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

Unable to handle wildcard support #10

Open
dmikushin opened this issue Oct 10, 2024 · 4 comments
Open

Unable to handle wildcard support #10

dmikushin opened this issue Oct 10, 2024 · 4 comments

Comments

@dmikushin
Copy link

Consider the following example:

►facad *.ttf
Error: Unexpected argument 'NotoMono-Regular.ttf'

Here I ask to list all .ttf files. Since the wildcard expansion is a part of the shell itself, facad sees expanded list of files. And fails, because it expects only one argument.

The expected correct behavior is to have the same as in ls:

►ls *.ttf
NotoMono-Regular.ttf  NotoSans-BoldItalic.ttf  NotoSans-Bold.ttf  NotoSans-Italic.ttf  NotoSansMono-Regular.ttf  NotoSans-Regular.ttf
@yellow-footed-honeyguide
Copy link
Owner

Thank you, @dmikushin ! Wildcard support added
scr_20241011_192211

@dmikushin
Copy link
Author

As of 62126de , there is an issue somewhere.

With '*' all files are correctly displayed:

~/f/f/build►./facad *                                                                                                                         (main|?↑1) 19:41
💾 facad                      🏝️ compile_commands.json         🏝️ intro-tests.json       🧩 src_emoji_mappings.c.o
🔬 sanitycheckc.c             🏝️ intro-benchmarks.json         🏝️ meson-info.json        🧩 src_emoji_utils.c.o   
❓ build.dat                  🏝️ intro-buildoptions.json       🔒 meson.lock             🧩 src_file_card.c.o     
❓ coredata.dat               🏝️ intro-buildsystem_files.json  🥷 build.ninja            🧩 src_longlisting.c.o   
❓ install.dat                🏝️ intro-dependencies.json       🧩 src_args_parser.c.o    🧩 src_main.c.o          
❓ meson_benchmark_setup.dat  🏝️ intro-install_plan.json       🧩 src_dev_dir_utils.c.o  ❓ coredata.dat.prev     
❓ meson_test_setup.dat       🏝️ intro-installed.json          🧩 src_dir_analytics.c.o  📝 cmd_line.txt          
💾 sanitycheckc.exe           🏝️ intro-projectinfo.json        🧩 src_dir_config.c.o     📝 install-log.txt       
🧢 config.h                   🏝️ intro-targets.json            🧩 src_display_utils.c.o  📝 meson-log.txt         

With *.json only one JSON file is listed, however there are many of them above:

~/f/f/build►./facad *.json                                                                                                                    (main|?↑1) 19:41
🏝️ compile_commands.json

But I'm not sure if it is related to wildcard, because -l * also misses a lot of files:

~/f/f/build►./facad -l *                                                                                                                      (main|?↑1) 19:40
  172.0K  4m ago  👑: marcusmae 📜: 👀✏️🚀  📁 facad.p          (0)
  119.2K  4m ago  👑: marcusmae 📜: 👀✏️🚀  📁 meson-private    (0)
   11.2K  4m ago  👑: marcusmae 📜: 👀✏️🚀  📁 meson-info       (0)
    3.0K  2d ago  👑: marcusmae 📜: 👀✏️🚀  📁 meson-logs       (0)
  108.0K  4m ago  👑: marcusmae 📜: 👀✏️🚀  💾 facad            
    5.6K  4m ago  👑: marcusmae 📜: 👀✏️❌  🥷 build.ninja      
    4.0K  4m ago  👑: marcusmae 📜: 👀✏️❌  🏝️ compile_commands.json
  135.0B  4m ago  👑: marcusmae 📜: 👀✏️❌  🧢 config.h         

What do you think?

@dmikushin
Copy link
Author

dmikushin commented Oct 11, 2024

So, negative: I cannot reproduce your output of multiple *go files, I always get only one file.

@yellow-footed-honeyguide
Copy link
Owner

@dmikushin , at least for fast checking can't reproduce....

scr_20241012_172852

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

2 participants