On 07/02/2021 18:19, Bruce Dubbs wrote:
On 2/7/21 8:11 AM, Pierre Labastie wrote:
On Sun, 2021-02-07 at 14:15 +0100, Pierre Labastie wrote:
On Sun, 2021-02-07 at 13:38 +0100, Frans de Boer wrote:
LS,

Finally had a change to finish the LFS build. Alas, the grub-
install
reported the old error "Decompression is too big".
Here is the solution to add to the section to compile grub:

# Patch gentpl.py to avoid an error from grub-mkinstall
"Decompressor
is
too big"
# Source:
https://www.mail-archive.com/grub-devel@gnu.org/msg29840.html
#
sed -i -E "s@(.*-R .note.gnu.gold-version -R)(.*)@\1
.note.gnu.property
-R\2@" gentpl.py
ln -fs /usr/bin/python3 /usr/bin/python  # grub needs 'python'
./autogen.sh
rm -f /usr/bin/python


Very interesting, thanks! There are several hundreds of commits in
grub
repositories since the last release. I hope we'll see a release
soon...

For us, instead of running autogen, can't the generated file be
patched
directly? I looks like gentpl.py generates Makefile.*.am, and my
guess
is that this ends up in Makefile.*.in after running autofu...

Will look more closely

Yes this can be simplified to (sorry for format, need to fit into 80
chars per line):
sed -i -E \
"s@(.*-R .note.gnu.gold-version -R)(.*)@\1 .note.gnu.property -R\2@" \
  Makefile.in grub-core/Makefile.in

I think it can be made shorter:

"/(version -R)/s/\1 .note.gnu.property -R/"

But not tested.

  -- Bruce



before running configure. Results:
- without the sed, after make:
------
$ ls -l grub-core/lzma*
-rwxr-xr-x 1 pierre pierre      4972  7 févr. 15:01 grub-
core/lzma_decompress.image
-rwxr-xr-x 1 pierre pierre 134479612  7 févr. 15:01 grub-
core/lzma_decompress.img
------
- with the sed, after make:
------
$ ls -l grub-core/lzma*
-rwxr-xr-x 1 pierre pierre 4972  7 févr. 15:09 grub-
core/lzma_decompress.image
-rwxr-xr-x 1 pierre pierre 2848  7 févr. 15:09 grub-
core/lzma_decompress.img
------

If nobody speaks up, will add that tomorrow to grub page.

Pierre


Pierre has a one line solution devised to be applied to the recipient file:

Can be further simplified to:
sed -e 's/\.note\.gnu\.gold-version/& -R .note.gnu.property/' \
    -i Makefile.in grub-core/Makefile.in

Just before configure (no need to run autogen, nor to relink python)

It works too and is confirmed by at least two people ;)

BTW: the sed -e ... can be changed to sed .... The -e is only required when there is more then one command to be executed by sed.

--- Frans.

--
A: Yes, just like that                            A: Ja, net zo
Q: Oh, Just like reading a book backwards         Q: Oh, net als een boek 
achterstevoren lezen
A: Because it upsets the natural flow of a story  A: Omdat het de natuurlijke 
gang uit het verhaal haalt
Q: Why is top-posting annoying?                   Q: Waarom is Top-posting zo 
irritant?

--
http://lists.linuxfromscratch.org/listinfo/lfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Do not top post on this list.

A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

http://en.wikipedia.org/wiki/Posting_style

Reply via email to