git.tremily.us Git - ikiwiki.git/log

Consume all stdin when rcs_receive short-circuits, to avoid git SIPIPE race.

We had a weird problem where, after moving to a new, faster server,
"git push" would sometimes fail like this:

Unpacking objects: 100% (3/3), done.
fatal: The remote end hung up unexpectedly
fatal: The remote end hung up unexpectedly

What turned out to be going on was that git-receive-pack was dying due
to an uncaught SIGPIPE. The SIGPIPE occurred when it tried to write to
the pre-receive hook's stdin. The pre-receive hook, in this case, was
able to do all the checks it needed to do without the input, and so did
exit(0) without consuming it.

Apparently that causes a race. Most of the time, git forks the hook,
writes output to the hook, and then the hook runs, ignores it, and exits.
But sometimes, on our new faster server, git forked the hook, and it
ran, and exited, before git got around to writing to it, resulting in
the SIGPIPE.

write(7, "c9f98c67d70a1cfeba382ec27d87644a"..., 100) = -1 EPIPE (Broken
pipe)
--- SIGPIPE (Broken pipe) @ 0 (0) ---

I think git should ignore SIGPIPE when writing to hooks. Otherwise,
hooks may have to go out of their way to consume all input, and as I've
seen, the races when they fail to do this can lurk undiscovered.

I have written to the git mailing list about this.

As a workaround, consume all stdin before exiting.

a different alternative

consider alternatives?

add wishlist tag to hopefully get the attention of Joey

add news item for ikiwiki 3.20111107

releasing version 3.20111107

Merge branch 'master' of ssh://git.ikiwiki.info

add missing license text

graphviz: Support wikilinks embedded in the graph.

(Sponsored by The TOVA Company.)

store imagemap in %pagestate, rather than a file

Using a file was sorta not right.

Note that when previewing, %pagestate is not saved, so
it has to rebuild the graph every time until that graph is saved;
then previews can use the cached data until the next time the graph
is changed.

Also note that it's stored in the destpage's pagestate. The imagemap
could vary between a page and an inlined page if wikilinks were supported.

graphviz: Support urls embedded in the graph, by having graphviz generate an imagemap.

Also, I let preview mode write real files, rather than using data: uri.
Which is ok these days, since ikiwiki tracks files created during
previewing, and cleans them up later.

Added a comment: thx

Merge branch 'master' of ssh://git.ikiwiki.info

Added a comment: comment on my own question

try a map instead of an inline

The umask setting can now be set to private, group, or public, avoiding the need to enter octal correctly which is particularly difficult in yaml setup files. (smcv)

Merge remote-tracking branch 'smcv/umask-keywords'

Merge branch 'master' of ssh://git.ikiwiki.info

I have no branch and must merge.

editpage: Fix FormattingHelp link on Discussion pages.

In 875d550f1278215e6c87d3b78ff87db24c6d76b3 I for some reason
made $page be changed when creating a discussion page, which
broke the link on the edit page. Changing page seems unnecessary,
so reverted that part of the change.