mirror of
https://git.yoctoproject.org/poky
synced 2026-04-26 00:32:12 +02:00
patchtest: set default repo and testdir targets
Since patchtest is in oe-core, the Python os module's methods can be used to retrieve the repository path and tests directory by default. This reduces the number of mandatory arguments for invocation of patchtest unless the user wants to use a custom test suite or test patches against a different repo. The REPO and TESTDIR arguments are likewise adjusted so that they are optional. Also, make it more obvious what the --startdir flag is meant for on the command line by renaming it to --testdir, and update the scripts/patchtest.README file to be consistent with the new usage. (From OE-Core rev: bae7421ece4806f5148f164293810b9fe75e0756) Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
committed by
Richard Purdie
parent
6e53a778f1
commit
26e3d58e71
@@ -35,6 +35,9 @@ import logging
|
||||
logger=logging.getLogger('patchtest')
|
||||
info=logger.info
|
||||
|
||||
default_testdir = os.path.abspath(os.path.dirname(__file__) + "/tests")
|
||||
default_repodir = os.path.abspath(os.path.dirname(__file__) + "/../../..")
|
||||
|
||||
# Data store commonly used to share values between pre and post-merge tests
|
||||
PatchTestDataStore = collections.defaultdict(str)
|
||||
|
||||
@@ -58,10 +61,12 @@ class PatchTestInput(object):
|
||||
target_patch_group.add_argument('--directory', metavar='DIRECTORY', dest='patch_path',
|
||||
help='The directory containing patches to be tested')
|
||||
|
||||
parser.add_argument('repodir', metavar='REPO',
|
||||
parser.add_argument('--repodir', metavar='REPO',
|
||||
default=default_repodir,
|
||||
help="Name of the repository where patch is merged")
|
||||
|
||||
parser.add_argument('startdir', metavar='TESTDIR',
|
||||
parser.add_argument('--testdir', metavar='TESTDIR',
|
||||
default=default_testdir,
|
||||
help="Directory where test cases are located")
|
||||
|
||||
parser.add_argument('--top-level-directory', '-t',
|
||||
|
||||
Reference in New Issue
Block a user