Allow using a system-installed giflib

[Brad Wetmore]

Thanks Omair,

Not sure how much I'll be able to help here since I use much of the 
internal /java mount point.  But essentially as long as you have your 
tools in the right places (or specify them through the ALT_* variables), 
I just do:

% cd jdk/make
% make381 all
           (or images)

Some of the big ones are:

     ALT_BOOTDIR
     ALT_JAVA_DEVTOOLS_DIR
     ALT_JDK_IMPORT_PATH

IIRC, ALT_JDK_IMPORT_PATH will set the HOTSPOT/etc import variables.

Hopefully this won't be an issue much longer.  There are at least two 
groups that are stuck on the old builds due to lack of time.

HTH,

Brad


On 4/9/2013 3:11 PM, Omair Majid wrote:
> On 04/09/2013 06:01 PM, Brad Wetmore wrote:
>> Mike/Omair/Eric,
>>
>> This putback has likely broken the old JDK build (i.e. in jdk/make)
>>
>>      JDK-8011812:  JDK-8011278 breaks the old build
>
> Sorry for this.
>
>> At least deploy and JCE are still stuck on the old build.
>
> I would like to avoid this in the future; any tips on how to run the old
> build? I couldn't get it to work last I tried (the error messages were
> redirecting me to the new build).
>
>> One workaround that is said to work:
>>
>> diff --git a/make/sun/splashscreen/Makefile
>> b/make/sun/splashscreen/Makefile
>> --- a/make/sun/splashscreen/Makefile
>> +++ b/make/sun/splashscreen/Makefile
>> @@ -89,7 +89,7 @@
>>                   -I$(OPENWIN_HOME)/include/X11/extensions
>>       OTHER_LDLIBS += -L$(OPENWIN_LIB) -lX11 -lXext $(LIBM) -pthread
>>     else
>> -    CPPFLAGS += -I$(OPENWIN_HOME)/include
>> -I$(OPENWIN_HOME)/include/X11/extensions
>> +    CPPFLAGS += -I$(OPENWIN_HOME)/include
>> -I$(OPENWIN_HOME)/include/X11/extensions
>> -I$(JDK_TOPDIR)/src/share/native/sun/awt/giflib
>>       OTHER_LDLIBS += -L$(OPENWIN_LIB) -lX11 -lXext $(LIBM) -lpthread
>>     endif
>>   else # PLATFORM
>
> The change looks sensible to me since it adds the same path to
> C(PP)FLAGS that's added in the new build:
> http://cr.openjdk.java.net/~omajid/webrevs/system-giflib/02/jdk/makefiles/CompileNativeLibraries.gmk.udiff.html
>
>> Since Eric is the RE for 8011278, I'm going to assign him this one.
>
> I will be happy to test this change too.
>
> Omair
>