Skip to content

Commit c1ce967

Browse files
committed
fix(pathfinder): simplify loader review comments
1 parent 32596ce commit c1ce967

1 file changed

Lines changed: 1 addition & 6 deletions

File tree

cuda_pathfinder/cuda/pathfinder/_dynamic_libs/load_dl_linux.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -164,20 +164,15 @@ def load_with_system_search(desc: LibDescriptor) -> LoadedDL | None:
164164
Returns:
165165
A LoadedDL object if successful, None if the library cannot be loaded
166166
167-
Raises:
168-
OSError: If the library is loaded but its absolute path cannot be
169-
resolved via dlinfo (surfaced from abs_path_for_dynamic_library).
170167
"""
171168
for soname in _candidate_sonames(desc):
172169
try:
173170
handle = _load_lib(desc, soname)
174171
except OSError:
175172
pass
176173
else:
177-
# abs_path_for_dynamic_library never returns None: it returns a
178-
# resolved path or raises OSError. Let that error surface rather
179-
# than masking it, consistent with the deterministic-loader policy.
180174
abs_path = abs_path_for_dynamic_library(desc.name, handle)
175+
assert abs_path
181176
return LoadedDL(abs_path, False, handle._handle, "system-search")
182177
return None
183178

0 commit comments

Comments
 (0)