[issue32773] distutils should NOT preserve timestamps

Steve Dower added the comment: Distutils is now deprecated (see PEP 632) and all tagged issues are being closed. From now until removal, only release blocking issues will be considered for distutils. If this issue does not relate to distutils, please remove the component and reopen it. If

[issue32773] distutils should NOT preserve timestamps

Jeroen Demeyer added the comment: I started a discussion at https://mail.python.org/mm3/archives/list/distutils-...@python.org/thread/4FHEGHZYWCDRWVPGYLAU5VUS5BAX73MO/ -- ___ Python tracker

[issue32773] distutils should NOT preserve timestamps

Change by Jay Yin : -- pull_requests: +5483 ___ Python tracker ___ ___

[issue32773] distutils should NOT preserve timestamps

Change by Jay Yin : -- keywords: +patch pull_requests: +5401 stage: -> patch review ___ Python tracker ___

[issue32773] distutils should NOT preserve timestamps

Jay Yin added the comment: would it be ok for me to make a patch for this as practice?, I will be making a PR though, I will be making the -- option available for backward compatibility. -- ___ Python tracker

[issue32773] distutils should NOT preserve timestamps

R. David Murray added the comment: We have a strong backward compatibility policy, especially for distutils, so even though you can't imagine a use case I think we should keep the default the same. You could try lobbying the folks at pypa and see if they disagree with

[issue32773] distutils should NOT preserve timestamps

Jeroen Demeyer added the comment: > if so wouldn't that still require an internal option? No, you just need to change the calls to the copy_file() function to add the keyword argument preserve_times=False If you agree, then I could easily provide a patch. --

[issue32773] distutils should NOT preserve timestamps

Jay Yin added the comment: so what you're proposing, is that the function copy_file() itself, use defaults, but make it so that any functions that reference it not save the timing?, if so wouldn't that still require an internal option? --

[issue32773] distutils should NOT preserve timestamps

Jeroen Demeyer added the comment: I am genuinely curious to hear a good reason why timestamps should be preserved by distutils. I cannot really imagine what could possibly break. As I said, this is the standard for many non-Python projects and it seems to work just

[issue32773] distutils should NOT preserve timestamps

Jay Yin added the comment: That's why I said to have the default be the old setting, so that any previous setups wouldn't need to changes (aka backward compat.) the option would give people the "option" to choose. -- ___

[issue32773] distutils should NOT preserve timestamps

Jay Yin added the comment: I can tackle this if it's alright? -- ___ Python tracker ___

[issue32773] distutils should NOT preserve timestamps

R. David Murray added the comment: That's the right thing in your opinion[*], but probably not in the opinion of others, thus the need for an option. There is also backward compatibility to be considered. -- nosy: +r.david.murray

[issue32773] distutils should NOT preserve timestamps

Jeroen Demeyer added the comment: This is a bug report and not a feature request. I don't think that there should be an option. It should just do the right thing, which is NOT preserving timestamps. -- ___ Python tracker

[issue32773] distutils should NOT preserve timestamps

Jay Yin added the comment: Will an option be added to copy_file() function? if we did we could probably make it preserve the times by default and have an -- option to make it update? -- nosy: +jayyin11043 ___ Python tracker

[issue32773] distutils should NOT preserve timestamps

Change by Stéphane Wirtel : -- nosy: +matrixise ___ Python tracker ___ ___

[issue32773] distutils should NOT preserve timestamps

New submission from Jeroen Demeyer : When a Python project is installed, distutils copies the files from the build to install directory using copy_file(). In this copy operation, timestamps are preserved. In other words, the timestamp of the installed file equals the