I've confirmed this bug is present on both windows and linux, the outputs below are from linux however.

Compare:

```
$ python3.4 --version
Python 3.4.3
$ python3.4 -c 'import os; print(os.unlink == os.remove)'
True
```

```
$ python3.5 --version
Python 3.5.0
$ python3.5 -c 'import os; print(os.unlink == os.remove)'
False
```

The docs say: https://docs.python.org/3/library/os.html#os.remove

"This function is identical to unlink()."

To me identity means `is` but I at least expect the `==` behaviour of previous versions.

This is due to using argument clinic in Modules/posixmodule.c:

    /*[clinic input]
    os.remove = os.unlink

builtin_function_or_method instances are equal if m_self (the module in this case) and m_ml->ml_meth (the C function) are the same. In 3.4, the function posix_unlink is used for both os.unlink and os.remove, which is why they compare as equal. In 3.5, argument clinic defines separate os_unlink and os_remove implementations in Modules/clinic/posixmodule.c.h.

Unless you can explain what bugs this is causing, I don't see any need to change the behavior.

Breaks this function:

```
def rmtree(path):
    """On windows, rmtree fails for readonly dirs."""
    def handle_remove_readonly(func, path, exc):  # pragma: no cover (windows)
        excvalue = exc[1]
        if func in (os.rmdir, os.remove) and excvalue.errno == errno.EACCES:
            os.chmod(path, stat.S_IRWXU | stat.S_IRWXG | stat.S_IRWXO)
            func(path)
        else:
            raise
    shutil.rmtree(path, ignore_errors=False, onerror=handle_remove_readonly)
```

How does it break?  Maybe you could explain more.

When calling shutil.rmtree on windows on a readonly directory, the error handler is called with os.unlink as the first argument `func` which fails the check `func in (os.rmdir, os.remove)` which succeeded on previous python versions

I think it's reasonable to say the documentation could be misconstrued -- as Anthony has shown -- and so a patch changing the os.remove docs to say "This function is semantically identical to os.unlink()" would be acceptable.

Please check this. Fixed according to previous comment.

New changeset 81c7b26f5b4e by Brett Cannon in branch 'default':
Issue #25930: Document that os.unlink and os.remove are *semantically* identical.
https://hg.python.org/cpython/rev/81c7b26f5b4e

New changeset 9f13322eba8e by Brett Cannon in branch '3.5':
Backport of fix for issue #25930
https://hg.python.org/cpython/rev/9f13322eba8e

New changeset e082519bc2c8 by Brett Cannon in branch 'default':
Merge for backport  of fix for issue #25930
https://hg.python.org/cpython/rev/e082519bc2c8

I tweaked Swati's patch as there was another spot where the "identical" line was used. Thanks to everyone who helped out with this!