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

[DRAFT] python310Packages.afdko: 3.9.0 -> 3.9.1 #182941

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions pkgs/development/python-modules/afdko/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,13 @@

buildPythonPackage rec {
pname = "afdko";
version = "3.9.0";
version = "3.9.1";

disabled = pythonOlder "3.7";

src = fetchPypi {
inherit pname version;
sha256 = "1fjsaz6bp028fbmry6fzfcih78mdzycqmky1wsz5y0bg4kfk4shh";
sha256 = "sha256-AtRNaZzn9BoNIjvKhS7rsYVfHw26JKKUFqO/6TcBIkw=";
};

format = "pyproject";
Expand All @@ -43,6 +43,9 @@ buildPythonPackage rec {
# Use antlr4 runtime from nixpkgs and link it dynamically
./use-dynamic-system-antlr4-runtime.patch

# Force use of CMake-shipped FindLibXml2 module
# This is needed to work around a nixpkgs bug
# https://github.com/NixOS/nixpkgs/issues/125008
./libxml2-cmake-find-package.patch
];

Expand All @@ -69,13 +72,11 @@ buildPythonPackage rec {
];

checkInputs = [ pytestCheckHook ];

preCheck = ''
export PATH=$PATH:$out/bin

# Update tests to match ufinormalizer-0.6.1 expectations:
# https://github.com/adobe-type-tools/afdko/issues/1418
find tests -name layerinfo.plist -delete
'';

disabledTests = lib.optionals (!runAllTests) [
# Disable slow tests, reduces test time ~25 %
"test_report"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,22 +1,13 @@
commit c423d1ddf0345aed7ecaf4c8b9e1fa5108aafc6f
Author: sternenseemann <[email protected]>
Date: Sat Jul 2 12:35:56 2022 +0200

Force use of CMake-shipped FindLibXml2 module

This is needed to work around a nixpkgs bug:
https://github.com/NixOS/nixpkgs/issues/125008

diff --git a/CMakeLists.txt b/CMakeLists.txt
index a75b6fb1..c1408283 100644
index a2423022..9a4f97a4 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -42,7 +42,7 @@ set(ANTLR4_TAG tags/4.9.3)
find_path(ANTLR4_HEADER antlr4-runtime.h PATH_SUFFIXES antlr4-runtime)
set(ANTLR4_INCLUDE_DIRS ${ANTLR4_HEADER})
@@ -43,7 +43,7 @@ set(ANTLR4_TAG tags/4.9.3)
include(ExternalAntlr4Cpp)


-FIND_PACKAGE(LibXml2)
+FIND_PACKAGE(LibXml2 REQUIRED MODULE)
IF (NOT ${LibXml2_FOUND})
MESSAGE(STATUS "Could not locate LibXml2, will install externally.")
IF ((NOT ${LibXml2_FOUND}) OR "${CMAKE_SYSTEM}" MATCHES "Linux")
MESSAGE(STATUS "Could not locate LibXml2 or system is Linux, will install externally & statically link")
set(LIBXML2_TAG tags/v2.9.13)
Original file line number Diff line number Diff line change
@@ -1,15 +1,5 @@
commit 286b9c6e69691292dce4f2b4beaac8f886da184d
Author: sternenseemann <[email protected]>
Date: Tue Oct 5 18:16:10 2021 +0200

Link against system antlr4 runtime, dynamically

Instead of cloning a antlr4 version from git, use the system one. Also
don't link it statically, but dynamically by default (the library is
called antlr4-runtime, not antlr4_static).

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 9ce80598..a75b6fb1 100644
index a2423022..2a9f7afa 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -36,11 +36,11 @@ list(APPEND CMAKE_MODULE_PATH ${CMAKE_CURRENT_SOURCE_DIR}/cmake)
Expand All @@ -24,10 +14,10 @@ index 9ce80598..a75b6fb1 100644
+find_path(ANTLR4_HEADER antlr4-runtime.h PATH_SUFFIXES antlr4-runtime)
+set(ANTLR4_INCLUDE_DIRS ${ANTLR4_HEADER})


FIND_PACKAGE(LibXml2)
IF (NOT ${LibXml2_FOUND})
diff --git a/c/makeotf/lib/cffread/CMakeLists.txt b/c/makeotf/lib/cffread/CMakeLists.txt
index 9a400fde..5452d987 100644
index 7d74e512..f340792f 100644
--- a/c/makeotf/lib/cffread/CMakeLists.txt
+++ b/c/makeotf/lib/cffread/CMakeLists.txt
@@ -13,6 +13,6 @@ else ()
Expand All @@ -39,7 +29,7 @@ index 9a400fde..5452d987 100644

target_compile_definitions(makeotf_cffread PRIVATE $<$<CONFIG:Debug>:CFF_DEBUG=1> CFF_T13_SUPPORT=0)
diff --git a/c/makeotf/lib/hotconv/CMakeLists.txt b/c/makeotf/lib/hotconv/CMakeLists.txt
index 3cceceea..9695ea21 100644
index 14e5c3cd..9b39f8b8 100644
--- a/c/makeotf/lib/hotconv/CMakeLists.txt
+++ b/c/makeotf/lib/hotconv/CMakeLists.txt
@@ -69,7 +69,7 @@ add_library(hotconv STATIC
Expand All @@ -49,5 +39,5 @@ index 3cceceea..9695ea21 100644
-target_link_libraries(hotconv PUBLIC antlr4_static)
+target_link_libraries(hotconv PUBLIC antlr4-runtime)

if (${LibXml2_FOUND})
if ((${LibXml2_FOUND}) AND (NOT "${CMAKE_SYSTEM}" MATCHES "Linux"))
target_link_libraries(hotconv PUBLIC ${LIBXML2_LIBRARY})