diff mbox series

[cip-kernel-sec,2/3] branch: fix remotes mapping

Message ID 20190618024025.8274-3-daniel.sangorrin@toshiba.co.jp (mailing list archive)
State Accepted
Headers show
Series [cip-kernel-sec,1/3] remotes: check that remotes exist | expand

Commit Message

Daniel Sangorrin June 18, 2019, 2:40 a.m. UTC
Probably a copy&paste error that never got checked because
nobody is using the mapping functionality.

Signed-off-by: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>
---
 scripts/kernel_sec/branch.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ben Hutchings June 18, 2019, 3:08 p.m. UTC | #1
On Tue, 2019-06-18 at 11:40 +0900, Daniel Sangorrin wrote:
> Probably a copy&paste error that never got checked because
> nobody is using the mapping functionality.

I thought I'd tested this. :-(

Ben.

> Signed-off-by: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>
> ---
>  scripts/kernel_sec/branch.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/kernel_sec/branch.py b/scripts/kernel_sec/branch.py
> index 6542415..fb8785c 100644
> --- a/scripts/kernel_sec/branch.py
> +++ b/scripts/kernel_sec/branch.py
> @@ -214,7 +214,7 @@ def get_remotes(mappings, mainline=None, stable=None):
>          _get_configured_remotes(
>              os.path.expanduser('~/.config/kernel-sec/remotes.yml')))
>      for mapping in mappings:
> -        left, right = arg.split(':', 1)
> +        left, right = mapping.split(':', 1)
>          remotes[left]['git_name'] = right
>      if mainline:
>          remotes['torvalds']['git_name'] = mainline
diff mbox series

Patch

diff --git a/scripts/kernel_sec/branch.py b/scripts/kernel_sec/branch.py
index 6542415..fb8785c 100644
--- a/scripts/kernel_sec/branch.py
+++ b/scripts/kernel_sec/branch.py
@@ -214,7 +214,7 @@  def get_remotes(mappings, mainline=None, stable=None):
         _get_configured_remotes(
             os.path.expanduser('~/.config/kernel-sec/remotes.yml')))
     for mapping in mappings:
-        left, right = arg.split(':', 1)
+        left, right = mapping.split(':', 1)
         remotes[left]['git_name'] = right
     if mainline:
         remotes['torvalds']['git_name'] = mainline