[PATCH] blackbox: add an option to customize the name of the log file

Yuya Nishihara yuya at tcha.org
Fri Jun 17 10:36:05 EDT 2016


On Thu, 16 Jun 2016 15:20:33 +0100, Jun Wu wrote:
> Excerpts from Yuya Nishihara's message of 2016-06-16 22:55:45 +0900:
> > It seems confusing that filesuffix isn't a suffix. Is there a reason to
> > not allow specifying the full file name? Invalid path components such as
> > '..' will be rejected by pathauditor.
> 
> I'd like to have flexible full names as well.
> 
> My main concern is names like "requires", "dirstate", "hgrc" are dangerous.
> But if we move them to ".hg/blackbox" and disallow "/", then everything will
> be fine.
> 
> If putting everything under "blackbox" subdirectory (including blackbox.log)
> makes sense, I can prepare the patch. If compatibilities are a concern, I
> can symlink blackbox.log.

Or use the "blackbox" directory only if the option is specified. Perhaps you'll
get better suggestion from your colleagues as they are main users of the
extension?

Either ways, symlink shouldn't be the solution. It doesn't work on Windows as
of Python 2.x.


More information about the Mercurial-devel mailing list