fix: Change the read_text implementation of CustomPosixPath to avoid

abaczek · abaczek · commit fed1ec8d1bb0 · 2025-11-25T09:57:18.000+01:00
hanging due to large output

Signed-off-by: Baczek, Arkadiusz &lt;arkadiusz.baczek@intel.com&gt;
diff --git a/mfd_connect/pathlib/path.py b/mfd_connect/pathlib/path.py
@@ -400,7 +400,8 @@ def samefile(self, other_path: CustomPath) -> bool:  # noqa:D102
         )
 
     def read_text(self, encoding: Optional[str] = None, errors: Optional["Iterable"] = None) -> str:  # noqa:D102
-        return self._owner.execute_command(f"cat {self}", expected_return_codes=None).stdout
+        proc = self._owner.start_process(f"cat {self}")
+        return "".join(chunk for chunk in proc.get_stdout_iter())
 
     def touch(self, mode: int = 0o666, exist_ok: bool = True) -> None:  # noqa:D102
         if not exist_ok and self.is_file():

Original file line number	Diff line number	Diff line change
`@@ -400,7 +400,8 @@ def samefile(self, other_path: CustomPath) -> bool: # noqa:D102`
`400`	`400`	`)`
`401`	`401`
`402`	`402`	`def read_text(self, encoding: Optional[str] = None, errors: Optional["Iterable"] = None) -> str: # noqa:D102`
`403`		`- return self._owner.execute_command(f"cat {self}", expected_return_codes=None).stdout`
	`403`	`+ proc = self._owner.start_process(f"cat {self}")`
	`404`	`+ return "".join(chunk for chunk in proc.get_stdout_iter())`
`404`	`405`
`405`	`406`	`def touch(self, mode: int = 0o666, exist_ok: bool = True) -> None: # noqa:D102`
`406`	`407`	`if not exist_ok and self.is_file():`