Adds building and testing of examples to CI. #599
Conversation
Superhepper
force-pushed
the
Ci-Examples
branch
from
65e4dbe to
f0335b4
Compare
Superhepper
requested review from
Firstyear,
hug-dev,
ionut-arm and
wiktor-k
tss-esapi/tests/examples.sh
Outdated
| ################## | ||
| # Build examples # | ||
| ################## | ||
| RUST_BACKTRACE=1 cargo build --examples |
There was a problem hiding this comment.
I think this shouldn't be necessary as cargo test --examples needs to build them first anyway... 🤔
There was a problem hiding this comment.
No, but its nice to separate build errors from running the examples errors.
There was a problem hiding this comment.
No, but its nice to separate build errors from running the examples errors.
There was a problem hiding this comment.
Then I suggest using cargo check which shows build errors but doesn't waste time creating output artifacts (that we wouldn't need in CI anyway). 👋
There was a problem hiding this comment.
Yeah, that could be done.
Oh I just noted that the script does not start the tpm server ... But everything is green anyway ..
There was a problem hiding this comment.
Yeah, that could be done.
Oh I just noted that the script does not start the tom server ... But everything is green anyway ..
There was a problem hiding this comment.
This was way more complicated then I expected. The examples requires a resource broker.
Superhepper
force-pushed
the
Ci-Examples
branch
5 times, most recently
from
5cfecc7 to
e1fd179
Compare
Superhepper
force-pushed
the
Ci-Examples
branch
17 times, most recently
from
f63f15e to
0186912
Compare
Superhepper
force-pushed
the
Ci-Examples
branch
3 times, most recently
from
6fd5b37 to
02fb471
Compare
Signed-off-by: Jesper Brynolf <[email protected]>
Superhepper
force-pushed
the
Ci-Examples
branch
from
02fb471 to
7d4c6c5
Compare
|
When I try to add the certify example as well I get an error I have not yet figured out. Is that something you have seen as well and have had any experience with? |
2 participants
Fixes #467