Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

@max-nextcloud
Copy link
Collaborator Author

I think current master is segfaulting with php 7.4, 8.0 and 8.1.

I'll try to spin up a current master in my local dev env.

@max-nextcloud
Copy link
Collaborator Author

I'll try to spin up a current master in my local dev env.

That worked fine 🤔

@max-nextcloud
Copy link
Collaborator Author

  • c31e0e3f45ce5fd70d4f74b8cb7a0a0a445ad5a3 was the server commit of the last CI run that succeeded.
  • b90ccaac0f7abe85be3f4efa974d96296c3a250f was the first that failed.

@max-nextcloud max-nextcloud force-pushed the fix/cypress-php-version branch from 567c79d to bba0343 Compare December 28, 2022 08:11
@cypress
Copy link

cypress bot commented Dec 28, 2022



Test summary

116 0 0 0Flakiness 0


Run details

Project Text
Status Passed
Commit af9d301
Started Dec 29, 2022 1:02 PM
Ended Dec 29, 2022 1:08 PM
Duration 05:24 💡
OS Linux Ubuntu -
Browser Electron 106

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@max-nextcloud max-nextcloud force-pushed the fix/cypress-php-version branch from bba0343 to 0453842 Compare December 28, 2022 08:47
@max-nextcloud max-nextcloud force-pushed the fix/cypress-php-version branch from c9ce2c1 to a38aa6e Compare December 28, 2022 16:19
lets see if the segfault happens in the
generateAvatarFromSvg function.

Signed-off-by: Max <[email protected]>
@max-nextcloud max-nextcloud marked this pull request as draft December 28, 2022 21:31
@max-nextcloud max-nextcloud force-pushed the fix/cypress-php-version branch 2 times, most recently from b841123 to af9d301 Compare December 29, 2022 09:23
@juliusknorr
Copy link
Member

Closing as the server PR was reverted

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