Skip to content

Commit

Permalink
Update glfw submodule to latest (#1237)
Browse files Browse the repository at this point in the history
* Update glfw submodule to latest

Old version was 3.3, released over two years ago
Fixes #1192

* Install libraries required by new GLFW version

* Fixing library names

* Fixing type

* Also add new GLFW required libs to v2 yml

* Also add new libraries to documentation ci step
Weird, but somehow required

* Update linux build docs with new libs required by updated GLFW

* Remove Wayland dependencies and conditionally build Antora

Wayland dependencies were removed from the workflows and build configurations to streamline the installation process. The Antora documentation build is now conditional, preventing unnecessary inclusion when VKB_GENERATE_ANTORA_SITE is not set. This improves build flexibility and reduces extraneous steps.

* Apply required changes to merged CI changes

* Trying to fix CI after merge (again)

* Trying to fix CI after merge (again and again)

---------

Co-authored-by: swinston <[email protected]>
  • Loading branch information
SaschaWillems and gpx1000 authored Jan 13, 2025
1 parent b3cb382 commit 271b1e9
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 8 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ jobs:
- if: ${{ matrix.platform != 'windows' }}
name: Configure and build
run: |
cmake -B"build/${{ matrix.platform }}" -DVKB_BUILD_TESTS=ON -DVKB_BUILD_SAMPLES=ON
cmake -B"build/${{ matrix.platform }}" -DVKB_BUILD_TESTS=ON -DVKB_BUILD_SAMPLES=ON -DGLFW_BUILD_WAYLAND=OFF
cmake --build "build/${{ matrix.platform }}" --target vulkan_samples --config ${{ matrix.build_type }} ${{ env.PARALLEL }}
build_v2:
Expand All @@ -76,7 +76,7 @@ jobs:
sudo apt install xorg-dev libglu1-mesa-dev
- name: Configure
run: cmake -B"build/${{ matrix.platform }}" -DVKB_BUILD_TESTS=ON
run: cmake -B"build/${{ matrix.platform }}" -DVKB_BUILD_TESTS=ON -DGLFW_BUILD_WAYLAND=OFF

- name: "Build Components ${{ matrix.platform }} in ${{ matrix.build_type }}"
run: cmake --build "build/${{ matrix.platform }}" --target vkb__components --config ${{ matrix.build_type }} ${{ env.PARALLEL }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ jobs:
with:
submodules: "recursive"
- run: git config --global --add safe.directory /__w/Vulkan-Samples/Vulkan-Samples
- run: cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -Bbuild/clang
- run: cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DGLFW_BUILD_WAYLAND=OFF -Bbuild/clang
- run: |
/usr/bin/run-clang-tidy -j $(($(nproc)/2+1)) -p build/clang -header-filter=framework,samples,app -checks=-*,google-*,-google-runtime-references -quiet ${{ needs.changed-files.outputs.all }}
Expand Down
6 changes: 4 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ cmake_minimum_required(VERSION 3.16)

project(vulkan_samples)

if(VKB_GENERATE_ANTORA_SITE)
add_subdirectory(antora)
else ()
# search for Vulkan SDK
find_package(Vulkan)

Expand Down Expand Up @@ -104,5 +107,4 @@ endif()

# Add vulkan app (runs all samples)
add_subdirectory(app)

add_subdirectory(antora)
endif ()
1 change: 0 additions & 1 deletion antora/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ function(gatherAntoraAssets)
samples
scripts
shaders
third_party
)
set(PAGES_DIR_SEARCH)
set(IMAGES_DIR_SEARCH)
Expand Down
2 changes: 1 addition & 1 deletion docs/build.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ build\windows\app\bin\Release\AMD64\vulkan_samples.exe
* <<3d-models,3D models>>

----
sudo apt-get install cmake g++ xorg-dev libglu1-mesa-dev
sudo apt-get install cmake g++ xorg-dev libglu1-mesa-dev libwayland-dev libxkbcommon-dev
----

=== Build with CMake
Expand Down
2 changes: 1 addition & 1 deletion third_party/glfw
Submodule glfw updated 111 files

0 comments on commit 271b1e9

Please sign in to comment.