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

sota_sanity.bbclass: don't use copy_data() #62

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

shr-project
Copy link
Collaborator

The oe-core change got backported today to langdale and kirkstone:
https://git.openembedded.org/openembedded-core/commit/?h=kirkstone&id=06e088ef6e961f05ca600612adcc71bff91f09be

so we need the changes from #61 to be applied there as well.

* fixes:
  uptane#60

  meta-updater/classes/sota_sanity.bbclass", line 100, in defaultsota_check_sanity_eventhandler(e=<bb.event.SanityCheck object at 0x7fa32faedf30>):
           if bb.event.getName(e) == "SanityCheck":
      >        sanity_data = copy_data(e)
               if e.generateevents:
  NameError: name 'copy_data' is not defined

  copy_data function was removed in:
  https://git.openembedded.org/openembedded-core/commit/?id=d3eb4531aae28a07cb7e52ed5fe1102445d2effd

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Copy link
Collaborator

@pattivacek pattivacek left a comment

Choose a reason for hiding this comment

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

Thanks!

@pattivacek pattivacek merged commit ffbbc2b into uptane:kirkstone Dec 2, 2022
cajun-rat pushed a commit to cajun-rat/meta-updater-1 that referenced this pull request Apr 17, 2023
* brings 37 new commits:
git log --oneline 637d1f19d14d3a99fad71d110f5018505b85d2c7..bf0494df6382dd26e3daa89d57a393c0ff77f7d8
bf0494df6 Speed up compiles with Include What You Use
2a00664ea Merge pull request uptane#72 from cajun-rat/build-fix2
dcbe347da Fix build breakage caused by git CVE-2022-24765 fixes
4345deec2 Merge pull request uptane#68 from uptane/hsm-test-failure-fix
5e6e4aef3 tests: Suppress a memory leak in libcrypto
81ab71daa crypto: Make the p11 engine singleton within a test suite
7ac53f631 Merge pull request uptane#67 from uptane/fix-codecov-badge
7700175cd Update codecov URLs.
cd373e6e1 secondary: Init structure to make clang-tidy happy
caf558b24 docker: Create a home directory of the test user
3cb7e2a2a scripts: Fix the script to run tests in docker
5c9ba5d40 Revert "Disable broken p11 tests until someone can sort them out."
a7983c56e Merge pull request uptane#63 from uptane/disable-broken-p11-tests
e8f7ce9de Disable broken p11 tests until someone can sort them out.
b6d60658f Merge pull request uptane#64 from cajun-rat/fix-aktualizr-kill
305e3219e Merge pull request uptane#62 from uptane/shellcheck-fixes
e84be427b Fix flakey aktualizr_kill test
987b52dc4 More shellcheck fixes, this time with v0.8.0.
17e71e0f8 Add Github Actions job to run shellcheck in CI.
c8e3ac84f More shellcheck fixes.
67408853d Fix easy shellcheck problems.
df868c6ff Merge pull request uptane#61 from cajun-rat/extra-keys-root-json
d2ab5e51e Add test for extra keys in root.json
9e14d6a74 Merge pull request uptane#60 from uptane/follow-redirects
1b6e9ae2d http: Remove redundant curl options.
068ea3573 [fio extras]: httpclient: follow redirects by default
6f75976e1 Merge pull request uptane#55 from cajun-rat/api-queue-refactor
6daaa0939 Rewrite ApiQueue as an explicit implementation
91daac983 Tidy up: std::move tasks in Aktualizr
813c1a68b Add test for ApiQueue
4249e9f53 Merge pull request uptane#59 from cajun-rat/import-root-keys
3b8fd67b5 Import initial root.json from the filesystem
718bd87e2 Better handling of existing paths in uptane-generator
9a298772a Remove STORAGE_TYPE option from CMakeLists and tests
d64fcb81b Remove dead code
f73c68e95 Merge pull request uptane#58 from uptane/aktualizr-info-keys
06c5a631b aktualizr-info: Add options to print Secondary ECU keys and all key IDs.

The last commit fixes the includes so that it builds OK with new glibc and new boost, replacing
uptane#38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants