Perhaps this is something the libgit2-supplement.json file can help with overriding but I've never added any content to that file myself.

@rcjsuen The overrides, from the supplement file, are applied after the Helpers.isCallbackFunction is called on the field. I tried the approach you put forward, but it failed to build with the following error:

            wrapper->raw->push_transfer_progress = (git_push_transfer_progress_cb)push_transfer_progress_cppCallback;
                                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                    git_push_transfer_progress
../vendor/libgit2/include/git2/remote.h:351:15: note: 'git_push_transfer_progress' declared here
typedef int (*git_push_transfer_progress)(```

I tried a couple of variations based on your code, but haven't managed to get past the build.
Perhaps there's somebody who better understands the generators that could help out?

@rcjsuen I've got it working using the supplement file and changing type to include _cb. I'm still not sure if this is the correct approach, but it is working.

I will get onto the test case shortly.

@rcjsuen I've setup the test using a local bare repository... Once again, I'm not sure if the approach is correct, but it works.

Update 2: Tried building from a clean repo and the build/test failed.

Update: Forgot to credit your advice, sorry... I'm now looking into the build errors.

It seems that the culprit is:

"push_transfer_progress": {
    "dupFunction": "git_transfer_progress_dup"
}

Have rewritten the isCallbackFunction to accept a second argument to override the regex test.

This means we can now declare a callback from the libgit2-supplement.json file using isCallback.

I hope this approach is alright, I can't see another way around it. I'm also not sure why libgit have named the function without _cb.

@rcjsuen Any suggestions for moving this forward?

@rcjsuen All checks have passed now, is there any chance of pushing this one through?

@implausible No worries, I've made the changes.

Thanks!

Any word on this merge going live?