view CONTRIBUTING @ 39820:68ea1f8dcb84

py3: proxy posixfile objects to re-add a useful 'name' attribute on Windows

This file object is used in the vfs layer, so there are many errors like this:

    ...
      File "mercurial\localrepo.py", line 2569, in savecommitmessage
        return self.pathto(fp.name[len(self.root) + 1:])
    TypeError: 'int' object is not subscriptable

It looks like the 'name' value is actually the fileno() value, and the
documentation says the name parameter to PyFile_FromFd() is ignored. [1]  I
tried just assigning the attribute after osutil.posixfile() returns, but that
crashes saying that it's read-only.

[1] https://docs.python.org/3.6/c-api/file.html

Our full contribution guidelines are in our wiki, please see:

https://www.mercurial-scm.org/wiki/ContributingChanges

If you just want a checklist to follow, you can go straight to

https://www.mercurial-scm.org/wiki/ContributingChanges#Submission_checklist

If you can't run the entire testsuite for some reason (it can be
difficult on Windows), please at least run `contrib/check-code.py` on
any files you've modified and run `python contrib/check-commit` on any
commits you've made (for example, `python contrib/check-commit
273ce12ad8f1` will report some style violations on a very old commit).