Skip to content
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

uwsgi: fix php plugin build #33420

Closed
wants to merge 1 commit into from
Closed

Conversation

schneefux
Copy link
Contributor

Motivation for this change

Workaround for #33400.

It does not compile with mysql.connector-c which was introduced in #30546.
It compiles with config.php.mysqlnd = true, but fails at runtime with "failed to connect to the database - an exception occured in driver - no such file or directory" (tested with Nextcloud).

Can someone with more C/Nix experience help me update the plugin the right way? I do not really understand what is happening and why it fails without the direct mysql dependency.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@globin
Copy link
Member

globin commented Jan 4, 2018

I'm checking this.

@globin globin closed this in 42af265 Jan 4, 2018
@schneefux
Copy link
Contributor Author

Follow-up on the "failed to connect to the database" error after mysqlnd is now the default (#44117): The solution is to set the host to 127.0.0.1 instead of localhost. This is apparently a Symfony (the php framework) issue because other applications like gogs work fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants