-
-
Notifications
You must be signed in to change notification settings - Fork 7
Image building fails #35
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
Comments
To be clear: The failing build instructions are in |
You are looking at the wrong repo, the correct one is https://packages.microsoft.com/debian/11/prod/pool/main/u/unixodbc/ (I see there is a newer repo for bookworm https://packages.microsoft.com/debian/12, but I'm not an expert in mssql.) Try -ARG odbc_version=2.3.7
+ARG odbc_version=2.3.11 Or any other version that has amd64 and arm64 packages available. |
Oh my… but there it should have 2.3.7. then. 🙄 Anyway, with 2.3.11 I do get further, indeed. And then:
Even though https://packages.microsoft.com/debian/11/prod/pool/main/m/mssql-tools/ is a thing. But with an added -RUN ACCEPT_EULA=Y DEBIAN_FRONTEND=noninteractive apt-get install -y msodbcsql18 mssql-tools
+RUN ACCEPT_EULA=Y DEBIAN_FRONTEND=noninteractive apt-get install -y msodbcsql18 mssql-tools18 it starts. 🎉 |
If this change works on amd64 as well (we'll see it in the tests), that's great news. |
Let's hope it does. 😇 |
Describe the bug
To Reproduce
With an empty directory, run
ddev config
, then add the addon, runddev start
This is what I got:
Now I install the add-on, my branch is from
main
and just allows to install on Apple Silicon, no other changes:Ok, now I start:
As you can see
2.3.7
is not a version available for:This is unchanged in my branch, so I assume something has changed for the PHP 8.3 image used as a base here?
Expected behavior
It should build and start without an error message. 😎
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: