hg notify error

[John Turner]

On Mon, 22 Dec 2014 14:12:40 -0500, Matt Mackall <mpm at selenic.com> wrote:

> On Mon, 2014-12-22 at 18:23 +0000, Becker, Mischa J wrote:
>> > -----Original Message-----
>> > From: Mercurial [mailto:mercurial-bounces at selenic.com] On Behalf Of  
>> John
>> > Turner
>> > Sent: Monday, December 22, 2014 10:19 AM
>> > To: mercurial
>> > Subject: hg notify error
>> >
>> > I attempted a push to my central repo and the notify hook raised the
>> > following error:
>> >    changegroup.notify hook raised an exception:
>> >   invalid literal for int() with base 10: '300 # max lines of diffs to
>> > include (0=none, -1=all)'
>
> I think that's pretty clear: the text "300 # max lines of diffs..." is
> not an integer. So where is it getting that text?
>
> That text last appeared in Mercurial's help in 1.9 (July 2011):
>
> http://www.selenic.com/hg/file/de9eb6b1da4f/hgext/notify.py#l34
>
> But as far as I can tell, we've never actually accepted trailing
> comments on config items, so anyone who copied this into their config
> would get this error.
>
> At any rate, it's not coming from Mercurial any more, which means it's
> coming from your machine somewhere. You might be able to spot it with
> 'hg config --debug'.

Ahh, thanks Matt - indeed, it was reading the config from my local repo  
which had the trailing comments, rather than from the central repo.

My actual intention was to kick off the notify from the central repo  
(whose notify config is in good form).  I suppose I'll need to iron-out  
the notify local/remote precedence a bit more..

Cheers,
John