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.

Sign up for GitHub

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

Jump to bottom

[qtbase] Fix configuration race #28155

Merged
merged 1 commit into from
Dec 5, 2022
Merged

[qtbase] Fix configuration race #28155

merged 1 commit into from
Dec 5, 2022

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Dec 4, 2022

Fixes build error

CMake Error at D:/installed/arm64-windows/share/Qt6/3rdparty/extra-cmake-modules/find-modules/FindEGL.cmake:135 (file):
  file COPY cannot set permissions on
  "D:/buildtrees/qtmultimedia/arm64-windows-dbg/../CMakeError.log":
  Permission denied.

https://dev.azure.com/vcpkg/public/_build/results?buildId=81873&view=logs&j=84da4f4a-968a-5b5c-16e9-f444c845396d

@dg0yt
Copy link
Contributor Author

dg0yt commented Dec 4, 2022

@Neumann-A I assume this was a temporary debugging aid (and a potential use case for a global script mode variable in #26754).

@Neumann-A
Copy link
Contributor

Yeah it was. Needed for debugging why libEGL didn't link on windows/linux

@Neumann-A Neumann-A mentioned this pull request Dec 5, 2022
Closed
@Cheney-W Cheney-W added category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines labels Dec 5, 2022
@dan-shaw dan-shaw merged commit d7c68ee into microsoft:master Dec 5, 2022
@dg0yt dg0yt deleted the qtbase-find-egl branch December 6, 2022 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions GitHub Actions github-actions approved these changes

Assignees

@Cheney-W Cheney-W

Labels
category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dg0yt @Neumann-A @Cheney-W @dan-shaw