On Sun, May 24, 2009 at 2:37 AM, Felipe Contreras <
felipe.contre...@gmail.com> wrote:

> However, there's one missing feature: now that I'm not testing any
> more I would like git_export to abort when the author is not listed.
> Can you do that?


By "not listed" do you mean not present in the authors-file?

Note that this doesn't have much, if anything, to do with the "<unknown>"
string used as the default author when there are *no* author certs. In that
case, there's not much else to do, some string needs to be used for the
author, and this string is looked up in the authors-file allowing you to
change it if you like.

At the moment, if the authors-file lookup fails the author string is checked
to ensure that it is wrapped in '<' and '>' characters which seems to work
ok for many historical monotone names and avoids the need to create a full
authors-file. It sounds like you want to require a successful authors-file
lookup and disable this name fixup.

I think it might be better to use a lua hook for doing whatever name fixups
are required if the authors-file lookup fails and if this hook fails we'll
cancel the export. This should allow for using either the authors-file, or
the hook or both.

IMHO putting some fake id such as 'unknown' is good for testing
> purposes so perhaps it should be turned on with --enable-author-guess
> or something.


As mentioned above, the "<unknown>" string is used when there are no author
certs available. We could provide an option to set the value of this string
to something else, but you can already change this string using the
authors-file so another option would be redundant.

Cheers,
Derek
_______________________________________________
Monotone-devel mailing list
Monotone-devel@nongnu.org
http://lists.nongnu.org/mailman/listinfo/monotone-devel

Reply via email to