On 10/09/2016 04:45 PM, Pierre-Yves David wrote:


On 10/09/2016 04:19 PM, Mads Kiilerich wrote:
diff --git a/hgext/eol.py b/hgext/eol.py
--- a/hgext/eol.py
+++ b/hgext/eol.py
@@ -312,10 +312,15 @@ def reposetup(ui, repo):
                 self._eolmatch = util.never
                 return

+            oldeol = None
             try:
                 cachemtime = os.path.getmtime(self.join("eol.cache"))

This seems like it should live in the "cache/" directory. Any reason not to ?


I don't know. I guess it predates the cache directory. I would probably agree it now would be better if it lived there.

This patch is backwards compatible and doesn't change that. Changing the location would be a separate independent change - and apparently trivial and backwards compatible.


/Mads

_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to