haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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
This commit is contained in:
Marek Roszko 2022-11-14 07:31:23 -05:00
parent daa6e84d74
commit e85a84b8a3
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
pcbnew/dialogs/dialog_export_step.cpp
View File

@ -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 // 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 // wxString::Format does not work. So use a %c format in string
int quote = '\''; int quote = '\'';
int dblquote = '"';
switch( GetOriginOption() ) switch( GetOriginOption() )
{ {
@ -450,11 +451,11 @@ void DIALOG_EXPORT_STEP::onExportButton( wxCommandEvent& aEvent )
// Input file path. // Input file path.
cmdK2S.Append( wxString::Format( wxT( " -f -o %c%s%c" ), cmdK2S.Append( wxString::Format( wxT( " -f -o %c%s%c" ),
quote, m_filePickerSTEP->GetPath(), quote ) ); dblquote, m_filePickerSTEP->GetPath(), dblquote ) );
// Output file path. // 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 ); wxLogTrace( traceKiCad2Step, wxT( "export step command: %s" ), cmdK2S );