(original) (raw)

Hi,

I'm currently working on exposing posix\_spawn in the posix module (and by extension in the os module). You can find the initial implementation in this PR:

https://github.com/python/cpython/pull/5109

As pointed out by Gregory P. Smith, some changes are needed in the way the file\_actions arguments is passed from Python. For context, posix\_spawn has the following declaration:

int posix\_spawn(pid\_t \*pid, const char \*path,
const posix\_spawn\_file\_actions\_t \*file\_actions,
const posix\_spawnattr\_t \*attrp,
char \*const argv\[\], char \*const envp\[\]);

Here, file\_actions is an object that represents a list of file actions (open, close or dup2) that is populated using helper functions on the C API.

The question is: what is the best way to deal with this argument?

Following Gregory's comment on the PR I understand that he is proposing to have three objects in the os module representing each action and pass a sequence of these objects to the Python API. What I am not sure about this is that there is no previous example of such classes in the os module for other similar APIs and therefore I am not sure if there is a better approach.

Thanks you very much for your time!