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

fix glib transfer for elements created with NewElementWithProperties #46

Merged
merged 1 commit into from Sep 27, 2023

Conversation

@RSWilli RSWilli self-assigned this Sep 25, 2023
@danjenkins
Copy link
Contributor

know whats up with windows this time?

@RSWilli
Copy link
Member Author

RSWilli commented Sep 26, 2023

no I haven't had time. Locally and in production I use Linux to compile and run it where it works fine.

I'm going to investigate it a bit.

@danjenkins
Copy link
Contributor

no I haven't had time. Locally and in production I use Linux to compile and run it where it works fine.

I'm going to investigate it a bit.

quite possible the windows runner image has been updated

@danjenkins
Copy link
Contributor

yup my builds for my application keep randomly failing - ubuntu failing, windows failing. My guess is theres been a change to the images thats being run by github actions

@danjenkins
Copy link
Contributor

danjenkins commented Sep 26, 2023

I recall having an issue on a windows machine in my office and downgrading it from mingw 12 to 11 because that's what github used and it all worked fine. actions/runner-images@a0f082f#diff-a1f67f5c4a583bf90c239743379ad9473fbf1fa9d11c5cb7739c8012404dcc7aL76 shows they've upgraded to mingw 12 in the past couple of days.

Likely that gcc has been upgraded - actions/runner-images#8266

@danjenkins
Copy link
Contributor

Check out the linked issues at the bottom of golang/go#51007 with lots of other people having issues with GitHub windows builds that rely on gcc. Not sure if it's connected as I'm on my phone. Just chucking this link in here for now

@danjenkins
Copy link
Contributor

looks like this https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/2731 is related

Sorry for the noise... just making notes of issues that are related

@RSWilli RSWilli force-pushed the element_factory_with_properties_memleak branch from 4f5b665 to 94e32cd Compare September 27, 2023 12:02
@RSWilli RSWilli merged commit ab1fffc into main Sep 27, 2023
4 checks passed
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

2 participants