[issue37152] Add AF_LOCAL alias for AF_UNIX

[Christian Heimes]

Christian Heimes  added the comment:

You are welcome! :)

--
resolution:  -> rejected
stage: patch review -> resolved
status: open -> closed
type:  -> enhancement

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37152] Add AF_LOCAL alias for AF_UNIX

[carlo]

carlo  added the comment:

> I'm -1 to add the alias. It just adds to the confusion. It's also against the 
> Zen of Python: "There should be one-- and preferably only one --obvious way 
> to do it."

Agreed with the zen of python.

This is my first PR / issue, thanks for the fast reply.

You may close this issue )

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37152] Add AF_LOCAL alias for AF_UNIX

[Christian Heimes]

Christian Heimes  added the comment:

The socket type is known as AF_UNIX, AF_LOCAL, Unix domain socket, and IPC 
(inter process communication) socket. The man page unix(7) 
http://man7.org/linux/man-pages/man7/unix.7.html uses three of those four terms 
to explain the socket type:

The AF_UNIX (also known as AF_LOCAL) socket family is used to
communicate between processes on the same machine efficiently.
Traditionally, UNIX domain sockets can be [...]

The strace command also uses the name AF_UNIX and not AF_LOCAL. HTTP over 
AF_UNIX often uses http+unix:// or unix:// to refer to a socket file.

I'm -1 to add the alias. It just adds to the confusion. It's also against the 
Zen of Python: "There should be one-- and preferably only one --obvious way to 
do it."

--
nosy: +christian.heimes
versions: +Python 3.9 -Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37152] Add AF_LOCAL alias for AF_UNIX

[carlo]

Change by carlo :


--
keywords: +patch
pull_requests: +13695
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/13810

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37152] Add AF_LOCAL alias for AF_UNIX

[carlo]

New submission from carlo :

both mean the same thing and are used to reference local socket files

if you do a `man socket`

one of the lines says

PF_LOCALHost-internal protocols, formerly called PF_UNIX,

(AF and PF mean the same thing here, but importantly it shows that PF_UNIX has 
been depreciated)

this change gives users the option to type socket.AF_LOCAL as well as 
socket.AF_UNIX when specifying the address family for the socket.

this is my first issue so lmk if there is anything I'm missing )

--
components: Library (Lib)
messages: 344585
nosy: frosty00
priority: normal
severity: normal
status: open
title: Add AF_LOCAL alias for AF_UNIX
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com