Re: [PATCH v5 6/8] gdbserver build-id attribute generator

Tom Tromey
2014-05-21T16:48:10Z

Jan> I wrote a never replied essay + [patch] fix about why the current handling
Jan> of 'version' is wrong:
Jan> [patch] gdbserver <library-list> and its #FIXED version="1.0"
Jan> https://sourceware.org/ml/gdb-patches/2011-11/msg00099.html

I read that note. The patch seems correct to me.

Tom> I really don't know; nor did I find anything in the documentation.
Tom> So at the very least this needs an update to the manual to warn
Tom> future users.

Jan> I do not understand this. The only changed (newly added) attribute is
Jan> build-id #IMPLIED - therefore optional - therefore backward/forward
Jan> compatible. Who needs to be warned about it?

I thought all attributes had to be specified in the DTD. Is that not
so?

If they must be then the problem comes when using an older gdb with a
newer gdbserver.

Anyway the point of my "warn" comment is that, ideally, our versioning
approach ought to be documented.

Tom