Thread: string comparision

Hi

I'm new programming in C and need help with a piece of code I need.

what I'm trying to do it to get a program to auto update, it will check for new version and if found it will download the new version and remove the old one.

here is the code

Code:

        system("cd /tmp ; wget http://xxx.com/smversion.data");
        FILE*dp ;
        char *fname;
        char c[20];
        dp = fopen ( "/tmp/smversion.data","r" ) ;
        fname=argv[0];
        fgets(c, 20, dp);
        if (strcmp(c, fname) != 0 )
        {       system("v=`cat /tmp/smversion.data`; wget http://xxx.com/$v; chmod +x $v");
                system("rm -rf /tmp/smversion.data");
                remove(fname);
                
        }
        else { execute the rest....... }

the name of the binary is program-v1.0 the content of smversion.data contains the new version, lets say (program-v1.1) so the first time run it works ok.... it will download version 1.1 and remove 1.0; however, if I run 1.1 it will download v1,1 again so the problem is that
if (strcmp(c, fname) != 0 ) is not working ok... I have tried to prinf the value of c and fname
and both are the same so have no idea whats going on

any help or any different idea how to accomplish this would be really appreciated

thx

PS: I use system() since I have to deal with bash commands

Originally Posted by ulises_FITX

Hi

I'm new programming in C and need help with a piece of code I need.

what I'm trying to do it to get a program to auto update, it will check for new version and if found it will download the new version and remove the old one.

here is the code

Code:

        system("cd /tmp ; wget http://xxx.com/smversion.data");
        FILE*dp ;
        char *fname;
        char c[20];
        dp = fopen ( "/tmp/smversion.data","r" ) ;
        fname=argv[0];
        fgets(c, 20, dp);
        if (strcmp(c, fname) != 0 )
        {       system("v=`cat /tmp/smversion.data`; wget http://xxx.com/$v; chmod +x $v");
                system("rm -rf /tmp/smversion.data");
                remove(fname);
                
        }
        else { execute the rest....... }

the name of the binary is program-v1.0 the content of smversion.data contains the new version, lets say (program-v1.1) so the first time run it works ok.... it will download version 1.1 and remove 1.0; however, if I run 1.1 it will download v1,1 again so the problem is that
if (strcmp(c, fname) != 0 ) is not working ok... I have tried to prinf the value of c and fname
and both are the same so have no idea whats going on

any help or any different idea how to accomplish this would be really appreciated

thx

PS: I use system() since I have to deal with bash commands

It runs once because you open the data file it needs, and it loads up the correct data. Now the next time you run it, it again opens the file, AND STARTS READING RIGHT FROM THE FRONT OF THE FILE AGAIN, and now your code is working from the wrong set of data.

You code is right, except (I believe), you need to either keep reading from the data file until you reach the right name, or you need to modify your program so somehow it knows "don't take that first file name, this is the second time, so take the second filename."

You may need for your program to write out a very small one integer data file of it's own, so it will know how many filenames to read through before it reaches the right one.

Adak

thx for the reply.... but the second time I run the new/downloaded program (program-v1.1)
if (strcmp(c, fname) != 0 ) should give = 0 so it should jump to the 'else' part....

beacuase c would be the same file which is program-v.1.1 and argv[0] (the name of the program) will be program-v1.1 --- am I missing something? why is taking like strcpm != 0 ?

I guess this is why computer scientists stick to using foo and bar instead of other symbols.

I am aware that you are new to 'C', but you should get a good book on 'C' which lists the standard functions that 'C' is bundled with it. It is not going to be easy at first to figure out how to do things in 'C', but the the things you are doing with the system command can be done in 'C'...the cat command is just reading a file. Look up the fopen, fread, fgets, fclose commands for that. 'C' has the equivalent of chmod as well. Another thing you should do is to ALWAYS check the return code of commands such as fopen, remove and chmod. This will save you from alot of headaches in trying to figure out why things go wrong. Otherwise, if you are not going to take advantage of all the 'C' functions, you should just write it in 'bash' like Salem says.

Good Luck
Sam

Originally Posted by OnionKnight

I guess this is why computer scientists stick to using foo and bar instead of other symbols.

You're SUPPOSED to use "example.com" when writing examples. It's a reserved domain (along with example.net, example.org, example.anything, really) especially for this purpose.

yeah I know it can be done on bash, in fact; I have already written the script but now I need a binary so I thought re-write it on C.... also, that is just a piece of code.... the whole thing is more than 500 lines and it works ok but now I need to get auto update incase I do any changes on the binary, all the servers can get the new binary automatically.

I have read already on fopen, fgets and fread -- I used them on that piece of code... I know I wont learn C in 3 days but my frustration is on why 'if (strcmp(c, fname) != 0 )' aint working if the strings are the same???

PS: lol... xxx.com was a baaaaaad example.... sorry about that