jcarr
/
git2go
1
0
Fork 0
Code Issues 52 Pull Requests 31 Packages Projects Releases Wiki Activity

Add support for creating signed commits and signing commits during a rebase #626

Merged
mbfr merged 25 commits from master into master 2020-08-18 11:25:31 -05:00
Conversation 1 Commits 25 Files Changed 8 +13 -1
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 971e18b585 - Show all commits

9
rebase.go
View File

@ -98,7 +98,14 @@ func commitSignCallback(_signature *C.git_buf, _signature_field *C.git_buf, _com
cstr := unsafe.Pointer(C.CString(bufData)) cstr := unsafe.Pointer(C.CString(bufData))
defer C.free(cstr) defer C.free(cstr)
if int(C.git_buf_set(buf, cstr, clen)) != 0 { // over-assign by a byte (see below)
lhchavez commented 2020-08-14 08:58:38 -05:00 (Migrated from github.com)
Review
Copy Link

in order to support callback authors being able to set a specific libgit2 error, we could do something like:

if gitError, ok := err.(*GitError); ok {
    return C.int(gitError.Code)
}
return C.int(-1)
in order to support callback authors being able to set a specific libgit2 error, we could do something like: ```go if gitError, ok := err.(*GitError); ok { return C.int(gitError.Code) } return C.int(-1) ```
mbfr commented 2020-08-14 09:53:17 -05:00 (Migrated from github.com)
Review
Copy Link

Good idea, added

Good idea, added
if int(C.git_buf_grow(buf, clen+1)) != 0 {
return errors.New("could not grow buffer")
}
C._go_git_buf_fill_null(buf)
lhchavez commented 2020-08-14 08:59:06 -05:00 (Migrated from github.com)
Review
Copy Link

this needs a C.free(...).

this needs a `C.free(...)`.
mbfr commented 2020-08-14 09:53:21 -05:00 (Migrated from github.com)
Review
Copy Link

Fixed

Fixed
if int(C.git_buf_set(buf, cstr, clen+1)) != 0 {
return errors.New("could not set buffer") return errors.New("could not set buffer")
} }

5
wrapper.c
View File

@ -12,6 +12,11 @@ void _go_git_populate_apply_cb(git_apply_options *options)
options->hunk_cb = (git_apply_hunk_cb)hunkApplyCallback; options->hunk_cb = (git_apply_hunk_cb)hunkApplyCallback;
} }
void _go_git_buf_fill_null(git_buf *buf)
{
memset(buf->ptr, '\0', buf->asize*sizeof(char));
}
void _go_git_populate_commit_sign_cb(git_rebase_options *opts) void _go_git_populate_commit_sign_cb(git_rebase_options *opts)
{ {
opts->signing_cb = (git_commit_signing_cb)commitSignCallback; opts->signing_cb = (git_commit_signing_cb)commitSignCallback;