koji

#4037 Fix temporary cg_import.log file path
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue4036  into  master

Download 4037.patch

Related: https://pagure.io/koji/issue/4036

 
-            self._cg_log_path = joinpath(log_dir, 'cg_import.log')
-            self._cg_log_file = open(self._cg_log_path, mode='wt')
+            fd, fpath = tempfile.mkstemp(dir=log_dir, suffix="-cg_import.log", text=True)
+            self._cg_log_path = fpath
+            self._cg_log_file = fd

mkstemp returns an os level file handle, not python level open file.

rebased onto b349f2abd3e2e1d9c05bc90255fdc537e28fbcf5

:thumbsup:

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

Commit 3d6b1090 fixes this pull-request

Pull-Request has been merged by tkopecek
Metadata
None

testing-ready

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.