bug#61292: define-public fontconfig vs. fontconfig-minimal as its name

bug#61292: define-public fontconfig vs. fontconfig-minimal as its name

[Attila Lendvai]

(define-public fontconfig
[...]
     (name "fontconfig-minimal")

this tripped me up. is this a mistake?

if not, then please add a comment that explains this anomaly.

-- 
• attila lendvai
• PGP: 963F 5D5F 45C7 DFCD 0A39
--
“We're all going to die, all of us, what a circus! That alone should make us love each other but it doesn't. We are terrorized and flattened by trivialities, we are eaten up by nothing.”
	— Charles Bukowski (1920–1994)

bug#61292: define-public fontconfig vs. fontconfig-minimal as its name

[Maxim Cournoyer]

Hi,

Attila Lendvai <attila@lendvai.name> writes:

> (define-public fontconfig
> [...]
>      (name "fontconfig-minimal")
>
> this tripped me up. is this a mistake?
>
> if not, then please add a comment that explains this anomaly.

There's already a comment above fontconfig-with-documentation that
reads:

  This is intentionally named 'fontconfig', as it's intended as the
  user-facing fontconfig package

The original fontconfig was renamed fontconfig-minimal but the variable
was not renamed to avoid having to adjust the rest of the code base.  It
could be done, and then the fontconfig-with-documentation *variable*
could be renamed to simply fontconfig.

That'd be more conventional.

Would you like to give it a try?  It shouldn't cause any rebuild, I
think.

-- 
Thanks,
Maxim

bug#61292: fontconfig -> fontconfig-minimal

[Attila Lendvai]

close 61292
stop

i started doing the rename of the variable, but the change quickly grew extensive, so i gave up. it's probably not worth the trouble.

there are 332 grep matches, and the majority requries a change.

-- 
• attila lendvai
• PGP: 963F 5D5F 45C7 DFCD 0A39
--
“Beware the stories you read or tell; subtly, at night, beneath the waters of consciousness, they are altering your world.”
	— Ben Okri (1959–)