You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Robert Muir (Jira)" <ji...@apache.org> on 2021/04/10 18:26:00 UTC

[jira] [Commented] (LUCENE-9923) remove timestamp from generated ASCIITLD.jflex

    [ https://issues.apache.org/jira/browse/LUCENE-9923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17318578#comment-17318578 ] 

Robert Muir commented on LUCENE-9923:
-------------------------------------

I think we just want to remove the {{generated on}} date here (which changes every time you run it regardless of inputs) and keep the {{Last-Modified}} date, which corresponds to the root.zone input file.

> remove timestamp from generated ASCIITLD.jflex
> ----------------------------------------------
>
>                 Key: LUCENE-9923
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9923
>             Project: Lucene - Core
>          Issue Type: Task
>            Reporter: Robert Muir
>            Priority: Major
>
> This causes the generation to always make changes to the file, even if the list of TLDs has changed. Let's avoid this as generating the resultant tokenizer is not trivial :)
> Also if we fix this, it allows us to run ./gradlew regenerate --rerun-tasks and have no local changes (idempotency everywhere)
> Diffs currently look like this:
> {noformat}
> --- a/lucene/analysis/common/src/java/org/apache/lucene/analysis/email/ASCIITLD.jflex
> +++ b/lucene/analysis/common/src/java/org/apache/lucene/analysis/email/ASCIITLD.jflex
> @@ -16,7 +16,7 @@
>   */
>  // Generated from IANA Root Zone Database <https://www.internic.net/zones/root.zone>
>  // file version from 2021 Apr 10, Sat 17:37:00 Coordinated Universal Time
> -// generated on 2021 Apr 10, Sat 17:55:26 Coordinated Universal Time
> +// generated on 2021 Apr 10, Sat 18:13:07 Coordinated Universal Time
>  // by org.apache.lucene.analysis.standard.GenerateJflexTLDMacros
>  // LUCENE-8278: None of the TLDs in {ASCIITLD} is a 1-character-shorter prefix of another TLD
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org