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

[libffi] Update libffi to 3.3 #3348

Closed

Conversation

predominant
Copy link
Collaborator

Signed-off-by: Graham Weldon [email protected]

Signed-off-by: Graham Weldon <[email protected]>
Copy link
Contributor

@stevendanna stevendanna left a comment

Choose a reason for hiding this comment

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

These code changes look good to me. Note that this has 118 rdeps, including a number that will probably time-out currently in bldr (like core/llvm and core/python2). We'll probably want to coordinate merging this with @MindNumbing or @gavindidrichsen

@predominant
Copy link
Collaborator Author

@MindNumbing @gavindidrichsen - Any comments?

@MindNumbing
Copy link
Contributor

Please refrain from merging until we come up with a solution to this problem internally. May need a work around or a temporary extension to the build timeout in the meantime.

@predominant
Copy link
Collaborator Author

Added DO NOT MERGE tag, and assigning to @MindNumbing

@predominant
Copy link
Collaborator Author

@MindNumbing Any update on this?

@MindNumbing
Copy link
Contributor

@predominant Haven't picked this up for a while, will discuss it this week with the builder team.

@predominant
Copy link
Collaborator Author

Thanks :)

@MindNumbing
Copy link
Contributor

The timeouts have been increased, I will run some checks to make sure that the build will still work with the latest libifi

@MindNumbing
Copy link
Contributor

Nope, timeout is still canceling this job.

@MindNumbing
Copy link
Contributor

Now that #3450 has been approved and #3472 has been merged, these plans are no longer available in this repository.

Please see Additional Plans for more information.

Closing this PR, feel free to re-open at the new location.

Sadly this won't fix the issue with bldr not being able to build the rdeps without going over the timeout limit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto-build timeouts meta issue
3 participants