Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] liberation-fonts-ttf: update to 2.1.5.
@ 2021-11-23 17:58 mobinmob
  2021-11-24 10:58 ` [PR REVIEW] " sgn
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: mobinmob @ 2021-11-23 17:58 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mobinmob/void-packages liberation-fonts-update
https://github.com/void-linux/void-packages/pull/34215

liberation-fonts-ttf: update to 2.1.5.
Also:
- update font metadata in post_extract().

<!-- Uncomment relevant sections and delete options which are not applicable -->

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

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration)
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-liberation-fonts-update-34215.patch --]
[-- Type: text/x-diff, Size: 1406 bytes --]

From 1f146831386bab0d6981b5ee95a9abd977e09297 Mon Sep 17 00:00:00 2001
From: mobinmob <mobinmob@disroot.org>
Date: Tue, 23 Nov 2021 19:55:14 +0200
Subject: [PATCH] liberation-fonts-ttf: update to 2.1.5.

Also:
- update font metadata in post_extract().
---
 srcpkgs/liberation-fonts-ttf/template | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/liberation-fonts-ttf/template b/srcpkgs/liberation-fonts-ttf/template
index 34312aaa9622..cc98a3086729 100644
--- a/srcpkgs/liberation-fonts-ttf/template
+++ b/srcpkgs/liberation-fonts-ttf/template
@@ -1,6 +1,6 @@
 # Template file for 'liberation-fonts-ttf'
 pkgname=liberation-fonts-ttf
-version=2.1.4
+version=2.1.5
 revision=1
 wrksrc=${pkgname%-*}-${version}
 build_style=gnu-makefile
@@ -12,9 +12,13 @@ license="OFL-1.1"
 homepage="https://github.com/liberationfonts/liberation-fonts"
 changelog="https://raw.githubusercontent.com/liberationfonts/liberation-fonts/devel/ChangeLog"
 distfiles="https://github.com/liberationfonts/liberation-fonts/archive/${version}.tar.gz"
-checksum=e76de918e7c98f4a992d2ab397196bde217d9aebc2ec305bf4d1d5bc765dbabc
+checksum=9a823ccb33c8a8a865e84b982bcdc44d03ba4914adb91e6000df035dc0e55936
 font_dirs="/usr/share/fonts/TTF"
 
+post_extract() {
+	make versionupdate
+}
+
 do_install() {
 	vmkdir usr/share/fonts/TTF
 	install -m644 ${pkgname}-${version}/*.ttf ${DESTDIR}/usr/share/fonts/TTF

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

* Re: [PR REVIEW] liberation-fonts-ttf: update to 2.1.5.
  2021-11-23 17:58 [PR PATCH] liberation-fonts-ttf: update to 2.1.5 mobinmob
@ 2021-11-24 10:58 ` sgn
  2021-11-24 13:50 ` [PR PATCH] [Updated] " mobinmob
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: sgn @ 2021-11-24 10:58 UTC (permalink / raw)
  To: ml

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

New review comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/34215#discussion_r755930847

Comment:
 Is it better done in `pre_build`

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

* Re: [PR PATCH] [Updated] liberation-fonts-ttf: update to 2.1.5.
  2021-11-23 17:58 [PR PATCH] liberation-fonts-ttf: update to 2.1.5 mobinmob
  2021-11-24 10:58 ` [PR REVIEW] " sgn
@ 2021-11-24 13:50 ` mobinmob
  2021-11-24 13:50 ` [PR REVIEW] " mobinmob
  2021-11-28  5:35 ` [PR PATCH] [Merged]: " ericonr
  3 siblings, 0 replies; 5+ messages in thread
From: mobinmob @ 2021-11-24 13:50 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mobinmob/void-packages liberation-fonts-update
https://github.com/void-linux/void-packages/pull/34215

liberation-fonts-ttf: update to 2.1.5.
Also:
- update font metadata in post_extract().

<!-- Uncomment relevant sections and delete options which are not applicable -->

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

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration)
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-liberation-fonts-update-34215.patch --]
[-- Type: text/x-diff, Size: 1400 bytes --]

From 7b44452f0a2b28386d09c0d0c79d39d44bf5814a Mon Sep 17 00:00:00 2001
From: mobinmob <mobinmob@disroot.org>
Date: Tue, 23 Nov 2021 19:55:14 +0200
Subject: [PATCH] liberation-fonts-ttf: update to 2.1.5.

Also:
- update font metadata in pre_build().
---
 srcpkgs/liberation-fonts-ttf/template | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/liberation-fonts-ttf/template b/srcpkgs/liberation-fonts-ttf/template
index 34312aaa9622..1e5b1997b0bd 100644
--- a/srcpkgs/liberation-fonts-ttf/template
+++ b/srcpkgs/liberation-fonts-ttf/template
@@ -1,6 +1,6 @@
 # Template file for 'liberation-fonts-ttf'
 pkgname=liberation-fonts-ttf
-version=2.1.4
+version=2.1.5
 revision=1
 wrksrc=${pkgname%-*}-${version}
 build_style=gnu-makefile
@@ -12,9 +12,13 @@ license="OFL-1.1"
 homepage="https://github.com/liberationfonts/liberation-fonts"
 changelog="https://raw.githubusercontent.com/liberationfonts/liberation-fonts/devel/ChangeLog"
 distfiles="https://github.com/liberationfonts/liberation-fonts/archive/${version}.tar.gz"
-checksum=e76de918e7c98f4a992d2ab397196bde217d9aebc2ec305bf4d1d5bc765dbabc
+checksum=9a823ccb33c8a8a865e84b982bcdc44d03ba4914adb91e6000df035dc0e55936
 font_dirs="/usr/share/fonts/TTF"
 
+pre_build() {
+	make versionupdate
+}
+
 do_install() {
 	vmkdir usr/share/fonts/TTF
 	install -m644 ${pkgname}-${version}/*.ttf ${DESTDIR}/usr/share/fonts/TTF

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

* Re: [PR REVIEW] liberation-fonts-ttf: update to 2.1.5.
  2021-11-23 17:58 [PR PATCH] liberation-fonts-ttf: update to 2.1.5 mobinmob
  2021-11-24 10:58 ` [PR REVIEW] " sgn
  2021-11-24 13:50 ` [PR PATCH] [Updated] " mobinmob
@ 2021-11-24 13:50 ` mobinmob
  2021-11-28  5:35 ` [PR PATCH] [Merged]: " ericonr
  3 siblings, 0 replies; 5+ messages in thread
From: mobinmob @ 2021-11-24 13:50 UTC (permalink / raw)
  To: ml

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

New review comment by mobinmob on void-packages repository

https://github.com/void-linux/void-packages/pull/34215#discussion_r756094522

Comment:
Thanks, fixed :)

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

* Re: [PR PATCH] [Merged]: liberation-fonts-ttf: update to 2.1.5.
  2021-11-23 17:58 [PR PATCH] liberation-fonts-ttf: update to 2.1.5 mobinmob
                   ` (2 preceding siblings ...)
  2021-11-24 13:50 ` [PR REVIEW] " mobinmob
@ 2021-11-28  5:35 ` ericonr
  3 siblings, 0 replies; 5+ messages in thread
From: ericonr @ 2021-11-28  5:35 UTC (permalink / raw)
  To: ml

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

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

liberation-fonts-ttf: update to 2.1.5.
https://github.com/void-linux/void-packages/pull/34215

Description:
Also:
- update font metadata in post_extract().

<!-- Uncomment relevant sections and delete options which are not applicable -->

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

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration)
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

end of thread, other threads:[~2021-11-28  5:35 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-23 17:58 [PR PATCH] liberation-fonts-ttf: update to 2.1.5 mobinmob
2021-11-24 10:58 ` [PR REVIEW] " sgn
2021-11-24 13:50 ` [PR PATCH] [Updated] " mobinmob
2021-11-24 13:50 ` [PR REVIEW] " mobinmob
2021-11-28  5:35 ` [PR PATCH] [Merged]: " ericonr

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).