Re: [oe] [meta-oe][PATCH v2] source-han-sans-*-fonts: fix SRC_URI

2024-05-19 Thread Khem Raj 
On Sat, May 18, 2024 at 1:22 PM Martin Jansa via
lists.openembedded.org 
wrote:
>
> On Fri, May 17, 2024 at 3:07 PM Alexandre Truong via
> lists.openembedded.org
>  wrote:
> >
> > We get fetch errors from the SRC_URI provided. See errors at [0].
> > svn protocol was used but it's deprecated.
>
> For the record the svn protocol itself isn't deprecated in bitbake
> fetcher, just github no longer provides a svn access as mentioned in:
> https://git.openembedded.org/bitbake/commit/?h=master=a735898abcf056f897c9350bb128a5637e6b4617
>
> * github no longer provides support for subversion clients:
> https://docs.github.com/en/enterprise-server@3.11/get-started/working-with-subversion-on-github/support-for-subversion-clients
> it was shut down on 2024-01-08:
> https://github.blog/2023-01-20-sunsetting-subversion-support/
>

Thanks for adding to information, I have added Martin's comments into
  commit msg on staged patch in master-next

  Thanks
  -Khem

> > So, a http url will be passed instead.
> >
> > The sha256sum has also been changed since the previous one was
> > for the 1.004 release's archive, while we are on 2.004.
> >
> > [0]: 
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/156/builds/353/steps/16/logs/errors
> >
> > Signed-off-by: Alexandre Truong 
> > Reviewed-by: Yoann Congal 
> > ---
> >  .../ttf-fonts/source-han-sans-cn-fonts_2.004.bb| 7 ++-
> >  .../ttf-fonts/source-han-sans-jp-fonts_2.004.bb| 7 ++-
> >  .../ttf-fonts/source-han-sans-kr-fonts_2.004.bb| 7 ++-
> >  .../ttf-fonts/source-han-sans-tw-fonts_2.004.bb| 7 ++-
> >  4 files changed, 8 insertions(+), 20 deletions(-)
> >
> > diff --git 
> > a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb 
> > b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
> > index b4a598527..391d46557 100644
> > --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
> > +++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
> > @@ -11,13 +11,10 @@ inherit allarch fontcache
> >  #EXCLUDE_FROM_WORLD = "1"
> >
> >  SRC_URI = " \
> > -
> > svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/CN;protocol=http;rev=82
> >  \
> > +
> > https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansCN.zip;subdir=${BP}
> >  \
> >  file://44-source-han-sans-cn.conf \
> >  "
> > -SRC_URI[md5sum] = "d16abc21f6575bb08894efedbed484a2"
> > -SRC_URI[sha256sum] = 
> > "0a0e1d8e52833bc352d454d8242da03b82c0efc41323fb66f7435e5b39734a4f"
> > -
> > -S = "${WORKDIR}/SourceHanSansCN"
> > +SRC_URI[sha256sum] = 
> > "6841fc13f1c0d255cfeb33d2a2c68d24bbebd94ae2c070347a2b2b200a1db4d6"
> >
> >  do_install() {
> >  install -d ${D}${sysconfdir}/fonts/conf.d/
> > diff --git 
> > a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb 
> > b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
> > index 6a4509048..f940478a8 100644
> > --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
> > +++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
> > @@ -11,13 +11,10 @@ inherit allarch fontcache
> >  #EXCLUDE_FROM_WORLD = "1"
> >
> >  SRC_URI = " \
> > -
> > svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/JP;protocol=http;rev=82
> >  \
> > +
> > https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansJP.zip;subdir=${BP}
> >  \
> >  file://44-source-han-sans-jp.conf \
> >  "
> > -SRC_URI[md5sum] = "908fbf97f3df04a6838708c093f1e900"
> > -SRC_URI[sha256sum] = 
> > "dc6dbae3fba35f220bac88ba7130b826c7efe1282f472788fae3628b79be3f54"
> > -
> > -S = "${WORKDIR}/SourceHanSansJP"
> > +SRC_URI[sha256sum] = 
> > "1ae9f62ad620d686c4a049ce25cf54e3afd8fefc954a678c644cf9802750c17e"
> >
> >  do_install() {
> >  install -d ${D}${sysconfdir}/fonts/conf.d/
> > diff --git 
> > a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb 
> > b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
> > index 5ab5057d8..f536d1b61 100644
> > --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
> > +++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
> > @@ -11,13 +11,10 @@ inherit allarch fontcache
> >  #EXCLUDE_FROM_WORLD = "1"
> >
> >  SRC_URI = " \
> > -
> > svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/TW;protocol=http;rev=82
> >  \
> > +
> > https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansKR.zip;subdir=${BP}
> >  \
> >  file://44-source-han-sans-kr.conf \
> >  "
> > -SRC_URI[md5sum] = "f8d1bd6c87d8575afdb25e2f46bd81d4"
> > -SRC_URI[sha256sum] = 
> > "38fd15c80f9980492faaa1af39ff873d8a38e45027023fb17d5b10d4b4b0e6af"
> > -
> > -S = "${WORKDIR}/SourceHanSansKR"
> > +SRC_URI[sha256sum] = 
> >

Re: [oe] [meta-oe][PATCH v2] source-han-sans-*-fonts: fix SRC_URI

2024-05-18 Thread Martin Jansa 
On Fri, May 17, 2024 at 3:07 PM Alexandre Truong via
lists.openembedded.org
 wrote:
>
> We get fetch errors from the SRC_URI provided. See errors at [0].
> svn protocol was used but it's deprecated.

For the record the svn protocol itself isn't deprecated in bitbake
fetcher, just github no longer provides a svn access as mentioned in:
https://git.openembedded.org/bitbake/commit/?h=master=a735898abcf056f897c9350bb128a5637e6b4617

* github no longer provides support for subversion clients:
https://docs.github.com/en/enterprise-server@3.11/get-started/working-with-subversion-on-github/support-for-subversion-clients
it was shut down on 2024-01-08:
https://github.blog/2023-01-20-sunsetting-subversion-support/

> So, a http url will be passed instead.
>
> The sha256sum has also been changed since the previous one was
> for the 1.004 release's archive, while we are on 2.004.
>
> [0]: 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/156/builds/353/steps/16/logs/errors
>
> Signed-off-by: Alexandre Truong 
> Reviewed-by: Yoann Congal 
> ---
>  .../ttf-fonts/source-han-sans-cn-fonts_2.004.bb| 7 ++-
>  .../ttf-fonts/source-han-sans-jp-fonts_2.004.bb| 7 ++-
>  .../ttf-fonts/source-han-sans-kr-fonts_2.004.bb| 7 ++-
>  .../ttf-fonts/source-han-sans-tw-fonts_2.004.bb| 7 ++-
>  4 files changed, 8 insertions(+), 20 deletions(-)
>
> diff --git 
> a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb 
> b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
> index b4a598527..391d46557 100644
> --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
> +++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
> @@ -11,13 +11,10 @@ inherit allarch fontcache
>  #EXCLUDE_FROM_WORLD = "1"
>
>  SRC_URI = " \
> -
> svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/CN;protocol=http;rev=82
>  \
> +
> https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansCN.zip;subdir=${BP}
>  \
>  file://44-source-han-sans-cn.conf \
>  "
> -SRC_URI[md5sum] = "d16abc21f6575bb08894efedbed484a2"
> -SRC_URI[sha256sum] = 
> "0a0e1d8e52833bc352d454d8242da03b82c0efc41323fb66f7435e5b39734a4f"
> -
> -S = "${WORKDIR}/SourceHanSansCN"
> +SRC_URI[sha256sum] = 
> "6841fc13f1c0d255cfeb33d2a2c68d24bbebd94ae2c070347a2b2b200a1db4d6"
>
>  do_install() {
>  install -d ${D}${sysconfdir}/fonts/conf.d/
> diff --git 
> a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb 
> b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
> index 6a4509048..f940478a8 100644
> --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
> +++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
> @@ -11,13 +11,10 @@ inherit allarch fontcache
>  #EXCLUDE_FROM_WORLD = "1"
>
>  SRC_URI = " \
> -
> svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/JP;protocol=http;rev=82
>  \
> +
> https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansJP.zip;subdir=${BP}
>  \
>  file://44-source-han-sans-jp.conf \
>  "
> -SRC_URI[md5sum] = "908fbf97f3df04a6838708c093f1e900"
> -SRC_URI[sha256sum] = 
> "dc6dbae3fba35f220bac88ba7130b826c7efe1282f472788fae3628b79be3f54"
> -
> -S = "${WORKDIR}/SourceHanSansJP"
> +SRC_URI[sha256sum] = 
> "1ae9f62ad620d686c4a049ce25cf54e3afd8fefc954a678c644cf9802750c17e"
>
>  do_install() {
>  install -d ${D}${sysconfdir}/fonts/conf.d/
> diff --git 
> a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb 
> b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
> index 5ab5057d8..f536d1b61 100644
> --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
> +++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
> @@ -11,13 +11,10 @@ inherit allarch fontcache
>  #EXCLUDE_FROM_WORLD = "1"
>
>  SRC_URI = " \
> -
> svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/TW;protocol=http;rev=82
>  \
> +
> https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansKR.zip;subdir=${BP}
>  \
>  file://44-source-han-sans-kr.conf \
>  "
> -SRC_URI[md5sum] = "f8d1bd6c87d8575afdb25e2f46bd81d4"
> -SRC_URI[sha256sum] = 
> "38fd15c80f9980492faaa1af39ff873d8a38e45027023fb17d5b10d4b4b0e6af"
> -
> -S = "${WORKDIR}/SourceHanSansKR"
> +SRC_URI[sha256sum] = 
> "02fe28a48c6381c49d61c27a1b173c77f0e6f2b9f2b68e79f076f10a6a8f4bfe"
>
>  do_install() {
>  install -d ${D}${sysconfdir}/fonts/conf.d/
> diff --git 
> a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb 
> b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb
> index a2b3fff07..0a4aff5e3 100644
> --- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb
> +++

[oe] [meta-oe][PATCH v2] source-han-sans-*-fonts: fix SRC_URI

2024-05-17 Thread Alexandre Truong 
We get fetch errors from the SRC_URI provided. See errors at [0].
svn protocol was used but it's deprecated.
So, a http url will be passed instead.

The sha256sum has also been changed since the previous one was
for the 1.004 release's archive, while we are on 2.004.

[0]: 
https://autobuilder.yoctoproject.org/typhoon/#/builders/156/builds/353/steps/16/logs/errors

Signed-off-by: Alexandre Truong 
Reviewed-by: Yoann Congal 
---
 .../ttf-fonts/source-han-sans-cn-fonts_2.004.bb| 7 ++-
 .../ttf-fonts/source-han-sans-jp-fonts_2.004.bb| 7 ++-
 .../ttf-fonts/source-han-sans-kr-fonts_2.004.bb| 7 ++-
 .../ttf-fonts/source-han-sans-tw-fonts_2.004.bb| 7 ++-
 4 files changed, 8 insertions(+), 20 deletions(-)

diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb 
b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
index b4a598527..391d46557 100644
--- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
+++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_2.004.bb
@@ -11,13 +11,10 @@ inherit allarch fontcache
 #EXCLUDE_FROM_WORLD = "1"
 
 SRC_URI = " \
-
svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/CN;protocol=http;rev=82
 \
+
https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansCN.zip;subdir=${BP}
 \
 file://44-source-han-sans-cn.conf \
 "
-SRC_URI[md5sum] = "d16abc21f6575bb08894efedbed484a2"
-SRC_URI[sha256sum] = 
"0a0e1d8e52833bc352d454d8242da03b82c0efc41323fb66f7435e5b39734a4f"
-
-S = "${WORKDIR}/SourceHanSansCN"
+SRC_URI[sha256sum] = 
"6841fc13f1c0d255cfeb33d2a2c68d24bbebd94ae2c070347a2b2b200a1db4d6"
 
 do_install() {
 install -d ${D}${sysconfdir}/fonts/conf.d/
diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb 
b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
index 6a4509048..f940478a8 100644
--- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
+++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_2.004.bb
@@ -11,13 +11,10 @@ inherit allarch fontcache
 #EXCLUDE_FROM_WORLD = "1"
 
 SRC_URI = " \
-
svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/JP;protocol=http;rev=82
 \
+
https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansJP.zip;subdir=${BP}
 \
 file://44-source-han-sans-jp.conf \
 "
-SRC_URI[md5sum] = "908fbf97f3df04a6838708c093f1e900"
-SRC_URI[sha256sum] = 
"dc6dbae3fba35f220bac88ba7130b826c7efe1282f472788fae3628b79be3f54"
-
-S = "${WORKDIR}/SourceHanSansJP"
+SRC_URI[sha256sum] = 
"1ae9f62ad620d686c4a049ce25cf54e3afd8fefc954a678c644cf9802750c17e"
 
 do_install() {
 install -d ${D}${sysconfdir}/fonts/conf.d/
diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb 
b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
index 5ab5057d8..f536d1b61 100644
--- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
+++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fonts_2.004.bb
@@ -11,13 +11,10 @@ inherit allarch fontcache
 #EXCLUDE_FROM_WORLD = "1"
 
 SRC_URI = " \
-
svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/TW;protocol=http;rev=82
 \
+
https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansKR.zip;subdir=${BP}
 \
 file://44-source-han-sans-kr.conf \
 "
-SRC_URI[md5sum] = "f8d1bd6c87d8575afdb25e2f46bd81d4"
-SRC_URI[sha256sum] = 
"38fd15c80f9980492faaa1af39ff873d8a38e45027023fb17d5b10d4b4b0e6af"
-
-S = "${WORKDIR}/SourceHanSansKR"
+SRC_URI[sha256sum] = 
"02fe28a48c6381c49d61c27a1b173c77f0e6f2b9f2b68e79f076f10a6a8f4bfe"
 
 do_install() {
 install -d ${D}${sysconfdir}/fonts/conf.d/
diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb 
b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb
index a2b3fff07..0a4aff5e3 100644
--- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb
+++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-tw-fonts_2.004.bb
@@ -11,13 +11,10 @@ inherit allarch fontcache
 #EXCLUDE_FROM_WORLD = "1"
 
 SRC_URI = " \
-
svn://github.com/adobe-fonts/source-han-sans;module=branches/release/SubsetOTF/TW;protocol=http;rev=82
 \
+
https://github.com/adobe-fonts/source-han-sans/releases/download/2.004R/SourceHanSansTW.zip;subdir=${BP}
 \
 file://44-source-han-sans-tw.conf \
 "
-SRC_URI[md5sum] = "6533b71b31c19e548768f0fc963202f3"
-SRC_URI[sha256sum] = 
"92ba161921c5cdec5a8f8d5711676f0865b50cee071c25eb4bd4125b5af59fd0"
-
-S = "${WORKDIR}/SourceHanSansTW"
+SRC_URI[sha256sum] = 
"11d78c8fbc1a4bb04a453cdd65c99db8d41ec5cd6ba6d68c8638e6ba170de806"
 
 do_install() {
 install -d ${D}${sysconfdir}/fonts/conf.d/
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this