Skip to content

Fix SGX-RA Building #4208

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Fix SGX-RA Building #4208

wants to merge 5 commits into from

Conversation

dzobbe
Copy link
Contributor

@dzobbe dzobbe commented Apr 22, 2025

Given the definition of the LIB_RATS vars, this line should be fixed in this way.

Given the definition of the LIB_RATS vars, this line should be fixed in this way.
@dzobbe dzobbe changed the title Fix Librats EDL Include Fix SGX-RA Compilation Apr 22, 2025
@dzobbe dzobbe changed the title Fix SGX-RA Compilation Fix SGX-RA Building Apr 22, 2025
Copy link
Collaborator

@lum1n0us lum1n0us left a comment

Choose a reason for hiding this comment

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

Could you please share which release of SGX and SGXSSL you are using?

@dzobbe
Copy link
Contributor Author

dzobbe commented Apr 25, 2025

Could you please share which release of SGX and SGXSSL you are using?

I am using the SGX sdk v2.25 SGXSSL and 3.0_Rev2

@lum1n0us
Copy link
Collaborator

Understood. Could you please update samples/sgx-ra/README.md based on the PR modifications?

@dzobbe
Copy link
Contributor Author

dzobbe commented May 5, 2025

Understood. Could you please update samples/sgx-ra/README.md based on the PR modifications?

done!

@dzobbe dzobbe requested a review from lum1n0us May 5, 2025 19:58
-L$(LIB_RATS_SRC)/verifiers/sgx-ecdsa-qve \
-L$(SGX_SSL)/lib64 \
-L$(VMLIB_BUILD_DIR)/external/libcbor/src/libcbor/lib \
-L$(LIB_RATS_SRC)/crypto_wrappers/openssl \
Copy link
Collaborator

Choose a reason for hiding this comment

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

duplicate line with 166

@@ -53,7 +53,7 @@ $ # Verify that the paths are correctly set
$ which ar as ld objcopy objdump ranlib
$ cd ../
$ git clone https://github.com/intel/intel-sgx-ssl.git
$ wget https://www.openssl.org/source/openssl-1.1.1v.tar.gz -O intel-sgx-ssl/openssl_source/openssl-1.1.1v.tar.gz
$ wget https://www.openssl.org/source/openssl-3.0.14.tar.gz -O intel-sgx-ssl/openssl_source/openssl-3.0.14.tar.gz
Copy link
Collaborator

Choose a reason for hiding this comment

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

When I try to make all, it requires openssl version to be 3.1.x, do you think it's a problem?

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.

3 participants