Skip to content

Installer#2

Merged
mzagozda merged 7 commits into
taskscape:mainfrom
MikolajSkaziak:installer
Jan 20, 2026
Merged

Installer#2
mzagozda merged 7 commits into
taskscape:mainfrom
MikolajSkaziak:installer

Conversation

@MikolajSkaziak

Copy link
Copy Markdown
Collaborator

No description provided.

@mzagozda mzagozda requested review from Copilot and mzagozda January 20, 2026 21:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds build artifacts and intermediate files for C++ module compilation across multiple Visual Studio plugins, along with an installation marker file.

Changes:

  • Addition of C++ STL module build artifacts (std and std.compat) for Visual Studio 2022
  • Addition of installation marker file for x64 platform

Reviewed changes

Copilot reviewed 137 out of 547 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/plugins/pak/vcxproj/salamander/Release_x64/plugins/pak/lang/Intermediate/microsoft/STL/* Build artifacts for C++ modules in pak plugin language build
src/plugins/pak/vcxproj/salamander/Release_x64/plugins/pak/Intermediate/microsoft/STL/* Build artifacts for C++ modules in pak plugin
src/plugins/nethood/vcxproj/salamander/Release_x64/plugins/nethood/lang/Intermediate/microsoft/STL/* Build artifacts for C++ modules in nethood plugin language build
src/plugins/nethood/vcxproj/salamander/Release_x64/plugins/nethood/Intermediate/microsoft/STL/* Build artifacts for C++ modules in nethood plugin
src/plugins/mmviewer/vcxproj/salamander/Release_x64/plugins/mmviewer/lang/Intermediate/microsoft/STL/* Build artifacts for C++ modules in mmviewer plugin language build
src/plugins/mmviewer/vcxproj/salamander/Release_x64/plugins/mmviewer/Intermediate/microsoft/STL/* Build artifacts for C++ modules in mmviewer plugin
src/plugins/ieviewer/vcxproj/salamander/Release_x64/plugins/ieviewer/lang/Intermediate/microsoft/STL/* Build artifacts for C++ modules in ieviewer plugin language build
src/plugins/ieviewer/vcxproj/salamander/Release_x64/plugins/ieviewer/Intermediate/microsoft/STL/* Build artifacts for C++ modules in ieviewer plugin
src/plugins/ftp/vcxproj/salamander/Release_x64/plugins/ftp/lang/Intermediate/microsoft/STL/* Build artifacts for C++ modules in ftp plugin language build
src/plugins/ftp/vcxproj/salamander/Release_x64/plugins/ftp/Intermediate/microsoft/STL/* Build artifacts for C++ modules in ftp plugin
src/plugins/folders/vcxproj/salamander/Release_x64/plugins/folders/lang/Intermediate/microsoft/STL/* Build artifacts for C++ modules in folders plugin language build
Instalator/x64 Marker file indicating x64 installation target

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Instalator/LICENSE
@@ -0,0 +1,67 @@
Software License Agreement for Altap Salamander.

This is a legal agreement ("Agreement") between you (either an individual or

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change to Taskscape Ltd instead of "ALTAP"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread Instalator/salamand.exe

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should never commit binary files. These should go to release (see github project releases)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh okey

Comment thread .gitignore
*.bak
*.backup

# Wyjątek dla gotowego instalatora

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no folder "instalator" or it should have been empty. Files should be part of release (see GitHub project releases). There should be no Polish titles or comments because it is very expensive to locate them and remove.

Comment thread src/debug_move.cpp

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this debug file is part of the installer task

Comment thread src/fileswn4.cpp
// search from the end for the character after which we can copy "..." and it fits in the column
while (fitChars > 0 && DrawItemAlpDx[fitChars - 1] + TextEllipsisWidth > textWidth)
fitChars--;
// copy a part of the original string to another buffer

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think removing those lines should be part of adding or updating the installer

@mzagozda mzagozda merged commit 760665f into taskscape:main Jan 20, 2026
3 checks passed
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.

3 participants