Skip to content

[osg] Update dependency sdl2 as sdl1#10455

Closed
NancyLi1013 wants to merge 2 commits intomicrosoft:masterfrom
NancyLi1013:dev/NancyLi/10320-fix-osg
Closed

[osg] Update dependency sdl2 as sdl1#10455
NancyLi1013 wants to merge 2 commits intomicrosoft:masterfrom
NancyLi1013:dev/NancyLi/10320-fix-osg

Conversation

@NancyLi1013
Copy link
Contributor

Since this file #include <SDL/SDL.h> is from sdl1 instead of sdl2.
I also tried to use SDL2 to build, but there were many errors in SDL2/SDL.h.
So I update the dependency sdl2 as sdl1 in CONTROL file.

Related issue #10320

Note: The feature tested pass in x86-windows triplet.

@dan-shaw
Copy link
Contributor

/azp run

@NancyLi1013
Copy link
Contributor Author

osg-qt failed on Linux platform due to this regression:

CMake Error at /ci/myagent/_work/1/s/installed/x64-linux/share/cmake/Qt5Gui/Qt5GuiConfig.cmake:15 (message):
  The imported target "Qt5::Gui" references the file

     "/ci/myagent/_work/3/s/installed/x64-linux/lib/libEGL.so"

  but this file does not exist.  Possible reasons include:

  * The file was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and contained

     "/ci/myagent/_work/1/s/installed/x64-linux/share/cmake/Qt5Gui/Qt5GuiConfigExtras.cmake"

  but not all the files it references.

@dan-shaw
Copy link
Contributor

Is this similar to the fix in #10082 ?

@NancyLi1013
Copy link
Contributor Author

@dan-shaw
Thanks for your reminder.
It seems that the PR #10082 has included this issue.
I close this PR now.

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