Skip to content

Commit 5794199

Browse files
committed
fully disable line tracing
1 parent 2fa2e53 commit 5794199

17 files changed

+38
-20
lines changed

.github/workflows/coverage.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
3030
# for cython tests inplace installation is required
3131
- name: build
32-
run: CXXFLAGS=-DCYTHON_TRACE_NOGIL=1 pip install -e . -v
32+
run: CMAKE_ARGS=-DCYTHON_TRACE_NOGIL=1 pip install -e . -v
3333

3434
- name: Test with pytest and generate coverage info
3535
run: |

CHANGELOG.rst

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,12 @@
11
Changelog
22
---------
33

4+
[3.14.1] - 2025-09-08
5+
^^^^^^^^^^^^^^^^^^^^^
6+
Fixed
7+
~~~~~
8+
* Fully disable line tracing in release builds
9+
410
[3.14.0] - 2025-08-27
511
^^^^^^^^^^^^^^^^^^^^^
612
Changed

src/rapidfuzz/CMakeLists.txt

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,20 @@ function(create_cython_target _name)
44
${CMAKE_CURRENT_LIST_DIR}/${_name}.cxx
55
PARENT_SCOPE)
66
else()
7+
set(cython_linetrace_options "")
8+
if(CYTHON_TRACE OR CYTHON_TRACE_NOGIL)
9+
set(cython_linetrace_options "-X" "linetrace=True")
10+
endif()
11+
712
add_custom_command(
813
OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx"
914
MAIN_DEPENDENCY "${CMAKE_CURRENT_LIST_DIR}/${_name}.pyx"
1015
VERBATIM
1116
COMMAND
12-
Python::Interpreter -m cython "${CMAKE_CURRENT_LIST_DIR}/${_name}.pyx"
13-
--cplus -I "${CMAKE_CURRENT_LIST_DIR}" --output-file
14-
"${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx")
17+
Python::Interpreter -m cython ${cython_linetrace_options}
18+
"${CMAKE_CURRENT_LIST_DIR}/${_name}.pyx" --cplus
19+
-I "${CMAKE_CURRENT_LIST_DIR}"
20+
--output-file "${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx")
1521

1622
set(${_name}
1723
${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx

src/rapidfuzz/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
__author__: str = "Max Bachmann"
88
__license__: str = "MIT"
9-
__version__: str = "3.14.0"
9+
__version__: str = "3.14.1"
1010

1111
from rapidfuzz import distance, fuzz, process, utils
1212

src/rapidfuzz/_feature_detector_cpp.pyx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# distutils: language=c++
2-
# cython: language_level=3, binding=True, linetrace=True
2+
# cython: language_level=3, binding=True
33

44
from libc.stdint cimport uint32_t
55

src/rapidfuzz/cpp_common.pxd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# distutils: language=c++
2-
# cython: language_level=3, binding=True, linetrace=True
2+
# cython: language_level=3, binding=True
33

44
from cpython.object cimport PyObject
55
from cpython.pycapsule cimport PyCapsule_GetPointer, PyCapsule_IsValid, PyCapsule_New

src/rapidfuzz/distance/CMakeLists.txt

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,20 @@ function(create_cython_target _name)
44
${CMAKE_CURRENT_LIST_DIR}/${_name}.cxx
55
PARENT_SCOPE)
66
else()
7+
set(cython_linetrace_options "")
8+
if(CYTHON_TRACE OR CYTHON_TRACE_NOGIL)
9+
set(cython_linetrace_options "-X" "linetrace=True")
10+
endif()
11+
712
add_custom_command(
813
OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx"
914
MAIN_DEPENDENCY "${CMAKE_CURRENT_LIST_DIR}/${_name}.pyx"
1015
VERBATIM
1116
COMMAND
12-
Python::Interpreter -m cython "${CMAKE_CURRENT_LIST_DIR}/${_name}.pyx"
13-
--cplus -I "${CMAKE_CURRENT_LIST_DIR}/.." --output-file
14-
"${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx")
17+
Python::Interpreter -m cython ${cython_linetrace_options}
18+
"${CMAKE_CURRENT_LIST_DIR}/${_name}.pyx" --cplus
19+
-I "${CMAKE_CURRENT_LIST_DIR}/.."
20+
--output-file "${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx")
1521

1622
set(${_name}
1723
${CMAKE_CURRENT_BINARY_DIR}/${_name}.cxx

src/rapidfuzz/distance/_initialize_cpp.pxd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# distutils: language=c++
2-
# cython: language_level=3, binding=True, linetrace=True
2+
# cython: language_level=3, binding=True
33

44
from cpp_common cimport RfEditops, RfOpcodes
55
from libcpp cimport bool

src/rapidfuzz/distance/_initialize_cpp.pyx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# distutils: language=c++
2-
# cython: language_level=3, binding=True, linetrace=True
2+
# cython: language_level=3, binding=True
33

44
from cpp_common cimport EditType, RfEditOp, RfOpcode, convert_string, is_valid_string
55
from cpython.list cimport PyList_New, PyList_SET_ITEM

src/rapidfuzz/distance/metrics_cpp.pyx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# distutils: language=c++
2-
# cython: language_level=3, binding=True, linetrace=True
2+
# cython: language_level=3, binding=True
33

44
from . import metrics_py
55
from ._initialize_cpp import Editops

0 commit comments

Comments
 (0)