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

Fix install error on Windows MINGW environment and drop Windows MSWIN environment support #1585

Closed
wants to merge 4 commits into from

Conversation

Watson1978
Copy link
Member

@Watson1978 Watson1978 commented Apr 24, 2024

Fix #1584

Previously ImageMagick provided CORE_RL_MagickCore_.lib where we statically resolve the symbols needed at build time.
The latest version no longer provides that file.

This has forced us to modify the build process to resolve symbols using .dll files.


In a WINGW environment, dlls can be linked directly.
However, in an MSWIN environment, a lib file is required to solve symbols or modify many source codes.
I think almost all users use RubyInstaller (MINGW environment), so I will drop MSWIN support


This fix will be released as v5.6.0.
After v5.6.0 is released, I let these changes to will be to the main branch as forward port.

@Watson1978 Watson1978 force-pushed the fix-install-error-windows branch 12 times, most recently from cf434c9 to d5194d0 Compare April 24, 2024 10:37
@Watson1978 Watson1978 changed the title Fix install error on Windows Fix install error on Windows WINGW environment and drop Windows MSWIN environment support Apr 25, 2024
@Watson1978 Watson1978 changed the title Fix install error on Windows WINGW environment and drop Windows MSWIN environment support Fix install error on Windows MINGW environment and drop Windows MSWIN environment support Apr 25, 2024
@Watson1978 Watson1978 marked this pull request as ready for review April 25, 2024 03:41
@Watson1978
Copy link
Member Author

@mockdeep I am going to drop it as it is difficult to support MSWIN environment. Do you have any concerns?

@Watson1978
Copy link
Member Author

Seems it would be better to release these changes as v6.0. hmm...

@Watson1978 Watson1978 closed this Apr 25, 2024
@Watson1978 Watson1978 deleted the fix-install-error-windows branch May 1, 2024 03:13
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.

None yet

1 participant