marcelvanherk / Conquest-DICOM-Server

Conquest DICOM server, aiming for complete source code release
112 stars 47 forks source link

error: ‘MYSQL_PORT’ was not declared in this scope #3

Closed nativecodeabuser closed 4 years ago

nativecodeabuser commented 4 years ago

Hi,

When compiling with mysql support I get:

In file included from ./src/dgate/src/total.cpp:149:
./src/dgate/src/odbci.cpp: In member function ‘virtual BOOL Database::Open(const char*, const char*, const char*, const char*)’:
./src/dgate/src/odbci.cpp:3278:15: error: ‘MYSQL_PORT’ was not declared in this scope
  { int port = MYSQL_PORT;
               ^~~~~~~~~~
./src/dgate/src/odbci.cpp:3278:15: note: suggested alternative: ‘MYSQL_STMT’
  { int port = MYSQL_PORT;
               ^~~~~~~~~~
               MYSQL_STMT

I think I have correct libraries installed.

# apt list --installed |grep mariadb

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

libmariadb-dev-compat/stable,now 1:10.3.18-0+deb10u1 amd64 [installed]
libmariadb-dev/stable,now 1:10.3.18-0+deb10u1 amd64 [installed]
libmariadb3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]
libmariadbclient-dev/stable,now 1:10.3.18-0+deb10u1 amd64 [installed]
mariadb-client-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]
mariadb-client-core-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]
mariadb-common/stable,now 1:10.3.18-0+deb10u1 all [installed,automatic]
mariadb-server-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]
mariadb-server-core-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]
mariadb-server/stable,now 1:10.3.18-0+deb10u1 all [installed]

OS is fully up to date Debian Buster.

Any ideas? Thanks...

marcelvanherk commented 4 years ago

Hi. Which versions?. Marcel

On Fri, 17 Jan 2020 1:25 pm nativecodeabuser, notifications@github.com wrote:

Hi,

When compiling with mysql support I get:

In file included from ./src/dgate/src/total.cpp:149:

./src/dgate/src/odbci.cpp: In member function ‘virtual BOOL Database::Open(const char, const char, const char, const char)’:

./src/dgate/src/odbci.cpp:3278:15: error: ‘MYSQL_PORT’ was not declared in this scope

{ int port = MYSQL_PORT;

           ^~~~~~~~~~

./src/dgate/src/odbci.cpp:3278:15: note: suggested alternative: ‘MYSQL_STMT’

{ int port = MYSQL_PORT;

           ^~~~~~~~~~

           MYSQL_STMT

I think I have correct libraries installed.

apt list --installed |grep mariadb

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

libmariadb-dev-compat/stable,now 1:10.3.18-0+deb10u1 amd64 [installed]

libmariadb-dev/stable,now 1:10.3.18-0+deb10u1 amd64 [installed]

libmariadb3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]

libmariadbclient-dev/stable,now 1:10.3.18-0+deb10u1 amd64 [installed]

mariadb-client-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]

mariadb-client-core-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]

mariadb-common/stable,now 1:10.3.18-0+deb10u1 all [installed,automatic]

mariadb-server-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]

mariadb-server-core-10.3/stable,now 1:10.3.18-0+deb10u1 amd64 [installed,automatic]

mariadb-server/stable,now 1:10.3.18-0+deb10u1 all [installed]

OS is fully up to date Debian Buster.

Any ideas? Thanks...

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/marcelvanherk/Conquest-DICOM-Server/issues/3?email_source=notifications&email_token=AAVDWJPYWIGS2NWRMDZOS3DQ6GPVLA5CNFSM4KIHRRP2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IG5P6AA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVDWJIN7TGW242CARTRDQ3Q6GPVLANCNFSM4KIHRRPQ .

nativecodeabuser commented 4 years ago

I tried 1419d1, 1419c, 150beta from .zip packages and git clone of this repo. Same error.

marcelvanherk commented 4 years ago

Ok,

so I assume this constant has been removed/renamed in the mysql header files. Can you mail me the fix. I will then change it in github.

Marcel

On Fri, Jan 17, 2020 at 1:36 PM nativecodeabuser notifications@github.com wrote:

I tried 1419d1, 1419c, 150beta from .zip packages and git clone of this repo. Same error.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/marcelvanherk/Conquest-DICOM-Server/issues/3?email_source=notifications&email_token=AAVDWJOKZJPTKEEBVJ2EZ2DQ6GQ6XA5CNFSM4KIHRRP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJHRUEA#issuecomment-575609360, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVDWJKH5UATRQTSPNM6YCDQ6GQ6XANCNFSM4KIHRRPQ .

nativecodeabuser commented 4 years ago

Unfortunately fixing this is beyond my knowledge. Suggested change breaks it further.

In file included from ./src/dgate/src/total.cpp:149:
./src/dgate/src/odbci.cpp: In member function ‘virtual BOOL Database::Open(const char*, const char*, const char*, const char*)’:
./src/dgate/src/odbci.cpp:3278:25: error: expected primary-expression before ‘;’ token
  { int port = MYSQL_STMT;
                         ^

Thanks anyway. Maybe someone will come up with a patch soon. :)

marcelvanherk commented 4 years ago

Try setting the value to 3306.

Marcel

On Fri, Jan 17, 2020 at 2:00 PM nativecodeabuser notifications@github.com wrote:

Unfortunately fixing this is beyond my knowledge. Suggested change breaks it further.

In file included from ./src/dgate/src/total.cpp:149:

./src/dgate/src/odbci.cpp: In member function ‘virtual BOOL Database::Open(const char, const char, const char, const char)’:

./src/dgate/src/odbci.cpp:3278:25: error: expected primary-expression before ‘;’ token

{ int port = MYSQL_STMT;

                     ^

Thanks anyway. Maybe someone will come up with a patch soon. :)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/marcelvanherk/Conquest-DICOM-Server/issues/3?email_source=notifications&email_token=AAVDWJO2P57QI23DVUDOKNDQ6GTWRA5CNFSM4KIHRRP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJHTKLY#issuecomment-575616303, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVDWJKQGJN3L3WSPMW6Y43Q6GTWRANCNFSM4KIHRRPQ .

marcelvanherk commented 4 years ago

I have changed the code to use constant 3306.