From: ebiederm@xmission.com (Eric W. Biederman)
To: Eric Wong <e@80x24.org>
Cc: meta@public-inbox.org
Subject: Re: Warnings from git fsck after lkml import
Date: Thu, 05 Jul 2018 19:36:20 -0500 [thread overview]
Message-ID: <87601turnf.fsf@xmission.com> (raw)
In-Reply-To: <20180705231346.GA6524@dcvr> (Eric Wong's message of "Thu, 5 Jul 2018 23:13:46 +0000")
Eric Wong <e@80x24.org> writes:
> "Eric W. Biederman" <ebiederm@xmission.com> wrote:
>> It looks like public-inbox has some challenges when importing some
>> questionable emails. The import of lkml has resulted in several commits
>> with bad dates that git fsck complains about. I have previously
>> reported this to Konstantin Ryabitsev who maintains kernel.org but since
>> I have not seen any discussion of this I thought I should report it
>> directly here as well.
>
> Thanks for bringing this up publically.
>
> Yes, I early during v2 development I noticed old mails had some
> -1400 timezone values (but the furthest is -1200). I opted to
> attempt to preserve the wonky timezones since fast-import
> happily accepts -1400 and I didn't anticipate problems...
I think 0.git was generated after your earlier fix.
Looking at the commits in question this is a different issue.
On some of the later ones I am really not certain what it is
but here is a representative sample you can look at.
email: Date: Wed, 13 Dec 2006 10:26:38 +1
git: Date: Wed Dec 13 09:27:38 2006 +0001
email: Date: Sun, 10 Sep 2006 23:18:30 +1
git: Date: Sun Sep 10 22:19:30 2006 +0001
email: Date: Fri, 3 Feb 2006 18:11:22 -00
git: Date: Fri Feb 3 18:11:22 2006 +0000
email: Date: Mon, 06 Jun 2005 13:59:56 +1
git: Date: Mon Jun 6 13:00:56 2005 +0001
email: Date: Thursday, 20 Feb 2003 01:14:34 +000
git: Date: Thu Feb 20 01:14:34 2003 +0000
# The error in this one does not immediate stand out
commit: 0f66db71a06bd7d651a0cd80877d8043b70fda20
email: Date: Fri, 28 Jun 2002 12:54:40 -700
git: Date: Fri Jun 28 12:54:40 2002 -0700
commit e8d3b19a91a2d86b6a91bd19dc811e851398b519
email: Date: Sat, 12 Jan 2002 12:52:57 -200
git: Date: Sat Jan 12 12:52:57 2002 -0200
commit afd9fc0cc87e56ed7736d633e17d0ef77817b3cc
email: Date: Thu, 10 Jan 2002 09:56:29 -200
git: Date: Thu Jan 10 09:56:29 2002 -0200
commit 811b3217708358cf1b75fba4602a64a426fce0f5
email: Date: Mon, 05 Nov 2001 10:36:16 -800
git: Date: Mon Nov 5 10:36:16 2001 -0800
commit e7a751a597c6f5e4770c61bdee6220d55a37cba9
email: Date: Sun, 09 Sep 2001 09:59:05 -800
git: Date: Sun Sep 9 09:59:05 2001 -0800
commit 3e32ad6192fe093e03e6b9346c3a90b16d9905c0
email: Date: Monday, 09 Apr 2001 23:50:10 +000
git: Date: Mon Apr 9 23:50:10 2001 +0000
>>
>> At a practical level these errors initially preventing me from cloning
>> the repos as in .gitconfig I had:
>> > [transfer]
>> > fsckobjects = true
>> > [fetch]
>> > fsckobjects = true
>> > [receive]
>> > fsckobjects = true
>
> ...But I didn't know people cared to set those :x
>
> Now I wonder if git should only warn for bad-but-still-usable
> objects on clone, as I wouldn't consider a malformed date to be
> on the level as actual FS corruption. Or at least complete
> the clone and fail with a special exit code.
I have added:
[fsck]
badTimezone = warn
To hopefully achieve that.
Eric
next prev parent reply other threads:[~2018-07-06 0:36 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-05 5:40 Warnings from git fsck after lkml import Eric W. Biederman
2018-07-05 23:13 ` Eric Wong
2018-07-06 0:36 ` Eric W. Biederman [this message]
2018-07-06 3:47 ` Eric W. Biederman
2018-07-06 21:32 ` [PATCH] MsgTime.pm: Use strptime to compute the time zone Eric W. Biederman
2018-07-06 22:22 ` Eric Wong
2018-07-07 18:18 ` Eric W. Biederman
2018-07-07 18:22 ` [PATCH] Import: Don't copy nulls from emails into git Eric W. Biederman
2018-07-08 0:07 ` Eric Wong
2018-07-08 1:52 ` Eric W. Biederman
2018-07-12 18:31 ` Warnings from git fsck after lkml import Konstantin Ryabitsev
2018-07-12 22:19 ` Eric W. Biederman
2018-07-12 22:29 ` Eric Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://public-inbox.org/README
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87601turnf.fsf@xmission.com \
--to=ebiederm@xmission.com \
--cc=e@80x24.org \
--cc=meta@public-inbox.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).