yuja added a comment.
> -names_tags = '{if(names % "{ifeq(name, 'tip', '', name)}", " ({label('log.{colorname}', join(names % "{ifeq(name, 'tip', '', name)}", ' '))})")}' > +names_tags = '{if(names % "{ifeq(name, 'tip', '', name)}", " ({label('log.{colorname}', separate(names % "{ifeq(name, 'tip', '', name)}", ' '))})")}' `separate()` can't be used here since it takes non-list arguments. `separate(sep, arg1, arg2, ...)` What we can do for now is to move the separator to the inner loop, and strip the leading/trailing separator. `strip(names % "{ifeq(..., "{name} ")}")` Maybe we'll need a function to filter out empty values? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D3648 To: tom.prince, #hg-reviewers Cc: yuja, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel