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

Add support for arm64 and update mozjpeg 4.1.3 #80

Merged
merged 2 commits into from
Jul 22, 2023

Conversation

MateuszKrawczuk
Copy link
Contributor

Windows building is not really implemented. Somebody should fix this in future. I recommend using vcpkg for providing zlib and libpng.
Mac Os X arm64 binary is placeholder now and it should be change to real arm64 binary.

Windows building is broken. It has to be fixed. I recommend using vcpkg.
Mac Os X arm64 binary should be change to real arm64 binary. I don't own mac, but everything builds correctly.
@MateuszKrawczuk MateuszKrawczuk mentioned this pull request Jul 10, 2023
2 tasks
@1000ch
Copy link
Member

1000ch commented Jul 18, 2023

@MateuszKrawczuk
Thank you for your work! I'd like to wait for someone's help to fix Windows build OR skip Windows build from source for now as temporary if prebuilt binary works correctly.

@MateuszKrawczuk
Copy link
Contributor Author

@1000ch
I fixed building on windows, but I didn't have time to fix building during test for windows.

@MateuszKrawczuk
Copy link
Contributor Author

@1000ch In mozjpeg-v4.1.3 and mozjpeg-v4.1.1 on windows it requires to copy libraries even if it uses static cjpeg. I think I should report issue in https://github.com/mozilla/mozjpeg/ with cjpeg-static.exe. That why I don't copy cjpeg-static.exe and copy smaller cjpeg.exe. Maybe it is issue related with building using Microsoft Visual Studio Build tools.

@1000ch 1000ch merged commit c0587fb into imagemin:main Jul 22, 2023
9 checks passed
@MateuszKrawczuk MateuszKrawczuk deleted the Arm64_and_cmake branch September 2, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants