BUILDING_DETAILS.md: extra packages section updates and fixes#141
Open
KanjiMonster wants to merge 5 commits into
Open
BUILDING_DETAILS.md: extra packages section updates and fixes#141KanjiMonster wants to merge 5 commits into
KanjiMonster wants to merge 5 commits into
Conversation
KanjiMonster
force-pushed
the
jogo_fix_update_extra_packages
branch
from
7e95128 to
3cbeed6
Compare
KanjiMonster
force-pushed
the
jogo_always_machine
branch
from
d9b6f3c to
f6151fe
Compare
Contributor
Author
|
What's missing (as a future exercise): description of package vs recipe (as bitbake takes recipe names, not package names), and how to find out what the recipe is named for a certain package. And maybe a link to the OpenEmbedded recipe index. |
We don't use $ in any other example commands, so drop them from the example here as well. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
"Building additional yocto packages" doesn't just build them, but actually includes them in the image, so update the heading to signify that. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
Yocto is always capitalized, so we should follow that. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
Add a decription on how to build additional packages and how to install them on a running switch. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
KanjiMonster
force-pushed
the
jogo_fix_update_extra_packages
branch
from
3cbeed6 to
50c2365
Compare
Adding missing whitespace to the beginning of the IMAGE_INSTALL:append section, to avoid package names being merged together. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
Contributor
Author
|
Found and fixed one additional small issue. |
ideaship
requested changes
Jan 10, 2025
| them by passing them via `--target` to kas: | ||
|
|
||
| ```shell | ||
| KAS_MACHINE=generic-x86-64 kas build --target lldpd bisdn-linux.yaml -- --runall=do_package_write_ipk |
Contributor
There was a problem hiding this comment.
Suggested change
| KAS_MACHINE=generic-x86-64 kas build --target lldpd bisdn-linux.yaml -- --runall=do_package_write_ipk | |
| KAS_MACHINE=generic-x86-64 kas-container build --target lldpd bisdn-linux.yaml -- --runall=do_package_write_ipk |
We have a separate section on building without containers, the other examples should use kas-container. I realize that I accidentally introduced an example with kas as well, I can fix that or you can add it to this PR, too.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The build extra packages section describes how to add packages to the image, so rename the section header accordingly, and add a section describing how to build individual packages to install on a running system.
While at it, remove the $ prefixes from the shell section to align it to all other sections whe have, and capitalize Yocto.