Skip to content

Commit 1961888

Browse files
tulinkryVladimir Kotal
authored andcommitted
moving repofactory to scm module
- removing the cross module dependency forcing the imports to be in particular order
1 parent e1f92fa commit 1961888

File tree

4 files changed

+11
-14
lines changed

4 files changed

+11
-14
lines changed

opengrok-tools/src/main/python/opengrok_tools/mirror.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,20 +40,17 @@
4040

4141
from filelock import Timeout, FileLock
4242

43+
from .scm.repofactory import get_repository
44+
from .scm.repository import RepositoryException
4345
from .utils.hook import run_hook
4446
from .utils.log import get_console_logger, get_class_basename, \
4547
print_exc_exit
4648
from .utils.opengrok import get_repos, get_config_value, get_repo_type
4749
from .utils.parsers import get_baseparser
4850
from .utils.readconfig import read_config
49-
from .utils.repofactory import get_repository
5051
from .utils.utils import is_exe, check_create_dir, get_int, diff_list, \
5152
is_web_uri
5253

53-
# do not reorder this import, it must be imported after utils.
54-
# (for me) idea reorders the import to the top, causing an import error
55-
from .scm.repository import RepositoryException
56-
5754
major_version = sys.version_info[0]
5855
if major_version < 3:
5956
print("Need Python 3, you are running {}".format(major_version))

opengrok-tools/src/main/python/opengrok_tools/scm/__init__.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
from .cvs import CVSRepository
22
from .git import GitRepository
33
from .mercurial import MercurialRepository
4+
from .repo import RepoRepository
5+
from .repofactory import get_repository
46
from .repository import Repository
57
from .svn import SubversionRepository
68
from .teamware import TeamwareRepository
7-
from .repo import RepoRepository
89

910
__all__ = [
1011
'CVSRepository',
@@ -14,4 +15,5 @@
1415
'TeamwareRepository',
1516
'Repository',
1617
'RepoRepository',
18+
'get_repository',
1719
]

opengrok-tools/src/main/python/opengrok_tools/utils/repofactory.py renamed to opengrok-tools/src/main/python/opengrok_tools/scm/repofactory.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@
2121
# Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
2222
#
2323

24-
from ..scm.cvs import CVSRepository
25-
from ..scm.git import GitRepository
26-
from ..scm.mercurial import MercurialRepository
27-
from ..scm.svn import SubversionRepository
28-
from ..scm.teamware import TeamwareRepository
29-
from ..scm.repo import RepoRepository
24+
from .cvs import CVSRepository
25+
from .git import GitRepository
26+
from .mercurial import MercurialRepository
27+
from .repo import RepoRepository
28+
from .svn import SubversionRepository
29+
from .teamware import TeamwareRepository
3030

3131

3232
def get_repository(logger, path, repo_type, project, commands, env, hooks,

opengrok-tools/src/main/python/opengrok_tools/utils/__init__.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
from . import opengrok
55
from . import parsers
66
from . import readconfig
7-
from . import repofactory
87
from . import utils
98
from . import webutil
109

@@ -14,7 +13,6 @@
1413
'command',
1514
'commandsequence',
1615
'hook',
17-
'repofactory',
1816
'webutil',
1917
'readconfig',
2018
'parsers',

0 commit comments

Comments
 (0)