vcs-fast-import-devs team mailing list archive
-
vcs-fast-import-devs team
-
Mailing list archive
-
Message #00038
Re: [PATCH v7 3/6] fast-import: add feature command
Heya,
On Sat, Sep 12, 2009 at 20:51, Shawn O. Pearce <spearce@xxxxxxxxxxx> wrote:
> Where is the documentation for 'feature date-format=<FORMAT>'?
D'oh, I forgot, my bad.
> Also, IIRC the fast-import list agreed that the <feature> name must
> match the re ^[a-zA-Z][a-zA-Z-]*$.
Ah, yes, of course, I'll update that.
> Saying that here does somewhat
> help another fast-import developer to use the same stream format,
> but it does not help a user to understand what features they can
> ask for in their stream.
No, you're absolutely right, there should be a section on supported features.
> So its legal to change the data format in the middle of a stream?
I'll add a !seen_non_option_command check to parse_feature, perhaps
renaming it to 'seen_data_command'?
--
Cheers,
Sverre Rabbelier
References