Re: [OpenJDK 2D-Dev] RFR: Allow using the system libjpeg

2014-05-21 Thread Omair Majid
* Andrew Hughes [2014-05-21 20:23]: > - Original Message - > > * Andrew Hughes [2014-05-21 12:22]: > > > I'm not keen on the hardcoding of '-ljpeg' > > > > > > + LIBJPEG_LIBS := -ljpeg > > > > There's no pkg-config files for it. Any suggestions on how to get > > something generic? > >

Re: [OpenJDK 2D-Dev] RFR: Allow using the system libjpeg

2014-05-21 Thread Andrew Hughes
- Original Message - > * Andrew Hughes [2014-05-21 12:22]: > > I'm not keen on the hardcoding of '-ljpeg' > > > > + LIBJPEG_LIBS := -ljpeg > > There's no pkg-config files for it. Any suggestions on how to get > something generic? > I know there isn't at present. I still think it's bet

Re: RFR: [9] 8043340: [macosx] Fix hard-wired paths to JavaVM.framework

2014-05-21 Thread David DeHaven
> Tested on 10.8 with Xcode 5.1 and Xcode 4.6.3 and on a clean 10.9 system with > Xcode 5.1 against jdk9/hs. I will submit a JPRT sanity run shortly. I just ran into an issue in hotspot with Xcode 5.1 on 10.8 with older command line tools (from Xcode 4) installed. We'll need to use -isysroot in

Re: [OpenJDK 2D-Dev] RFR: Allow using the system libjpeg

2014-05-21 Thread Omair Majid
* Andrew Hughes [2014-05-21 12:22]: > I'm not keen on the hardcoding of '-ljpeg' > > + LIBJPEG_LIBS := -ljpeg There's no pkg-config files for it. Any suggestions on how to get something generic? > Could this not at least be conditionally assigned? (?=) It can still be overridden on the make c

Re: [OpenJDK 2D-Dev] RFR: Allow using the system libjpeg

2014-05-21 Thread Phil Race
On 05/21/2014 08:06 AM, Omair Majid wrote: I will need to push that closed generated-configure.sh afterwards however so that closed builds don't break. Anything I need to do to ensure the pushes are co-ordinated correctly? Just make sure I notice the push and I'll fix ASAP .. -phil.

Re: RFR: [9] 8043340: [macosx] Fix hard-wired paths to JavaVM.framework

2014-05-21 Thread David DeHaven
As long as XCODEBUILD=@XCODEBUILD@ doesn't result in an error on other platforms, that was my only concern there. -DrD- > Hello David, > > Mostly looks good. I don't see a reason to add ifeqs on OS around variables > in spec.gmk.in. We usually just leave them there for all platforms. The only

Re: [OpenJDK 2D-Dev] RFR: Allow using the system libjpeg

2014-05-21 Thread Andrew Hughes
- Original Message - > Hi, > > Updated webrevs: > http://cr.openjdk.java.net/~omajid/webrevs/system-libjpeg/01/ > http://cr.openjdk.java.net/~omajid/webrevs/system-libjpeg/01.jdk/ > > * Anthony Petrov [2014-05-19 14:42]: > > make/lib/Awt2dLibraries.gmk > > >1236 LIBJPEG_CFLAGS := > >

Re: [OpenJDK 2D-Dev] RFR: Allow using the system libjpeg

2014-05-21 Thread Omair Majid
Hi, * Phil Race [2014-05-20 18:34]: > I only tried the default (not specifying the system libjpeg) but my builds > all succeeded and SwingSet & Java2D demo ran fine on Mac & Windows > with the closed builds (after generating a closed generated-configure.sh) > so this all looks good to push. Than

Re: RFR: [9] 8043340: [macosx] Fix hard-wired paths to JavaVM.framework

2014-05-21 Thread Erik Joelsson
Hello David, Mostly looks good. I don't see a reason to add ifeqs on OS around variables in spec.gmk.in. We usually just leave them there for all platforms. The only difference that will make is the variable will be undefined instead of empty. Variable name as suggested by Mike seems fine to