Bug 10070

Summary: Make it clear that PARALLEL_MAKE(INST) won't work unless EXTRA_OEMAKE is passed to make
Product: [Documentation] Reference Reporter: ulfalizer
Component: handbookAssignee: Scott Rifenbark <srifenbark>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Undecided CC: bluelightning, poky.doc.watcher, poky.watcher, srifenbark
Version: 2.2   
Target Milestone: 2.2 M4   
Hardware: x86   
OS: Multiple   
Whiteboard: 03 August 2016: RESOLVED
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: Done (doc changes complete)

Description ulfalizer 2016-08-02 22:10:19 UTC
Not passing EXTRA_OEMAKE to make is a mistake I've seen a few times. The following proposed changes should hopefully clear it up:


Add between "...represents the maximum number of parallel threads make can run" and "by default, the OpenEmbedded..." in the glossary entry for PARALLEL_MAKE:

  Caution {
  In order for PARALLEL_MAKE to be effective, 'make' must be called with
  ${EXTRA_OEMAKE}. An easy way to ensure this is to use the oe_runmake function.
  }


Add before the Note in the glossary entry for PARALLEL_MAKEINST:

  Caution {
  In order for PARALLEL_MAKEINST to be effective, 'make' must be called with
  ${EXTRA_OEMAKE}. An easy way to ensure this is to use the oe_runmake function.
  }


Add the following to the end of the EXTRA_OEMAKE glossary entry:

  [PARALLEL_MAKE] and [PARALLEL_MAKEINST] also make use of EXTRA_OEMAKE to pass
  the required flags.


Cheers,
Ulf
Comment 1 Scott Rifenbark 2016-08-03 17:42:11 UTC
Ulf, 

See http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#var-PARALLEL_MAKE for the change to PARALLEL_MAKE.

See http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#var-PARALLEL_MAKEINST for the combined note in PARALLEL_MAKEINST.

See http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#var-EXTRA_OEMAKE for the addition to EXTRA_OEMAKE.

Scott
Comment 2 ulfalizer 2016-08-03 18:45:17 UTC
Looks good to me!

Thanks,
Ulf
Comment 3 ulfalizer 2016-08-03 20:53:25 UTC
One nit:

  In order for PARALLEL_MAKE to be effective...

should say

  In order for PARALLEL_MAKEINST to be effective...

in the note for PARALLEL_MAKEINST.
Comment 4 Scott Rifenbark 2016-08-03 21:22:39 UTC
oops... see http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#var-PARALLEL_MAKEINST.  Cut and paste error :)

thanks
Comment 5 ulfalizer 2016-08-03 21:55:06 UTC
Yeah, I figured. Fine now. :)

Cheers,
Ulf
Comment 6 Scott Rifenbark 2016-08-04 00:05:40 UTC
Thanks, 

Setting to RESOLVED and putting doc flag to "done."

Scott