* [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts.
@ 2020-04-19 5:24 Raghav Gururajan
2020-04-23 2:30 ` Leo Famulari
0 siblings, 1 reply; 6+ messages in thread
From: Raghav Gururajan @ 2020-04-19 5:24 UTC (permalink / raw)
To: 40708
[-- Attachment #1: Type: text/plain, Size: 0 bytes --]
[-- Attachment #2: 0002-gnu-font-gnu-unifont-Unify-outputs-for-fonts.patch --]
[-- Type: text/x-patch, Size: 2161 bytes --]
From dff7acc350ba6f2e8d6ba3b561fb58acdc1a9c00 Mon Sep 17 00:00:00 2001
From: Raghav Gururajan <raghavgururajan@disroot.org>
Date: Sun, 19 Apr 2020 01:18:41 -0400
Subject: [PATCH 2/2] gnu: font-gnu-unifont: Unify outputs for fonts.
* gnu/package/fonts.scm (font-gnu-unifont): Unify outputs for fonts.
Let us not over-do multiple outputs. Separating fonts and binaries is
sane enough. Also, GNU Unifont only have very few fonts in each format.
---
gnu/packages/fonts.scm | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/gnu/packages/fonts.scm b/gnu/packages/fonts.scm
index 0356d50c7c..77103bdd39 100644
--- a/gnu/packages/fonts.scm
+++ b/gnu/packages/fonts.scm
@@ -681,9 +681,7 @@ for use at smaller text sizes")))
(sha256
(base32 "1svzm3xahb2m8r79ha9gb1z3zlckykx9d87cghswj7dxn9868j4b"))))
(build-system gnu-build-system)
- (outputs '("out" ; TrueType version
- "pcf" ; PCF (bitmap) version
- "psf" ; PSF (console) version
+ (outputs '("out" ; Fonts
"bin")) ; Utilities to manipulate '.hex' format
(arguments
'(#:tests? #f ; no check target
@@ -696,11 +694,11 @@ for use at smaller text sizes")))
'install
(lambda* (#:key outputs #:allow-other-keys)
(let* ((ttf (string-append (assoc-ref outputs "out")
- "/share/fonts/truetype"))
- (pcf (string-append (assoc-ref outputs "pcf")
- "/share/fonts/misc"))
- (psf (string-append (assoc-ref outputs "psf")
- "/share/consolefonts"))
+ "/share/fonts/ttf"))
+ (pcf (string-append (assoc-ref outputs "out")
+ "/share/fonts/pcf"))
+ (psf (string-append (assoc-ref outputs "out")
+ "/share/fonts/psf"))
(bin (assoc-ref outputs "bin")))
(invoke "make"
(string-append "PREFIX=" bin)
--
2.26.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts.
2020-04-19 5:24 [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts Raghav Gururajan
@ 2020-04-23 2:30 ` Leo Famulari
2020-04-23 2:44 ` Raghav Gururajan
0 siblings, 1 reply; 6+ messages in thread
From: Leo Famulari @ 2020-04-23 2:30 UTC (permalink / raw)
To: Raghav Gururajan; +Cc: 40708
On Sun, Apr 19, 2020 at 01:24:41AM -0400, Raghav Gururajan wrote:
> From dff7acc350ba6f2e8d6ba3b561fb58acdc1a9c00 Mon Sep 17 00:00:00 2001
> From: Raghav Gururajan <raghavgururajan@disroot.org>
> Date: Sun, 19 Apr 2020 01:18:41 -0400
> Subject: [PATCH 2/2] gnu: font-gnu-unifont: Unify outputs for fonts.
>
> * gnu/package/fonts.scm (font-gnu-unifont): Unify outputs for fonts.
>
> Let us not over-do multiple outputs. Separating fonts and binaries is
> sane enough. Also, GNU Unifont only have very few fonts in each format.
There is a significant size difference between these outputs. Is there
any prior discussion about what they are used for? Is there something to
gain by combining them?
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts.
2020-04-23 2:30 ` Leo Famulari
@ 2020-04-23 2:44 ` Raghav Gururajan
2020-04-23 16:05 ` Leo Famulari
0 siblings, 1 reply; 6+ messages in thread
From: Raghav Gururajan @ 2020-04-23 2:44 UTC (permalink / raw)
To: Leo Famulari; +Cc: 40708
Hi Leo!
> There is a significant size difference between these outputs. Is there
> any prior discussion about what they are used for? Is there something to
> gain by combining them?
Not sure about discussion, but it is better to combine fonts and keep just
binaries separate. Because if a user install unifont to use in their
applications, not all of them gonna use ttf. Some apps gonna look for otf
formats. It would be confusing to install three different outputs for get all
formats.
Regards,
RG.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts.
2020-04-23 2:44 ` Raghav Gururajan
@ 2020-04-23 16:05 ` Leo Famulari
2020-05-11 4:54 ` Raghav Gururajan
0 siblings, 1 reply; 6+ messages in thread
From: Leo Famulari @ 2020-04-23 16:05 UTC (permalink / raw)
To: Raghav Gururajan; +Cc: 40708
On Wed, Apr 22, 2020 at 10:44:09PM -0400, Raghav Gururajan wrote:
> Not sure about discussion, but it is better to combine fonts and keep just
> binaries separate. Because if a user install unifont to use in their
> applications, not all of them gonna use ttf. Some apps gonna look for otf
> formats. It would be confusing to install three different outputs for get all
> formats.
Unifont provides TrueType, PCF, and PSF. It does not have OTF fonts.
Currently, if a user installs 'font-gnu-unifont' they get TrueType
fonts.
On the other hand, the PSF fonts can be used in the bootloader and the
Linux console, and they are only 68 kilobytes, rather than ~22 megabytes
if you combine all the font outputs. I don't see a compelling reason for
that increase.
If the problem is that packages with multiple outputs are confusing, we
should try to improve the UI, and we shouldn't use multiple outputs if
there isn't a good reason. But this is a case where I think it makes
sense.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts.
2020-04-23 16:05 ` Leo Famulari
@ 2020-05-11 4:54 ` Raghav Gururajan
2020-05-11 5:32 ` Raghav Gururajan
0 siblings, 1 reply; 6+ messages in thread
From: Raghav Gururajan @ 2020-05-11 4:54 UTC (permalink / raw)
To: Leo Famulari; +Cc: 40708
[-- Attachment #1.1.1: Type: text/plain, Size: 1398 bytes --]
Hi Leo!
Sorry for the delayed response.
> Unifont provides TrueType, PCF, and PSF. It does not have OTF fonts.
That's correct. I was suggesting a scenario of using more than one type. :-)
> Currently, if a user installs 'font-gnu-unifont' they get TrueType
> fonts.
>
> On the other hand, the PSF fonts can be used in the bootloader and the
> Linux console, and they are only 68 kilobytes, rather than ~22 megabytes
> if you combine all the font outputs. I don't see a compelling reason for
> that increase.
OK. I think of two reasons: 1) If one installs this package with
suggested patch, the user can use unifont for any purpose as they wish.
It may be application (ttf), console/grub (psf) and/or display server
(pcf). 2) The setup will be consistent with other font packages, where
the package installs fonts with all the types it can provide.
> If the problem is that packages with multiple outputs are confusing, we
> should try to improve the UI, and we shouldn't use multiple outputs if
> there isn't a good reason. But this is a case where I think it makes
> sense.
Improving UI would be good too. Regarding multiple outputs, I think it
is good enough to separate libs, bins and docs. In this packages, we
already separated bin and fonts. I suggest that we shall not separate
fonts further into it's subtypes. That is all I ask. :-)
Regards,
RG.
[-- Attachment #1.1.2: 0xAE6EF3046D6F7B57.asc --]
[-- Type: application/pgp-keys, Size: 2877 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts.
2020-05-11 4:54 ` Raghav Gururajan
@ 2020-05-11 5:32 ` Raghav Gururajan
0 siblings, 0 replies; 6+ messages in thread
From: Raghav Gururajan @ 2020-05-11 5:32 UTC (permalink / raw)
To: Leo Famulari; +Cc: 40708
[-- Attachment #1.1.1: Type: text/plain, Size: 1499 bytes --]
Hi Leo!
> Sorry for the delayed response.
>
>> Unifont provides TrueType, PCF, and PSF. It does not have OTF fonts.
>
> That's correct. I was suggesting a scenario of using more than one type. :-)
>
>> Currently, if a user installs 'font-gnu-unifont' they get TrueType
>> fonts.
>>
>> On the other hand, the PSF fonts can be used in the bootloader and the
>> Linux console, and they are only 68 kilobytes, rather than ~22 megabytes
>> if you combine all the font outputs. I don't see a compelling reason for
>> that increase.
>
> OK. I think of two reasons: 1) If one installs this package with
> suggested patch, the user can use unifont for any purpose as they wish.
> It may be application (ttf), console/grub (psf) and/or display server
> (pcf). 2) The setup will be consistent with other font packages, where
> the package installs fonts with all the types it can provide.
>
>> If the problem is that packages with multiple outputs are confusing, we
>> should try to improve the UI, and we shouldn't use multiple outputs if
>> there isn't a good reason. But this is a case where I think it makes
>> sense.
>
> Improving UI would be good too. Regarding multiple outputs, I think it
> is good enough to separate libs, bins and docs. In this packages, we
> already separated bin and fonts. I suggest that we shall not separate
> fonts further into it's subtypes. That is all I ask. :-)
I have attached revised patch with this email. :-)
Regards,
RG.
[-- Attachment #1.1.2: 0001-gnu-font-gnu-unifont-Unify-outputs-for-fonts.patch --]
[-- Type: text/x-patch, Size: 2052 bytes --]
From 70037b0f0f640c64aa2323a0bc8ba85a17c5559e Mon Sep 17 00:00:00 2001
From: Raghav Gururajan <raghavgururajan@disroot.org>
Date: Mon, 11 May 2020 01:26:43 -0400
Subject: [PATCH] gnu: font-gnu-unifont: Unify outputs for fonts.
* gnu/package/fonts.scm (font-gnu-unifont): Combine font types.
[arguments]<'install>: Change outputs for pcf and psf to "out".
---
gnu/packages/fonts.scm | 12 +++++-------
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/gnu/packages/fonts.scm b/gnu/packages/fonts.scm
index 6e9b080299..a79bdf72a6 100644
--- a/gnu/packages/fonts.scm
+++ b/gnu/packages/fonts.scm
@@ -689,9 +689,7 @@ for use at smaller text sizes")))
(sha256
(base32 "1svzm3xahb2m8r79ha9gb1z3zlckykx9d87cghswj7dxn9868j4b"))))
(build-system gnu-build-system)
- (outputs '("out" ; TrueType version
- "pcf" ; PCF (bitmap) version
- "psf" ; PSF (console) version
+ (outputs '("out" ; Fonts
"bin")) ; Utilities to manipulate '.hex' format
(arguments
'(#:tests? #f ; no check target
@@ -705,10 +703,10 @@ for use at smaller text sizes")))
(lambda* (#:key outputs #:allow-other-keys)
(let* ((ttf (string-append (assoc-ref outputs "out")
"/share/fonts/truetype"))
- (pcf (string-append (assoc-ref outputs "pcf")
- "/share/fonts/misc"))
- (psf (string-append (assoc-ref outputs "psf")
- "/share/consolefonts"))
+ (pcf (string-append (assoc-ref outputs "out")
+ "/share/fonts/portablecompiled"))
+ (psf (string-append (assoc-ref outputs "out")
+ "/share/fonts/pcscreen"))
(bin (assoc-ref outputs "bin")))
(invoke "make"
(string-append "PREFIX=" bin)
--
2.26.2
[-- Attachment #1.1.3: 0xAE6EF3046D6F7B57.asc --]
[-- Type: application/pgp-keys, Size: 2877 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-05-11 5:32 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-19 5:24 [bug#40708] gnu: font-gnu-unifont: Unify outputs for fonts Raghav Gururajan
2020-04-23 2:30 ` Leo Famulari
2020-04-23 2:44 ` Raghav Gururajan
2020-04-23 16:05 ` Leo Famulari
2020-05-11 4:54 ` Raghav Gururajan
2020-05-11 5:32 ` Raghav Gururajan
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).