From e85a84b8a39059734483a992b0953e7376a61190 Mon Sep 17 00:00:00 2001 From: Marek Roszko Date: Mon, 14 Nov 2022 07:31:23 -0500 Subject: [PATCH] Fix the step export for now Need to fix argparse to handle single quotes and double quotes better Fixes https://gitlab.com/kicad/code/kicad/-/issues/12899 --- pcbnew/dialogs/dialog_export_step.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pcbnew/dialogs/dialog_export_step.cpp b/pcbnew/dialogs/dialog_export_step.cpp index 7fa11dcc12..52cb767169 100644 --- a/pcbnew/dialogs/dialog_export_step.cpp +++ b/pcbnew/dialogs/dialog_export_step.cpp @@ -396,6 +396,7 @@ void DIALOG_EXPORT_STEP::onExportButton( wxCommandEvent& aEvent ) // Note: for some reason, using \" to insert a quote in a format string, under MacOS // wxString::Format does not work. So use a %c format in string int quote = '\''; + int dblquote = '"'; switch( GetOriginOption() ) { @@ -450,11 +451,11 @@ void DIALOG_EXPORT_STEP::onExportButton( wxCommandEvent& aEvent ) // Input file path. cmdK2S.Append( wxString::Format( wxT( " -f -o %c%s%c" ), - quote, m_filePickerSTEP->GetPath(), quote ) ); + dblquote, m_filePickerSTEP->GetPath(), dblquote ) ); // Output file path. - cmdK2S.Append( wxString::Format( wxT( " %c%s%c" ), quote, m_boardPath, quote ) ); + cmdK2S.Append( wxString::Format( wxT( " %c%s%c" ), dblquote, m_boardPath, dblquote ) ); wxLogTrace( traceKiCad2Step, wxT( "export step command: %s" ), cmdK2S );