Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] [WIP] grub: update to 2.06
@ 2021-06-12  3:09 FollieHiyuki
  2021-06-12  3:12 ` ericonr
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: FollieHiyuki @ 2021-06-12  3:09 UTC (permalink / raw)
  To: ml

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

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

https://github.com/FollieHiyuki/void-packages grub-update
https://github.com/void-linux/void-packages/pull/31432

[WIP] grub: update to 2.06
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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
-->

I opened the PR early for reviewing. The only thing I have done is reviewing and re-applying patches to fit the new tarball.
I will try to build and test whether the package works in the next few days when I have time to spin up a Void VM.

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

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

From 85743e7059692da5cfa6a1977d57858f98ac2c11 Mon Sep 17 00:00:00 2001
From: FollieHiyuki <folliekazetani@protonmail.com>
Date: Wed, 9 Jun 2021 01:32:08 +0300
Subject: [PATCH] grub: update to 2.06

---
 .../patches/0004-add-GRUB_COLOR_variables.patch     |  6 +++---
 srcpkgs/grub/patches/grub-install-pvxen-paths.patch |  4 ++--
 .../grub/patches/install_powerpc_machtypes.patch    | 12 ++++++------
 srcpkgs/grub/patches/os-prober-osx86.patch          |  2 +-
 srcpkgs/grub/template                               | 13 +++----------
 5 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch b/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch
index f899717c4d99..1e6b406d1b5c 100644
--- a/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch
+++ b/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch
@@ -1,11 +1,11 @@
 diff --git a/util/grub-mkconfig.in b/util/grub-mkconfig.in
-index 3390ba9..c416489 100644
+index f8cbb8d..f271608 100644
 --- util/grub-mkconfig.in
 +++ util/grub-mkconfig.in
-@@ -218,6 +218,8 @@ export GRUB_DEFAULT \
+@@ -246,6 +246,8 @@ export GRUB_DEFAULT \
+   GRUB_BACKGROUND \
    GRUB_THEME \
    GRUB_GFXPAYLOAD_LINUX \
-   GRUB_DISABLE_OS_PROBER \
 +  GRUB_COLOR_NORMAL \
 +  GRUB_COLOR_HIGHLIGHT \
    GRUB_INIT_TUNE \
diff --git a/srcpkgs/grub/patches/grub-install-pvxen-paths.patch b/srcpkgs/grub/patches/grub-install-pvxen-paths.patch
index 0522bcd60efe..0965c058f4c2 100644
--- a/srcpkgs/grub/patches/grub-install-pvxen-paths.patch
+++ b/srcpkgs/grub/patches/grub-install-pvxen-paths.patch
@@ -31,7 +31,7 @@ diff --git a/util/grub-install.c b/util/grub-install.c
 index b82c14d41..caadada98 100644
 --- a/util/grub-install.c
 +++ b/util/grub-install.c
-@@ -2057,6 +2057,28 @@ main (int argc, char *argv[])
+@@ -1962,6 +1962,28 @@ main (int argc, char *argv[])
  	}
        break;
  
@@ -60,7 +60,7 @@ index b82c14d41..caadada98 100644
      case GRUB_INSTALL_PLATFORM_MIPSEL_LOONGSON:
      case GRUB_INSTALL_PLATFORM_MIPSEL_QEMU_MIPS:
      case GRUB_INSTALL_PLATFORM_MIPS_QEMU_MIPS:
-@@ -2066,8 +2088,6 @@ main (int argc, char *argv[])
+@@ -1971,8 +1971,6 @@ main (int argc, char *argv[])
      case GRUB_INSTALL_PLATFORM_MIPSEL_ARC:
      case GRUB_INSTALL_PLATFORM_ARM_UBOOT:
      case GRUB_INSTALL_PLATFORM_I386_QEMU:
diff --git a/srcpkgs/grub/patches/install_powerpc_machtypes.patch b/srcpkgs/grub/patches/install_powerpc_machtypes.patch
index ce5e36cb4a57..68f050878524 100644
--- a/srcpkgs/grub/patches/install_powerpc_machtypes.patch
+++ b/srcpkgs/grub/patches/install_powerpc_machtypes.patch
@@ -28,7 +28,7 @@ diff --git a/grub-core/osdep/basic/platform.c b/grub-core/osdep/basic/platform.c
 index 4b5502aeb..2ab907976 100644
 --- grub-core/osdep/basic/platform.c
 +++ grub-core/osdep/basic/platform.c
-@@ -24,3 +24,8 @@ grub_install_get_default_x86_platform (void)
+@@ -30,3 +30,8 @@ grub_install_get_default_x86_platform (void)
    return "i386-pc";
  }
  
@@ -49,7 +49,7 @@ index 35f1bcc0e..9805c36d4 100644
  #include <string.h>
  
  #include <sys/utsname.h>
-@@ -145,3 +146,74 @@ grub_install_get_default_x86_platform (void)
+@@ -154,3 +154,74 @@ grub_install_get_default_x86_platform (void)
    grub_util_info ("... not found");
    return "i386-pc";
  }
@@ -128,7 +128,7 @@ diff --git a/grub-core/osdep/unix/platform.c b/grub-core/osdep/unix/platform.c
 index a3fcfcaca..28cb37e15 100644
 --- grub-core/osdep/unix/platform.c
 +++ grub-core/osdep/unix/platform.c
-@@ -212,13 +212,29 @@ grub_install_register_ieee1275 (int is_prep, const char *install_device,
+@@ -218,13 +218,29 @@ grub_install_register_ieee1275 (int is_prep, const char *install_device,
    else
      boot_device = get_ofpathname (install_device);
  
@@ -185,21 +185,21 @@ diff --git a/include/grub/util/install.h b/include/grub/util/install.h
 index 5ca4811cd..9f517a1bb 100644
 --- include/grub/util/install.h
 +++ include/grub/util/install.h
-@@ -206,6 +206,9 @@ grub_install_create_envblk_file (const char *name);
+@@ -223,6 +223,9 @@ grub_install_get_default_arm_platform (void);
  const char *
  grub_install_get_default_x86_platform (void);
  
 +const char *
 +grub_install_get_default_powerpc_machtype (void);
 +
- void
+ init
  grub_install_register_efi (grub_device_t efidir_grub_dev,
  			   const char *efifile_path,
 diff --git a/util/grub-install.c b/util/grub-install.c
 index e1a0202da..70b22eec4 100644
 --- util/grub-install.c
 +++ util/grub-install.c
-@@ -1155,7 +1155,18 @@ main (int argc, char *argv[])
+@@ -1179,7 +1179,18 @@ main (int argc, char *argv[])
  
    if (platform == GRUB_INSTALL_PLATFORM_POWERPC_IEEE1275)
      {
diff --git a/srcpkgs/grub/patches/os-prober-osx86.patch b/srcpkgs/grub/patches/os-prober-osx86.patch
index 5fabbeb6d993..061797fd976f 100644
--- a/srcpkgs/grub/patches/os-prober-osx86.patch
+++ b/srcpkgs/grub/patches/os-prober-osx86.patch
@@ -2,7 +2,7 @@ Patches OS X detection out of os-prober hook on non-x86 architectures. The
 menu entries generated for those are invalid for non-x86 Mac stuff.
 --- util/grub.d/30_os-prober.in
 +++ util/grub.d/30_os-prober.in
-@@ -42,6 +42,11 @@ if [ -z "${OSPROBED}" ] ; then
+@@ -45,6 +45,11 @@ if [ -z "${OSPROBED}" ] ; then
  fi
  
  osx_entry() {
diff --git a/srcpkgs/grub/template b/srcpkgs/grub/template
index 69cea9ae19a3..bf10fc663483 100644
--- a/srcpkgs/grub/template
+++ b/srcpkgs/grub/template
@@ -1,7 +1,7 @@
 # Template file for 'grub'
 pkgname=grub
-version=2.04
-revision=6
+version=2.06
+revision=1
 hostmakedepends="python3 pkg-config flex freetype-devel font-unifont-bdf help2man"
 makedepends="libusb-compat-devel ncurses-devel freetype-devel
  liblzma-devel device-mapper-devel fuse-devel"
@@ -12,7 +12,7 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="GPL-3.0-or-later"
 homepage="https://www.gnu.org/software/grub/"
 distfiles="${GNU_SITE}/grub/grub-${version}.tar.xz"
-checksum=e5292496995ad42dabe843a0192cf2a2c502e7ffcc7479398232b10a472df77d
+checksum=b79ea44af91b93d17cd3fe80bdae6ed43770678a9a5ae192ccea803ebb657ee1
 
 archs="i686* x86_64* aarch64* ppc ppc-musl ppc64*"
 nopie=yes
@@ -43,10 +43,6 @@ case "$XBPS_TARGET_MACHINE" in
 		;;
 esac
 
-post_extract() {
-	sed -i -e 's,freetype/ftsynth.h,freetype2/ftsynth.h,' util/grub-mkfont.c
-}
-
 do_configure() {
 	unset CC AS LD RANLIB CPP
 	local freestanding_cflags="-fno-stack-protector"
@@ -61,9 +57,6 @@ do_configure() {
 		--enable-grub-mkfont --enable-grub-mount --disable-werror
 		--sbindir=/usr/bin"
 
-	# fix unifont.bdf location so grub-mkfont can create *.pf2 files
-	sed -i 's|/usr/share/fonts/unifont|/usr/share/fonts/misc|' configure
-
 	# build tools
 	msg_normal "Configuring grub tools...\n"
 	mkdir $wrksrc/build

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

* Re: [WIP] grub: update to 2.06
  2021-06-12  3:09 [PR PATCH] [WIP] grub: update to 2.06 FollieHiyuki
@ 2021-06-12  3:12 ` ericonr
  2021-06-12  3:14 ` [PR PATCH] [Updated] " FollieHiyuki
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-06-12  3:12 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31432#issuecomment-859989109

Comment:
I think we should revert https://git.savannah.gnu.org/cgit/grub.git/commit/?id=e346414725a70e5c74ee87ca14e580c66f517666 for Void. os-prober is relied upon by a bunch of people.

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

* Re: [PR PATCH] [Updated] [WIP] grub: update to 2.06
  2021-06-12  3:09 [PR PATCH] [WIP] grub: update to 2.06 FollieHiyuki
  2021-06-12  3:12 ` ericonr
@ 2021-06-12  3:14 ` FollieHiyuki
  2021-06-12  3:20 ` FollieHiyuki
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: FollieHiyuki @ 2021-06-12  3:14 UTC (permalink / raw)
  To: ml

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

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

https://github.com/FollieHiyuki/void-packages grub-update
https://github.com/void-linux/void-packages/pull/31432

[WIP] grub: update to 2.06
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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
-->

I opened the PR early for reviewing. The only thing I have done is reviewing and re-applying patches to fit the new tarball.
I will try to build and test whether the package works in the next few days when I have time to spin up a Void VM.

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

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

From ec95c9aee3e8c22b3ba43a57ab63612b2b44a39e Mon Sep 17 00:00:00 2001
From: FollieHiyuki <folliekazetani@protonmail.com>
Date: Wed, 9 Jun 2021 01:32:08 +0300
Subject: [PATCH] grub: update to 2.06

---
 .../patches/0004-add-GRUB_COLOR_variables.patch     |  6 +++---
 srcpkgs/grub/patches/grub-install-pvxen-paths.patch |  4 ++--
 .../grub/patches/install_powerpc_machtypes.patch    | 12 ++++++------
 srcpkgs/grub/patches/os-prober-osx86.patch          |  2 +-
 srcpkgs/grub/template                               | 13 +++----------
 5 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch b/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch
index f899717c4d99..1e6b406d1b5c 100644
--- a/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch
+++ b/srcpkgs/grub/patches/0004-add-GRUB_COLOR_variables.patch
@@ -1,11 +1,11 @@
 diff --git a/util/grub-mkconfig.in b/util/grub-mkconfig.in
-index 3390ba9..c416489 100644
+index f8cbb8d..f271608 100644
 --- util/grub-mkconfig.in
 +++ util/grub-mkconfig.in
-@@ -218,6 +218,8 @@ export GRUB_DEFAULT \
+@@ -246,6 +246,8 @@ export GRUB_DEFAULT \
+   GRUB_BACKGROUND \
    GRUB_THEME \
    GRUB_GFXPAYLOAD_LINUX \
-   GRUB_DISABLE_OS_PROBER \
 +  GRUB_COLOR_NORMAL \
 +  GRUB_COLOR_HIGHLIGHT \
    GRUB_INIT_TUNE \
diff --git a/srcpkgs/grub/patches/grub-install-pvxen-paths.patch b/srcpkgs/grub/patches/grub-install-pvxen-paths.patch
index 0522bcd60efe..0965c058f4c2 100644
--- a/srcpkgs/grub/patches/grub-install-pvxen-paths.patch
+++ b/srcpkgs/grub/patches/grub-install-pvxen-paths.patch
@@ -31,7 +31,7 @@ diff --git a/util/grub-install.c b/util/grub-install.c
 index b82c14d41..caadada98 100644
 --- a/util/grub-install.c
 +++ b/util/grub-install.c
-@@ -2057,6 +2057,28 @@ main (int argc, char *argv[])
+@@ -1962,6 +1962,28 @@ main (int argc, char *argv[])
  	}
        break;
  
@@ -60,7 +60,7 @@ index b82c14d41..caadada98 100644
      case GRUB_INSTALL_PLATFORM_MIPSEL_LOONGSON:
      case GRUB_INSTALL_PLATFORM_MIPSEL_QEMU_MIPS:
      case GRUB_INSTALL_PLATFORM_MIPS_QEMU_MIPS:
-@@ -2066,8 +2088,6 @@ main (int argc, char *argv[])
+@@ -1971,8 +1971,6 @@ main (int argc, char *argv[])
      case GRUB_INSTALL_PLATFORM_MIPSEL_ARC:
      case GRUB_INSTALL_PLATFORM_ARM_UBOOT:
      case GRUB_INSTALL_PLATFORM_I386_QEMU:
diff --git a/srcpkgs/grub/patches/install_powerpc_machtypes.patch b/srcpkgs/grub/patches/install_powerpc_machtypes.patch
index ce5e36cb4a57..37f52460f6f3 100644
--- a/srcpkgs/grub/patches/install_powerpc_machtypes.patch
+++ b/srcpkgs/grub/patches/install_powerpc_machtypes.patch
@@ -28,7 +28,7 @@ diff --git a/grub-core/osdep/basic/platform.c b/grub-core/osdep/basic/platform.c
 index 4b5502aeb..2ab907976 100644
 --- grub-core/osdep/basic/platform.c
 +++ grub-core/osdep/basic/platform.c
-@@ -24,3 +24,8 @@ grub_install_get_default_x86_platform (void)
+@@ -30,3 +30,8 @@ grub_install_get_default_x86_platform (void)
    return "i386-pc";
  }
  
@@ -49,7 +49,7 @@ index 35f1bcc0e..9805c36d4 100644
  #include <string.h>
  
  #include <sys/utsname.h>
-@@ -145,3 +146,74 @@ grub_install_get_default_x86_platform (void)
+@@ -154,3 +154,74 @@ grub_install_get_default_x86_platform (void)
    grub_util_info ("... not found");
    return "i386-pc";
  }
@@ -128,7 +128,7 @@ diff --git a/grub-core/osdep/unix/platform.c b/grub-core/osdep/unix/platform.c
 index a3fcfcaca..28cb37e15 100644
 --- grub-core/osdep/unix/platform.c
 +++ grub-core/osdep/unix/platform.c
-@@ -212,13 +212,29 @@ grub_install_register_ieee1275 (int is_prep, const char *install_device,
+@@ -218,13 +218,29 @@ grub_install_register_ieee1275 (int is_prep, const char *install_device,
    else
      boot_device = get_ofpathname (install_device);
  
@@ -185,21 +185,21 @@ diff --git a/include/grub/util/install.h b/include/grub/util/install.h
 index 5ca4811cd..9f517a1bb 100644
 --- include/grub/util/install.h
 +++ include/grub/util/install.h
-@@ -206,6 +206,9 @@ grub_install_create_envblk_file (const char *name);
+@@ -223,6 +223,9 @@ grub_install_get_default_arm_platform (void);
  const char *
  grub_install_get_default_x86_platform (void);
  
 +const char *
 +grub_install_get_default_powerpc_machtype (void);
 +
- void
+ int
  grub_install_register_efi (grub_device_t efidir_grub_dev,
  			   const char *efifile_path,
 diff --git a/util/grub-install.c b/util/grub-install.c
 index e1a0202da..70b22eec4 100644
 --- util/grub-install.c
 +++ util/grub-install.c
-@@ -1155,7 +1155,18 @@ main (int argc, char *argv[])
+@@ -1179,7 +1179,18 @@ main (int argc, char *argv[])
  
    if (platform == GRUB_INSTALL_PLATFORM_POWERPC_IEEE1275)
      {
diff --git a/srcpkgs/grub/patches/os-prober-osx86.patch b/srcpkgs/grub/patches/os-prober-osx86.patch
index 5fabbeb6d993..061797fd976f 100644
--- a/srcpkgs/grub/patches/os-prober-osx86.patch
+++ b/srcpkgs/grub/patches/os-prober-osx86.patch
@@ -2,7 +2,7 @@ Patches OS X detection out of os-prober hook on non-x86 architectures. The
 menu entries generated for those are invalid for non-x86 Mac stuff.
 --- util/grub.d/30_os-prober.in
 +++ util/grub.d/30_os-prober.in
-@@ -42,6 +42,11 @@ if [ -z "${OSPROBED}" ] ; then
+@@ -45,6 +45,11 @@ if [ -z "${OSPROBED}" ] ; then
  fi
  
  osx_entry() {
diff --git a/srcpkgs/grub/template b/srcpkgs/grub/template
index 69cea9ae19a3..bf10fc663483 100644
--- a/srcpkgs/grub/template
+++ b/srcpkgs/grub/template
@@ -1,7 +1,7 @@
 # Template file for 'grub'
 pkgname=grub
-version=2.04
-revision=6
+version=2.06
+revision=1
 hostmakedepends="python3 pkg-config flex freetype-devel font-unifont-bdf help2man"
 makedepends="libusb-compat-devel ncurses-devel freetype-devel
  liblzma-devel device-mapper-devel fuse-devel"
@@ -12,7 +12,7 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="GPL-3.0-or-later"
 homepage="https://www.gnu.org/software/grub/"
 distfiles="${GNU_SITE}/grub/grub-${version}.tar.xz"
-checksum=e5292496995ad42dabe843a0192cf2a2c502e7ffcc7479398232b10a472df77d
+checksum=b79ea44af91b93d17cd3fe80bdae6ed43770678a9a5ae192ccea803ebb657ee1
 
 archs="i686* x86_64* aarch64* ppc ppc-musl ppc64*"
 nopie=yes
@@ -43,10 +43,6 @@ case "$XBPS_TARGET_MACHINE" in
 		;;
 esac
 
-post_extract() {
-	sed -i -e 's,freetype/ftsynth.h,freetype2/ftsynth.h,' util/grub-mkfont.c
-}
-
 do_configure() {
 	unset CC AS LD RANLIB CPP
 	local freestanding_cflags="-fno-stack-protector"
@@ -61,9 +57,6 @@ do_configure() {
 		--enable-grub-mkfont --enable-grub-mount --disable-werror
 		--sbindir=/usr/bin"
 
-	# fix unifont.bdf location so grub-mkfont can create *.pf2 files
-	sed -i 's|/usr/share/fonts/unifont|/usr/share/fonts/misc|' configure
-
 	# build tools
 	msg_normal "Configuring grub tools...\n"
 	mkdir $wrksrc/build

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

* Re: [WIP] grub: update to 2.06
  2021-06-12  3:09 [PR PATCH] [WIP] grub: update to 2.06 FollieHiyuki
  2021-06-12  3:12 ` ericonr
  2021-06-12  3:14 ` [PR PATCH] [Updated] " FollieHiyuki
@ 2021-06-12  3:20 ` FollieHiyuki
  2021-06-16 10:45 ` FollieHiyuki
  2021-07-02 18:28 ` [PR PATCH] [Closed]: " abenson
  4 siblings, 0 replies; 6+ messages in thread
From: FollieHiyuki @ 2021-06-12  3:20 UTC (permalink / raw)
  To: ml

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

New comment by FollieHiyuki on void-packages repository

https://github.com/void-linux/void-packages/pull/31432#issuecomment-859989874

Comment:
> I think we should revert https://git.savannah.gnu.org/cgit/grub.git/commit/?id=e346414725a70e5c74ee87ca14e580c66f517666 for Void. os-prober is relied upon by a bunch of people.

Let me download the patch and add it (reverted) into the commit.
I personally don't use `os-prober` though (there is only Void in my main machine). Shouldnt we add the option (as written in the description) rather than revert the patch?

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

* Re: [WIP] grub: update to 2.06
  2021-06-12  3:09 [PR PATCH] [WIP] grub: update to 2.06 FollieHiyuki
                   ` (2 preceding siblings ...)
  2021-06-12  3:20 ` FollieHiyuki
@ 2021-06-16 10:45 ` FollieHiyuki
  2021-07-02 18:28 ` [PR PATCH] [Closed]: " abenson
  4 siblings, 0 replies; 6+ messages in thread
From: FollieHiyuki @ 2021-06-16 10:45 UTC (permalink / raw)
  To: ml

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

New comment by FollieHiyuki on void-packages repository

https://github.com/void-linux/void-packages/pull/31432#issuecomment-862263597

Comment:
There is also this patch:
https://git.savannah.gnu.org/cgit/grub.git/commit/?id=54e0a1bbf1e9106901a557195bb35e5e20fb3925

added later on, in which they encouraged to override the value (if we really want os-prober).

I just borked 2 VMs (BIOS x86_64) so the package is not working properly. Investigating.

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

* Re: [PR PATCH] [Closed]: [WIP] grub: update to 2.06
  2021-06-12  3:09 [PR PATCH] [WIP] grub: update to 2.06 FollieHiyuki
                   ` (3 preceding siblings ...)
  2021-06-16 10:45 ` FollieHiyuki
@ 2021-07-02 18:28 ` abenson
  4 siblings, 0 replies; 6+ messages in thread
From: abenson @ 2021-07-02 18:28 UTC (permalink / raw)
  To: ml

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

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

[WIP] grub: update to 2.06
https://github.com/void-linux/void-packages/pull/31432

Description:
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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
-->

I opened the PR early for reviewing. The only thing I have done is reviewing and re-applying patches to fit the new tarball.
I will try to build and test whether the package works in the next few days when I have time to spin up a Void VM.

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

end of thread, other threads:[~2021-07-02 18:28 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-12  3:09 [PR PATCH] [WIP] grub: update to 2.06 FollieHiyuki
2021-06-12  3:12 ` ericonr
2021-06-12  3:14 ` [PR PATCH] [Updated] " FollieHiyuki
2021-06-12  3:20 ` FollieHiyuki
2021-06-16 10:45 ` FollieHiyuki
2021-07-02 18:28 ` [PR PATCH] [Closed]: " abenson

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