diff options
author | Thibault Saunier <tsaunier@igalia.com> | 2021-08-04 22:15:19 -0400 |
---|---|---|
committer | GStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org> | 2021-08-12 13:20:15 +0000 |
commit | c24958c7813fcc151253f6985d6dee83ec6090d0 (patch) | |
tree | a8db6d72e5373001a3bf5db6ba10f7b6eb41c69a | |
parent | b29665f3bd645310ba299d45f1734e2151a9899d (diff) |
git-update: Fix passing fetch_args
Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-build/-/merge_requests/255>
-rwxr-xr-x | git-update | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -61,7 +61,7 @@ def update_subprojects(manifest, no_interaction=False, status=False): continue revision, args = repos_commits.get(repo_name, [None, []]) - if not update_repo(repo_name, repo_dir, revision, no_interaction, args, status=status): + if not update_repo(repo_name, repo_dir, revision, no_interaction, fetch_args=args, status=status): return False return True @@ -103,7 +103,7 @@ def update_repo(repo_name, repo_dir, revision, no_interaction, fetch_args=None, revision = ensure_revision_if_necessary(repo_dir, revision) git("config", "rebase.autoStash", "true", repository_path=repo_dir) - fetch_args = fetch_args if fetch_args is None else [] + fetch_args = fetch_args if fetch_args is not None else [] fetch_args.append('--tags') fatal = fatal_git_fetches(repo_dir) try: @@ -142,7 +142,7 @@ def update_repo(repo_name, repo_dir, revision, no_interaction, fetch_args=None, if recurse_i < 3: return update_repo(repo_name, repo_dir, revision, no_interaction, - recurse_i + 1) + recurse_i=recurse_i + 1) return False else: print("\nCould not rebase %s, please fix and try again." @@ -214,7 +214,7 @@ if __name__ == "__main__": repos_commits = {} revision, args = repos_commits.get('gst-build', [None, []]) - if not update_repo('gst-build', SCRIPTDIR, revision, options.no_interaction, args, status=options.status): + if not update_repo('gst-build', SCRIPTDIR, revision, options.no_interaction, fetch_args=args, status=options.status): exit(1) if not update_subprojects(options.manifest, options.no_interaction, status=options.status): exit(1) |