Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli: uboot: Include PYTHONPATH in env for compiling U-Boot #6819

Merged
merged 1 commit into from
Jun 30, 2024

Conversation

ColorfulRhino
Copy link
Collaborator

@ColorfulRhino ColorfulRhino commented Jun 28, 2024

Description

  • Also group all u-boot make environment variables together for easier maintainability
  • Fixes u-boot requiring pyelftools
  • Fixes commit 04f619d

GitHub issue reference: #6799 (comment)

How Has This Been Tested?

  • sudo apt remove python3-pyelftools and then ./compile.sh uboot ARTIFACT_IGNORE_CACHE=yes BOARD=nanopi-r6c BRANCH=edge BUILD_DESKTOP=no BUILD_MINIMAL=no EXPERT=yes KERNEL_CONFIGURE=no RELEASE=trixie
  • ./compile.sh uboot ARTIFACT_IGNORE_CACHE=yes BOARD=radxa-zero3 BRANCH=edge BUILD_DESKTOP=no BUILD_MINIMAL=no EXPERT=yes KERNEL_CONFIGURE=no RELEASE=trixie

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@ColorfulRhino
Copy link
Collaborator Author

I can confirm that this fixes the issue that #6818 tries to solve in another way, since I have just stumbled over the issue on another branch (without this fix applied). After applying the fix from this PR, everything works. Tested compiling U-Boot for NanoPi R5C.

- Also group all u-boot make environment variables together for easier maintainability
- Fixes u-boot requiring `pyelftools`
- Fixes commit armbian@04f619d
@rpardini rpardini merged commit 792d9d9 into armbian:main Jun 30, 2024
5 checks passed
@ColorfulRhino ColorfulRhino deleted the uboot-python-env branch July 2, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Framework components size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants