Fix warnings in Dunfell OE build. #26
Merged
Conversation
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
amstewart
requested changes
Aug 24, 2020
Contributor
amstewart
left a comment
amstewart
left a comment
There was a problem hiding this comment.
Looks good overall. It's nice to see that we're able to drop so many .patches!
shruthi-ravi
force-pushed
the
dev/dunfell_build_3
branch
from
5d62d57 to
675460d
Compare
amstewart
requested changes
Aug 26, 2020
Contributor
amstewart
left a comment
amstewart
left a comment
There was a problem hiding this comment.
Though is old/deprecated/unmaintained
s/is/it is/
:)
amstewart
requested changes
Aug 26, 2020
Contributor
|
Looks good; pending the nits above. |
shruthi-ravi
force-pushed
the
dev/dunfell_build_3
branch
from
5e91ea7 to
b9cdac6
Compare
Contributor
Author
|
I had updated the iproute2 version to 2.5 in another commit. I need to rebase this branch on top of those commits, which probably means another round of force pushes. |
amstewart
approved these changes
Aug 26, 2020
NI ARM targets will not be supported in Dunfell. u-boot has updated recipe files upstream and it will take significant effort to rework these bbappends. Drop all u-boot files and patches. Signed-off-by: Shruthi Ravichandran <[email protected]>
Bump version to match OE version 1.1.1g. Drop patches for Pharlap and VxWorks. These OSes will not be supported in Dunfell. Drop patches for nibuild Windows. These patches have already been applied in this version.
Python2 is no longer supported upstream. Signed-off-by: Shruthi Ravichandran <[email protected]>
Patch has been merged upstream since v2.5. Signed-off-by: Shruthi Ravichandran <[email protected]>
Signed-off-by: Shruthi Ravichandran <[email protected]>
Address multiple warnings related to Python requiring 4 spaces indentation instead of tabs. Signed-off-by: Shruthi Ravichandran <[email protected]>
ARM targets not supported in Dunfell. Signed-off-by: Shruthi Ravichandran <[email protected]>
Signed-off-by: Shruthi Ravichandran <[email protected]>
docker-registry not recommended anymore in OE recipe. Signed-off-by: Shruthi Ravichandran <[email protected]>
Signed-off-by: Shruthi Ravichandran <[email protected]>
Signed-off-by: Shruthi Ravichandran <[email protected]>
Signed-off-by: Shruthi Ravichandran <[email protected]>
Signed-off-by: Shruthi Ravichandran <[email protected]>
Change priority of packages to a value higher than the priority of coreutils. This fixes a warning that the binaries have multiple providers with the same priority. Update RDEPENDS variable name to fix warning that this key replaces the original RDEPENDS_coreutils key in coreutils instead of appending to it. Signed-off-by: Shruthi Ravichandran <[email protected]>
This variable gets defined in oe-core if not already defined. We shouldn't have to maintain this version separately. Signed-off-by: Shruthi Ravichandran <[email protected]>
Instead of carrying this patch forward every rebase in
meta-openembedded, move recipe to meta-nilrt.
commit 9dcf44f ("libglade: remove") dropped libglade because it
was broken by recipe specific sysroots (glib-2.0-native was missing).
Though it is old/deprecated/unmaintained, libglade is useful because
it's a dependency for the florence virtual keyboard which is the
smallest virtual keyboard in terms of disk space / memory usage.
Signed-off-by: Shruthi Ravichandran <[email protected]>
shruthi-ravi
force-pushed
the
dev/dunfell_build_3
branch
from
b9cdac6 to
c5a8a4c
Compare
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.
There are still a few warnings left that I haven't figured out. Pushing what I have.
@ni/rtos