cryptop icon indicating copy to clipboard operation
cryptop copied to clipboard

Allow user to specify alternate wallet and config filenames as arguments

Open ptsand opened this issue 7 years ago • 3 comments

Issue #43

ptsand avatar Jan 17 '18 13:01 ptsand

Awesome! I look forward to this being merged into the main!

jgoggan avatar Jan 17 '18 14:01 jgoggan

You can simplify your function like this:

# Allow user to specify wallet and config filenames as arguments
def handleargs():
    global DATAFILE, CONFFILE
    parser = argparse.ArgumentParser()
    parser.add_argument("-w", "--wallet", type=str, help="Specify wallet filename")
    parser.add_argument("-c", "--config", type=str, help="Specify config filename")
    args = parser.parse_args()
    
    DATAFILE = os.path.join(BASEDIR, args.wallet or 'wallet.json')
    CONFFILE = os.path.join(BASEDIR, args.config or 'config.ini')
    
    print ('Wallet path:', str(DATAFILE))
    print ('Config path:', str(CONFFILE))

Or use default value with argparse

# Allow user to specify wallet and config filenames as arguments
def handleargs():
    global DATAFILE, CONFFILE
    parser = argparse.ArgumentParser()
    parser.add_argument("-w", "--wallet", type=str, default='wallet.json' help="Specify wallet filename")
    parser.add_argument("-c", "--config", type=str, default='config.ini' help="Specify config filename")
    args = parser.parse_args()
    
    print ('Wallet path:', str(DATAFILE))
    print ('Config path:', str(CONFFILE))

rpesche avatar Feb 18 '18 14:02 rpesche

Thanks rpeche. I just pushed a shorter cleaner version of the feature, which are using the globals DATAFILE and CONFFILE as default arguments.

ptsand avatar Jun 07 '18 12:06 ptsand