Ubuntu 18.04 snap issue with -c flag sh and nano

[jtparrish]

Issue Label:

• Bug
• New feature
• Enhancement

Description:
When trying to run cryptowatch -c I am simply presented with the line
sh: 1: nano: not found in the terminal and immediately returned to the terminal prompt

If its a bug make sure to include this section.

OS and OS version:

• Mac
• Linux 32 Bit
• Linux 64 Bit
• Windows 32 Bit
• Windows 64 Bit

OS Version:
Ubuntu 18.04 LTS

[sarlacpit]

I get the same message on Ubuntu 18.04 when trying to generate the config.
I have generated a file manually but I don't think it helps as I cannot see my crypto balances.

[alexanderepstein]

Hmm if you run nano from the terminal does that not work? I was not aware the Ubuntu stopped shipping nano with the distribution. If this is the case you should be able to manually make the file at ~/.crypto.cfg The offending L.O.C. can be found here https://github.com/alexanderepstein/cryptowatch/blob/master/cryptoUtils/cwconfig.py#L125-L134. I should be able to make the editor a configurable option if Ubuntu did stop shipping nano with the distribution.

[sarlacpit]

Nano is installed works from the terminal - I don't think it has stopped shipping, it's located in /bin/nano

dpkg -l | grep nano
ii  nano                                    2.9.3-2                                      amd64        small, friendly text editor inspired by Pico

My skills in programming are weak, however - would it not be worthwhile calling the full path for nano:

    def openFile(self,filePath):
        if platform == "linux" or platform == "linux2":
            system("/bin/nano " + filePath)

[jtparrish]

I think it might be something to do with an invocation through dash? Is there somewhere in the code where nano is called using sh? If so, maybe this is where the issue lies. I got a similar (but not identical, so forgive the information dump if this is extraneous) message when I tried the command sh nano through my terminal. The correct formatting is sh -c nano because I think sh tries to run a shell script by default.

[alexanderepstein]

@sarlacpit did that work? If so submit a pull request and I will happily merge it.