GeezerVison

Rebase?

I’ll rebase and change the call to os.path.abspath().

Edit: Done. I ended up adding the os.path.abspath() call later in the function, since the non-absolute s is needed in early parts of the function. I also took the liberty to refactor the function slightly to avoid very deep indentations in the function.

OK, I’ve

1. Move the unit test under _BasePathTest so it is run for all platforms.
2. Remove the _getfinalpathname check since it should now be available everywhere. This also allows some additional cleanups that aren’t exactly related to the .resolve() fix.
3. Use _getfullpathname instead of abspath as suggested.

I split each of these changes into individual commits so they can be easily reverted if needed.

Will this be in 3.9?

Any progress?

Is this just blocked on reviews?

Is this a case of letting the best be the enemy of the good? This bug is a real time-waster and can easily slip into cross-platform code. If the issues @eryksun highlights are pre-existing then perhaps you should merge it and then fix those separately?

@prescod, I just created bpo-43455 for the os.getcwd() issue. It should not hold this issue up.

Is it too late for this to be in 3.10?

@ofek, this is a bug fix, not a new feature. Even if it were, new features can be added to 3.10 until the first beta release in May. Hopefully it gets merged into the 3.8 branch in time for the 3.8.9 release in May, since it's the last bug-fix release for 3.8.

Strong nudge.

Hopefully it gets merged into the 3.8 branch in time for the 3.8.9 release in May

This is our last chance.

Needs one final rebase

I’ve rebased this multiple times and this time the new incoming changes is too much for me. I give up. Someone else can take over.