-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow empty name_suffix
#11001
base: master
Are you sure you want to change the base?
allow empty name_suffix
#11001
Conversation
Dot is still present at end of file name if suffix is empty. More work needed. |
Wouldn't the correct file extension for kernel modules actually be |
This is true for Linux, but not necessary for another operating systems. For example in Haiku kernel modules have no any prefix or suffix and module file name is used to find requested module. Haiku also do not use any prefix or suffix for various system extension plugins (add-ons) and userspace drivers. |
It is required for plugins and kernel modules that are looked up by file name and require to specify exact file name. Some plugin systems do not use any prefix or suffix.
@@ -1818,25 +1816,27 @@ def post_init(self) -> None: | |||
if not hasattr(self, 'suffix'): | |||
# Executable for Windows or C#/Mono | |||
if machine.is_windows() or machine.is_cygwin() or 'cs' in self.compilers: | |||
self.suffix = 'exe' | |||
self.suffix = '.exe' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be best not to change this. Instead where the suffix is concatenated to the file name, conditionally add a dot.
It is required for plugins and kernel modules that are looked up by file name and require to specify exact file name. Some plugin systems do not use any prefix or suffix.