Use Bitwise AND to set permissions in utils/unpacking.py

[deveshks]

Originally triaged in #8164

We are using 0o777 - current_umask() to set file permissions, but we should be using a Bitwise AND (0o777 & ~current_umask())

pip/src/pip/_internal/utils/unpacking.py

Line 148 in 43426ee

os.chmod(fn, (0o777 - current_umask() | 0o111))

pip/src/pip/_internal/utils/unpacking.py

Line 232 in 43426ee

os.chmod(path, (0o777 - current_umask() | 0o111))

We would also want to move the two os.chmod calls into a helper function.

[deveshks]

Hi @sbidoul , @pradyunsg

Do we still want to make the above mentioned changes? If yes, I can go ahead and make a PR for the same.

[sbidoul]

@deveshks yes, please go ahead.

[deveshks]

@deveshks yes, please go ahead.

Thanks, I have filed the PR. Please have a look