Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing import "parse_date_or_fatal" to cmd/split-cmd.py #36

Closed
wants to merge 1 commit into from
Closed

Add missing import "parse_date_or_fatal" to cmd/split-cmd.py #36

wants to merge 1 commit into from

Conversation

xx4h
Copy link
Contributor

@xx4h xx4h commented Aug 1, 2017

If using bup split -n NAME -d STAMP FILE we currently get an error:

Traceback (most recent call last):
  File "/usr/lib/bup/cmd/bup-split", line 67, in <module>
    date = parse_date_or_fatal(opt.date, o.fatal)
NameError: name 'parse_date_or_fatal' is not defined

this patch does add the missing parse_date_or_fatal to from bup.helpers import

to cmd/split-cmd.py

Signed-off-by: Fabian 'xx4h' Melters <xx4h@xx4h.de>
@xx4h
Copy link
Contributor Author

xx4h commented Nov 2, 2017

Anything missing here?

@rlbdv
Copy link
Member

rlbdv commented Dec 28, 2017

Thanks for the help. In accordance with HACKING I've redirected this pull request to the list for discussion: https://groups.google.com/d/msg/bup-list/hwHfXQz4Fwg/A3kS5Nw-BAAJ

@rlbdv
Copy link
Member

rlbdv commented Dec 28, 2017

Pushed to master. Thanks.

@rlbdv rlbdv closed this Dec 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants