rlarranaga / bafprp

Automatically exported from code.google.com/p/bafprp
0 stars 0 forks source link

Errors compiling with GCC 4.3.3 #5

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Download source
2. $ make

What is the expected output? What do you see instead?
Program doesn't compile and several errors appear

I had the following issues:
> error: ‘memset’ was not declared in this scope
    on the following files:
        * src/baffile.cpp

    Solved adding to those files:
        #include <cstring>

> error: ‘strlen’ was not declared in this scope
    on the following files:
        *src/bafprp.cpp

    Solved adding to those files:
        #include <cstring>

> error: ‘atoi’ was not declared in this scope
    on the following files:
        * src/emailoutput.o
        * src/duration_dt.cpp
        * src/number_dt.cpp
        * src/number_switch_dt.cpp

    Solved adding to those files:
        #include <cstdlib>

> error: ‘strchr’ was not declared in this scope
    on the following files:
        * src/mysqloutput.cpp

    Solved adding to those files:
        #include <cstring>

What version of the product are you using? On what operating system?
Source: bafprp_1_2_1_source.zip
OS: Debian
Compiler:
 GNU C++ (Debian 4.3.3-10) version 4.3.3 (i486-linux-gnu)
    compiled by GNU C version 4.3.3, GMP version 4.2.4, MPFR version 2.4.1-p2.
 GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
 GNU assembler version 2.19.1 (i486-linux-gnu) using BFD version (GNU
Binutils for Debian) 2.19.1

Please provide any additional information below.

Solution is in issue description but no idea how to create a patch (though
it's just adding 1 line to 9 files)

Original issue reported on code.google.com by matiasbe...@gmail.com on 19 Jun 2009 at 4:30

GoogleCodeExporter commented 8 years ago
Forgot about these ones

> error: ‘strcopy’ was not declared in this scope
    on the following files:
        * src/mailer.cpp

    Solved adding to those files:
        #include <cstring>

> error: ‘atoi’ was not declared in this scope
    on the following files:
        * src/phonenumber_dt.cpp

    Solved adding to those files:
        #include <cstdlib>

Original comment by matiasbe...@gmail.com on 20 Jun 2009 at 6:59

GoogleCodeExporter commented 8 years ago
Ok, thanks for pointing these out.  I will make sure the next version compiles
correctly. :)

Original comment by charless...@gmail.com on 20 Jun 2009 at 7:27

GoogleCodeExporter commented 8 years ago
Added the include files, need to test still.

Original comment by charless...@gmail.com on 23 Jul 2009 at 3:32

GoogleCodeExporter commented 8 years ago
These are still occurring: 

> error: ‘memset’ was not declared in this scope
    on the following files:
        * src/baffile.cpp

    Solved adding to those files:
        #include <cstring>

> error: ‘atoi’ was not declared in this scope
    on the following files:
        * src/emailoutput.o
        * src/duration_dt.cpp
        * src/number_dt.cpp
        * src/phonenumber_dt.cpp

    Solved adding to those files:
        #include <cstdlib> (i think that all of those files include "ifield.h")

> error: ‘strcopy’ was not declared in this scope
    on the following files:
        * src/mailer.cpp

    Solved adding to those files:
        #include <cstring>

Original comment by matiasbe...@gmail.com on 26 Jul 2009 at 8:58

GoogleCodeExporter commented 8 years ago
I fixed the header files in the latest trunk revision.
I just tried compiling with GCC 4.3.3-13
GNU C++ (Debian 4.3.3-13) version 4.3.3 (i486-linux-gnu)
and it worked fine.  

If you are still having problems with the latest revision let me know

Original comment by charless...@gmail.com on 26 Jul 2009 at 10:40

GoogleCodeExporter commented 8 years ago
My bad... I thought that the latest zipped file had that changeset too so 
that's why
I updated this.

I just did an checkout of the trunk and it compiled perfectly.

Original comment by matiasbe...@gmail.com on 26 Jul 2009 at 11:29

GoogleCodeExporter commented 8 years ago
no problem, thanks for double checking

Original comment by charless...@gmail.com on 26 Jul 2009 at 11:34

GoogleCodeExporter commented 8 years ago

Original comment by charless...@gmail.com on 17 Nov 2009 at 4:38