Bug 10017 - base.bbclass: Having ${D} in do_install[dirs] is redundant, since it's already in do_install[cleandirs]
Summary: base.bbclass: Having ${D} in do_install[dirs] is redundant, since it's alread...
Status: RESOLVED FIXED
Alias: None
Product: Meta-yocto
Classification: Build System, Metadata & Runtime
Component: meta-yocto (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 2.2
Assignee: Richard Purdie
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-07-25 11:26 UTC by ulfalizer
Modified: 2016-09-22 10:22 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description ulfalizer 2016-07-25 11:26:02 UTC 
Hello,

do_install[cleandirs] = "${D}" will remove and recreate ${D} already if it does not exist. Therefore, putting ${D} in do_install[dirs] is redundant.

If ${D} was the last entry in do_install[dirs], then it would become the working directory of do_install. But it is not, so removing it makes no difference.

The most important side-effect of fixing it might be to clarify how [cleandirs] works to people reading base.bbclass.

Cheers,
Ulf