Skip to content

Fix windows version - #1670

Open
dtristany wants to merge 16 commits into
ahrm:developmentfrom
dtristany:development
Open

Fix windows version#1670
dtristany wants to merge 16 commits into
ahrm:developmentfrom
dtristany:development

Conversation

@dtristany

Copy link
Copy Markdown

Streamline build process with yaml.

Pin to latest compatible mupdf, openssl and zlib for latest security fixes.

Fix issue that would not allow building windows portable version.

Fix issue #1669 without changing c++ version for other platforms.

dtristany and others added 16 commits August 16, 2026 00:12
Update actions to latest to support Node 24. Node 20 gets fully removed from GitHub runners on September 16, 2026  so older file will hard-fail once that lands.
qt 6.11 introduces subfolders for each architecture, which is not supported by older versions of aqtinstall
The Windows MSVC build currently fails with error C7555 because `main_widget.cpp` uses designated initializers  but `pdf_viewer_build_config.pro` restricts the build to C++17. 

Unlike other compilers, MSVC strictly enforces standard flags and hard-stops when encountering C++20 syntax in C++17 mode.

This commit appends `/std:c++20` to `QMAKE_CXXFLAGS` within the `win32` block of the .pro file.
by not building mupdf debug needlessly, 10min per build are saved,
Since that is what Qt 6 expects
NON-PORTABLE was previously hardcoded for all releases
Latest security fixes
* Update the build files to use c++20 (addresses ahrm#1669)

* Fix a bug where page in shrink_selection was uninitialized

* Fix the issue where the overview did not respect inverted_preserved_image_colors (addresses ahrm#1674)

---------

Co-authored-by: ahrm <a.hr.mostafavi@gmail.com>
remove automatic releases
remove automatic releases
No longer required as project now uses c++20 for all builds
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.

1 participant