Changeset: 9561998e40fa for MonetDB URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=9561998e40fa Removed Files: sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.err.oid32 sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.out.oid32 sql/test/pg_regress/Tests/oid.stable.err.64bit.oid32 sql/test/pg_regress/Tests/oid.stable.err.Windows.oid32 sql/test/pg_regress/Tests/oid.stable.err.oid32.int128 sql/test/pg_regress/Tests/oid.stable.out.oid32 sql/test/pg_regress/Tests/without_oid.stable.out.oid32 sql/test/testdb-upgrade-chain-hge/Tests/upgrade.stable.out.oid32 sql/test/testdb-upgrade-hge/Tests/upgrade.stable.out.oid32 Modified Files: MonetDB.spec NT/monetdb_config.h.in NT/rules.msc buildtools/ChangeLog clients/Tests/All configure.ag gdk/gdk.h monetdb5/modules/mal/remote.c sql/test/emptydb/Tests/All testing/Mtest.py.in Branch: default Log Message:
Removed configure option --enable-oid32 and associated code. diffs (truncated from 10910 to 300 lines): diff --git a/MonetDB.spec b/MonetDB.spec --- a/MonetDB.spec +++ b/MonetDB.spec @@ -12,12 +12,7 @@ %define with_int128 1 %endif -# only add .oidXX suffix if oid size differs from bit size -%if %{bits} == 64 && %{?oid32:1}%{!?oid32:0} -%define oidsuf .oid32 -%endif - -%define release %{buildno}%{?dist}%{?oidsuf} +%define release %{buildno}%{?dist} # On RedHat Enterprise Linux and derivatives, if the Extra Packages # for Enterprise Linux (EPEL) repository is available, you can define @@ -905,7 +900,6 @@ developer, but if you do want to test, t --enable-monetdb5=yes \ --enable-netcdf=no \ --enable-odbc=yes \ - --enable-oid32=%{?oid32:yes}%{!?oid32:no} \ --enable-optimize=yes \ --enable-profile=no \ --enable-rintegration=%{?with_rintegration:yes}%{!?with_rintegration:no} \ diff --git a/NT/monetdb_config.h.in b/NT/monetdb_config.h.in --- a/NT/monetdb_config.h.in +++ b/NT/monetdb_config.h.in @@ -799,9 +799,6 @@ /* Release name or "unreleased" */ #define MONETDB_RELEASE "unreleased" -/* Define if the oid type should use 32 bits on a 64-bit architecture */ -/* #undef MONET_OID32 */ - /* Suffix for C++ files */ #define MX_CXX_SUFFIX "cxx" diff --git a/NT/rules.msc b/NT/rules.msc --- a/NT/rules.msc +++ b/NT/rules.msc @@ -280,19 +280,12 @@ create_winconfig_conds_new_py: !IFDEF BITS32 $(ECHO) BITS32_FALSE='#' >> "$(TOPDIR)\winconfig_conds_new.py" $(ECHO) BITS64_FALSE='' >> "$(TOPDIR)\winconfig_conds_new.py" - $(ECHO) BITS64OID32_FALSE='' >> "$(TOPDIR)\winconfig_conds_new.py" !ELSE $(ECHO) BITS32_FALSE='' >> "$(TOPDIR)\winconfig_conds_new.py" !IFDEF BITS64 $(ECHO) BITS64_FALSE='#' >> "$(TOPDIR)\winconfig_conds_new.py" -!IFDEF MONET_OID32 - $(ECHO) BITS64OID32_FALSE='#' >> "$(TOPDIR)\winconfig_conds_new.py" -!ELSE - $(ECHO) BITS64OID32_FALSE='' >> "$(TOPDIR)\winconfig_conds_new.py" -!ENDIF !ELSE $(ECHO) BITS64_FALSE='' >> "$(TOPDIR)\winconfig_conds_new.py" - $(ECHO) BITS64OID32_FALSE='' >> "$(TOPDIR)\winconfig_conds_new.py" !ENDIF !ENDIF !IFDEF CROSS_COMPILING @@ -521,16 +514,6 @@ MAKEDEBUG=DEBUG=1 PSAPILIB = psapi.lib -# When compiled with plain `nmake`, -# 64-bit OIDs are used by default on 64-bit Windows -# (see monetdb_config.h.in for details); -# compile with `nmake MONET_OID32=1` to use 32-bit OIDs on 64-bit Windows. -!IF $(bits) == 64 -!IFDEF MONET_OID32 -CFLAGS = $(CFLAGS) -DMONET_OID32 -!ENDIF -!ENDIF - CONFIG_H = monetdb_config.h {$(srcdir)}.c.obj: diff --git a/buildtools/ChangeLog b/buildtools/ChangeLog --- a/buildtools/ChangeLog +++ b/buildtools/ChangeLog @@ -1,3 +1,7 @@ # ChangeLog file for buildtools # This file is updated with Maddlog +* Mon Jul 18 2016 Sjoerd Mullender <sjo...@acm.org> +- Removed configure option --enable-oid32 to compile with 32 bit OIDs + on a 64 bit architecture. + diff --git a/clients/Tests/All b/clients/Tests/All --- a/clients/Tests/All +++ b/clients/Tests/All @@ -1,4 +1,4 @@ exports HAVE_FITS&HAVE_GEOM&HAVE_GSL&HAVE_LIBPY&HAVE_LIBR&!HAVE_LIDAR&HAVE_NETCDF&HAVE_SAMTOOLS&HAVE_SHP&HAVE_SPHINXCLIENT?MAL-signatures -!BITS32&!BITS64OID32&HAVE_FITS&HAVE_GEOM&HAVE_GSL&!HAVE_LIDAR&HAVE_NETCDF&HAVE_PCRE&HAVE_SHP&HAVE_SAMTOOLS?SQL-dump +!BITS32&HAVE_FITS&HAVE_GEOM&HAVE_GSL&!HAVE_LIDAR&HAVE_NETCDF&HAVE_PCRE&HAVE_SHP&HAVE_SAMTOOLS?SQL-dump MERCURIAL?malcheck diff --git a/configure.ag b/configure.ag --- a/configure.ag +++ b/configure.ag @@ -499,22 +499,6 @@ AC_SUBST(bits) AM_CONDITIONAL(BITS32, test x"$bits" = x32) AM_CONDITIONAL(BITS64, test x"$bits" = x64) -oids=$bits -AC_ARG_ENABLE(oid32, - AS_HELP_STRING([--enable-oid32], - [use 32 bits for OIDs on a 64-bit architecture]), - enable_oid32=$enableval, - enable_oid32=no) -AC_MSG_CHECKING(size of OIDs) -case $bits-$enable_oid32 in -64-yes) AC_DEFINE(MONET_OID32, 1, [Define if the oid type should use 32 bits on a 64-bit architecture]) - oids=32 - ;; -esac -AM_CONDITIONAL(BITS64OID32, test x"$bits-$enable_oid32" = x64-yes) -AC_SUBST(oids) -AC_MSG_RESULT($oids bits) - AC_C_BIGENDIAN() LINUX_DIST='' @@ -3804,7 +3788,7 @@ else fi echo echo "* Important options:" -echo " OID size: ${oids} bits" +echo " OID size: ${bits} bits" echo " largest integer type: ${hge_bits} bits" echo " largest decimal type: ${hge_digits} digits" echo diff --git a/gdk/gdk.h b/gdk/gdk.h --- a/gdk/gdk.h +++ b/gdk/gdk.h @@ -543,22 +543,9 @@ typedef signed char bit; typedef signed char bte; typedef short sht; -#ifdef MONET_OID32 -#define SIZEOF_OID SIZEOF_INT -typedef unsigned int oid; -#else #define SIZEOF_OID SIZEOF_SIZE_T typedef size_t oid; -#endif -#if SIZEOF_OID == SIZEOF_SIZE_T #define OIDFMT SZFMT -#else -#if SIZEOF_OID == SIZEOF_INT -#define OIDFMT "%u" -#else -#define OIDFMT ULLFMT -#endif -#endif typedef int bat; /* Index into BBP */ typedef void *ptr; /* Internal coding of types */ diff --git a/monetdb5/modules/mal/remote.c b/monetdb5/modules/mal/remote.c --- a/monetdb5/modules/mal/remote.c +++ b/monetdb5/modules/mal/remote.c @@ -411,7 +411,7 @@ str RMTprelude(void *ret) { #else type |= RMTT_32_BITS; #endif -#if SIZEOF_SIZE_T == SIZEOF_INT || defined(MONET_OID32) +#if SIZEOF_SIZE_T == SIZEOF_INT type |= RMTT_32_OIDS; #else type |= RMTT_64_OIDS; @@ -1297,7 +1297,7 @@ str RMTbintype(Client cntxt, MalBlkPtr m #else type |= RMTT_32_BITS; #endif -#if SIZEOF_SIZE_T == SIZEOF_INT || defined(MONET_OID32) +#if SIZEOF_SIZE_T == SIZEOF_INT type |= RMTT_32_OIDS; #else type |= RMTT_64_OIDS; diff --git a/sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.err.oid32 b/sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.err.oid32 deleted file mode 100644 --- a/sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.err.oid32 +++ /dev/null @@ -1,89 +0,0 @@ -stderr of test 'offset_limited_32bit.SF-2950579` in directory 'sql/test/BugTracker-2010` itself: - - -# 09:37:07 > -# 09:37:07 > mserver5 "--config=/ufs/niels/scratch/rc/Linux-x86_64/etc/monetdb5.conf" --debug=10 --set gdk_nr_threads=0 --set "monet_mod_path=/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5:/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5/lib:/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5/bin" --set "gdk_dbfarm=/ufs/niels/scratch/rc/Linux-x86_64/var/MonetDB5/dbfarm" --set mapi_open=true --set xrpc_open=true --set mapi_port=33568 --set xrpc_port=42121 --set monet_prompt= --trace "--dbname=mTests_src_test_BugTracker-2010" --set mal_listing=0 "--dbinit= include sql;" ; echo ; echo Over.. -# 09:37:07 > - -# builtin opt gdk_arch = 64bitx86_64-unknown-linux-gnu -# builtin opt gdk_version = 1.36.0 -# builtin opt prefix = /ufs/niels/scratch/rc/Linux-x86_64 -# builtin opt exec_prefix = ${prefix} -# builtin opt gdk_dbname = tst -# builtin opt gdk_dbfarm = ${prefix}/var/MonetDB -# builtin opt gdk_debug = 8 -# builtin opt gdk_alloc_map = yes -# builtin opt gdk_vmtrim = yes -# builtin opt monet_admin = adm -# builtin opt monet_prompt = > -# builtin opt monet_welcome = yes -# builtin opt monet_mod_path = ${exec_prefix}/lib/MonetDB -# builtin opt monet_daemon = yes -# builtin opt host = localhost -# builtin opt mapi_port = 50000 -# builtin opt mapi_noheaders = no -# builtin opt mapi_debug = 0 -# builtin opt mapi_clients = 2 -# builtin opt sql_debug = 0 -# builtin opt standoff_ns = http://monetdb.cwi.nl/standoff -# builtin opt standoff_start = start -# builtin opt standoff_end = end -# config opt prefix = /ufs/niels/scratch/rc/Linux-x86_64 -# config opt config = ${prefix}/etc/monetdb5.conf -# config opt prefix = /ufs/niels/scratch/rc/Linux-x86_64 -# config opt exec_prefix = ${prefix} -# config opt gdk_dbfarm = ${prefix}/var/MonetDB5/dbfarm -# config opt gdk_dbname = demo -# config opt gdk_alloc_map = no -# config opt gdk_embedded = no -# config opt gdk_debug = 0 -# config opt monet_mod_path = ${exec_prefix}/lib/MonetDB5:${exec_prefix}/lib/MonetDB5/lib:${exec_prefix}/lib/MonetDB5/bin -# config opt monet_daemon = no -# config opt monet_welcome = yes -# config opt mero_msglog = ${prefix}/var/log/MonetDB/merovingian.log -# config opt mero_errlog = ${prefix}/var/log/MonetDB/merovingian.log -# config opt mero_pidfile = ${prefix}/var/run/MonetDB/merovingian.pid -# config opt mero_controlport = 50001 -# config opt mal_init = ${exec_prefix}/lib/MonetDB5/mal_init.mal -# config opt mal_listing = 2 -# config opt mapi_port = 50000 -# config opt mapi_autosense = false -# config opt mapi_open = false -# config opt sql_optimizer = default_pipe -# config opt default_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,mitosis,mergetable,deadcode,commonTerms,joinPath,reorder,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# config opt nov2009_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# config opt replication_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,garbageCollector,dataflow,history,replication,multiplex -# config opt accumulator_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,accumulators,garbageCollector,dataflow,history,multiplex -# config opt recycler_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,deadcode,constants,commonTerms,joinPath,deadcode,recycle,reduce,garbageCollector,dataflow,history,multiplex -# config opt cracker_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,selcrack,deadcode,constants,commonTerms,joinPath,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# config opt sidcrack_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,sidcrack,deadcode,constants,commonTerms,joinPath,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# config opt octopus_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,mitosis,mergetable,deadcode,constants,commonTerms,joinPath,octopus,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# config opt datacyclotron_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,datacyclotron,mergetable,deadcode,constants,commonTerms,joinPath,reorder,deadcode,reduce,garbageCollector,dataflow,history,replication,multiplex -# config opt derive_pipe = inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,derivePath,joinPath,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# config opt dictionary_pipe = inline,remap,dictionary,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,garbageCollector,dataflow,history,multiplex -# cmdline opt config = /ufs/niels/scratch/rc/Linux-x86_64/etc/monetdb5.conf -# cmdline opt gdk_nr_threads = 0 -# cmdline opt monet_mod_path = /ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5:/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5/lib:/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5/bin -# cmdline opt gdk_dbfarm = /ufs/niels/scratch/rc/Linux-x86_64/var/MonetDB5/dbfarm -# cmdline opt mapi_open = true -# cmdline opt xrpc_open = true -# cmdline opt mapi_port = 33568 -# cmdline opt xrpc_port = 42121 -# cmdline opt monet_prompt = -# cmdline opt gdk_dbname = mTests_src_test_BugTracker-2010 -# cmdline opt mal_listing = 0 -#warning: please don't forget to set your vault key! -#(see /ufs/niels/scratch/rc/Linux-x86_64/etc/monetdb5.conf) - -# 09:37:07 > -# 09:37:07 > mclient -lsql -ftest -i -e --host=alf --port=33568 -# 09:37:07 > - -MAPI = monetdb@WIN2012:39044 -QUERY = SELECT * from tables OFFSET 2147483647; -- never finishes -ERROR = !Illegal argument - -# 09:37:07 > -# 09:37:07 > Done. -# 09:37:07 > - diff --git a/sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.out.oid32 b/sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.out.oid32 deleted file mode 100644 --- a/sql/test/BugTracker-2010/Tests/offset_limited_32bit.SF-2950579.stable.out.oid32 +++ /dev/null @@ -1,37 +0,0 @@ -stdout of test 'offset_limited_32bit.SF-2950579` in directory 'sql/test/BugTracker-2010` itself: - - -# 09:37:07 > -# 09:37:07 > mserver5 "--config=/ufs/niels/scratch/rc/Linux-x86_64/etc/monetdb5.conf" --debug=10 --set gdk_nr_threads=0 --set "monet_mod_path=/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5:/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5/lib:/ufs/niels/scratch/rc/Linux-x86_64/lib/MonetDB5/bin" --set "gdk_dbfarm=/ufs/niels/scratch/rc/Linux-x86_64/var/MonetDB5/dbfarm" --set mapi_open=true --set xrpc_open=true --set mapi_port=33568 --set xrpc_port=42121 --set monet_prompt= --trace "--dbname=mTests_src_test_BugTracker-2010" --set mal_listing=0 "--dbinit= include sql;" ; echo ; echo Over.. -# 09:37:07 > - -# MonetDB server v5.18.1, based on kernel v1.36.0 -# Serving database 'mTests_src_test_BugTracker-2010', using 4 threads -# Compiled for x86_64-unknown-linux-gnu/64bit with 64bit OIDs dynamically linked -# Found 7.754 GiB available main-memory. -# Copyright (c) 1993-July 2008 CWI. -# Copyright (c) August 2008-2015 MonetDB B.V., all rights reserved -# Visit http://monetdb.cwi.nl/ for further information -# Listening for connection requests on mapi:monetdb://alf.ins.cwi.nl:33568/ -# MonetDB/SQL module v2.36.1 loaded - -Ready. -#function user.main():void; -# clients.quit(); -#end main; _______________________________________________ checkin-list mailing list checkin-list@monetdb.org https://www.monetdb.org/mailman/listinfo/checkin-list