On 2018-03-01 23:17, Erik Joelsson wrote:
This is a nice fix!
I would prefer if the shell expression wasn't executed on every
re-make. In this case it can easily be fixed by changing := to =.
There is only one use of DTRACE_ELFEDIT_COMMANDS so it will only be
evaluated once, and since it's in a recipe line, it will only get
evaluated if that recipe is run. For better styling, perhaps it should
be renamed with camel humps to make it more apparent that it's a macro.
Stylewise, I would also argue that the indentation on 149 should be 4
spaces. One could say foreach qualifies as a logic indent, but I would
say this is a broken up one liner. If foreach had started on a new
line, then 2 space would have been ok, and in that case I would also
have liked the closing brace on a new line.
I had already pushed the changeset after Tim's review. I will fold your
suggested changes into the patch for JDK-8198862.
On 2018-03-01 02:01, Magnus Ihse Bursie wrote:
Solaris builds have always produced a lot of warnings when linking,
ld: fatal: symbol '__JvmOffsets' has differing types:
This is due to an unresolved bug in dtrace. This bug has been
reported on the dtrace team in 2014, but no solution have been coming
forth. :-( However, I just discovered that we can actually use
elfedit to fix the type of the fields that the linker is complaining
about on $(DTRACE_JHELPER_OBJ).
That will make the linker quiet.