[PATCH 2/2] z/OS support: Introducing z/OS support

[David Lanzendörfer]

Hi Alan

> > create mode 100644 bfd/po-bfd.h
> > create mode 100644 bfd/po64-s390.c
> > create mode 100644 include/po/common.h
> > create mode 100644 include/po/external.h
> > create mode 100644 include/po/internal.h
> > create mode 100644 ld/emulparams/po64_s390.sh
> > create mode 100644 ld/scripttempl/po64_s390.sc
> 
> These are really jarring names to anyone who is accustomed to po files
> as Gettext Portable Object files.  Please change them.  zpo maybe.
> You also have a lot of identifiers starting with po, those also would
> be better as zpo or something else.
Ok. I will change it to zpo

> > +    _bfd_dummy_target,
> > +    _bfd_dummy_target, /* TODO: bfd_po_object_p */
> > +    _bfd_dummy_target,
> > +    _bfd_dummy_target
> 
> Without an object_p function BFD can't read your object files!  So
> this is a write-only format.  You can't inspect symbols, you can't
> objcopy or objdump the files.  How do you test anything?
I know that it's a write only format for now, I'm not yet done with the
reading part in objdump/objcopy and bfd.
For now I'm just testing on the mainframe I've got access to.

Cheers
David
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: This is a digitally signed message part.
URL: <https://sourceware.org/pipermail/binutils/attachments/20200128/d229aa01/attachment.sig>