luke/yosys
1
0
Fork 0
mirror of https://github.com/YosysHQ/yosys.git synced 2026-10-06 10:03:36 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6265 from SimonEbner/fix-run-command-eintr

Browse Source 
handle EINTR and signal termination in run_command
This commit is contained in:
Emil J
2026-09-30 16:10:35 +00:00
committed by GitHub
parent c09944d8ee aa6108d70f
commit 5b4a2edb8a
1 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
kernel/yosys.cc
+13 -4
View File
@@ -192,10 +192,17 @@ int run_command(const std::string &command, std::function<void(const std::string
std::string line;
char logbuf[128];
while (fgets(logbuf, 128, f) != NULL) {
line += logbuf;
if (!line.empty() && line.back() == '\n')
process_line(line), line.clear();
while (true) {
if (fgets(logbuf, 128, f) != NULL) {
line += logbuf;
if (!line.empty() && line.back() == '\n')
process_line(line), line.clear();
} else if (ferror(f) && errno == EINTR) {
clearerr(f);
continue;
} else {
break;
}
}
if (!line.empty())
process_line(line);
@@ -206,6 +213,8 @@ int run_command(const std::string &command, std::function<void(const std::string
#ifdef _WIN32
return ret;
#else
if (!WIFEXITED(ret))
return -1;
return WEXITSTATUS(ret);
#endif
}