Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

"Undefined reference to" Error while linking object files [duplicate]

Tags:

c++

c

gcc

g++

linker

I realize this question has been asked in a number of ways including this very comprehensive answer but I have looked at many and tried fixing my error to no avail.

I am using .cpp and .c files to create a program. I compiled all files with g++, it seemed to have no more linking errors, though it gave me a bunch of C++ errors related to the C syntax. This was the command I used:

g++ -o program main.cpp /data/*.c -l[...libs]

The main.cpp calls functions in the .c files. I then understood that one should not try to compile both .c and .cpp files with one compiler, instead to use gcc for one, g++ for the other, and thereafter to simply link the object files.

So I did this (the .c files are part of a library, and already have .o files)

g++ -c main.cpp
g++ -o program main.o /data/*.o -l[..libs]

But then here I will get "undefined reference to" errors for functions called from main.cpp to the precompiled .c files, errors which I didn't get previously.

Could anyone help? Or do I maybe need to provide more information?

EDIT (a more in depth excerpt of code, I've tried to simplify otherwise this will be impossible to read, but let me know if I still need to add stuff, I'll try to remove unnecessary code):

main.cpp :

#include "header.h"

int main(int argc, char** argv) {
    string s1 = argv[2];
    fn1(s1)
}

header.h

void fn1(string s1)

mycfile.c

#include "header.h"

void fn1(string s1){
    fprintf(stdout, " you entered %s", s1);
    fflush(stdout);
}

ANSWER:

@Smeehey helped me figure out the solution, it was actually that I was still including the old headers in an .hpp file I was using. But the core solution was indeed to use the external C{}.

like image 632
Victor.dMdB Avatar asked Aug 04 '16 07:08

Victor.dMdB


2 Answers

This is highly likely to do with C-vs-C++ linkage. In your main.cpp, you probably have something like this:

#include <data/header.h>

where header.h refers to your c library. Replace it as follows:

extern "C" {
#include <data/header.h>
}

This tells your c++ compiler not to use c++-style name mangling when defining the required symbols from the header, allowing the linker to successfully find them in the c-compiled .o files.

like image 163
Smeeheey Avatar answered Oct 06 '22 00:10

Smeeheey


You have to compile C files with the gcc command, C++ files with either gcc or g++, and link with the g++ command. This sequence will probably work:

 gcc -c data/*.c main.cpp
 g++ -o program *.o -l <libs...>

Next step: learn to write proper makefiles.

like image 39
n. 1.8e9-where's-my-share m. Avatar answered Oct 06 '22 00:10

n. 1.8e9-where's-my-share m.