init shell failed? about mcfly

[cantino]

Did you add mcfly init fish | source to your fish config file?

[4t8dd]

Did you add mcfly init fish | source to your fish config file?

yes, but nothing got written as I inspected.
Something wrong here? But the key bingding works and I can search.

[4t8dd]

should I paste all the init shell to config.fish manually?

EDIT:

@cantino sorry, I missed the line and I should have add the init result to config.fish

But even so, I still got a panic with mcfly dump in fish.

McFly error: Please ensure that MCFLY_HISTORY is set (environment variable not found)

but I found one line in the mcfly init shell:

# We don't need a MCFLY_HISTORY file because we can get the last command in fish_postexec. 
set -g __MCFLY_CMD $MCFLY_PATH --mcfly_history /dev/null --history_format fish             

But in settiings.rs: if MCFLY_HISTORY not found there will be panic. Why? Is this a bug? Or I missed the config.
But I did not see any config like this in readme.

And what would the config.toml file look like? And it should be in XDG_DATA_DIR in MAC where locates
the history.db by default. But I did not find this config file too.

I think I figure it out here.
When to search, the MCFLY_PATH is set to /dev/null, because __MCFLY_COMMAND is set to
`
set -g __MCFLY_CMD $MCFLY_PATH --mcfly_history /dev/null --history_format fish

Then search and add are good from this init shell. But if I just usemcfly dump` or any other command it just failed.
because MCFLY_HISTORY is not set.

So the simplest fix is add line set -g MCFLY_HISTORY /dev/null and it is fix then.

[4t8dd]

PR: #414

[cantino]

Good fix, thank you!