Skip to content

Conversation

@akihikodaki
Copy link

No description provided.

@lbibass
Copy link

lbibass commented May 29, 2021

Any updates on getting this pulled in?

@lbibass
Copy link

lbibass commented Aug 29, 2021

Checking in on getting this merged into master again! Would really love to see it.

@ebassi
Copy link
Collaborator

ebassi commented Aug 30, 2021

This will not get merged, unless the issue on Windows is solved.

Copy link

@t0rr3sp3dr0 t0rr3sp3dr0 left a comment

Choose a reason for hiding this comment

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

Would these suggested changes be enough to get this PR merged?

@akihikodaki
Copy link
Author

Would these suggested changes be enough to get this PR merged?

I need an answer for #239 (comment).

@akihikodaki akihikodaki force-pushed the macos branch 6 times, most recently from 344c91e to d1e471b Compare December 11, 2021 02:11
@akihikodaki
Copy link
Author

This pull request should be now ready for reviews by maintainers.

@melroy89
Copy link

melroy89 commented Mar 2, 2022

@ebassi please take a second look.

@akihikodaki
Copy link
Author

8611b07: rebased to c84bc94.

@akihikodaki
Copy link
Author

@ebassi Hi, it's been three years since this was submitted. Can you take a look at this?

@ebassi
Copy link
Collaborator

ebassi commented Mar 20, 2024

Two minor changes to limit DLL checks on Windows.

@akihikodaki
Copy link
Author

Two minor changes to limit DLL checks on Windows.

Applied with commit b0d6d60.

@akihikodaki
Copy link
Author

9fdedad: Replaced host_machine with host_system.

@avoidik
Copy link

avoidik commented Jun 15, 2024

@ebassi could you please approve this workflow?

@AkihiroSuda
Copy link

This will not get merged, unless the issue on Windows is solved.

@ebassi Is this issue solved?

cc @anholt

@AkihiroSuda
Copy link

What's the current status of this?

@mohd-akram
Copy link

It seems all the feedback was addressed. Is something pending here? Would love to see this merged.

@mohd-akram mohd-akram mentioned this pull request Oct 25, 2025
Now it automatically finds files from Qt 5 and supports the upstream
ANGLE on Windows and macOS.
@akihikodaki
Copy link
Author

Based on feedback from @mohd-akram, commit 47d975d simplifies the code by assuming modern ANGLE.

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.

10 participants