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

add-on dependency updates #241

Closed
wants to merge 12 commits into from
Closed

add-on dependency updates #241

wants to merge 12 commits into from

Conversation

AlwinEsch
Copy link
Member

This update mostly all dependencies to latest versions, two now includes cmake and no more needs separated between POSIX and Windows.

As following steps comes to change to SSL 3.0.

@AlwinEsch AlwinEsch requested a review from phunkyfish May 24, 2023 21:53
@AlwinEsch AlwinEsch force-pushed the update-ssl branch 10 times, most recently from 2c2da62 to 5eb0186 Compare May 27, 2023 16:44
Now also includes CMakeLists.txt and on Windows depends no more needed.
Before was it not present and failed if perl not available inside system itself.
Copy link
Collaborator

@phunkyfish phunkyfish left a comment

Choose a reason for hiding this comment

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

Nice work!!!

@phunkyfish
Copy link
Collaborator

Appears to run well. I’d say just add the version and merge.

@AlwinEsch
Copy link
Member Author

Appears to run well. I’d say just add the version and merge.

Still some depends missing on update. Think the most strange are done and hopefully the others not also produce a fight to become usable on all OS.

@AlwinEsch AlwinEsch force-pushed the update-ssl branch 13 times, most recently from 40451e7 to 238ecdf Compare June 3, 2023 23:14
@AlwinEsch AlwinEsch changed the title [WIP] add-on dependency updates add-on dependency updates Jun 3, 2023
Now also includes CMakeLists.txt and on Windows depends no more needed.
On following changes becomes it cleaned up and only be used on "common".
By this change the Android folder no more available.
About Android a bad fix patch was needed as Jenkins still use Android API 21, but gnutls include __fpending where is available in API 23.
@AlwinEsch
Copy link
Member Author

The builds on all OS now OK, only needs it runtime tests to confirm correct operation’s.
If someone can look on his OS would be good. Have currently only my Windows laptop available, will try on this, but no good internet in my hotel in England.

@howie-f
Copy link
Contributor

howie-f commented Aug 27, 2023

deleted: sorry the comment belongs to #247

@phunkyfish
Copy link
Collaborator

phunkyfish commented Aug 27, 2023

deleted: sorry the comment belongs to #247

We should likely fix the issue on this PR as the others are already merged. This one needs a rebase anyway.

@phunkyfish
Copy link
Collaborator

Superseded by #267

@phunkyfish phunkyfish closed this Jan 30, 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.

None yet

3 participants