[Buildroot] [PATCH] syslinux: Drop patch 0003, not needed after gnu-efi update

[Buildroot] [PATCH] syslinux: Drop patch 0003, not needed after gnu-efi update

[Benoît Allard]

Signed-off-by: Beno?t Allard <benoit.allard@greenbone.net>
---
 ....elf-Not-enough-room-for-program-headers-.patch | 35 ----------------------
 1 file changed, 35 deletions(-)
 delete mode 100644 boot/syslinux/0003-Fix-ldlinux.elf-Not-enough-room-for-program-headers-.patch

diff --git a/boot/syslinux/0003-Fix-ldlinux.elf-Not-enough-room-for-program-headers-.patch b/boot/syslinux/0003-Fix-ldlinux.elf-Not-enough-room-for-program-headers-.patch
deleted file mode 100644
index 83a87ee..0000000
--- a/boot/syslinux/0003-Fix-ldlinux.elf-Not-enough-room-for-program-headers-.patch
+++ /dev/null
@@ -1,35 +0,0 @@
-From 61de7762389d460da7ffdd644f50c60175cce23b Mon Sep 17 00:00:00 2001
-From: Steve McIntyre <93sam@debian.org>
-Date: Wed, 5 Apr 2017 22:09:37 +0200
-Subject: [PATCH] Fix 'ldlinux.elf: Not enough room for program headers, try
- linking with -N'
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-Fix for https://bugs.debian.org/846679: syslinux: FTBFS: ld:
-ldlinux.elf: Not enough room for program headers, try linking with -N
-
-https://bugs.debian.org/cgi-bin/bugreport.cgi?att=1;bug=846679;filename=syslinux_6.03%2Bdfsg-14.1.debdiff;msg=10
-
-Signed-off-by: Beno?t Allard <benoit.allard@greenbone.net>
----
- core/Makefile | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/core/Makefile b/core/Makefile
-index ad0acb5..58a3545 100644
---- a/core/Makefile
-+++ b/core/Makefile
-@@ -165,7 +165,7 @@ LDSCRIPT = $(SRC)/$(ARCH)/syslinux.ld
- 
- %.elf: %.o $(LIBDEP) $(LDSCRIPT) $(AUXLIBS)
- 	$(LD) $(LDFLAGS) -Bsymbolic $(LD_PIE) -E --hash-style=gnu -T $(LDSCRIPT) -M -o $@ $< \
--		--start-group $(LIBS) $(subst $(*F).elf,lib$(*F).a,$@) --end-group \
-+		--start-group $(LIBS) $(subst $(*F).elf,lib$(*F).a,$@) --end-group --no-dynamic-linker \
- 		> $(@:.elf=.map)
- 	$(OBJDUMP) -h $@ > $(@:.elf=.sec)
- 	$(PERL) $(SRC)/lstadjust.pl $(@:.elf=.lsr) $(@:.elf=.sec) $(@:.elf=.lst)
--- 
-2.7.4
-
-- 
2.1.4

[Buildroot] [PATCH] syslinux: Drop patch 0003, not needed after gnu-efi update

[Thomas Petazzoni]

Hello,

On Mon,  3 Jul 2017 11:11:03 +0200, Beno?t Allard wrote:
> Signed-off-by: Beno?t Allard <benoit.allard@greenbone.net>
> ---
>  ....elf-Not-enough-room-for-program-headers-.patch | 35 ----------------------
>  1 file changed, 35 deletions(-)
>  delete mode 100644 boot/syslinux/0003-Fix-ldlinux.elf-Not-enough-room-for-program-headers-.patch

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] syslinux: Drop patch 0003, not needed after gnu-efi update

[Arnout Vandecappelle]

 Hi Benoit,

On 03-07-17 11:11, Beno?t Allard wrote:
> Signed-off-by: Beno?t Allard <benoit.allard@greenbone.net>

 Is this patch really not needed anymore after the gnu-efi update? Isn't it
still needed for BIOS (and perhaps others as well, e.g. isolinux)? I'm looking
at the build failure of ts5x00_defconfig and it seems to point at this issue.
And indeed, reinstating this patch fixes the build failure.

[snip]
> -Fix for https://bugs.debian.org/846679: syslinux: FTBFS: ld:
> -ldlinux.elf: Not enough room for program headers, try linking with -N

 This is exactly the error we see.

 I sent a patch to revert this, could you check it?

 Regards,
 Arnout

[1] https://gitlab.com/arnout/buildroot/-/jobs/28979377

[snip]

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF