On Sat, 4 Aug 2018 at 10:05, Jeroen Demeyer <j.deme...@ugent.be> wrote: > > On 2018-08-04 10:34, Paul Moore wrote: > > Jeroen seemed to say he agreed with this, but > > I'm not sure I see how that matches his stated requirement for > > installs to not preserve timestamps... > > The way how pip currently works (I assume that this stays the case) is > that it uses a temporary build directory for everything, for example > /tmp/pip-build-uvfWuR > > So concretely for a wheel install: > > (A) Extract the wheel in build directory > > (B) Generate .pyc files in build directory > > (C) Copy files from build directory to final installation directory in > site-packages. > > The way I understand it, https://github.com/pypa/pip/issues/5648 is > about (A) while my proposal is about (C). So I think that timestamps > should be preserved in (A) but not for (C).
Ah, OK. At the moment I believe we preserve in (C) - see https://github.com/pypa/pip/blob/master/src/pip/_internal/wheel.py#L304. The original code here just used shutil.move, but there were changes over time to restrict it to only copying certain metadata (specifically timestamps). But it's always preserved timestamps back to the original implementation in 2012. I still have no opinion on what's "right" here. Paul -- Distutils-SIG mailing list -- distutils-sig@python.org To unsubscribe send an email to distutils-sig-le...@python.org https://mail.python.org/mm3/mailman3/lists/distutils-sig.python.org/ Message archived at https://mail.python.org/mm3/archives/list/distutils-sig@python.org/message/IJRLGMZQZTX2DL2NVECWDVSP2WCM5D6T/
