Source-Changes-D archive

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

Re: CVS commit: src/usr.bin/elf2aout



> Modified Files:
> 	src/usr.bin/elf2aout: elf2aout.1 elf2aout.c
> 
> Log Message:
> - add flag to produce OMAGIC
> - factor out complicated inline code into simpler functions
> - add options parsing

Thank you for your changes.

But you also put more functional changes not in the log,
i.e. you added new MID values:
 - EM_AARCH64	(in #ifdef notyet)
 - EM_ALPHA	(in #ifdef notyet)
 - EM_PARISC	(in #ifdef notyet)
 - EM_OR1K
 - EM_PPC64	(in #ifdef notyet)
 - EM_RISCV
 - EM_SH
 - EM_X86_64	(in #ifdef notyet)
and it causes build failure:
---
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c: In function 'get_mid':
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c:134:7: error: 'EM_OR1K' undeclared (first use in this function)
  case EM_OR1K:
       ^
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c:134:7: note: each undeclared identifier is reported only once for each function it appears in
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c:135:10: error: 'MID_OR1K' undeclared (first use in this function)
   return MID_OR1K;
          ^
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c:148:7: error: 'EM_RISCV' undeclared (first use in this function)
  case EM_RISCV:
       ^
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c:149:10: error: 'MID_RISCV' undeclared (first use in this function)
   return MID_RISCV;
          ^
---

If we need to check such new MID values, we have to sync aout_mids.h
and exec_aout.h in src/tools/arm-elf2aout/sys
and src/tools/m68k-elf2aout/sys dirs with -current ones.
But I doubt such newer CPUs actually had a.out binaries..

Furthermore, you also add a new reference of "PT_TLS":
---
+		if (ph[i].p_type == PT_TLS) {
+			if (debug)
+				warnx("Can't handle TLS section");
+			continue;
+		}
---
and it also causes an error, at least in src/tools/m68k-elf2aout:
---
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c: In function 'main':
/usr/src/tools/m68k-elf2aout/../../usr.bin/elf2aout/elf2aout.c:318:23: error: 'PT_TLS' undeclared (first use in this function)
   if (ph[i].p_type == PT_TLS) {
                       ^
---
If we need to check it, we also have to prepare host tool version.

Thanks,

---
Izumi Tsutsui


Home | Main Index | Thread Index | Old Index