Hi Matthias, the change looks good. Can you please also update the copyright year before pushing (no need for a new webrev).
Thanks, Volker On Tue, Mar 20, 2018 at 2:12 PM, Baesken, Matthias <matthias.baes...@sap.com> wrote: > Hello , could you please review this small fix for Solaris x86_64 ? > > We were running into build errors for unpack200 on Solaris x86_64 > because of missing symbol "environ" . > The patch adds environ to the mapfile . > > ( seems this is a similar / same problem already solved for Solaris Sparc > with > > https://bugs.openjdk.java.net/browse/JDK-8172548 > > JDK-8172548<https://bugs.openjdk.java.net/browse/JDK-8172548> : unpack200 > fails linking with new update of SS12u4 ) > > > > Bug : > > https://bugs.openjdk.java.net/browse/JDK-8199858 > > > Webrev : > > > http://cr.openjdk.java.net/~mbaesken/webrevs/8199858/ > > > > Thanks, Matthias
