[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

elfutils not building its HPPA/PARISC port?



HI,

I'm not a Debian user but Gentoo (from where I hail) is the only other distro AFAIK still supporting HPPA, so I like
to look at what Debian is doing for some packages.

I noticed elfutils upstream lacks HPPA support but saw Debian has a HPPA backend patch [0].

Two questions:
1. Has this been attempted to be upstreamed before? Any known problems with it?
2. I'm not sure it's actually being used in Debian despite being applied(?)

The backends/Makefile.am changes seem to be missing a hunk(?):
```
--- a/backends/Makefile.am
+++ b/backends/Makefile.am
@@ -102,7 +102,7 @@ libebl_backends_a_SOURCES = $(i386_SRCS) $(sh_SRCS) $(x86_64_SRCS) \
                            $(ia64_SRCS) $(alpha_SRCS) $(arm_SRCS) \
                            $(aarch64_SRCS) $(sparc_SRCS) $(ppc_SRCS) \
                            $(ppc64_SRCS) $(s390_SRCS) \
-                           $(m68k_SRCS) $(bpf_SRCS) $(riscv_SRCS) $(csky_SRCS)
+                           $(m68k_SRCS) $(bpf_SRCS) $(riscv_SRCS) $(csky_SRCS) $(parisc_SRCS)

 libebl_backends_pic_a_SOURCES =
 am_libebl_backends_pic_a_OBJECTS = $(libebl_backends_a_SOURCES:.c=.os)
```

Without adding this to a local build in Gentoo, the PARISC files added by the patch
aren't built at all.

I don't see any reference to the files in the build log for Debian either [1].

But when I add this t Makefile.am anyway, I end up with a build failure:
```
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:41:1: error: conflicting types for ‘parisc_init’; have ‘const char *(Elf *, GElf_Half,  Ebl *, size_t)’ {aka ‘const char *
(Elf *, short unsigned int,  struct ebl *, unsigned int)’}
   41 | parisc_init (Elf *elf __attribute__ ((unused)),
      | ^~~~~~~~~~~
In file included from /var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:33:
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:31:25: note: previous declaration of ‘parisc_init’ with type ‘Ebl *(Elf *, GElf_Half,  Ebl *)’ {aka ‘struct ebl *(Elf *, short unsigned int,  struct ebl *)’}
   31 | #define BACKEND         parisc_
      |                         ^~~~~~~
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/libebl_CPU.h:37:25: note: in definition of macro ‘EBLHOOK_2’
   37 | #define EBLHOOK_2(a, b) a##b
      |                         ^
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/libebl_CPU.h:35:25: note: in expansion of macro ‘EBLHOOK_1’
   35 | #define EBLHOOK(name)   EBLHOOK_1(BACKEND, name)
      |                         ^~~~~~~~~
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/libebl_CPU.h:35:35: note: in expansion of macro ‘BACKEND’
   35 | #define EBLHOOK(name)   EBLHOOK_1(BACKEND, name)
      |                                   ^~~~~~~
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/libebl_CPU.h:40:13: note: in expansion of macro ‘EBLHOOK’
   40 | extern Ebl *EBLHOOK(init) (Elf *elf, GElf_Half machine, Ebl *eh);
      |             ^~~~~~~
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c: In function ‘parisc_init’:
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:59:5: error: ‘Ebl’ {aka ‘struct ebl’} has no member named ‘name’
   59 |   eh->name = "PA-RISC";
      |     ^~
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:72:10: error: ‘MODVERSION’ undeclared (first use in this function); did you mean ‘OLD_VERSION’?
   72 |   return MODVERSION;
      |          ^~~~~~~~~~
      |          OLD_VERSION
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:72:10: note: each undeclared identifier is reported only once for each function it appears in
/var/tmp/portage/dev-libs/elfutils-0.186/work/elfutils-0.186/backends/parisc_init.c:73:1: warning: control reaches end of non-void function [-Wreturn-type]
   73 | }
      | ^
```

[0] https://sources.debian.org/src/elfutils/0.186-1/debian/patches/hppa_backend.diff/
[1] https://buildd.debian.org/status/fetch.php?pkg=elfutils&arch=hppa&ver=0.186-1&stamp=1637187997&raw=0

Best,
sam

Attachment: signature.asc
Description: Message signed with OpenPGP


Reply to: