Skip to content
This repository was archived by the owner on Jul 24, 2020. It is now read-only.

Commit d24b871

Browse files
authored
Merge pull request #92 from nsoranzo/python3_fixes
Python3 fixes
2 parents 3cfdd1f + eea2b79 commit d24b871

2 files changed

Lines changed: 17 additions & 7 deletions

File tree

galaxy/tools/deps/conda_util.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import six
1313
from six.moves import shlex_quote
1414

15+
from galaxy.util import unicodify
1516
from . import (
1617
commands,
1718
installable
@@ -165,6 +166,7 @@ def ensure_conda_build_installed_if_needed(self):
165166
def conda_info(self):
166167
if self.conda_exec is not None:
167168
info_out = commands.execute([self.conda_exec, "info", "--json"])
169+
info_out = unicodify(info_out)
168170
info = json.loads(info_out)
169171
return info
170172
else:
@@ -494,6 +496,7 @@ def best_search_result(conda_target, conda_context, channels_override=None, offl
494496
search_cmd.extend(conda_context._override_channels_args)
495497
search_cmd.append(conda_target.package)
496498
res = commands.execute(search_cmd)
499+
res = unicodify(res)
497500
hits = json.loads(res).get(conda_target.package, [])
498501
hits = sorted(hits, key=lambda hit: packaging.version.parse(hit['version']), reverse=True)
499502

galaxy/tools/deps/containers.py

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,15 @@
88

99
import six
1010

11-
from galaxy.util import asbool
12-
from galaxy.util import in_directory
13-
from galaxy.util import plugin_config
11+
from galaxy.util import (
12+
asbool,
13+
in_directory,
14+
plugin_config
15+
)
16+
from . import (
17+
docker_util,
18+
singularity_util
19+
)
1420
from .container_resolvers.explicit import ExplicitContainerResolver
1521
from .container_resolvers.mulled import (
1622
BuildMulledDockerContainerResolver,
@@ -19,10 +25,11 @@
1925
CachedMulledSingularityContainerResolver,
2026
MulledDockerContainerResolver,
2127
)
22-
from .requirements import ContainerDescription
23-
from .requirements import DEFAULT_CONTAINER_RESOLVE_DEPENDENCIES, DEFAULT_CONTAINER_SHELL
24-
from ..deps import docker_util
25-
from ..deps import singularity_util
28+
from .requirements import (
29+
ContainerDescription,
30+
DEFAULT_CONTAINER_RESOLVE_DEPENDENCIES,
31+
DEFAULT_CONTAINER_SHELL
32+
)
2633

2734
log = logging.getLogger(__name__)
2835

0 commit comments

Comments
 (0)