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

nss: fix build after e60fe20fdf94e829ba5fce33f7a9d6c281149f7d #1391

Closed
wants to merge 1 commit into from

Conversation

MarcelRaad
Copy link
Member

Curl_llist_alloc is now Curl_llist_init.

Curl_llist_alloc is now Curl_llist_init.
@bagder
Copy link
Member

bagder commented Apr 5, 2017

Ouch, I had totally not done my homework properly there! I presume this can run the test suite fine after this change? Our travis jobs unfortunately don't do enough different-TLS tests.

Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

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

looks fine to me, anything extra we should look out for you think @kdudka ?

@MarcelRaad
Copy link
Member Author

I presume this can run the test suite fine after this change?

I'm currently running it on Linux and MinGW, will know tomorrow morning.

Our travis jobs unfortunately don't do enough different-TLS tests.

Yes, this is exactly why I thought I should try a different one today :-)

@MarcelRaad
Copy link
Member Author

MarcelRaad commented Apr 6, 2017

I'm currently running it on Linux and MinGW, will know tomorrow morning.

Almost all of the 6xx tests failed, but from a cursory look, it seems unrelated.

@kdudka
Copy link
Contributor

kdudka commented Apr 6, 2017

Looks good to me. Thanks!

@MarcelRaad MarcelRaad closed this in c59fcda Apr 6, 2017
@MarcelRaad MarcelRaad deleted the fix_nss branch April 6, 2017 10:55
@MarcelRaad
Copy link
Member Author

Thanks for the reviews!

@lock lock bot locked as resolved and limited conversation to collaborators May 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants