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

GH Actions automatic MSI builds are failing #1

Closed
ischoegl opened this issue Apr 9, 2023 · 2 comments
Closed

GH Actions automatic MSI builds are failing #1

ischoegl opened this issue Apr 9, 2023 · 2 comments

Comments

@ischoegl
Copy link
Member

ischoegl commented Apr 9, 2023

MSI builder for the Matlab toolbox are currently failing due to

cantera_shared.def : error LNK2001: unresolved external symbol <…>

Failures started in Fall 2022.

@speth
Copy link
Member

speth commented Jun 15, 2023

After the fix introduced in Cantera/cantera#1505 for a different MSI build error, we're back to this one. For the sake of posterity, a complete build log is saved here: x86-msi.log.

At present, only the 32-bit build is failing, and I think the relevant aspect is what was elided in the report above. All of the "unresolved external symbol" names are for decorated "C" methods, which includes the Sundials methods and the Cantera clib methods, e.g.:

cantera_shared.def : error LNK2001: unresolved external symbol _CVDlsSetLinearSolver
...
cantera_shared.def : error LNK2001: unresolved external symbol _thermo_meanMolecularWeight
...

Documentation from Microsoft indicates that there is a difference in how (or whether) "C" methods are decorated between 32 and 64 bit platforms.

Suggested resolution: Given that the main purpose of this package is to provide the Matlab toolbox, and that Mathworks stopped releasing 32-bit versions of Matlab for Windows in 2015, I think we can safely drop the 32-bit builds.

@ischoegl
Copy link
Member Author

I agree that dropping support for 32-bit versions is an adequate resolution.

@speth speth closed this as completed in cb2a7f3 Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

No branches or pull requests

2 participants