Actions
Forum Thread Details
  • Replies 3 replies
  • Subscribers 176 subscribers
  • Views 980 views
  • Users 0 members are here
Related

system("cmd.exe /c path..."); fails if path include white space, even quoted

Morosh

Hello:
I made a small ulp which invoke eaglecon from inside eagle to get eps drawing, using system("cmd.exe /c path\eaglecon etc.....");
I noticed that this command fails if the path even quoted include space (in my case "C:\EAGLE-7.7.0 x32\bin\eaglecon")

I got: Windows cannot fond '-X', Make sure you typed the name correctly and then try again.

my ulp:
string cmd,fname,bname,path,path2,rot,quick;
int i;

string convert(string dumy)
{
string dumy2,ch;
int i;

rot="";
quick="";

dumy2="";
for(i=0;i<strlen(dumy);i++) {
   ch=strsub(dumy,i,1);
   if(ch=="/") ch="\\";
   dumy2=dumy2+ch;
   }
return dumy2;
}

path=path_doc[0];
path="\""+strsub(path,0,strlen(path)-4)+"\\bin\\eaglecon"+"\"";
path=convert(path);


if(board){
   board(B){
      bname=B.name;;
      //dlgMessageBox("bname1="+bname);
      bname=convert(bname);
      //dlgMessageBox("bname2="+bname);
      fname=bname;
      fname[strlen(fname)-4]=0;
      for(i=1;i< argc; i++) {
         if(argv[i]=="rot") rot=" -r ";
         else if(argv[i]=="quick") quick=" -q+ ";
         else if(argv[i]=="sol") {
            cmd="cmd.exe /c start echo " + path + rot+quick+ " -X -f- -deps -o\""+fname+"_sol.eps\" \""+bname+"\" 16 17 18 20";
            system(cmd);
            cmd="cmd.exe /c start " + path + rot+quick+ " -X -f- -deps -o\""+fname+"_sol.eps\" \""+bname+"\" 16 17 18 20";
            system(cmd);
            }
         }

      //dlgMessageBox(cmd);
      }
   }

I've added "echo" for verification, typing the echoed string in a command window succeed.
Everything worked OK if I remove the space in "C:\EAGLE-7.7.0 x32", like "C:\EAGLE-7.7.0x32"


is there a way to resolve that?

Thanks




--
Cet email a fait l'objet d'une analyse antivirus par AVG.
http://www.avg.com

 

Parents

  • Thanks all for reply.

    Finally it wasn't a problem of white space, I removed it and the problem persists mysteriously (although it seems to me that it works once earlier).

    So I changed my ulp and now I create a batch file, then I run it and it's working good:

    here it's if anyone is interested:

     

    string convert(string dumy)

    {

    string dumy2,ch;

    int i;

     

    dumy2="";

    for(i=0;i<strlen(dumy);i++) {

       ch=strsub(dumy,i,1);

       if(ch=="/") ch="\\";

       dumy2=dumy2+ch;

       }

    return dumy2;

    }

     

    string fname,bname,path,rot,quick;

    int i;

     

    rot="";

    quick="";

     

    for(i=1;i< argc; i++) {

       if(argv[i]=="rot") rot=" -r ";

       if(argv[i]=="quick") quick=" -q- ";

       }

     

    path=path_doc[0];

    path="\""+strsub(path,0,strlen(path)-4)+"\\bin\\eaglecon"+"\"";

    path=convert(path);

     

    output("toto.bat"){

     

       if(board){

          board(B){

             bname=B.name;;

             bname=convert(bname);

             fname=bname;

             fname[strlen(fname)-4]=0;

             for(i=1;i< argc; i++) {

                if(argv[i]=="sol") {

                   printf("%s%s%s -X -f- -deps -o\"%s_sol.eps\" \"%s\" 16 17 18 20\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="cmp") {

                   printf("%s%s%s -X -f- -deps -o\"%s_cmp.eps\" \"%s\" 1 17 18 20\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="pls") {

                   printf("%s%s%s -X -f- -deps -o\"%s_pls.eps\" \"%s\" 20 22 26\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="plc") {

                   printf("%s%s%s -X -f- -deps -o\"%s_plc.eps\" \"%s\" 20 21 25\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="vls") {

                   printf("%s%s%s -X -f- -deps -o\"%s_vls.eps\" \"%s\" 20 22 28\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="vlc") {

                   printf("%s%s%s -X -f- -deps -o\"%s_vlc.eps\" \"%s\" 20 21 27\n", path,rot,quick,fname,bname);

                   }

                }  //for(i=1;i< argc; i++)

                printf("del toto.bat\n");

             }  //board(B){

          }  //if(board)

       }  //output("toto.bat")

    system("cmd.exe /c toto.bat");

     

    I'm using this way also to generate gerber files, I found it's more comfortable than cam processor

     

    Regards

Reply

  • Thanks all for reply.

    Finally it wasn't a problem of white space, I removed it and the problem persists mysteriously (although it seems to me that it works once earlier).

    So I changed my ulp and now I create a batch file, then I run it and it's working good:

    here it's if anyone is interested:

     

    string convert(string dumy)

    {

    string dumy2,ch;

    int i;

     

    dumy2="";

    for(i=0;i<strlen(dumy);i++) {

       ch=strsub(dumy,i,1);

       if(ch=="/") ch="\\";

       dumy2=dumy2+ch;

       }

    return dumy2;

    }

     

    string fname,bname,path,rot,quick;

    int i;

     

    rot="";

    quick="";

     

    for(i=1;i< argc; i++) {

       if(argv[i]=="rot") rot=" -r ";

       if(argv[i]=="quick") quick=" -q- ";

       }

     

    path=path_doc[0];

    path="\""+strsub(path,0,strlen(path)-4)+"\\bin\\eaglecon"+"\"";

    path=convert(path);

     

    output("toto.bat"){

     

       if(board){

          board(B){

             bname=B.name;;

             bname=convert(bname);

             fname=bname;

             fname[strlen(fname)-4]=0;

             for(i=1;i< argc; i++) {

                if(argv[i]=="sol") {

                   printf("%s%s%s -X -f- -deps -o\"%s_sol.eps\" \"%s\" 16 17 18 20\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="cmp") {

                   printf("%s%s%s -X -f- -deps -o\"%s_cmp.eps\" \"%s\" 1 17 18 20\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="pls") {

                   printf("%s%s%s -X -f- -deps -o\"%s_pls.eps\" \"%s\" 20 22 26\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="plc") {

                   printf("%s%s%s -X -f- -deps -o\"%s_plc.eps\" \"%s\" 20 21 25\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="vls") {

                   printf("%s%s%s -X -f- -deps -o\"%s_vls.eps\" \"%s\" 20 22 28\n", path,rot,quick,fname,bname);

                   }

                else if (argv[i]=="vlc") {

                   printf("%s%s%s -X -f- -deps -o\"%s_vlc.eps\" \"%s\" 20 21 27\n", path,rot,quick,fname,bname);

                   }

                }  //for(i=1;i< argc; i++)

                printf("del toto.bat\n");

             }  //board(B){

          }  //if(board)

       }  //output("toto.bat")

    system("cmd.exe /c toto.bat");

     

    I'm using this way also to generate gerber files, I found it's more comfortable than cam processor

     

    Regards

Children
No Data