Bug 10070 - Make it clear that PARALLEL_MAKE(INST) won't work unless EXTRA_OEMAKE is passed to make
Summary: Make it clear that PARALLEL_MAKE(INST) won't work unless EXTRA_OEMAKE is pass...
Status: RESOLVED FIXED
Alias: None
Product: Reference
Classification: Documentation
Component: handbook (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: Undecided normal
Target Milestone: 2.2 M4
Assignee: Scott Rifenbark
QA Contact:
URL:
Whiteboard: 03 August 2016: RESOLVED
Depends on:
Blocks:
 
Reported: 2016-08-02 22:10 UTC by ulfalizer
Modified: 2016-08-04 00:05 UTC (History)
4 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Done (doc changes complete)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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