GithubHelp home page GithubHelp logo

Comments (16)

cantino avatar cantino commented on August 12, 2024 8

Should hopefully be fixed by #416 and https://github.com/cantino/mcfly/releases/tag/v0.9.0.

from mcfly.

mikionakajima avatar mikionakajima commented on August 12, 2024 3

Thank you.
I cloned the repository and compiled.
mcfly works perfect for a user and the root.

from mcfly.

slothyrulez avatar slothyrulez commented on August 12, 2024 1

Something similar here:

❯ uname -a
Linux micri 6.8.0-31-generic #31-Ubuntu SMP PREEMPT_DYNAMIC Sat Apr 20 00:40:06 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
❯ mcfly -V
mcfly 0.8.5

CTRL+r works as always, but, once I hit enter, there is fraction of a second of a text being shown, and nothing happens.

Grabacion.de.pantalla.desde.2024-05-30.14-26-43.mp4

from mcfly.

nuke-web3 avatar nuke-web3 commented on August 12, 2024 1

I have one feature suggestion. Harmful and common destructive commands should be filtered out. So there should be prompting just like cloud companies have when we deleter a bucket or object e.g. type 'DELETE'.

e.g. "rm -rf *"

sounds like a new feat req issue to me 😉

from mcfly.

cantino avatar cantino commented on August 12, 2024 1

I believe this is fixed! Please re-open if not. @dineshdharme, feel free to make a feature request or PR for blacklisting rm -rf commands, I think that's a good idea (if hard to do comprehensively).

from mcfly.

cantino avatar cantino commented on August 12, 2024

Did you also add it to your ~/.bashrc, ~/.zshrc, or ~/.config/fish/config.fish file?

from mcfly.

dineshdharme avatar dineshdharme commented on August 12, 2024

Yes. It is already there. I merely upgraded from 22.04 to 24.04 and this error started showing up.

If I set MCFLY_HISTORY , the error goes away.

from mcfly.

cantino avatar cantino commented on August 12, 2024

This means that mcfly's setup shell code isn't being executed, I suspect. If you're in a new shell and getting this error, and then you run the appropriate one of these, does it work?

Bash:
eval "$(mcfly init bash)"

Zsh:
eval "$(mcfly init zsh)"

Fish:
mcfly init fish | source

from mcfly.

dineshdharme avatar dineshdharme commented on August 12, 2024

I started a new shell. Executed the command.

mcfly init fish | source

Still same error. CTRL+R is not binded.

mcfly search throws the same error.

from mcfly.

cantino avatar cantino commented on August 12, 2024

I'm not sure what's going on. Does it work in Zsh?

from mcfly.

dineshdharme avatar dineshdharme commented on August 12, 2024

I tried it in Bash shell and it works in Bash.

Switched to Bash shell,

executed : eval "$(mcfly init bash)"

CTRL+R is binded properly and no error is thrown.

from mcfly.

nuke-web3 avatar nuke-web3 commented on August 12, 2024

Yes. It is already there. I merely upgraded from 22.04 to 24.04 and this error started showing up.

If I set MCFLY_HISTORY , the error goes away.

This could be related to #333 ? see also #404
I am betting kernel changes breaking a few things: https://itsfoss.com/ubuntu-24-04-vs-22-04/#8-linux-kernel-version

from mcfly.

dineshdharme avatar dineshdharme commented on August 12, 2024

I am using kernel 6.8.0-31-generic #31-Ubuntu SMP PREEMPT_DYNAMIC

from mcfly.

mikionakajima avatar mikionakajima commented on August 12, 2024

I got the same trouble.

deskmini with AMD Ryzen 2200G, 16G

> uname -a
Linux deskmini 6.8.0-31-generic #31-Ubuntu SMP PREEMPT_DYNAMIC Sat Apr 20 00:40:06 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
> mcfly -V
mcfly 0.8.6

CTRL-R works and MCFLY_HISTORY is correct.
When I push Enter, nothing happen.
IF I push TAB, nothing will display.

When I try CTRL-R as a root, it works.
I think the trouble is related with a permission.

I checked the temp files and I think the permission for them are set correctly.

# echo $MCFLY_HISTORY
/tmp/mcfly.SBmH3qQ5
> echo $MCFLY_HISTORY
/tmp/mcfly.FxOY7CBm
> ls -l /tmp
-rw------- 1 root  root  1872  6月  1 09:49 mcfly.SBmH3qQ5
-rw------- 1 myusr myusr 1251  6月  1 09:55 mcfly.FxOY7CBm

from mcfly.

dineshdharme avatar dineshdharme commented on August 12, 2024

I did the same. I followed the cargo installation steps. It works now.

from mcfly.

dineshdharme avatar dineshdharme commented on August 12, 2024

I have one feature suggestion. Harmful and common destructive commands should be filtered out. So there should be prompting just like cloud companies have when we deleter a bucket or object e.g. type 'DELETE'.

e.g. "rm -rf *"

from mcfly.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.