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

Can you make an artifact for a release that does not include the gigantic other_lib directory? #763

Closed
GasimGasimzada opened this issue Sep 8, 2023 · 3 comments · Fixed by #931

Comments

@GasimGasimzada
Copy link
Contributor

GasimGasimzada commented Sep 8, 2023

I am downloading source code of KTX from the public release to be able to build the loader and write in my own project. Downloading the source code of KTX is 250mb. Can we have an artifact in KTX that does not bundle the other_lib directory and provides only the source code (the file size gets reduced to 25mb due to KTX files in tests but it is at least downloadable in CI environments)?

@MarkCallow
Copy link
Collaborator

MarkCallow commented Aug 1, 2024

The linked PR removes other_lib from the repo and will close this issue. tests/testimages will remain but will be greatly reduced when the legacy tools and their tests are removed in the release after next. A few images will remain that are used by {es3,gl3,vk}loadtests.

@GasimGasimzada
Copy link
Contributor Author

Thank you!

@MarkCallow
Copy link
Collaborator

I prefer to let this be closed when the PR is merged. Re-opening.

@MarkCallow MarkCallow reopened this Aug 4, 2024
MarkCallow added a commit that referenced this issue Aug 6, 2024
1.3.290 is the first release of official iOS support and first with support
for Windows arm64.

Stop using binaries of SDL2, assimp and glew libraries stored in repo
because an updated SDL2 is needed for 1.3.290 and more platforms are
now supported. Dependencies for iOS, macOS and Windows are now obtained
through vcpkg. Those for Linux through the package manager as before.

Thus remove other_lib, other_include/{SDL2,assimp} and
other_include/GL/{glew,glxew,wglew}.h. Move the modified Vulkan
manifest files to tests/loadtests/appfwSDL/VulkanAppSDL.

This incidentally fixes #763.

These changes plus removal of the hacks for finding the iOS components
in older Vulkan SDKs means a major revamp of the load tests-related
CMake files.

vcpkg is used in manifest mode so installation of dependencies is
automatic once vcpkg is installted. This use of vcpkg led to a reshuffle
of the root CMakeLists.txt to put some options before the project()
comment so their values can be used to set VCPKG-related variables.

Deployment target for iOS is bumped to 12.0 and for macOS to 11.0 to
match what is supported by this Vulkan SDK.

The PR also fixes some unrelated problems:

1. A syntax error in scripts/mkversion most likely introduced by PR
    #922. It is unclear why CI did not fail with this.
2. Use of a query in `scripts/build_win.ps1 that returned process
    architecture not machine architecture. VS2022 Developer PowerShells
    are x86 processes leading to failure with the old query. Use
    Get-ComputerInfo instead.
3. Also in `scripts/build_win.ps1`, a warning when `PYTHON` is defined
    on the CMake configure command line if FEATURE_PYTHON is not ON.
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 a pull request may close this issue.

2 participants