Skip to content

Commit

Permalink
Merge pull request #6146 from rouault/FindOracle_libnnz19
Browse files Browse the repository at this point in the history 
FindOracle.cmake: add support for OCI 19
  • Loading branch information
@rouault
rouault committed Sep 24, 2020
2 parents 5f09279 + 5eedd4d commit 39adc75
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmake/FindOracle.cmake
View file
Expand Up @@ -37,7 +37,7 @@ if(DEFINED ENV{ORACLE_HOME})
${ORACLE_HOME}/OCI/include) # Oracle XE on Windows

set(ORACLE_OCI_NAMES clntsh libclntsh oci)
set(ORACLE_NNZ_NAMES nnz10 libnnz10 nnz11 libnnz11 nnz12 libnnz12 nnz18 libnnz18 ociw32)
set(ORACLE_NNZ_NAMES nnz10 libnnz10 nnz11 libnnz11 nnz12 libnnz12 nnz18 libnnz18 ociw32 libnnz19)
set(ORACLE_OCCI_NAMES libocci occi oraocci10 oraocci11 oraocci12)

set(ORACLE_LIB_DIR
Expand Down

4 comments on commit 39adc75

@kurotych
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget to add nnz19

@rouault
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget to add nnz19

there's only libnnz19.so in instantclient-basic-linux.x64-19.8.0.0.0dbru.zip. What would be "nnz19" : a Windows .lib ?

@kurotych
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget to add nnz19

there's only libnnz19.so in instantclient-basic-linux.x64-19.8.0.0.0dbru.zip. What would be "nnz19" : a Windows .lib ?

Definitely not Windows.
I came here from another project just looking for a solution.

We use Oracle LInux and oracle-instantclient19.8-devel/oracle-instantclient19.8-sqlplus.
And only nnz19 or libnnz19.so helps me to find and link the new library.

@rouault
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nnz19 added per 9bd402c

Please sign in to comment.