Skip to content

Commit cbdb1f4

Browse files
fix(ci): guard setNativeArguments behind Q_OS_WIN
QProcess::setNativeArguments is Windows-only API. Wrap with #ifdef so Linux and macOS builds use setArguments unconditionally. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 1dcd368 commit cbdb1f4

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/AcpConnection.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -181,12 +181,12 @@ void AcpConnection::spawn(const AcpAgentDefinition &agent, const QString &workin
181181
#endif
182182

183183
m_process->setProgram(argv.program);
184+
#ifdef Q_OS_WIN
184185
if (!argv.nativeArgumentsLine.isEmpty()) {
185-
// Windows-only path. The CMD `/D /S /C "<line>"` form must be passed
186-
// verbatim — QProcess::setArguments would re-quote our hand-built
187-
// command line and break paths with spaces.
188186
m_process->setNativeArguments(argv.nativeArgumentsLine);
189-
} else {
187+
} else
188+
#endif
189+
{
190190
m_process->setArguments(argv.arguments);
191191
}
192192

0 commit comments

Comments
 (0)