Implements the profile handling functions from the TerminalInterface.
Also registered Profile::Property with the Qt Meta Type System in order
to allow to query these properties from the interface.
Depends on D20169
hindenburg |
Konsole |
Implements the profile handling functions from the TerminalInterface.
Also registered Profile::Property with the Qt Meta Type System in order
to allow to query these properties from the interface.
Depends on D20169
No Linters Available |
No Unit Test Coverage |
Buildable 12400 | |
Build 12418: arc lint + arc unit |
I believe that once the KParts code is committed to frameworks, Konsole needs to add checks the frameworks version. I'd rather not require the new framework version for all of Konsole, just this new code.
Yeah, agreed. I'm just waiting for the Interface Api to be finished until I update this. Just opened this to have some kind of reference of how the implementation might look like and also for testing purpose.
Update to the new terminalinterfacev2 version
Use TerminalInterfacV2 depending on kf5 version (>= 5.60.0)
Thanks, do you have time to add some tests? They would be added to autotests/TerminalInterfaceTest.cpp TerminalInterfaceTest.h
KDE uses spaces instead of tabs. Does the tests past if you manually install the new kde_terminal_interface.h and remove the #IF? I get failures in the QVERIFY(terminal).
If you effectively compile with the InterfaceV2 enabled and an updated kde_terminal_interface.h it should pass. Only issue I had is that the tests don't load the konsolepart.so from the build dir directly which means the actually loaded konsolepart.so is still the old version. This results in the test failure.
Forgot to add the line to close this.
https://invent.kde.org/kde/konsole/commit/12eb5c193465f1bee44f1aa6bbc6c4efc7e166d5