kristopolous / mansnip

📺 "The new way to search through manpages that everyone is talking about!"
MIT License
68 stars 4 forks source link

No result in most cases #7

Closed SmartFinn closed 3 years ago

SmartFinn commented 4 years ago

Hello.

I like the idea of this utility, but in most cases it does not return a result, even for the examples in README.

$ > ./mansnip tar -x -z -v -f
$ > ./mansnip tar -x
$ > ./mansnip ping -w
$ > ./mansnip ping -c
$ > ./mansnip bash -e
$ > ./mansnip bash -z
$ > ./mansnip find -printf
547  EXPRESSION
     ACTIONS
       -printf format
...
787  UNUSUAL FILENAMES
       -printf, -fprintf
...
$ > ./mansnip journalctl -x
$ > ./mansnip awk -f
$ > ./mansnip sed -i
$ > ./mansnip grep -E
$ > 

All of these queries request valid manpages and options.

Disto: Fedora 32 Python version: 3.8 (with 3.7 the same) Mansnip version: master branch

kristopolous commented 3 years ago

thanks i just saw this ... let me look

kristopolous commented 3 years ago

these all except for awk works for me. could you give me ssh access if you're still interested?

SmartFinn commented 3 years ago

could you give me ssh access if you're still interested?

It's hard to do because my machine behind NAT but you can get the same environment with docker/podman with the command:

docker run --rm -ti registry.fedoraproject.org/f33/fedora-toolbox:33 bash
[root@a6ae691285d9 /]# pip install mansnip-kristopolous
[root@a6ae691285d9 /]# mansnip tar -x
[root@a6ae691285d9 /]# mansnip ping -c
[root@a6ae691285d9 /]# mansnip awk -f
[root@a6ae691285d9 /]# mansnip mansnip sed -i
kristopolous commented 3 years ago

wonderful. That's a great starting point. The actual manpage pipeline is pretty crazy so I'll look into this, I'm probably hooking in a way that's not universal. Thanks a lot! I really appreciate it.

kristopolous commented 3 years ago

oh they brought back the auto-closer .... fuck that feature

kristopolous commented 3 years ago

i hate how github imposes a workflow. anyway that commit should fix it. feel free to confirm.

SmartFinn commented 3 years ago

Now it works wonderfully. Thank you :+1: