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

Handle -R in mysql_config output #54

Merged
merged 2 commits into from
Aug 16, 2024
Merged

Conversation

citrus-it
Copy link
Contributor

When building on OmniOS, the build panics with:

  --- stderr
  thread 'main' panicked at /data/omnios-build/.cargo/registry/src/index.crates.io-6f17d22bba15001f/mysqlclient-sys-0.4.0/build.rs:80:17:
  Unexpected output from mysql_config: `-L/opt/ooce/mariadb-10.7/lib/amd64 -R/opt/ooce/mariadb-10.7/lib/amd64 -lmariadb -lnsl -lsocket -ldl -lm -lssl -lcrypto -lz`

This is because the output of mysql_config includes a -R link option because the libraries are in a non-standard location. This patch handles converting -R to an appropriate option.

@weiznich weiznich force-pushed the illumos branch 2 times, most recently from 30e8367 to 77a6e82 Compare August 16, 2024 11:54
@weiznich
Copy link
Collaborator

Thanks 👍

@weiznich weiznich merged commit f44157d into sgrif:master Aug 16, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants