Skip to content

Commit

Permalink
Merge pull request #1 from JeffAlyanak/issue-185
Browse files Browse the repository at this point in the history
Resolves Issue uskudnik#185.

Setting the `logfile=` option in the `[glacier]` section of a config had no effect. The logfile variable was simply being silently discarded. After this patch, the option works as expected.

Signed-off-by: Jeff Alyanak jeff@alyanak.ca
  • Loading branch information
JeffAlyanak authored Jan 31, 2020
2 parents ba3b49d + d166b53 commit bf44e8b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion glacier/glacier.py
Original file line number Diff line number Diff line change
Expand Up @@ -614,7 +614,7 @@ def main():
help="Amazon SimpleDB domain name for bookkeeping.")
group.add_argument('--logfile',
required=False,
default=os.path.expanduser('~/.glacier-cmd.log'),
default=default('logfile') if default('logfile') else os.path.expanduser('~/.glacier-cmd.log'),
help='File to write log messages to.')
group.add_argument('--loglevel',
required=False,
Expand Down

0 comments on commit bf44e8b

Please sign in to comment.