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

aml-s9xx-box: bsp-cli hashing fixes #6839

Merged

Conversation

rpardini
Copy link
Member

@rpardini rpardini commented Jun 30, 2024

aml-s9xx-box: bsp-cli hashing fixes

  • aml-s9xx-box: inline bsp-cli preinst function so it's hashed together with the hook; half fixes [Bug]: bsp-cli package isn't being rebuilt #6817
    • this is technically cheating, but works
    • add TODO's ref the manual include'ing of files that are not hashed in the bsp-cli
  • aml-s9xx-box: move files included in bsp-cli from packages/bsp/aml-s9xx-box to config/optional/boards/aml-s9xx-box/_packages/bsp-cli; fixes [Bug]: bsp-cli package isn't being rebuilt #6817
    • with this:
      • those files are automatically hashed by the bsp-cli hashing mechanism
      • no need to copy them manually any more

@github-actions github-actions bot added size/medium PR with more then 50 and less then 250 lines Hardware Hardware related - kernel, u-boot, patches labels Jun 30, 2024
@rpardini rpardini marked this pull request as ready for review June 30, 2024 11:40
@rpardini rpardini requested review from SteeManMI, a team and igorpecovnik as code owners June 30, 2024 11:40
@rpardini rpardini force-pushed the pr/aml-s9xx-box-bsp-cli-hashing-fixes branch 3 times, most recently from 36e0433 to 8655b83 Compare June 30, 2024 18:07
… with the hook; half fixes armbian#6817

- this is technically cheating, but works
- add TODO's ref the manual include'ing of files that are not hashed in the bsp-cli
…9xx-box` to `config/optional/boards/aml-s9xx-box/_packages/bsp-cli`; fixes armbian#6817

- with this:
  - those files are automatically hashed by the bsp-cli hashing mechanism
  - no need to copy them manually any more
@rpardini rpardini force-pushed the pr/aml-s9xx-box-bsp-cli-hashing-fixes branch from 8655b83 to 769ab72 Compare June 30, 2024 19:34
Copy link
Contributor

@SteeManMI SteeManMI left a comment

Choose a reason for hiding this comment

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

@rpardini Thank you very much. You didn't need to do this, this is really going above and beyond. I would have been happy with a couple of.pointers.on what needed to be done and left to do the legwork myself. Again, thank you.

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 08 Milestone: Third quarter release labels Jun 30, 2024
@igorpecovnik igorpecovnik merged commit ffe83ab into armbian:main Jul 1, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
08 Milestone: Third quarter release Hardware Hardware related - kernel, u-boot, patches Ready to merge Reviewed, tested and ready for merge size/medium PR with more then 50 and less then 250 lines
Development

Successfully merging this pull request may close these issues.

[Bug]: bsp-cli package isn't being rebuilt
3 participants