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

Re: elfutils not building its HPPA/PARISC port?



Hi Sam,

I don't know whether the files were ever submitted upstream.  The files date from 2002.  HP stopped
supporting PA-RISC work at Redhat somewhere around that time.

Anyway, the errors don't look hard to fix.  Do you want to take a crack at a patch?

Regards,
Dave

On 2022-02-01 9:38 p.m., Sam James wrote:
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


--
John David Anglin  dave.anglin@bell.net


Reply to: