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

Update realm patch and build script with latest wolfssl support #210

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gojimmypi
Copy link

@gojimmypi gojimmypi commented Nov 19, 2024

The PR updates the existing realm-commit-a5e87a39.patch file with my dev-consolidated changes (a pseudo commit-squash), with my actual development branch on my realm-core/tree/dev, improving wolfSSL integration with the realm-core repository.

The update to realm-core is still a patch file here and applied in the build_wolfssl_with_realm.sh script, pending merge of wolfSSL changes at that repository.

Changes are consistently applied from the upstream a5e87a39 commit at this time. Sync to upstream master is planned.

Also included are some of the initial changes for Windows / Visual Studio build support. See also my osp dev branch that contains WIP realm/VS2022 updates, not included in this PR.

The build_wolfssl_with_realm.sh is also updated with many changes including additional parameterized settings, output status verbosity, a new -u option to default to user-specific fork clones & naming, error checking, and more.

  • edit: Note my wolfssl was built and installed to my WSL ~/wolfssl-install-dir directory.

See WOLFSSL_INSTALL_DIR="$HOME/wolfssl-install-dir" setting in build_wolfssl_with_realm.sh script and adjust as needed.

@dgarske dgarske requested review from JacobBarthelmeh and removed request for dgarske November 19, 2024 23:08
@dgarske dgarske assigned wolfSSL-Bot and unassigned gojimmypi Nov 19, 2024
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