Comment 9 for bug 1078454

Thank you Elan!

I'm going to try to get the hook in my production next week and let
you know how things work out.

When I wrote the shell script the deal was rebase everyting on master.
No branches pushed, no merges. I wonder what happens now with eventum
and merges at my end happening all the time.

Maybe the 'module' column should contain the branchname being pushed.
So people watching a bugreport can see work happening on a branch. And
somehow there should also be a notification when work is merged from a
dev branch to a production branch.

Cool that it is in. Thanks again!

Regards,
Robbert-Jan

On 30/11/2014 12:40, Elan Ruusamäe wrote:
> yeah. git is more difficult than cvs, because of the merges
>
> if svn integration was more or less convertible to cvs model (but it's
> not perfect and i don't really use it), then git is total horror
> comparing to that.
>
> for example this history containing 1 merge created 4 checkins to issue:
>
> * 00fabd1 - (HEAD, origin/merge-tip, merge-tip) add setup. issue 1 (16 minutes ago) <Elan Ruusamäe>
> * 4d06d21 - Merge branch 'newroot' (23 hours ago) <Elan Ruusamäe>
> |\
> | * af4d053 - (origin/newroot, newroot) added (23 hours ago) <Elan Ruusamäe>
> * 484ff0a - added (23 hours ago) <Elan Ruusamäe>
> * 31de9fd - add some files. issue 1 (4 weeks ago) <Elan Ruusamäe>
>
>
>
> ** Attachment added: "Screenshot-1.png"
> https://bugs.launchpad.net/eventum/+bug/1078454/+attachment/4271136/+files/Screenshot-1.png
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1078454
>
> Title:
> git hashes don't fit in eventum_issue_checkin columns
>
> Status in Eventum:
> Fix Released
>
> Bug description:
> I'd like to write a git scm hook for eventum. It seems doable when
> linking to viewgit. My problem is the columns isc_old_version and
> isc_new_version are only 32 characters wide while git hashes are 40
> characters.
>
> This query solves that problem:
>
> alter table eventum_issue_checkin
> modify column isc_old_version varchar(40) default null,
> modify column isc_new_version varchar(40) default null;
>
> Is it possible to include it in a feature update as a database patch?
>
> Thanks,
> Robbert-Jan
>
> PS: If there is any progress on my scm hook I'll let you know.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/eventum/+bug/1078454/+subscriptions