forked from apache/arrow
-
Notifications
You must be signed in to change notification settings - Fork 0
[Paused] Enable ODBC build in macOS Intel CI #138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
alinaliBQ
wants to merge
27
commits into
apache-odbc
Choose a base branch
from
enable-macos-ci
base: apache-odbc
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2575182 to
2671b6d
Compare
36282dd to
fb25307
Compare
e6dfb52 to
7df62c8
Compare
TEMP - disable Windows CI to reduce work load
macOS 14 is on ARM. ODBC cannot build on ARM mac 14 yet.
fb25307 to
ccc79ef
Compare
Updated readme. The build works locally on my Intel mac machine.
8b38903 to
da2e4f5
Compare
On macOS, sqltypes.h defines `BOOL` which causes a conflict on macOS
This reverts commit de22953.
Add explicit `undef` commands in .cc files where `type::BOOL` is used.
Move include of odbc_impl header to after utf8.
9bbdae7 to
b73c118
Compare
Push local changes for a copy online, will likely remove the script from this PR later.
unixodbc is not needed for building ODBC.
Use command to find actual libiodbc path
Getting errors
```
In file included from /usr/local/include/sqlucode.h:784:
/usr/local/include/sqlext.h:1790:19: error: conflicting types for 'SQLDriverConnectW'
1790 | SQLRETURN SQL_API SQLDriverConnect(
| ^
/usr/local/include/sqlucode.h:765:26: note: expanded from macro 'SQLDriverConnect'
765 | #define SQLDriverConnect SQLDriverConnectW
| ^
/usr/local/include/sqlucode.h:257:19: note: previous declaration is here
257 | SQLRETURN SQL_API SQLDriverConnectW(
| ^
```
trying to see if switching to ARM macOS will fix the issue.
9fece32 to
d677087
Compare
4375cc9 to
421aa00
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Enable Flight SQL ODBC build in macOS CI. Note that Flight SQL is also enabled as a result.