Skip to content
This repository was archived by the owner on Jan 27, 2019. It is now read-only.

Conversation

@hundeboll
Copy link

Use use flags instead of a variable override in machine.conf or whatever.
Variables in machine.conf will propagate to all signatures whereas use
flags is only relavant for specific recipes.


## @var UBOOT_IMAGE Name of the image to build. Is u-boot.bin by default.
UBOOT_IMAGE ?= "u-boot.bin"
UBOOT_IMAGE ?= "${USE_uboot_image}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add

DEFAULT_USE_uboot_image = "u-boot.bin"

Which should be a sane default, and give us backwards compatibility.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See line 13 above...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bah :)

@hundeboll
Copy link
Author

I am wondering if I should add a use flag for UBOOT_IMAGE_FILENAME (and possibly UBOOT_IMAGE_BASE) as those are likely to change if USE_uboot_image is configured to something else than the default...

@esben
Copy link
Contributor

esben commented Mar 14, 2016

Sounds like a good idea.

Use use flags instead of a variable override in machine.conf or whatever.
Variables in machine.conf will propagate to all signatures whereas use
flags is only relavant for specific recipes.
@hundeboll
Copy link
Author

Updated with more use flags

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants