WolframResearch / wolfram-app-discovery-rs

Locate local installations of the Wolfram Language.
Apache License 2.0
14 stars 5 forks source link

bugfix: Add Linux support to build_scripts::wstp_static_library_path() #40

Closed ConnorGray closed 1 year ago

ConnorGray commented 1 year ago

This is preventing wstp-sys's build.rs script from working on Linux.

This should have been included in PR #28, which had significantly expanded Linux support for wolfram-app-discovery.

The mistake was that this match OperatingSystem::target_os() was copy-paste duplicated between WolframApp::wstp_static_library_path() and build_scripts::wstp_static_library_path(), but only the former was updated in #28.

I've partially refactor this logic out into a new wstp_static_library_file_name() function so that this particular mismatch doesn't reoccur.