I use this context manager in my code:

@contextmanager
def in_directory(path):
    pwd = str(Path().absolute())
    if not path.is_dir():
        path = path.parent
    os.chdir(str(path))
    yield path.absolute()
    os.chdir(pwd)

I thought it would be nice to have something like this in the pathlib as a method of Path library, I find it quite convenient, especially when dealing with subprocesses.

I'm not a Python core developer, but a few thoughts:
* this is not thread safe
* subprocess.Popen already has a cwd parameter for similar purpose

I did not know about cwd for Popen. It seems like the better way to go.
Thank you.

2015-11-14 19:58 GMT+01:00 desbma <report@bugs.python.org>:

>
> desbma added the comment:
>
> I'm not a Python core developer, but a few thoughts:
> * this is not thread safe
> * subprocess.Popen already has a cwd parameter for similar purpose
>
> ----------
> nosy: +desbma
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <http://bugs.python.org/issue25625>
> _______________________________________
>

What's the relation with subprocess.Popen ?

I agree with desbma.

> What's the relation with subprocess.Popen ?

In the context of subprocesses there is better way to run a program in specified working directory that temporary change current working directory in parent process.

> In the context of subprocesses there is better way to run a program in specified working directory that temporary change current working directory in parent process.

Ok, haven't read the last line of the description. thanks for the
clarification.
-- 
Stéphane Wirtel - http://wirtel.be - @matrixise

So moving way from subprocess.Process; what about having this for other use cases? I know there is no thread-safe way to get this working, given it's a process state variable. However, can we have it as a non-thread-safe feature?

New changeset 3592980f9122ab0d9ed93711347742d110b749c2 by Filipe Laíns in branch 'main':
bpo-25625: add contextlib.chdir (GH-28271)
https://github.com/python/cpython/commit/3592980f9122ab0d9ed93711347742d110b749c2

Thanks for perservering, Filipe! ✨ 🍰 ✨ 

Per Steering Council decision, the context manager is approved: https://github.com/python/steering-council/issues/77

New changeset a774285e7d3e63c24dbaaee0b0d9e59ded5c49bf by Thomas Grainger in branch 'main':
bpo-25625: [doc] fix async/aync typo (GH-29091)
https://github.com/python/cpython/commit/a774285e7d3e63c24dbaaee0b0d9e59ded5c49bf

There is renewed discussion on python-dev about placing this in contextlib.

Can you share the link? I haven't seen anything recent. Is it under other thread?

On 15Nov2021 01:04, Python Bug Reports <report@bugs.python.org> wrote:
>Can you share the link? I haven't seen anything recent. Is it under 
>other thread?

It's in the discuss-ideas part of discuss.python.org during a 
discussions about a possible new context manager to atomically prepare a 
filename. Latest comment:

    https://discuss.python.org/t/adding-atomicwrite-in-stdlib/11899/15

and I'm with Inada-san here: I think shutil is a better fit for both the 
chdir context manager and the thing under discussion. They're both "high 
level shell-like things" to my mind.