Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
@ 2023-09-18 20:42 chrysos349
  2023-09-18 20:46 ` classabbyamp
                   ` (10 more replies)
  0 siblings, 11 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 20:42 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1359 bytes --]

There is a new pull request by chrysos349 against master on the void-packages repository

https://github.com/chrysos349/void-packages ttf-nerd-fonts-symbols
https://github.com/void-linux/void-packages/pull/46118

New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
@cinerea0

#### Testing the changes
- I tested the changes in this PR: **YES**

#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64

WHY?
`nerd-fonts` is a bloated package (2GB, REALLY?) which includes a ton of fonts, and scripts which can't be even executed by a non-root user due to permissions. BTW, these scripts have nothing to do with patching fonts, but with cheat sheet functionality. I don't even understand how it it was allowed into the repository in the first place!

`ttf-nerd-fonts-symbols` is the only part that's truly necessary. It provides unique glyphs.  My template includes also `fontconfig` preset.

A patch file from https://github.com/void-linux/void-packages/pull/46118.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-ttf-nerd-fonts-symbols-46118.patch --]
[-- Type: text/x-diff, Size: 3062 bytes --]

From 0b752a3c1d076fff480b3a3bb135cee0e60c84a3 Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 18 Sep 2023 23:22:35 +0300
Subject: [PATCH 1/2] removed-packages: revbump due to nerd-fonts{,-ttf,-otf}
 removal

---
 srcpkgs/removed-packages/template | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/removed-packages/template b/srcpkgs/removed-packages/template
index edf3aa545694b..420fb5673995d 100644
--- a/srcpkgs/removed-packages/template
+++ b/srcpkgs/removed-packages/template
@@ -1,7 +1,7 @@
 # Template file for 'removed-packages'
 pkgname=removed-packages
 version=0.1.20230918
-revision=2
+revision=3
 build_style=meta
 short_desc="Uninstalls packages removed from repository"
 maintainer="Piotr Wójcik <chocimier@tlen.pl>"
@@ -416,6 +416,9 @@ replaces="
  namecoin<=0.18.1_9
  nautilus-sendto<=3.8.6_2
  ncdns<=0.0.2_12
+ nerd-fonts-otf<=3.0.2_1
+ nerd-fonts-ttf<=3.0.2_1
+ nerd-fonts<=3.0.2_1
  newt-python<=0.52.21_5
  nodejs-lts-10<=10.24.0_2
  nvidia340-dkms<=340.108_4

From 39ba8540250b1ad944c748a133f7b9d47931f88f Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 18 Sep 2023 23:40:39 +0300
Subject: [PATCH 2/2] New package: ttf-nerd-fonts-symbols-3.0.2

---
 srcpkgs/ttf-nerd-fonts-symbols/template | 38 +++++++++++++++++++++++++
 1 file changed, 38 insertions(+)
 create mode 100644 srcpkgs/ttf-nerd-fonts-symbols/template

diff --git a/srcpkgs/ttf-nerd-fonts-symbols/template b/srcpkgs/ttf-nerd-fonts-symbols/template
new file mode 100644
index 0000000000000..ba030c3509187
--- /dev/null
+++ b/srcpkgs/ttf-nerd-fonts-symbols/template
@@ -0,0 +1,38 @@
+# Template file for 'ttf-nerd-fonts-symbols'
+pkgname=ttf-nerd-fonts-symbols
+version=3.0.2
+revision=1
+short_desc="High number of extra glyphs from popular 'iconic fonts'"
+maintainer="chrysos349 <chrysostom349@gmail.com>"
+license="MIT"
+homepage="https://github.com/ryanoasis/nerd-fonts"
+_url="https://raw.githubusercontent.com/ryanoasis/nerd-fonts/v${version}"
+distfiles="${_url}/patched-fonts/NerdFontsSymbolsOnly/SymbolsNerdFont-Regular.ttf
+ ${_url}/patched-fonts/NerdFontsSymbolsOnly/SymbolsNerdFontMono-Regular.ttf
+ ${_url}/10-nerd-font-symbols.conf
+ ${_url}/LICENSE"
+checksum="adc6d3e4a7cd6ac90eb5355d9a75f4f3078d9516eb1d35b1b4c640cac15264c8
+ 14ece2cc91d5a4762ab3e7b76a19a489a1be9f0ec9e13dfadd81cd49fe6a1ad7
+ 8bea9c77f30dfde59f91ca1c4d860a3bf8b228b9e6f9f71afd1701d902e9cce7
+ 1f6ad4edae6479aaace3112ede5279a23284ae54b2a34db66357aef5f64df160"
+font_dirs="/usr/share/fonts/TTF"
+replaces="nerd-fonts-ttf>=0"
+
+do_extract() {
+	cp ${XBPS_SRCDISTDIR}/${pkgname}-${version}/* \
+		${XBPS_BUILDDIR}/${pkgname}-${version}
+}
+
+do_install() {
+	vmkdir usr/share/fonts/TTF
+	install -m644 *.ttf ${DESTDIR}/usr/share/fonts/TTF
+
+	vmkdir usr/share/fontconfig/conf.avail
+	install -m644 *.conf ${DESTDIR}/usr/share/fontconfig/conf.avail
+
+	vmkdir etc/fonts/conf.d
+	ln -sf /usr/share/fontconfig/conf.avail/10-nerd-font-symbols.conf \
+		${DESTDIR}/etc/fonts/conf.d/
+
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
@ 2023-09-18 20:46 ` classabbyamp
  2023-09-18 20:48 ` chrysos349
                   ` (9 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: classabbyamp @ 2023-09-18 20:46 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 408 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724347086

Comment:
1. will this break things people who use these fonts by name?
2. it would make more sense for the old nerd fonts packages to be transitional dummy subpackages (grep for transitional in srcpkgs for examples) so that people don't get the old packages uninstalled

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
  2023-09-18 20:46 ` classabbyamp
@ 2023-09-18 20:48 ` chrysos349
  2023-09-18 20:49 ` classabbyamp
                   ` (8 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 20:48 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 184 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724350097

Comment:
@classabbyamp good idea! working on it.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
  2023-09-18 20:46 ` classabbyamp
  2023-09-18 20:48 ` chrysos349
@ 2023-09-18 20:49 ` classabbyamp
  2023-09-18 21:03 ` [PR PATCH] [Updated] " chrysos349
                   ` (7 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: classabbyamp @ 2023-09-18 20:49 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 401 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724347086

Comment:
1. will this break things people who use these fonts by name?
2. it would make more sense for the old nerd fonts packages to be transitional dummy subpackages (grep for transitional in srcpkgs for examples) so that people don't get the fonts uninstalled

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PR PATCH] [Updated] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (2 preceding siblings ...)
  2023-09-18 20:49 ` classabbyamp
@ 2023-09-18 21:03 ` chrysos349
  2023-09-18 21:06 ` New-package: ttf-nerd-fonts-symbols-3.0.2 chrysos349
                   ` (6 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 21:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1364 bytes --]

There is an updated pull request by chrysos349 against master on the void-packages repository

https://github.com/chrysos349/void-packages ttf-nerd-fonts-symbols
https://github.com/void-linux/void-packages/pull/46118

New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal
@cinerea0

#### Testing the changes
- I tested the changes in this PR: **YES**

#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64

WHY?
`nerd-fonts` is a bloated package (2GB, REALLY?) which includes a ton of fonts, and scripts which can't be even executed by a non-root user due to permissions. BTW, these scripts have nothing to do with patching fonts, but with cheat sheet functionality. I don't even understand how it it was allowed into the repository in the first place!

`ttf-nerd-fonts-symbols` is the only part that's truly necessary. It provides unique glyphs.  My template includes also `fontconfig` preset.

A patch file from https://github.com/void-linux/void-packages/pull/46118.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-ttf-nerd-fonts-symbols-46118.patch --]
[-- Type: text/x-diff, Size: 5176 bytes --]

From 0d2cac4d8f3b590d71d2d32105e92c84ecf81769 Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Tue, 19 Sep 2023 00:03:12 +0300
Subject: [PATCH] New package: ttf-nerd-fonts-symbols-3.0.2

---
 srcpkgs/nerd-fonts                      |  1 +
 srcpkgs/nerd-fonts-otf                  |  2 +-
 srcpkgs/nerd-fonts-ttf                  |  2 +-
 srcpkgs/nerd-fonts/template             | 47 ---------------------
 srcpkgs/ttf-nerd-fonts-symbols/template | 55 +++++++++++++++++++++++++
 5 files changed, 58 insertions(+), 49 deletions(-)
 create mode 120000 srcpkgs/nerd-fonts
 delete mode 100644 srcpkgs/nerd-fonts/template
 create mode 100644 srcpkgs/ttf-nerd-fonts-symbols/template

diff --git a/srcpkgs/nerd-fonts b/srcpkgs/nerd-fonts
new file mode 120000
index 0000000000000..31ae545cd9de0
--- /dev/null
+++ b/srcpkgs/nerd-fonts
@@ -0,0 +1 @@
+ttf-nerd-fonts-symbols
\ No newline at end of file
diff --git a/srcpkgs/nerd-fonts-otf b/srcpkgs/nerd-fonts-otf
index e5a8856321374..31ae545cd9de0 120000
--- a/srcpkgs/nerd-fonts-otf
+++ b/srcpkgs/nerd-fonts-otf
@@ -1 +1 @@
-nerd-fonts
\ No newline at end of file
+ttf-nerd-fonts-symbols
\ No newline at end of file
diff --git a/srcpkgs/nerd-fonts-ttf b/srcpkgs/nerd-fonts-ttf
index e5a8856321374..31ae545cd9de0 120000
--- a/srcpkgs/nerd-fonts-ttf
+++ b/srcpkgs/nerd-fonts-ttf
@@ -1 +1 @@
-nerd-fonts
\ No newline at end of file
+ttf-nerd-fonts-symbols
\ No newline at end of file
diff --git a/srcpkgs/nerd-fonts/template b/srcpkgs/nerd-fonts/template
deleted file mode 100644
index f4409116aa628..0000000000000
--- a/srcpkgs/nerd-fonts/template
+++ /dev/null
@@ -1,47 +0,0 @@
-# Template file for 'nerd-fonts'
-pkgname=nerd-fonts
-version=3.0.2
-revision=1
-depends="nerd-fonts-ttf nerd-fonts-otf"
-short_desc="Iconic font aggregator, collection and patcher"
-maintainer="cinerea0 <cinerea0@protonmail.com>"
-license="MIT"
-homepage="https://nerdfonts.com"
-changelog="https://raw.githubusercontent.com/ryanoasis/nerd-fonts/master/changelog.md"
-distfiles="https://github.com/ryanoasis/nerd-fonts/archive/refs/tags/v${version}.tar.gz"
-checksum=3b18b305451b0b22cb6484197f8029efe54e5c459e8ed3adbb0fb1697c0c13a6
-
-do_install() {
-	vmkdir usr/share/fonts/NerdFonts/otf
-	vmkdir usr/share/fonts/NerdFonts/ttf
-	vmkdir usr/lib/NerdFonts
-
-	# get all patched fonts
-	find patched-fonts -name '*.otf' -exec install -Dm644 '{}' ${DESTDIR}/usr/share/fonts/NerdFonts/otf \;
-	find patched-fonts -name '*.ttf' -exec install -Dm644 '{}' ${DESTDIR}/usr/share/fonts/NerdFonts/ttf \;
-
-	# install patching scripts
-	for sh in bin/scripts/lib/i_*.sh; do
-		vinstall $sh 744 usr/lib/NerdFonts
-	done
-
-	vlicense LICENSE
-}
-
-nerd-fonts-otf_package() {
-	short_desc="Iconic font aggregator, collection and patcher - otf fonts"
-	font_dirs="/usr/share/fonts/NerdFonts/otf"
-	depends="font-util"
-	pkg_install() {
-		vmove usr/share/fonts/NerdFonts/otf
-	}
-}
-
-nerd-fonts-ttf_package() {
-	short_desc="Iconic font aggregator, collection and patcher - ttf fonts"
-	font_dirs="/usr/share/fonts/NerdFonts/ttf"
-	depends="font-util"
-	pkg_install() {
-		vmove usr/share/fonts/NerdFonts/ttf
-	}
-}
diff --git a/srcpkgs/ttf-nerd-fonts-symbols/template b/srcpkgs/ttf-nerd-fonts-symbols/template
new file mode 100644
index 0000000000000..575b7b2b9a24e
--- /dev/null
+++ b/srcpkgs/ttf-nerd-fonts-symbols/template
@@ -0,0 +1,55 @@
+# Template file for 'ttf-nerd-fonts-symbols'
+pkgname=ttf-nerd-fonts-symbols
+version=3.0.2
+revision=1
+short_desc="High number of extra glyphs from popular 'iconic fonts'"
+maintainer="chrysos349 <chrysostom349@gmail.com>"
+license="MIT"
+homepage="https://github.com/ryanoasis/nerd-fonts"
+_url="https://raw.githubusercontent.com/ryanoasis/nerd-fonts/v${version}"
+distfiles="${_url}/patched-fonts/NerdFontsSymbolsOnly/SymbolsNerdFont-Regular.ttf
+ ${_url}/patched-fonts/NerdFontsSymbolsOnly/SymbolsNerdFontMono-Regular.ttf
+ ${_url}/10-nerd-font-symbols.conf
+ ${_url}/LICENSE"
+checksum="adc6d3e4a7cd6ac90eb5355d9a75f4f3078d9516eb1d35b1b4c640cac15264c8
+ 14ece2cc91d5a4762ab3e7b76a19a489a1be9f0ec9e13dfadd81cd49fe6a1ad7
+ 8bea9c77f30dfde59f91ca1c4d860a3bf8b228b9e6f9f71afd1701d902e9cce7
+ 1f6ad4edae6479aaace3112ede5279a23284ae54b2a34db66357aef5f64df160"
+font_dirs="/usr/share/fonts/TTF"
+
+do_extract() {
+	cp ${XBPS_SRCDISTDIR}/${pkgname}-${version}/* \
+		${XBPS_BUILDDIR}/${pkgname}-${version}
+}
+
+do_install() {
+	vmkdir usr/share/fonts/TTF
+	install -m644 *.ttf ${DESTDIR}/usr/share/fonts/TTF
+
+	vmkdir usr/share/fontconfig/conf.avail
+	install -m644 *.conf ${DESTDIR}/usr/share/fontconfig/conf.avail
+
+	vmkdir etc/fonts/conf.d
+	ln -sf /usr/share/fontconfig/conf.avail/10-nerd-font-symbols.conf \
+		${DESTDIR}/etc/fonts/conf.d/
+
+	vlicense LICENSE
+}
+
+nerd-fonts_package() {
+	build_style=meta
+	depends=${sourcepkg}>=${version}_${revision}
+	short_desc+=" (transitional dummy package)"
+}
+
+nerd-fonts-otf_package() {
+	build_style=meta
+	depends=${sourcepkg}>=${version}_${revision}
+	short_desc+=" (transitional dummy package)"
+}
+
+nerd-fonts-ttf_package() {
+	build_style=meta
+	depends=${sourcepkg}>=${version}_${revision}
+	short_desc+=" (transitional dummy package)"
+}

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (3 preceding siblings ...)
  2023-09-18 21:03 ` [PR PATCH] [Updated] " chrysos349
@ 2023-09-18 21:06 ` chrysos349
  2023-09-18 22:52 ` cinerea0
                   ` (5 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 21:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 164 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724415842

Comment:
@classabbyamp done!

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (4 preceding siblings ...)
  2023-09-18 21:06 ` New-package: ttf-nerd-fonts-symbols-3.0.2 chrysos349
@ 2023-09-18 22:52 ` cinerea0
  2023-09-18 23:45 ` [PR PATCH] [Updated] " chrysos349
                   ` (4 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: cinerea0 @ 2023-09-18 22:52 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 580 bytes --]

New comment by cinerea0 on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724569988

Comment:
This is a bad idea:

1. This completely removes the OTF fonts.
2. There are 17 font families that are only packaged as their nerd-font patched variants. This change would remove them from users systems with no alternative except to manually install them.
3. This breaks things for people who use the fonts by their full name.
4. Even in situations where the "base" font is separately available, it doesn't seem to fallback properly.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PR PATCH] [Updated] New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (5 preceding siblings ...)
  2023-09-18 22:52 ` cinerea0
@ 2023-09-18 23:45 ` chrysos349
  2023-09-18 23:47 ` chrysos349
                   ` (3 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 23:45 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1299 bytes --]

There is an updated pull request by chrysos349 against master on the void-packages repository

https://github.com/chrysos349/void-packages ttf-nerd-fonts-symbols
https://github.com/void-linux/void-packages/pull/46118

New-package: ttf-nerd-fonts-symbols-3.0.2
@cinerea0

#### Testing the changes
- I tested the changes in this PR: **YES**

#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64

WHY?
`nerd-fonts` is a bloated package (2GB, REALLY?) which includes a ton of fonts, and scripts which can't be even executed by a non-root user due to permissions. BTW, these scripts have nothing to do with patching fonts, but with cheat sheet functionality. I don't even understand how it it was allowed into the repository in the first place!

`ttf-nerd-fonts-symbols` is the only part that's truly necessary. It provides unique glyphs.  My template includes also `fontconfig` preset.

A patch file from https://github.com/void-linux/void-packages/pull/46118.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-ttf-nerd-fonts-symbols-46118.patch --]
[-- Type: text/x-diff, Size: 6940 bytes --]

From 495b7ce710a3b3ff16154324d9f4d1257c170fce Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Tue, 19 Sep 2023 02:45:30 +0300
Subject: [PATCH] New package: ttf-nerd-fonts-symbols-3.0.2

---
 srcpkgs/nerd-fonts                            |  1 +
 srcpkgs/nerd-fonts-otf                        |  2 +-
 srcpkgs/nerd-fonts-ttf                        |  2 +-
 srcpkgs/nerd-fonts/template                   | 47 ----------------
 .../nerd-fonts-otf.INSTALL.msg                |  3 +
 .../nerd-fonts-ttf.INSTALL.msg                |  3 +
 .../nerd-fonts.INSTALL.msg                    |  3 +
 srcpkgs/ttf-nerd-fonts-symbols/template       | 55 +++++++++++++++++++
 8 files changed, 67 insertions(+), 49 deletions(-)
 create mode 120000 srcpkgs/nerd-fonts
 delete mode 100644 srcpkgs/nerd-fonts/template
 create mode 100644 srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-otf.INSTALL.msg
 create mode 100644 srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-ttf.INSTALL.msg
 create mode 100644 srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts.INSTALL.msg
 create mode 100644 srcpkgs/ttf-nerd-fonts-symbols/template

diff --git a/srcpkgs/nerd-fonts b/srcpkgs/nerd-fonts
new file mode 120000
index 0000000000000..31ae545cd9de0
--- /dev/null
+++ b/srcpkgs/nerd-fonts
@@ -0,0 +1 @@
+ttf-nerd-fonts-symbols
\ No newline at end of file
diff --git a/srcpkgs/nerd-fonts-otf b/srcpkgs/nerd-fonts-otf
index e5a8856321374..31ae545cd9de0 120000
--- a/srcpkgs/nerd-fonts-otf
+++ b/srcpkgs/nerd-fonts-otf
@@ -1 +1 @@
-nerd-fonts
\ No newline at end of file
+ttf-nerd-fonts-symbols
\ No newline at end of file
diff --git a/srcpkgs/nerd-fonts-ttf b/srcpkgs/nerd-fonts-ttf
index e5a8856321374..31ae545cd9de0 120000
--- a/srcpkgs/nerd-fonts-ttf
+++ b/srcpkgs/nerd-fonts-ttf
@@ -1 +1 @@
-nerd-fonts
\ No newline at end of file
+ttf-nerd-fonts-symbols
\ No newline at end of file
diff --git a/srcpkgs/nerd-fonts/template b/srcpkgs/nerd-fonts/template
deleted file mode 100644
index f4409116aa628..0000000000000
--- a/srcpkgs/nerd-fonts/template
+++ /dev/null
@@ -1,47 +0,0 @@
-# Template file for 'nerd-fonts'
-pkgname=nerd-fonts
-version=3.0.2
-revision=1
-depends="nerd-fonts-ttf nerd-fonts-otf"
-short_desc="Iconic font aggregator, collection and patcher"
-maintainer="cinerea0 <cinerea0@protonmail.com>"
-license="MIT"
-homepage="https://nerdfonts.com"
-changelog="https://raw.githubusercontent.com/ryanoasis/nerd-fonts/master/changelog.md"
-distfiles="https://github.com/ryanoasis/nerd-fonts/archive/refs/tags/v${version}.tar.gz"
-checksum=3b18b305451b0b22cb6484197f8029efe54e5c459e8ed3adbb0fb1697c0c13a6
-
-do_install() {
-	vmkdir usr/share/fonts/NerdFonts/otf
-	vmkdir usr/share/fonts/NerdFonts/ttf
-	vmkdir usr/lib/NerdFonts
-
-	# get all patched fonts
-	find patched-fonts -name '*.otf' -exec install -Dm644 '{}' ${DESTDIR}/usr/share/fonts/NerdFonts/otf \;
-	find patched-fonts -name '*.ttf' -exec install -Dm644 '{}' ${DESTDIR}/usr/share/fonts/NerdFonts/ttf \;
-
-	# install patching scripts
-	for sh in bin/scripts/lib/i_*.sh; do
-		vinstall $sh 744 usr/lib/NerdFonts
-	done
-
-	vlicense LICENSE
-}
-
-nerd-fonts-otf_package() {
-	short_desc="Iconic font aggregator, collection and patcher - otf fonts"
-	font_dirs="/usr/share/fonts/NerdFonts/otf"
-	depends="font-util"
-	pkg_install() {
-		vmove usr/share/fonts/NerdFonts/otf
-	}
-}
-
-nerd-fonts-ttf_package() {
-	short_desc="Iconic font aggregator, collection and patcher - ttf fonts"
-	font_dirs="/usr/share/fonts/NerdFonts/ttf"
-	depends="font-util"
-	pkg_install() {
-		vmove usr/share/fonts/NerdFonts/ttf
-	}
-}
diff --git a/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-otf.INSTALL.msg b/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-otf.INSTALL.msg
new file mode 100644
index 0000000000000..6ae1fe453c4bb
--- /dev/null
+++ b/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-otf.INSTALL.msg
@@ -0,0 +1,3 @@
+The nerd-fonts-otf package has been replaced by ttf-nerd-fonts-symbols,
+which doesn't include the full versions of fonts anymore, but symbols
+and a fontconfig preset only.
diff --git a/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-ttf.INSTALL.msg b/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-ttf.INSTALL.msg
new file mode 100644
index 0000000000000..b6b5a3ce169ff
--- /dev/null
+++ b/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts-ttf.INSTALL.msg
@@ -0,0 +1,3 @@
+The nerd-fonts-ttf package has been replaced by ttf-nerd-fonts-symbols,
+which doesn't include the full versions of fonts anymore, but symbols
+and a fontconfig preset only.
diff --git a/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts.INSTALL.msg b/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts.INSTALL.msg
new file mode 100644
index 0000000000000..d33079fa6c417
--- /dev/null
+++ b/srcpkgs/ttf-nerd-fonts-symbols/nerd-fonts.INSTALL.msg
@@ -0,0 +1,3 @@
+The nerd-fonts package has been replaced by ttf-nerd-fonts-symbols,
+which doesn't include the full versions of fonts anymore, but symbols
+and a fontconfig preset only .
diff --git a/srcpkgs/ttf-nerd-fonts-symbols/template b/srcpkgs/ttf-nerd-fonts-symbols/template
new file mode 100644
index 0000000000000..575b7b2b9a24e
--- /dev/null
+++ b/srcpkgs/ttf-nerd-fonts-symbols/template
@@ -0,0 +1,55 @@
+# Template file for 'ttf-nerd-fonts-symbols'
+pkgname=ttf-nerd-fonts-symbols
+version=3.0.2
+revision=1
+short_desc="High number of extra glyphs from popular 'iconic fonts'"
+maintainer="chrysos349 <chrysostom349@gmail.com>"
+license="MIT"
+homepage="https://github.com/ryanoasis/nerd-fonts"
+_url="https://raw.githubusercontent.com/ryanoasis/nerd-fonts/v${version}"
+distfiles="${_url}/patched-fonts/NerdFontsSymbolsOnly/SymbolsNerdFont-Regular.ttf
+ ${_url}/patched-fonts/NerdFontsSymbolsOnly/SymbolsNerdFontMono-Regular.ttf
+ ${_url}/10-nerd-font-symbols.conf
+ ${_url}/LICENSE"
+checksum="adc6d3e4a7cd6ac90eb5355d9a75f4f3078d9516eb1d35b1b4c640cac15264c8
+ 14ece2cc91d5a4762ab3e7b76a19a489a1be9f0ec9e13dfadd81cd49fe6a1ad7
+ 8bea9c77f30dfde59f91ca1c4d860a3bf8b228b9e6f9f71afd1701d902e9cce7
+ 1f6ad4edae6479aaace3112ede5279a23284ae54b2a34db66357aef5f64df160"
+font_dirs="/usr/share/fonts/TTF"
+
+do_extract() {
+	cp ${XBPS_SRCDISTDIR}/${pkgname}-${version}/* \
+		${XBPS_BUILDDIR}/${pkgname}-${version}
+}
+
+do_install() {
+	vmkdir usr/share/fonts/TTF
+	install -m644 *.ttf ${DESTDIR}/usr/share/fonts/TTF
+
+	vmkdir usr/share/fontconfig/conf.avail
+	install -m644 *.conf ${DESTDIR}/usr/share/fontconfig/conf.avail
+
+	vmkdir etc/fonts/conf.d
+	ln -sf /usr/share/fontconfig/conf.avail/10-nerd-font-symbols.conf \
+		${DESTDIR}/etc/fonts/conf.d/
+
+	vlicense LICENSE
+}
+
+nerd-fonts_package() {
+	build_style=meta
+	depends=${sourcepkg}>=${version}_${revision}
+	short_desc+=" (transitional dummy package)"
+}
+
+nerd-fonts-otf_package() {
+	build_style=meta
+	depends=${sourcepkg}>=${version}_${revision}
+	short_desc+=" (transitional dummy package)"
+}
+
+nerd-fonts-ttf_package() {
+	build_style=meta
+	depends=${sourcepkg}>=${version}_${revision}
+	short_desc+=" (transitional dummy package)"
+}

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (6 preceding siblings ...)
  2023-09-18 23:45 ` [PR PATCH] [Updated] " chrysos349
@ 2023-09-18 23:47 ` chrysos349
  2023-09-18 23:47 ` chrysos349
                   ` (2 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 23:47 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1405 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724629725

Comment:
I included INSTALL.msg to the affected packages to explain the change.

@cinerea0 

> This is a bad idea:

making this bloated package a part of the official repo WAS a bad idea. replacing it with a leaner version which includes only unique glyphs (as per package requirements concerning fonts) IS NOT!

> 1\. This completely removes the OTF fonts.

so what? the fonts can be downloaded manually.

> 2\. There are 17 font families that are only packaged as their nerd-font patched variants.

do you mean the fonts with included nerd symbols and changed names? it doesn't matter. again, they fonts can be downloaded manually.

> 2\. This change would remove them from users systems with no alternative except to manually install them.

the previous commits would. this one would leave all the fonts in `/usr/share/fonts/NerdFonts`, thanks to @classabbyamp 's suggestion.

> 3\. This breaks things for people who use the fonts by their full name.

again, it won't. the only change is the fonts won't be updated anymore. they'll be left intact in the system dir.

> 4\. Even in situations where the "base" font is separately available, it doesn't seem to fallback properly.

people can download the full versions of fonts themselves.



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (7 preceding siblings ...)
  2023-09-18 23:47 ` chrysos349
@ 2023-09-18 23:47 ` chrysos349
  2023-09-19  1:48 ` ahesford
  2023-09-19  2:40 ` [PR PATCH] [Closed]: " chrysos349
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-18 23:47 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1407 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724629725

Comment:
I included `INSTALL.msg` to the affected packages to explain the change.

@cinerea0 

> This is a bad idea:

making this bloated package a part of the official repo WAS a bad idea. replacing it with a leaner version which includes only unique glyphs (as per package requirements concerning fonts) IS NOT!

> 1\. This completely removes the OTF fonts.

so what? the fonts can be downloaded manually.

> 2\. There are 17 font families that are only packaged as their nerd-font patched variants.

do you mean the fonts with included nerd symbols and changed names? it doesn't matter. again, they fonts can be downloaded manually.

> 2\. This change would remove them from users systems with no alternative except to manually install them.

the previous commits would. this one would leave all the fonts in `/usr/share/fonts/NerdFonts`, thanks to @classabbyamp 's suggestion.

> 3\. This breaks things for people who use the fonts by their full name.

again, it won't. the only change is the fonts won't be updated anymore. they'll be left intact in the system dir.

> 4\. Even in situations where the "base" font is separately available, it doesn't seem to fallback properly.

people can download the full versions of fonts themselves.



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (8 preceding siblings ...)
  2023-09-18 23:47 ` chrysos349
@ 2023-09-19  1:48 ` ahesford
  2023-09-19  2:40 ` [PR PATCH] [Closed]: " chrysos349
  10 siblings, 0 replies; 12+ messages in thread
From: ahesford @ 2023-09-19  1:48 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 756 bytes --]

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/46118#issuecomment-1724712185

Comment:
Whether you consider the addition of `nerd-fonts` to the repo a bad idea is irrelevant, because it is not your decision. We will not accept a PR that removes well-maintained packages from the distribution when the maintainer of those packages objects. Furthermore, you can avoid downloading the large font package and "download the [reduced] versions of fonts [yourself]" if you like.

If your PR offers some additional fonts that are not already present in the `nerd-fonts` packages, please separate the useful additions from the proposed removal and replacement. Otherwise, the PR is not suitable for merging.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PR PATCH] [Closed]: New-package: ttf-nerd-fonts-symbols-3.0.2
  2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
                   ` (9 preceding siblings ...)
  2023-09-19  1:48 ` ahesford
@ 2023-09-19  2:40 ` chrysos349
  10 siblings, 0 replies; 12+ messages in thread
From: chrysos349 @ 2023-09-19  2:40 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]

There's a closed pull request on the void-packages repository

New-package: ttf-nerd-fonts-symbols-3.0.2
https://github.com/void-linux/void-packages/pull/46118

Description:
@cinerea0

#### Testing the changes
- I tested the changes in this PR: **YES**

#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64

WHY?
`nerd-fonts` is a bloated package (2GB, REALLY?) which includes a ton of fonts, and scripts which can't be even executed by a non-root user due to permissions. BTW, these scripts have nothing to do with patching fonts, but with cheat sheet functionality. I don't even understand how it it was allowed into the repository in the first place!

`ttf-nerd-fonts-symbols` is the only part that's truly necessary. It provides unique glyphs.  My template includes also `fontconfig` preset.

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2023-09-19  2:41 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-18 20:42 [PR PATCH] New-package: ttf-nerd-fonts-symbols-3.0.2, removed-packages: revbump due to nerd-fonts{,-ttf,-otf} removal chrysos349
2023-09-18 20:46 ` classabbyamp
2023-09-18 20:48 ` chrysos349
2023-09-18 20:49 ` classabbyamp
2023-09-18 21:03 ` [PR PATCH] [Updated] " chrysos349
2023-09-18 21:06 ` New-package: ttf-nerd-fonts-symbols-3.0.2 chrysos349
2023-09-18 22:52 ` cinerea0
2023-09-18 23:45 ` [PR PATCH] [Updated] " chrysos349
2023-09-18 23:47 ` chrysos349
2023-09-18 23:47 ` chrysos349
2023-09-19  1:48 ` ahesford
2023-09-19  2:40 ` [PR PATCH] [Closed]: " chrysos349

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 NNTP newsgroup(s).