On 3/18/08, Oliver Deakin <[EMAIL PROTECTED]> wrote: > Sorry about this - there was a missing "is.windows" check for the > dt_shmem clean target. Should be fixed at r638284. >
Unfortunately, this didn't fully fix the Linux build - the makefile also had to be adjusted (I fixed it at r638711). Oliver, if you don't have access to the Linux host why you don't use integrity status page [1] to check that your updates don't break anything? The page is updated regularly and it is possible to know effect of your changes in hours. Also there is an alerts list ([EMAIL PROTECTED]). The list is used for alerts notifications for suites included into integrity testing. So if you were subscribed to the list, updated the build and get notification about broken build then that would mean that probably your changes broke the build. And the best way to avoid broken builds is to check changes before committing on both platforms :-) [1] http://people.apache.org/~varlax/harmony-integrity/ Thanks, Stepan. > Regards, > Oliver > > Stepan Mishura wrote: > > Hi Oliver, > > > > Your commit broke Linux build. Could you fix it or roll it back? > > > > The build fails with: > > > > [java] -clean-native-windows: > > [java] [echo] Making "clean" in > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/modules/jpda/src/main/native/jdwp/unix/transport/dt_shmem > > [java] [echo] LOCAL MAKE DEFN L > > /export/users/cc/cc-common/build/checkouts/hdk/working_vm/deploy > > [java] > > [java] BUILD FAILED > > [java] /export/users/cc/cc-common/build/checkouts/hdk/build.xml:444: > > The following error occurred while executing this line: > > [java] > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/build.xml:200: > > The following error occurred while executing this line: > > [java] > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/make/build-native.xml:69: > > The following error occurred while executing this line: > > [java] > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/make/properties.xml:336: > > The following error occurred while executing this line: > > [java] > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/modules/jpda/build.xml:185: > > The following error occurred while executing this line: > > [java] > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/modules/jpda/build.xml:189: > > The following error occurred while executing this line: > > [java] > > /export/users/cc/cc-common/build/checkouts/hdk/working_jdktools/make/properties.xml:354: > > The directory you specified does not exist > > > > Thanks, > > Stepan. > > > > -----Original Message----- > > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > > Sent: Monday, March 17, 2008 9:18 PM > > To: [EMAIL PROTECTED] > > Subject: svn commit: r637937 - in > > /harmony/enhanced/jdktools/trunk/modules/jpda: ./ > > src/main/native/jdwp/common/transport/common/ > > src/main/native/jdwp/unix/transport/ > > src/main/native/jdwp/unix/transport/dt_socket/ > > src/main/native/jdwp/windows/transport/ src/mai... > > > > Author: odeakin > > Date: Mon Mar 17 08:17:35 2008 > > New Revision: 637937 > > > > URL: http://svn.apache.org/viewvc?rev=637937&view=rev > > Log: > > Some changes to start implementation of JDWP shared memory transport: > > - Reorganisation of socket transport makefiles into their own > > subdirectories. > > - Initial check-in of shared memory transport code. > > - Alteration to build.xml to start building dt_shmem.dll on Windows. > > > > Added: > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/unix/transport/dt_socket/exports.txt > > - copied unchanged from r636747, > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/unix/transport/exports.txt > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/unix/transport/dt_socket/makefile > > - copied, changed from r636747, > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/unix/transport/makefile > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/dt_shmem/ > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/dt_shmem/SharedMemTransport.cpp > > (with props) > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/dt_shmem/SharedMemTransport.h > > (with props) > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/dt_shmem/dt_shmem.def > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/dt_shmem/makefile > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/dt_socket/makefile > > - copied, changed from r636747, > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/makefile > > Removed: > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/unix/transport/exports.txt > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/unix/transport/makefile > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/windows/transport/makefile > > Modified: > > harmony/enhanced/jdktools/trunk/modules/jpda/build.xml > > > > harmony/enhanced/jdktools/trunk/modules/jpda/src/main/native/jdwp/common/transport/common/LastTransportError.h > > > > Modified: harmony/enhanced/jdktools/trunk/modules/jpda/build.xml > > URL: > > http://svn.apache.org/viewvc/harmony/enhanced/jdktools/trunk/modules/jpda/build.xml?rev=637937&r1=637936&r2=637937&view=diff > > ============================================================================== > > > > <SNIP> > > > > > > -- > Oliver Deakin > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU > >
