Skip to content

Commit

Permalink
Build msodbc drivers into arm64 images (#190)
Browse files Browse the repository at this point in the history
## Description
msodbc18 can now be installed on ARM64 (eg Mac M1) platforms so we can
remove the if block in the Dockerfile to exclude this process for those
platforms.


## Related Issues
#184 

## Additional Notes
I'm not sure if we have a good way to test the success of this change
but if I followed the Teams conversation correctly, we have confirmed
that NBS was able to get this working.
  • Loading branch information
m-goggins authored Feb 4, 2025
1 parent 8ea5fda commit 04578b2
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,7 @@ RUN apt-get update && apt-get upgrade -y && apt-get install curl -y
RUN pip install --upgrade pip

# Conditionally install ODBC driver for SQL Server.
# There is no ODBC driver for linux/arm64 architecture, so SQL Server support
# is limited to linux/amd64 architecture
RUN if [ "$USE_MSSQL" = "true" ] && [ "$(dpkg --print-architecture)" = "amd64" ]; then \
RUN if [ "$USE_MSSQL" = "true" ]; then \
apt-get install -y gnupg2 apt-transport-https && \
curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor -o /etc/apt/trusted.gpg.d/microsoft.gpg && \
curl https://packages.microsoft.com/config/debian/11/prod.list | tee /etc/apt/sources.list.d/mssql-release.list && \
Expand Down

0 comments on commit 04578b2

Please sign in to comment.