We need some commit hooks for vcs_git. For this we maybe need to enhance the internal API. I think for normal commits it makes most sense to use the whole API, load the whole cache file and process the commit as normal to make testing easier, and we'd need the cache file anyways as we need to update it, so I think we don't need that little performance gain which is hidden here.

Comments

CorniI’s picture

tagging for release...

CorniI’s picture

Title: Create git hooks/xgit scripts for vcs_git » Create git commit hooks for vcs_git
sdboyer’s picture

Depending on the scope and difficulty of the changes, there's probably a point at which it would be worth waiting until we can generate a gitlib (analogue to svnlib) that'll make doing those operations easier. I dunno how many or how complex changes you have in mind, though, so I'm just throwing that out there as a general note.

sdboyer’s picture

Issue tags: +git phase 2

need these, BIIIIG time, for phase 2 of the migration.

well actually, we may end up implementing the hooks we want in the drupalorg module, or something less generic than vcsapi. but until that decision is made, i'm tagging this issue accordingly.

sdboyer’s picture

Priority: Normal » Critical
CorniI’s picture

well, this is already in the vcs_git queue, not the vcsapi one, so i think it's quite right here ;)
The most dumbed-down version of the hooks would just fire off the log parser, so it shouldn't be that complicated.
It get's more complicated when one tries to incorporate the information given by the commit hook so that the log parser is more performant.

marvil07’s picture

related: #973944: Integrate GSoC 2009 Daniel Hackney work, IIRC that was one of the main goals for Daniel

marvil07’s picture

marvil07’s picture

Version: » 6.x-2.x-dev
Issue tags: -vcs_git-6.x-1.0-release
sdboyer’s picture

Title: Create git commit hooks for vcs_git » Create serverside git hooks
Issue tags: +git sprint 9

We're going to be writing some serverside git hooks in sprint 8 - though they'll probably end up being entirely d.o specific. Hence this issue in infra: #1012988: Create serverside git hooks that interact with our unique server environment. We need to update these before we can make a stable release, though.

eliza411’s picture

Assigned: Unassigned » marvil07

If you can't help Sam with this, would you please reassign it to him?

sdboyer’s picture

We don't need this for launch, since we use our own custom set of hooks that are really quite distinct from anything that would be useful generically. Untagging, and not tagging for phase 3.

We need something in vc_git for it to be set for a stable release, I think, but that's a separate priority.

eliza411’s picture

Issue tags: -git sprint 9

removing sprint 9 tag

marvil07’s picture

Assigned: marvil07 » sdboyer

@sdboyer: Probably related with your current work? (IIRC it would be generalizing it?)

sdboyer’s picture

Yeah, this dovetails with generalizing the history sync mechanism and some of the new methods added to that interface. We may need to decide on what our "out of the box" target use case is for these to be really useful...maybe not. We might be able to get away with a single generic set.

marvil07’s picture

Not sure about the status of this, but before git hook scripts for d.o ended up managed with puppet directly they were on the drupalorg project, so pasting here a: link there.

marvil07’s picture

Status: Fixed » Closed (fixed)
Issue tags: -git phase 2 leftovers

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 271fdde on 7.x-1.x, fix-invalid-default-branches, fullsync-memory by marvil07:
    Issue #484366: Improve and document the serverside git hook script...