[PATCH] devtool: include do_patch in SRCTREECOVEREDTASKS

[PATCH] devtool: include do_patch in SRCTREECOVEREDTASKS

[Tzu-Jung Lee]

The external source of kernel has been patched during the
construction of git repository. Include the do_patch task in the
SRCTREECOVEREDTASKS.

Signed-off-by: Tzu-Jung Lee <roylee17@currantlabs.com>

diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index 7b87558..a7747c7 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -623,7 +623,7 @@ def modify(args, config, basepath, workspace):
             f.write('EXTERNALSRC_BUILD_pn-%s = "%s"\n' % (pn, srctree))
 
         if bb.data.inherits_class('kernel', rd):
-            f.write('SRCTREECOVEREDTASKS = "do_validate_branches do_kernel_checkout do_fetch do_unpack"\n')
+            f.write('SRCTREECOVEREDTASKS = "do_validate_branches do_kernel_checkout do_fetch do_unpack do_patch"\n')
         if initial_rev:
             f.write('\n# initial_rev: %s\n' % initial_rev)
             for commit in commits:
-- 
2.6.2

Re: [PATCH] devtool: include do_patch in SRCTREECOVEREDTASKS

[Paul Eggleton]

On Monday 02 November 2015 19:21:25 Tzu-Jung Lee wrote:
> The external source of kernel has been patched during the
> construction of git repository. Include the do_patch task in the
> SRCTREECOVEREDTASKS.
> 
> Signed-off-by: Tzu-Jung Lee <roylee17@currantlabs.com>
> 
> diff --git a/scripts/lib/devtool/standard.py
> b/scripts/lib/devtool/standard.py index 7b87558..a7747c7 100644
> --- a/scripts/lib/devtool/standard.py
> +++ b/scripts/lib/devtool/standard.py
> @@ -623,7 +623,7 @@ def modify(args, config, basepath, workspace):
>              f.write('EXTERNALSRC_BUILD_pn-%s = "%s"\n' % (pn, srctree))
> 
>          if bb.data.inherits_class('kernel', rd):
> -            f.write('SRCTREECOVEREDTASKS = "do_validate_branches
> do_kernel_checkout do_fetch do_unpack"\n') +           
> f.write('SRCTREECOVEREDTASKS = "do_validate_branches do_kernel_checkout
> do_fetch do_unpack do_patch"\n') if initial_rev:
>              f.write('\n# initial_rev: %s\n' % initial_rev)
>              for commit in commits:

Seems reasonable - I can't think why this would have been legitimately 
excluded, though I wonder if it had something to do with linux-yocto vs. more 
traditional OE kernel recipes.

Acked-by: Paul Eggleton <paul.eggleton@linux.intel.com>

(This will get queued for master, but we can backport to jethro after the 
release.)

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre