linux-ext4 - Re: [PATCH] e2fsprogs: Introduce script used to generate utf8data.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20181208181324.GC20708@thunk.org>
Date:   Sat, 8 Dec 2018 13:13:24 -0500
From:   "Theodore Y. Ts'o" <tytso@....edu>
To:     Gabriel Krisman Bertazi <krisman@...labora.com>
Cc:     linux-ext4@...r.kernel.org, kernel@...labora.com
Subject: Re: [PATCH] e2fsprogs: Introduce script used to generate utf8data.h

On Mon, Dec 03, 2018 at 06:04:28PM -0500, Gabriel Krisman Bertazi wrote:
> The mkutf8data program was written by SGI for the kernel. We integrate
> it here, but it should be kept in sync with the kernel code, when
> applicable.  The header generated by this script is also included in
> this repository, such that we don't have to carry the UCD files too.
> They are only necessary when the header has to be recreated (for
> instance, to update the Unicode version supported)
> 
> The mkutf8data is not a dependency of utf8data.h because we don't want
> the utf8data.h rule to be called after a make clean that would remove
> mkutf8data.  It is simpler to call it from inside the rule, and make
> sure we only trigger the utf8data.h rule if we really need to,
> particularly if we decide not to integrate the ucd files in the tree.
> 
> The UCD files are not distributed along with this commit, but we add
> information on how to fetch them.
> 
> Signed-off-by: Gabriel Krisman Bertazi <krisman@...labora.com>

Thanks, applied, with one minor fixup:

> diff --git a/util/ucd/README b/util/ucd/README
> new file mode 100644
> index 000000000000..9fed514084b7
> --- /dev/null
> +++ b/util/ucd/README
...
> +md5sums
> +
> +  414436796cf097df55f798e1585448ee  CaseFolding-11.0.0.txt
> +  6032a595fbb782694456491d86eecfac  DerivedAge-11.0.0.txt
> +  3240997d671297ac754ab0d27577acf7  DerivedCombiningClass-11.0.0.txt
> +  d41d8cd98f00b204e9800998ecf8427e  DerivedCombiningClass.txt
                                       ^^^^^^^^^^^^^^^^^^^^^^^^^

This looks like a leftover, so I removed it.

     	   	  	       	 - Ted