It was actually down to spacing in the patch (not sure if it's on my end or the AUR has done it). I manually fixed that up and got it applied now. Thanks for your help.
Search Criteria
Package Details: uefi-shell-git 28272.edk2.stable201903.2535.gd150439b72-3
Git Clone URL: | https://aur.archlinux.org/uefi-shell-git.git (read-only, click to copy) |
---|---|
Package Base: | uefi-shell-git |
Description: | UEFI Shell v2 - from Tianocore EDK2 - GIT Version |
Upstream URL: | https://github.com/tianocore/edk2 |
Licenses: | |
Conflicts: | |
Provides: | |
Submitter: | ridikulusrat |
Maintainer: | Flubbadub |
Last Packager: | Flubbadub |
Votes: | 50 |
Popularity: | 0.124079 |
First Submitted: | 2015-07-22 00:17 |
Last Updated: | 2020-11-10 23:52 |
Flubbadub commented on 2020-11-10 23:51
gesh commented on 2020-11-10 21:48
I think the sticking point is the pkgver edit, try removing it. Can't test right now.
Flubbadub commented on 2020-11-10 08:32
gesh: Thanks for the patch but I couldn't get it to apply. I could obviously apply the same changes but that would then lose your attribution. Any chance you can update the patch for the current state?
For now I applied mcmodder's suggestion which at least fixes the immediate issue.
gesh commented on 2020-11-02 22:03
Below is a patch to implement mcmodder's suggestion. I go a little further, having all module fetching managed by makepkg instead of git directly
From 9117c0006bfc42e8c55db40710640cc7beec189c Mon Sep 17 00:00:00 2001
From: Gesh <gesh@gesh.uni.cx>
Date: Mon, 2 Nov 2020 21:35:24
Subject: [PATCH] Fix build after submodulification
---
PKGBUILD | 35 ++++++++++++++++++++++++++++++-----
1 file changed, 30 insertions(+), 5 deletions(-)
diff --git a/PKGBUILD b/PKGBUILD
index d3ba187..2362500 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -29,8 +29,8 @@ _GUID="EA4BB293-2D7F-4456-A681-1F22F42CD0BC"
_pkgname="uefi-shell"
pkgname="${_pkgname}-git"
-pkgver=26946.edk2.stable201903.1209.gf8dd7c7018
-pkgrel=1
+pkgver=28247.edk2.stable201903.2510.gffddac3e0f
+pkgrel=2
pkgdesc="UEFI Shell v2 - from Tianocore EDK2 - GIT Version"
url="https://github.com/tianocore/edk2"
arch=('x86_64' 'i686')
@@ -45,9 +45,28 @@ provides=('uefi-shell')
install="${_pkgname}.install"
-source=("${_TIANO_DIR_}::git+https://github.com/tianocore/edk2.git#branch=master")
-
-sha1sums=('SKIP')
+declare -A _submod_path
+_submod_path["CryptoPkg/Library/OpensslLib/openssl"]="openssl"
+_submod_path["SoftFloat"]="softfloat"
+_submod_path["UnitTestFrameworkPkg/Library/CmockaLib/cmocka"]="cmocka"
+_submod_path["MdeModulePkg/Universal/RegularExpressionDxe/oniguruma"]="oniguruma"
+_submod_path["MdeModulePkg/Library/BrotliCustomDecompressLib/brotli"]="brotli"
+_submod_path["BaseTools/Source/C/BrotliCompress/brotli"]="brotli"
+
+source=("${_TIANO_DIR_}::git+https://github.com/tianocore/edk2.git#branch=master"
+ brotli::git+https://github.com/google/brotli
+ softfloat::git+https://github.com/ucb-bar/berkeley-softfloat-3.git
+ cmocka::git+https://git.cryptomilk.org/projects/cmocka.git
+ oniguruma::git+https://github.com/kkos/oniguruma
+ openssl::git+https://github.com/openssl/openssl
+ )
+
+sha1sums=('SKIP'
+ 'SKIP'
+ 'SKIP'
+ 'SKIP'
+ 'SKIP'
+ 'SKIP')
pkgver() {
cd "${srcdir}/${_TIANO_DIR_}/"
@@ -63,6 +82,12 @@ _setup_env_vars() {
_prepare_tianocore_sources() {
cd "${_UDK_DIR}/"
+ git submodule init
+ for _module in "${!_submod_path[@]}"; do
+ git config submodule."$_module".url "$srcdir/${_submod_path[$_module]}"
+ done
+ git submodule update
+
msg "Cleanup UDK config files"
rm -rf "${_UDK_DIR}/Build/" || true
rm -rf "${_UDK_DIR}/Conf/" || true
--
2.29.2
mcmodder commented on 2020-05-14 17:11
Build fails due to non-initialized git submodules: https://github.com/tianocore/tianocore.github.io/wiki/Common-instructions#initialize-submodules https://bugzilla.tianocore.org/show_bug.cgi?id=2677
Fixes with adding git submodule update --init
to _prepare_tianocore_sources()
Flubbadub commented on 2019-11-11 22:03
Yes, I've done that now, thanks for the suggestion. Seems sensible and it even simplifies the PKGBUILD a bit as a nice little bonus.
hardfalcon commented on 2019-11-11 09:58
Can you remove the python2 references, and add the normal "python" package (for python3) to the "makedepends" array? Upstream has supported python3 since February 2019: https://bugzilla.tianocore.org/show_bug.cgi?id=55
GeneArch commented on 2019-11-06 16:08
Thank you for adopting :)
Flubbadub commented on 2019-11-06 09:05
I adopted this just so that I could apply the fix suggested by GeneArch (after testing it) but I'm happy to pass it over to someone else who's interested & more knowledgeable than myself. If no one contacts me I will do my best to keep things working as expected.
GeneArch commented on 2019-10-01 18:05
By looking at the file src/edk2/ShellPkg/ShellPkg.dsc
It is clear 2 versions of the shell are built. The 2nd one is :
# # Build a second version of the shell with all commands integrated # ShellPkg/Application/Shell/Shell.inf { <Defines> FILE_GUID = EA4BB293-2D7F-4456-A681-1F22F42CD0BC
So this is the one we will use
i.e. the shell to rename is Shell_EA4BB293-2D7F-4456-A681-1F22F42CD0BC.efi
Which means the change to PKGBUILD is simple:
In package()
Add a variable:
GUID="EA4BB293-2D7F-4456-A681-1F22F42CD0BC"
change install to use above guid
install -D ..../Shell_${GUID}.efi" ...
Pinned Comments
Hello71 commented on 2016-09-20 14:10
If you have any problems (e.g. does not compile) with this package, consider the binary shells: https://github.com/tianocore/edk2/releases.