On Fri, 17 Feb 2023 at 03:39, Junio C Hamano <gitster@xxxxxxxxx> wrote: > > I do wonder if people would complain (both with the patch above and with > > brian's proposal) that the resulting tarballs extract everything with a > > date in 1970. That's not functionally a problem, but it looks kind of > > weird in "ls -l". > > And owned by root:root ;-) > > I am sure people would complain. What matters is if these > complaints have merit, and in this case, I doubt it. I especially > like your "it has been already changing once per second" reasoning > for this change. I don't really get the argument as far as it is presented as a reason /not/ to support a --mtime option as was requested in this thread. You yourself say "people will complain", and you seem to be planning to just ignore those complaints as lacking merit. I honestly don't understand that, wouldn't it be less noise, less stress, and better "customer service" to let the user choose what constant is in the archive? Then no complaints, and you don't have to explain/argue to/with everybody why their complaint is without merit over and over. I could understand this position if letting the user set the --mtime implied some harm that must be mitigated, but it seems like an odd choice if there is none. Especially given it would also future proof git against people coming up with a good reason not to use the 0 epoch in the future that you haven't thought of right now. It is not like epoch's and unix time have a totally uncontroversial and stable history. 0 has meant multiple dates over time, and the current definition of 1 Jan 1970, 00:00:00 UTC is problematic as UTC didn't exist until 1972! Given it clearly wouldn't be hard to allow users to select the epoch in these archives then why not do so? I have seen devs have issues with stuff like this in the past. Unpacking an archive on one machine showing a different date than one another, or other weird artifacts. Mac used to use a different 0 epoch than windows and linux as I recall, etc etc. I dont remember the gory details, but i have definitely seen people gnash their teeth over these kind of decisions before. Why not side-step that if you can and let people choose their own defaults? Just my $0.02. cheers, Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"
