Skip to content

refactor: remove obsolete Makefile and script for attestation verifier #56

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hemanthsm
Copy link
Contributor

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • Ready to merge

Description

Fixes # (issue)

Any Newly Introduced Dependencies

How Has This Been Tested?

@hemanthsm hemanthsm force-pushed the dev/hemanthk/verifier_build_cleanup branch from 72a1154 to 7a87a79 Compare May 8, 2025 05:01
@hemanthsm
Copy link
Contributor Author

build is working
ci is catching make test error now, think earlier it was missing this check as test was being run inside docker
@lsubash can u take look on go test failures comment on how to disposition them

@hemanthsm hemanthsm requested a review from lsubash May 8, 2025 05:35
docker logs -f $(CONTAINER_NAME); \
docker images; \
fi
cd src && make download-eca
Copy link
Contributor

Choose a reason for hiding this comment

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

There were certain Build tool dependencies as cabextract, tpmtools for trustagent band others. User has to install these tools on host systems if we bring these builds on hosts.

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