keyman icon indicating copy to clipboard operation
keyman copied to clipboard

feat(windows): insthelper Delphi to C++ conversion

Open chenjieen opened this issue 2 years ago • 4 comments

Fixes: #8700 This creates a new C++ project to create a DLL to replace the current Delphi insthelper DLL.

chenjieen avatar Dec 14 '23 06:12 chenjieen

User Test Results

Test specification and instructions

ERROR: user tests have not yet been defined

keymanapp-test-bot[bot] avatar Dec 14 '23 06:12 keymanapp-test-bot[bot]

I will review this on my return from leave in January

mcdurdin avatar Dec 14 '23 06:12 mcdurdin

@chenjieen From a quick slack chat with Marc re logging. logging for that particular project should be to msi logs. https://resources.flexera.com/web/pdf/archive/msi_writing_to_the_log_file.pdf (really, pdf...) shows something like this:

// standard MSI DLL custom action signature
UINT __stdcall LoggingTestCpp(MSIHANDLE hInstall)
{
PMSIHANDLE hRecord = MsiCreateRecord(1);
// field 0 is the template
 MsiRecordSetString(hRecord, 0, "Log: [1]");
// field 1, to be placed in [1] placeholder
MsiRecordSetString(hRecord, 1, "Calling LoggingTestCpp...");
// send message to running installer
MsiProcessMessage(hInstall, INSTALLMESSAGE_INFO, hRecord);
return ERROR_SUCCESS;
} 

rc-swag avatar Feb 16 '24 08:02 rc-swag

This pull request is from an external repo and will not automatically be built. The build must still be passed before it can be merged. Ask one of the team members to make a manual build of this PR.

keyman-server avatar Apr 22 '24 14:04 keyman-server

@rc-swag, @chenjieen what is the status of this PR?

mcdurdin avatar Nov 13 '24 03:11 mcdurdin

@mcdurdin. @chenjieen has addressed the review comments ready for a re-review except noting he could not get it to compile anymore. It has been sitting on my todo list as I was not quickly able to determin the issue. I may need your input. Or at least longer time to determine why needs to be changed.

rc-swag avatar Nov 14 '24 01:11 rc-swag

@rc-swag, do you have time to get this over the line or should we abandon it?

mcdurdin avatar Jan 20 '25 06:01 mcdurdin

We're going to abandon this for now. We don't have the capacity to continue on the Delphi -> C++ conversion internally. This code will still be here if anyone wants to pick it up and finish it.

mcdurdin avatar Jan 31 '25 07:01 mcdurdin