Skip to content

Use quantlib from the system when available via pkg-config. #192

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

Merged
merged 2 commits into from
Mar 25, 2025

Conversation

Image for: Conversation
Copy link
Contributor

This changes Makevars.win to use QuantLib on Windows from the system when available via pkg-config, otherwise falling back to what has been done before. This will allow using QuantLib from Rtools - it will be available there from the next update. The package passes its checks with QuantLib in the development version of Rtools on my system.

Copy link
Owner

eddelbuettel commented Mar 25, 2025

Fabulous! Thank you so much.

Two small asks:

  • Can you add yourself to ChangeLog? Two spaces between entries in the header line, eight spaces to the '*'. Or C-x 4 a in the one true editor.
  • Is the change 'r-oldrel safe' once we switch to R 4.5.0? [ Edit: I guess CI just showed me that ... ]
Copy link
Contributor Author

Yes, it should work with older versions of R. I've tested myself with R 4.4.0. When QuantLib is not found via pkg-config, it falls back to what has been done before my change.

Copy link
Owner

Beautiful. Really really appreciate this.

eddelbuettel merged commit e64c1e8 into eddelbuettel:master Mar 25, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants