1 |
On nie, 2017-04-30 at 06:03 +1200, Kent Fredric wrote: |
2 |
> On Fri, 28 Apr 2017 16:39:45 +0200 |
3 |
> Michał Górny <mgorny@g.o> wrote: |
4 |
> |
5 |
> > Change the unset value tag to '@DEFAULT-UNSET' to ensure consistent |
6 |
> > use of hyphen/underscore throughout eclassdoc. Before, one tag |
7 |
> > (@ECLASS-VARIABLE) has used hyphen while also one (@DEFAULT_UNSET) |
8 |
> > used underscore. Unify them to use the former since @ECLASS-VARIABLE |
9 |
> > tag is more common (and hyphens do not require holding shift). |
10 |
> > |
11 |
> > Fixing all existing uses is perfectly within our power; however, I |
12 |
> > think it would be reasonable to delay it and combine with other |
13 |
> > eclass changes to avoid unnecessary cache regen. The script still |
14 |
> > allows the old tag name for compatibility. |
15 |
> |
16 |
> I have a counter suggestion: |
17 |
> |
18 |
> 1. Leave @ECLASS-VARIABLE as-is |
19 |
> 2. Leave @DEFAULT_UNSET as is |
20 |
> 3. Document that underscores are to be used for all new tags |
21 |
|
22 |
Don't forget to write a complete policy on what names for ebuild |
23 |
variables should developers use. And then a policy on making new |
24 |
policies regarding vocabulary. And maybe document some on preferred |
25 |
words to be used in mailing list messages. |
26 |
|
27 |
> 4. Add support for @ECLASS_VARIABLE that works the same as |
28 |
> @ECLASS-VARIABLE |
29 |
> 5. Don't go out of our way to migrate to @ECLASS_VARIABLE, just let it |
30 |
> occur over time, particularly in conjunction with other major |
31 |
> changes. |
32 |
> |
33 |
> Mostly because @FOO_VARIABLE is suspiciously similar syntax to me as |
34 |
> other ALL CAPS variables used as ENV tokens in Bash. |
35 |
> |
36 |
> And I can't think of a single instance where I've seen a language with |
37 |
> a convention that used ALL CAPS terms in conjunctions with hyphens, |
38 |
> ( and regex with _ are ultimately simpler to reason about than ones |
39 |
> with - ) |
40 |
|
41 |
From now on I'm not going to listen to any suggestions that do not come |
42 |
with a patch. |
43 |
|
44 |
-- |
45 |
Best regards, |
46 |
Michał Górny |