In D21261#608154, @shubham wrote:

@stikonas I am a bit confused, can you please help a bit?

@stikonas I am a bit confused, can you please help a bit?

Sorry for typos, What I meant was:

In D21261#595885, @shubham wrote:

@stikonas Ping?

@stikonas Ping?

Do you have commit rights or do I need to push it for you?

Fix unrelated change

Keep comment at the right place

Remove unneeded change

Remove trailing whitespaces

Some cleanup and add unittest

Rebase

Rebase wrt parent

const at the end of startHelper() function

Use sync version for checking authorization

Remove left over commented code

Fix documentation

More changes

Fix linker error

Remove unused parameters

Success starting the kauth_externalcommand helper

Remove commented code

Refactor

1. Use checkAuthorizationSync() to authenticate
2. Fix infinite recursion in startHelper()

Try to start the helper(Atleast polkit-qt-1)

Remove unnecessary linking

1. Fix runtime warnings inside initPolkitAgent()
2. Use "new" inside ctor to instantiate object

Move helperMain() outside Q_SLOTS

some more changes

Try fixing the inline comments

Try to fix the broken things

QDBus communication

Update wrt parent

Rebase wrt parent

Remove unwanted signals

Rename function

Refactor

@stikonas Yes, I am planning to re utilize the code, my first intention was just to make it compile.

connect to newData signal in the application

Fix errors, make it compile
@stikonas can you have a look please?

Yes, and you need to add it to cmake. That part of code is autogenerated.

@stikonas It says error: ‘org’ does not name a type

auto interface = new org::kde::kpmcore::externalcommand(QStringLiteral("org.kde.kpmcore.externalcommand"),

I think I need to add ExternalCommand as an interface.

Refactor

Before it's pushed (once it compiles) also refactor code a bit so that
there isn't that much code duplication.

Add test to revoke authorization

Rebase wrt parent

Remove extra line changes

Proper rebase

.

Rebase

Rebase wrt master

Rebase wrt parent

Merged this with the main patch ie. authorization backend

Compile helper as standalone