Skip to content

Commit

Permalink
iconbrowser test: URI-encode the remote path
Browse files Browse the repository at this point in the history
Otherwise the download fails. We need to provide both the encoded path,
and the name for the local file, as cmake doesn't provide an encoding
helper.

Pick-to: 6.8
Change-Id: Id2d1c197f9ee1326ee229ebd32e5af156c970aed
Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
  • Loading branch information
vohi committed Nov 13, 2024
1 parent 2c8c17e commit 5ecbba1
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/manual/iconbrowser/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -29,14 +29,15 @@ target_link_libraries(iconbrowser PRIVATE
endif()

if (ANDROID)
set(font_uri "MaterialSymbolsOutlined%5BFILL%2CGRAD%2Copsz%2Cwght%5D.ttf")
set(font_filename "MaterialSymbolsOutlined[FILL,GRAD,opsz,wght].ttf")
if (QT_ALLOW_DOWNLOAD)
include(FetchContent)

FetchContent_Declare(
MaterialIcons
URL
"https://github.com/google/material-design-icons/raw/master/variablefont/${font_filename}"
"https://github.com/google/material-design-icons/raw/master/variablefont/${font_uri}"
DOWNLOAD_DIR ${CMAKE_CURRENT_BINARY_DIR}
DOWNLOAD_NAME "${font_filename}"
DOWNLOAD_NO_EXTRACT TRUE
Expand Down

0 comments on commit 5ecbba1

Please sign in to comment.