Skip to content

Commit 21d5669

Browse files
authored
confusables: no more google forms for suggestions (#1197)
1 parent 1399fcc commit 21d5669

File tree

1 file changed

+7
-24
lines changed

1 file changed

+7
-24
lines changed

docs/security/index.md

Lines changed: 7 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,7 @@ source/confusables-winFonts.txt. The main file is confusables-source.txt.
1515
results. Sometimes you may get an unexpected union of two equivalence sets.
1616
Look at Testing below for help.***
1717

18-
Look at the following spreadsheets / bugs to see if there are any additional
19-
suggestions.
20-
21-
* **[Confusable
22-
Suggestions](https://docs.google.com/spreadsheet/ccc?key=0ArRWBHdd5mx-dHRXelRVbXRYSVp2QTNDdTBlV1I5X1E&usp=drive_web#gid=0)**
23-
* **[Identifier Restriction
24-
Suggestions](https://docs.google.com/spreadsheet/ccc?key=0ArRWBHdd5mx-dEJJWkdzZzk4cDRYbEVLTmhraGN0Q3c&usp=drive_web#gid=0)**
25-
* *[Sample PRs](https://github.com/unicode-org/unicodetools/pull/841)
26-
27-
If so, assess and add to unicodetools/data/security/{version}/data/source/confusables-source.txt — *if needed.*
28-
Then in the spreadsheets, move the "new stuff" line to the end.
18+
[Sample PR for confusables changes](https://github.com/unicode-org/unicodetools/pull/841)
2919

3020
### File Format
3121
There is a brief description of the file format at the top.
@@ -61,6 +51,9 @@ http://cldr.unicode.org/development/updating-codes/updating-script-metadata
6151

6252
The identifier type & status take this data into account.
6353

54+
We need a recent CLDR repo snapshot, and update our Maven dependency to use that.
55+
When there is a blocker for updating our CLDR dependency, we sometimes need to hack new script data into our tools code.
56+
6457
## Generating
6558

6659
Fix the version string (which will appear inside GenerateConfusables.java) and
@@ -78,7 +71,7 @@ Run GenerateConfusables -c -b to generate the files. They will appear in two pla
7871
The TestSecurity.java test is part of the unit test suite, run by a github CI.
7972
It verifies that the confusable mappings are idempotent.
8073

81-
Copy the following from the output directory to the top level of the revision directory, and check in.
74+
Copy the following from the output directory to the top level of the revision directory, and check in. Omit files where only the time stamp changes, unless this is the first set of files in a year.
8275

8376
* confusables.txt
8477
* confusablesSummary.txt
@@ -109,7 +102,7 @@ Markus 2020-feb-07 for Unicode 13.0:
109102

110103
You may see Identifier_Type=Recommended for characters/scripts/blocks that should not be recommended.
111104
For example, the initial generation for Unicode 14 "recommended" Znamenny combining marks.
112-
Add these to unicodetools/data/security/{version}/data/source/removals.txt.
105+
Add these to unicodetools/data/security/dev/data/source/removals.txt.
113106
You can use block properties like
114107
```
115108
\p{block=Znamenny_Musical_Notation} ; technical
@@ -202,14 +195,4 @@ with reason `[[arabic]] plus [[arabic]→𞻰, [arabic]]`
202195

203196
## Posting
204197

205-
Once you've resolved all the problems, copy certain generated files to
206-
https://www.unicode.org/Public/security/{version}/
207-
208-
* confusables.txt
209-
* confusablesSummary.txt
210-
* confusablesWholeScript.txt
211-
* intentional.txt
212-
* ReadMe.txt
213-
* xidmodifications.txt
214-
215-
Check that the files are copied to https://www.unicode.org/Public/security/{version}/.
198+
The security files are published together with other data files for beta & final, see [Data Files Workflow](../data-workflow.md).

0 commit comments

Comments
 (0)