Skip to content
This repository has been archived by the owner on Jun 9, 2024. It is now read-only.

Feat/mod management #12

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Trico-Everfire
Copy link
Contributor

@Trico-Everfire Trico-Everfire commented Dec 25, 2022

I added a zip extractor based using the minizip-ng library and some help from the internet. I integrated this into the SDK Launcher to allow the creation of P2CE Source mods.
This also upgrades the application from QT5 to QT6
string_builder and zip_handler are C code and are used in the zip extraction process. Feel free to tear the code apart.
Resolves #2

@Trico-Everfire
Copy link
Contributor Author

The command line handler is a bug introduced in the last PR where it'd ignore " and ' in command line arguments, this fixes that :)

@Trico-Everfire Trico-Everfire marked this pull request as ready for review December 31, 2022 13:30
src/mainview.cpp Show resolved Hide resolved
src/modmanager.cpp Outdated Show resolved Hide resolved
src/modmanager.cpp Outdated Show resolved Hide resolved
src/modmanager.h Outdated Show resolved Hide resolved
src/string_builder.h Outdated Show resolved Hide resolved
src/modmanager.cpp Show resolved Hide resolved
src/editconfig.h Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated
@@ -11,7 +13,7 @@ endif()
if(UNIX)
set(CMAKE_CXX_STANDARD 20)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fPIC")
set(QT_INCLUDE "/usr/include/x86_64-linux-gnu/qt5")
set(QT_INCLUDE "$ENV{HOME}/Qt/6.4.1/gcc_64/include/")
Copy link
Owner

Choose a reason for hiding this comment

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

This shouldn't be required anymore. CMake should figure it out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wasn't it that this was required for people who don't have QT installed in the place CMake expects it to be? Like you said with the Chaos QT6 install repo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right?

src/main.cpp Outdated Show resolved Hide resolved
@Trico-Everfire
Copy link
Contributor Author

Erin, I want to ask, I've made a window for adding the newly made mod to P2CE Hammer's config, should I add that into this PR or is it not required within the scope of this PR/Project?

@AWildErin
Copy link
Owner

Add it into another PR.

@Trico-Everfire
Copy link
Contributor Author

Will do after this PR, as it relies on this PR to function (provided it's part of the mod management)

@Trico-Everfire
Copy link
Contributor Author

Hey... I was wondering, when will it'll be reviewed? I know you're busy with P2CE and other things, so no haste.

@AWildErin
Copy link
Owner

I don't really have a time frame for when it will be reviewed other than when I have time. I have several projects taking up a bunch of my time, and I'm focused on getting on with them rather than my open-source projects.

@Trico-Everfire
Copy link
Contributor Author

That's all right, I'll just await the next review. Thank you for your time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mod management
2 participants