* [gentoo-catalyst] [PATCH] resume: fix missing sys import
@ 2015-10-06 3:48 Mike Frysinger
2015-10-06 6:51 ` Brian Dolbec
0 siblings, 1 reply; 2+ messages in thread
From: Mike Frysinger @ 2015-10-06 3:48 UTC (permalink / raw
To: gentoo-catalyst
This module uses the sys module but didn't actually import it.
---
catalyst/base/resume.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/catalyst/base/resume.py b/catalyst/base/resume.py
index 2c006ab..a03adf3 100644
--- a/catalyst/base/resume.py
+++ b/catalyst/base/resume.py
@@ -14,6 +14,7 @@ set, unset, is_set, is_unset, enabled, clear_all
import os
import shutil
from stat import ST_UID, ST_GID, ST_MODE
+import sys
import traceback
from catalyst.fileops import ensure_dirs, pjoin, listdir_files, clear_dir
--
2.5.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [gentoo-catalyst] [PATCH] resume: fix missing sys import
2015-10-06 3:48 [gentoo-catalyst] [PATCH] resume: fix missing sys import Mike Frysinger
@ 2015-10-06 6:51 ` Brian Dolbec
0 siblings, 0 replies; 2+ messages in thread
From: Brian Dolbec @ 2015-10-06 6:51 UTC (permalink / raw
To: gentoo-catalyst
On Mon, 5 Oct 2015 23:48:17 -0400
Mike Frysinger <vapier@gentoo.org> wrote:
> This module uses the sys module but didn't actually import it.
> ---
> catalyst/base/resume.py | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/catalyst/base/resume.py b/catalyst/base/resume.py
> index 2c006ab..a03adf3 100644
> --- a/catalyst/base/resume.py
> +++ b/catalyst/base/resume.py
> @@ -14,6 +14,7 @@ set, unset, is_set, is_unset, enabled, clear_all
> import os
> import shutil
> from stat import ST_UID, ST_GID, ST_MODE
> +import sys
> import traceback
>
> from catalyst.fileops import ensure_dirs, pjoin, listdir_files,
> clear_dir
uh oh, this one is my bad. I usually run pyflakes on the code to pick
up these little things...
Thank you, merge please
--
Brian Dolbec <dolsen>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-10-06 6:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-06 3:48 [gentoo-catalyst] [PATCH] resume: fix missing sys import Mike Frysinger
2015-10-06 6:51 ` Brian Dolbec
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox