Skip to content

Allow more granular selection of system dependencies (inc. SheenBidi)#3723

Open
solbjorn wants to merge 1 commit into
SFML:masterfrom
solbjorn:deps-granularity
Open

Allow more granular selection of system dependencies (inc. SheenBidi)#3723
solbjorn wants to merge 1 commit into
SFML:masterfrom
solbjorn:deps-granularity

Conversation

@solbjorn
Copy link
Copy Markdown
Contributor

  • Has this change been discussed on the forum or in an issue before?
  • Does the code follow the SFML Code Style Guide?
  • Have you provided some example/test code for your changes?

Description

"System deps" might be used not only when the corresponding packages are installed by the OS package manager, but also when compiling them separately. In either case, it would be useful to allow selecting which system/external dependencies shoud be used and which should be built within SFML more granularly.

Add the corresponding options. Freetype and Harfbuzz are combined into one option due to their circular dependencies, same with FLAC and Vorbis, and MbedTLS and libssh. Allow using external SheenBidi. The default behaves just like before the change, and the global SFML_USE_SYSTEM_DEPS switch is still here.

Tasks

  • Tested on Linux
  • Tested on Windows
  • Tested on macOS
  • Tested on iOS
  • Tested on Android

How to test this PR?

Compile test on different OSes and different combinations of the system dependencies options.

"System deps" might be used not only when the corresponding
packages are installed by the OS package manager, but also when
compiling them separately. In either case, it would be useful to
allow selecting which system/external dependencies shoud be used
and which should be built within SFML more granularly.

Add the corresponding options. Freetype and Harfbuzz are combined
into one option due to their circular dependencies, same with FLAC
and Vorbis, and MbedTLS and libssh. Allow using external SheenBidi.
The default behaves just like before the change, and the global
`SFML_USE_SYSTEM_DEPS` switch is still here.

Signed-off-by: Alexander Lobakin <alobakin@mailbox.org>
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.

1 participant