This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] [BZ 14094] Update locale data to Unicode 7.0.0


On Wed, 25 Jun 2014, Pravin Satpute wrote:

> These classes now disable in gen-unicode-ctype.c and also moved this 
> file to scripts/

If you move and change a file at the same time, you should use "git diff 
-M" so the changes to the file are shown, rather than separate deletion 
and addition of the file.  But I don't think moving to scripts/ is a good 
idea.  I think scripts/ should be for scripts relating to glibc as a 
whole, and something specific to a particular subdirectory should go in 
that subdirectory - so the new scripts should go in localedata/, and the 
existing file stay there.

(That principle of scripts/ only being for scripts relating to glibc as a 
whole would imply that some existing files in scripts/ should go 
elsewhere, e.g. those that are only used by elf/ testcases.  That's a 
separate issue, nothing to do with your patch.)

-- 
Joseph S. Myers
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]