bump packaging versions for 2016.10 final

Revert "[bugfix] since HAVE_DUNE_ISTL is used in opm-grid it should b…

.travis.yml

@@ -17,7 +17,6 @@ addons:
       - libgmp3-dev
       - libsuitesparse-dev
       - libeigen3-dev
-      - bc
 
 before_script:
     - export CXX="g++-4.8" CC="gcc-4.8" FC="gfortran-4.8"

CMakeLists.txt

@@ -12,7 +12,7 @@ endmacro (dir_hook)
 set(OPM_PROJECT_EXTRA_CODE_INSTALLED  "set(OPM_MACROS_ROOT ${CMAKE_INSTALL_PREFIX}/share/opm)
                                        list(APPEND CMAKE_MODULE_PATH \${OPM_MACROS_ROOT}/cmake/Modules)")
 
-set(OPM_PROJECT_EXTRA_CODE_INTREE  "set(OPM_MACROS_ROOT ${OPM_MACROS_ROOT})
+set(OPM_PROJECT_EXTRA_CODE_INTREE  "set(OPM_MACROS_ROOT ${OPM_COMMON_ROOT})
                                     list(APPEND CMAKE_MODULE_PATH \${OPM_MACROS_ROOT}/cmake/Modules)")
 
 # project information is in dune.module. Read this file and set variables.

CMakeLists_files.cmake

@@ -54,9 +54,6 @@ list (APPEND PROGRAM_SOURCE_FILES
 list( APPEND PUBLIC_HEADER_FILES
       opm/common/ErrorMacros.hpp
       opm/common/Exceptions.hpp
-      opm/common/ResetLocale.hpp
-      opm/common/Unused.hpp
-      opm/common/Valgrind.hpp
       opm/common/data/SimulationDataContainer.hpp
       opm/common/OpmLog/CounterLog.hpp
       opm/common/OpmLog/EclipsePRTLog.hpp

cmake/Modules/FindCXX11Features.cmake

@@ -33,34 +33,23 @@ include(CheckIncludeFileCXX)
 # macro to only add option once
 include(AddOptions)
 
-# Force CXX Standard cross platfrom for CMakeVersion >=3.1
-set(CMAKE_CXX_EXTENSIONS OFF)
-
-if(NOT CMAKE_CXX_STANDARD)
-  set(CMAKE_CXX_STANDARD 11)
-
-
-  if(NOT MSVC)
-    foreach(_flag "14" "11")
-      set(_FLAG "CXX_FLAG_CXX${_flag}")
-      string(TOUPPER ${_FLAG} _FLAG)
-      # try to use compiler flag -std=c++${_flag}
-      CHECK_CXX_ACCEPTS_FLAG("-std=c++${_flag}" ${_FLAG})
-
-      if(${_FLAG})
-        if(CMAKE_VERSION VERSION_LESS 3.1)
-          add_options (CXX ALL_BUILDS "-std=c++${_flag}")
-        endif()
-        set(CXX_STD0X_FLAGS "-std=c++${_flag}")
-        #Use vriables for CMake > 3.1 set the standard with no extensions
-        set(CMAKE_CXX_STANDARD ${_flag})
-        break()
-      endif()
-    endforeach()
-  endif(NOT MSVC)
-endif()
-
 if(CMAKE_VERSION VERSION_LESS 3.1)
+  if(NOT MSVC)
+    # try to use compiler flag -std=c++11
+    CHECK_CXX_ACCEPTS_FLAG("-std=c++11" CXX_FLAG_CXX11)
+    if(CXX_FLAG_CXX11)
+      add_options (CXX ALL_BUILDS "-std=c++11")
+      set(CXX_STD0X_FLAGS "-std=c++11")
+    else()
+      # try to use compiler flag -std=c++0x for older compilers
+      CHECK_CXX_ACCEPTS_FLAG("-std=c++0x" CXX_FLAG_CXX0X)
+      if(CXX_FLAG_CXX0X)
+        add_options (CXX ALL_BUILDS "-std=c++0x")
+        set(CXX_STD0X_FLAGS "-std=c++0x")
+      endif(CXX_FLAG_CXX0X)
+    endif(CXX_FLAG_CXX11)
+  endif(NOT MSVC)
+
   # if we are building with an Apple toolchain in MacOS X,
   # we cannot use the old GCC 4.2 fork, but must use the
   # new runtime library
@@ -69,8 +58,8 @@ if(CMAKE_VERSION VERSION_LESS 3.1)
   if (APPLE AND (_comp_id MATCHES "CLANG"))
     CHECK_CXX_ACCEPTS_FLAG ("-stdlib=libc++" CXX_FLAG_STDLIB_LIBCXX)
     if (CXX_FLAG_STDLIB_LIBCXX)
-      add_options (CXX ALL_BUILDS "-stdlib=libc++")
+          add_options (CXX ALL_BUILDS "-stdlib=libc++")
-      set (CXX_STDLIB_FLAGS "-stdlib=libc++")
+          set (CXX_STDLIB_FLAGS "-stdlib=libc++")
     endif (CXX_FLAG_STDLIB_LIBCXX)
   endif (APPLE AND (_comp_id MATCHES "CLANG"))
 
@@ -80,9 +69,13 @@ if(CMAKE_VERSION VERSION_LESS 3.1)
   else (CXX_STD0X_FLAGS AND CXX_STDLIB_FLAGS)
     set (CXX_SPACE)
   endif (CXX_STD0X_FLAGS AND CXX_STDLIB_FLAGS)
-endif()
+else()
+  if(NOT CMAKE_CXX_STANDARD)
+    set(CMAKE_CXX_STANDARD 11)
+  endif()
+  set(CMAKE_CXX_STANDARD_REQUIRED ON)
+  set(CMAKE_CXX_EXTENSIONS OFF)
 
-if(NOT CMAKE_VERSION VERSION_LESS 3.1)
   # Workaround bug in cmake:
   # cxx standard flags are not applied in CheckCXXSourceCompiles
   if (CMAKE_CXX_STANDARD EQUAL 14)
@@ -92,7 +85,6 @@ if(NOT CMAKE_VERSION VERSION_LESS 3.1)
   endif()
 endif()
 
-
 # perform tests
 include(CheckCXXSourceCompiles)
 

cmake/Modules/FindERT.cmake

@@ -242,10 +242,9 @@ if (NOT (ERT_INCLUDE_DIR MATCHES "-NOTFOUND" OR ERT_LIBRARIES MATCHES "-NOTFOUND
   set (CMAKE_REQUIRED_INCLUDES ${ERT_INCLUDE_DIR})
   set (CMAKE_REQUIRED_LIBRARIES ${ERT_LIBRARIES})
   check_cxx_source_compiles (
-"#include <ert/ecl/ecl_grid.h>
+"#include <ert/ecl/EclKW.hpp>
 int main ( ) {
-    ecl_grid_type * grid = ecl_grid_alloc_rectangular( 10,10,10,1,1,1, NULL);
+    ERT::EclKW< int > kw( ecl_kw_alloc( \"SATNUM\", 0, ECL_INT_TYPE ) );
-    ecl_grid_free( grid );
     return 0;
 }" HAVE_ERT)
   cmake_pop_check_state ()

cmake/Modules/Findcjson.cmake

@@ -20,8 +20,8 @@ endif (CJSON_ROOT)
 find_path (CJSON_INCLUDE_DIR
   NAMES "cjson/cJSON.h"
   HINTS "${CJSON_ROOT}"
-  PATHS "${PROJECT_SOURCE_DIR}"
+  PATHS "${PROJECT_SOURCE_DIR}" "${PROJECT_SOURCE_DIR}/../opm-parser"
-  PATH_SUFFIXES "include" "external"
+  PATH_SUFFIXES "include" "opm/json"
   DOC "Path to cjson library header files"
   ${_no_default_path} )
 

cmake/Modules/Finddune-grid.cmake

@@ -41,7 +41,7 @@ find_opm_package (
 "#include <dune/grid/onedgrid.hh>
 int main (void) {
   Dune::OneDGrid grid(1, 0., 1.);
-  return grid.size(0);
+  return grid.lbegin<0>(0) == grid.lend<0>(0);
 }
 "
   # config variables

cmake/Modules/Findopm-core.cmake

@@ -21,7 +21,7 @@ find_opm_package (
   "${opm-core_DEPS}"
   
   # header to search for
-  "opm/core/wells.h"
+  "opm/core/grid.h"
 
   # library to search for
   "opmcore"
@@ -30,13 +30,12 @@ find_opm_package (
   ""
 
   # test program
-"
+"#include <opm/core/grid.h>
-#include <opm/core/utility/parameters/ParameterGroup.hpp>
+int main (void) {
-int main()
+  struct UnstructuredGrid *g;
-{
+  g = create_grid_empty ();
-    Opm::parameter::ParameterGroup parameters;
+  destroy_grid (g);
-    parameters.insertParameter(\"number\", \"7\");
+  return 0;  
-    return 0;
 }
 "
   # config variables

cmake/Modules/Findopm-data.cmake

@@ -17,8 +17,8 @@ endif()
 if (EXISTS "${_opm_data_root}/norne/NORNE_ATW2013.DATA")
    set( HAVE_OPM_DATA True )
    set( OPM_DATA_ROOT ${_opm_data_root} )
-   message(STATUS "Setting OPM_DATA_ROOT: ${OPM_DATA_ROOT}")
+   message( "-- Setting OPM_DATA_ROOT: ${OPM_DATA_ROOT}")
 else()
    set( HAVE_OPM_DATA False )
-   message(WARNING "opm-data not found - integration tests using opm-data will be skipped.")
+   message( "opm-data not found - integration tests using opm-data will be skipped.")
-endif()
+endif()

cmake/Modules/Findopm-parser.cmake

@@ -137,21 +137,6 @@ if (ERT_FOUND AND Boost_FOUND AND
     ${Boost_LIBRARIES}
     ${ERT_LIBRARIES})
 
-  # We might be using an external cJSON library
-  # but we have to unset the OPM_PARSER_ROOT stuff to find it
-  # (other NO_DEFAULT_PATH will be set).
-  set(_OPM_PARSER_ROOT_bak ${OPM_PARSER_ROOT})
-  set(OPM_PARSER_ROOT "")
-  find_package(cjson)
-  set(OPM_PARSER_ROOT ${OPM_PARSER_ROOT_bak})
-
-  if (CJSON_FOUND)
-    # If we do we need to add it to the libs.
-    set (opm-parser_LIBRARIES
-      ${opm-parser_LIBRARIES}
-      ${CJSON_LIBRARY})
-  endif (CJSON_FOUND)
-
   # see if we can compile a minimum example
   # CMake logical test doesn't handle lists (sic)
   include (CMakePushCheckState)

cmake/Modules/OpmCompile.cmake

@@ -13,19 +13,11 @@ macro (opm_compile opm)
   # all public header files are together with the source. prepend our own
   # source path to the one of the dependencies so that our version of any
   # ambigious paths are used.
-
+  set (${opm}_INCLUDE_DIR "${PROJECT_SOURCE_DIR}")
-  option(SILENCE_CROSSMODULE_WARNINGS "Disable warnings from cross-module includes" OFF)
+  set (${opm}_INCLUDE_DIRS ${${opm}_INCLUDE_DIR} ${${opm}_INCLUDE_DIRS})
-  if (SILENCE_CROSSMODULE_WARNINGS)
-     include_directories("${PROJECT_SOURCE_DIR}")
-     include_directories (SYSTEM ${${opm}_INCLUDE_DIRS})
-  else()
-     set (${opm}_INCLUDE_DIR "${PROJECT_SOURCE_DIR}")
-     set (${opm}_INCLUDE_DIRS ${${opm}_INCLUDE_DIR} ${${opm}_INCLUDE_DIRS})
-     include_directories (${${opm}_INCLUDE_DIRS})
-  endif()
-
 
   # create this library, if there are any compilation units
+  include_directories (${${opm}_INCLUDE_DIRS})
   link_directories (${${opm}_LIBRARY_DIRS})
   add_definitions (${${opm}_DEFINITIONS})
   set (${opm}_VERSION "${${opm}_VERSION_MAJOR}.${${opm}_VERSION_MINOR}")

cmake/Modules/OpmLibMain.cmake

@@ -285,6 +285,10 @@ opm_doc (${project} ${doxy_dir})
 
 # provide compatibility with using this build in dunecontrol
 include (DuneCompat)
+include (LibtoolArchives)
+if (${project}_TARGET)
+	configure_la (${project} ${${project}_TARGET} ${project}_LIBTOOL_ARCHIVE)
+endif ()
 
 ### clean in-source builds ###
 include (OpmDistClean)

cmake/Modules/ewoms-prereqs.cmake

@@ -35,7 +35,6 @@ set (ewoms_DEPS
 	"dune-fem"
 	"opm-parser"
 	"opm-grid"
-	"opm-core"
 	# librt (on some systems necessary for clock_gettime())
 	"librt REQUIRED"
 	# valgrind client requests

cmake/Modules/opm-common-prereqs.cmake

@@ -3,7 +3,7 @@
 
 # defines that must be present in config.h for our headers
 set (opm-common_CONFIG_VAR
-	"HAS_ATTRIBUTE_UNUSED")
+	)
 
 # dependencies
 set (opm-common_DEPS

cmake/Modules/opm-core-prereqs.cmake

@@ -8,9 +8,6 @@ set (opm-core_CONFIG_VAR
 	HAVE_DUNE_ISTL
 	HAVE_MPI
 	HAVE_PETSC
-        DUNE_ISTL_VERSION_MAJOR
-        DUNE_ISTL_VERSION_MINOR
-        DUNE_ISTL_VERSION_REVISION
 	)
 
 # dependencies
@@ -45,8 +42,5 @@ set (opm-core_DEPS
 	"opm-parser REQUIRED"
 	# the code which implements the material laws
 	"opm-material REQUIRED"
-	# the code which implements the output routines
 	"opm-output REQUIRED"
-	# the code which implements grids
-	"opm-grid REQUIRED"
 	)

cmake/Modules/opm-flowdiagnostics-applications-prereqs.cmake

@@ -12,7 +12,7 @@ set (opm-flowdiagnostics-applications_DEPS
 	# compile with C++0x/11 support if available
 	"CXX11Features REQUIRED"
 	"Boost 1.44.0
-		COMPONENTS filesystem regex system unit_test_framework REQUIRED"
+		COMPONENTS filesystem system unit_test_framework REQUIRED"
 	"ERT REQUIRED"
 	# prerequisite OPM modules
 	"opm-common REQUIRED;

cmake/Modules/opm-grid-prereqs.cmake

@@ -9,8 +9,6 @@ set (opm-grid_CONFIG_VAR
 	DUNE_COMMON_VERSION_MAJOR
 	DUNE_COMMON_VERSION_MINOR
 	DUNE_COMMON_VERSION_REVISION
-	HAVE_DUNE_ISTL
-	HAVE_MPI
 	HAVE_ZOLTAN
 	)
 
@@ -23,11 +21,12 @@ set (opm-grid_DEPS
 	# various runtime library enhancements
 	"Boost 1.44.0
 		COMPONENTS date_time filesystem system unit_test_framework REQUIRED"
-	"MPI"
+	# DUNE dependency
-	"dune-common"
+	"dune-common REQUIRED;
-	"dune-grid REQUIRED"
+	dune-grid REQUIRED;
-	"dune-istl"
+	dune-geometry REQUIRED"
-	"opm-common REQUIRED"
+	# OPM dependency
-	"opm-parser REQUIRED"
+        "opm-common REQUIRED;
+        opm-core REQUIRED"
 	"ZOLTAN"
 	)

cmake/Modules/opm-material-prereqs.cmake

@@ -20,6 +20,4 @@ set (opm-material_DEPS
 	"opm-common REQUIRED"
 	# DUNE dependency
 	"dune-common REQUIRED"
-	# valgrind client requests
-	"Valgrind"
 	)

cmake/Modules/opm-simulators-prereqs.cmake

@@ -5,10 +5,6 @@
 set (opm-simulators_CONFIG_VAR
 	HAVE_OPM_GRID
 	HAVE_PTHREAD
-	HAVE_EWOMS
-        DUNE_ISTL_VERSION_MAJOR
-        DUNE_ISTL_VERSION_MINOR
-        DUNE_ISTL_VERSION_REVISION
 	)
 
 # dependencies
@@ -24,13 +20,12 @@ set (opm-simulators_DEPS
 	"dune-common REQUIRED;
 	dune-istl REQUIRED"
 	"ERTPython"
-  # OPM dependency
+        # OPM dependency
 	"opm-common REQUIRED;
-   opm-parser REQUIRED;
+        opm-parser REQUIRED;
-	 opm-grid REQUIRED;
+	opm-core REQUIRED;
-	 opm-core REQUIRED;
+	opm-output REQUIRED;
-	 opm-output REQUIRED;
+	opm-grid"
-	 ewoms REQUIRED"
 	# Eigen
 	"Eigen3 3.2.0"
 	)

cmake/Modules/opm-upscaling-prereqs.cmake

@@ -26,7 +26,7 @@ set (opm-upscaling_DEPS
 	dune-geometry REQUIRED;
 	dune-grid REQUIRED;
 	opm-common REQUIRED;
-  opm-grid REQUIRED;
 	opm-core REQUIRED;
+        opm-grid REQUIRED;
 	opm-output REQUIRED"
 	)

debian/changelog

@@ -1,4 +1,10 @@
-opm-common (2017.04-rfinal-1~xenial) xenial; urgency=medium
+opm-common (2016.10-rfinal-1~xenial) xenial; urgency=medium
+
+  * New release
+
+ -- Arne Morten Kvarving <arne.morten.kvarving@sintef.no>  Wed, 26 Oct 2016 09:55:11 +0200
+
+opm-common (2015.10-1~trusty) trusty; urgency=medium
 
   * New release
 

dune.module

@@ -5,8 +5,8 @@
 
 Module: opm-common
 Description: Open Porous Media Initiative shared infrastructure
-Version: 2017.04
+Version: 2016.10
-Label: 2017.04
+Label: 2016.10
 Maintainer: opm@opm-project.org
 MaintainerName: OPM community
 Url: http://opm-project.org

jenkins/build-opm-module.sh

@@ -83,12 +83,7 @@ function build_module {
   test $? -eq 0 || exit 1
   if test $2 -eq 1
   then
-    if [ ! -z $BUILDTHREADS ]
+    cmake --build .
-    then
-      cmake --build . -- -j$BUILDTHREADS
-    else
-      cmake --build .
-    fi
     test $? -eq 0 || exit 2
     ctest -T Test --no-compress-output
 
@@ -101,12 +96,7 @@ function build_module {
       sed -e "s/classname=\"TestSuite\"/classname=\"${configuration}\"/g" testoutput.xml > $WORKSPACE/$configuration/testoutput.xml
     fi
   else
-    if [ ! -z $BUILDTHREADS ]
+    cmake --build . --target install
-    then
-      cmake --build . --target install -- -j$BUILDTHREADS
-    else
-      cmake --build . --target install
-    fi
   fi
 }
 
@@ -183,13 +173,7 @@ function build_downstreams {
     # Installation for downstream
     pushd .
     cd $WORKSPACE/$configuration/build-$downstream
-
+    cmake --build . --target install
-    if [ ! -z $BUILDTHREADS ]
-    then
-      cmake --build . --target install -- -j$BUILDTHREADS
-    else
-      cmake --build . --target install
-    fi
     popd
     egrep_cmd="$egrep_cmd $WORKSPACE/$configuration/build-$downstream/testoutput.xml"
   done

jenkins/build.sh

@@ -12,12 +12,11 @@ downstreams=(ert
              opm-parser
              opm-output
              opm-material
-             opm-grid
              opm-core
-             ewoms
+             opm-grid
              opm-simulators
              opm-upscaling
-             )
+             ewoms)
 
 declare -A downstreamRev
 downstreamRev[ert]=master
@@ -26,9 +25,9 @@ downstreamRev[opm-material]=master
 downstreamRev[opm-core]=master
 downstreamRev[opm-grid]=master
 downstreamRev[opm-output]=master
-downstreamRev[ewoms]=master
 downstreamRev[opm-simulators]=master
 downstreamRev[opm-upscaling]=master
+downstreamRev[ewoms]=master
 
 parseRevisions
 printHeader opm-common

jenkins/update-opm-data.sh

@@ -1,58 +0,0 @@
-#!/bin/bash
-
-MAIN_REPO=$1 # The repo the update was triggered from
-
-source $WORKSPACE/deps/opm-common/jenkins/build-opm-module.sh
-
-declare -a upstreams # Everything is considered an upstream to aid code reuse
-upstreams=(ert
-           opm-common
-           opm-parser
-           opm-output
-           opm-material
-           opm-grid
-           opm-core
-           ewoms
-           opm-simulators
-           opm-upscaling
-          )
-
-declare -A upstreamRev
-upstreamRev[ert]=master
-upstreamRev[opm-common]=master
-upstreamRev[opm-parser]=master
-upstreamRev[opm-material]=master
-upstreamRev[opm-core]=master
-upstreamRev[opm-grid]=master
-upstreamRev[opm-output]=master
-upstreamRev[ewoms]=master
-upstreamRev[opm-simulators]=master
-upstreamRev[opm-upscaling]=master
-
-# Setup revision tables
-parseRevisions
-upstreamRev[$MAIN_REPO]=$sha1
-
-# Create branch name
-BRANCH_NAME="update"
-for repo in ${upstreams[*]}
-do
-  if [ "${upstreamRev[$repo]}" != "master" ]
-  then
-    rev=${upstreamRev[$repo]}
-    prnumber=${rev//[!0-9]/}
-    BRANCH_NAME="${BRANCH_NAME}_${repo}_$prnumber"
-    test -n "$REASON" && REASON+="        "
-    REASON+="https://github.com/OPM/$repo/pull/$prnumber\n"
-  fi
-done
-
-# Do the commit
-export REASON
-export BRANCH_NAME
-$WORKSPACE/deps/opm-simulators/tests/update_reference_data.sh $OPM_DATA_ROOT
-
-# Finally open the pull request
-cd $OPM_DATA_ROOT
-git remote add jenkins4opm git@github.com:jenkins4opm/opm-data
-git-open-pull -u jenkins4opm --base-account OPM --base-repo opm-data -r /tmp/cmsg $BRANCH_NAME

opm/common/OpmLog/LogUtil.cpp

@@ -85,9 +85,8 @@ namespace Log {
             return message; // No color coding, not even the code for default color.
         case MessageType::Warning:
             return AnsiTerminalColors::blue_strong + message + AnsiTerminalColors::none;
+        case MessageType::Error:
         case MessageType::Problem:
-            return AnsiTerminalColors::magenta_strong + message + AnsiTerminalColors::none;  
-        case MessageType::Error:  
         case MessageType::Bug:
             return AnsiTerminalColors::red_strong + message + AnsiTerminalColors::none;
         default:

opm/common/OpmLog/LogUtil.hpp

@@ -49,8 +49,6 @@ namespace Log {
         const std::string yellow_strong = "\033[33;1m";
         const std::string blue = "\033[34m";
         const std::string blue_strong = "\033[34;1m";
-        const std::string magenta = "\033[35m";
-        const std::string magenta_strong = "\033[35;1m";
         const std::string default_color = "\033[39m";
     }
 

opm/common/OpmLog/OpmLog.cpp

@@ -21,27 +21,9 @@
 #include <opm/common/OpmLog/Logger.hpp>
 #include <opm/common/OpmLog/StreamLog.hpp>
 #include <iostream>
-#include <errno.h>  // For errno
-#include <stdio.h>  // For fileno() and stdout
-#include <unistd.h> // For isatty()
 
 namespace Opm {
 
-    namespace {
-        bool stdoutIsTerminal()
-        {
-            const int errno_save = errno; // For playing nice with C error handling.
-            const int file_descriptor = fileno(stdout);
-            if (file_descriptor == -1) {
-                // stdout is an invalid stream
-                errno = errno_save;
-                return false;
-            } else {
-                return isatty(file_descriptor);
-            }
-        }
-    }
-
 
     std::shared_ptr<Logger> OpmLog::getLogger() {
         if (!m_logger)
@@ -194,14 +176,12 @@ namespace Opm {
 
 
 
-    void OpmLog::setupSimpleDefaultLogging(const bool use_prefix,
+    void OpmLog::setupSimpleDefaultLogging(const bool use_prefix)
-                                           const bool use_color_coding,
-                                           const int message_limit)
     {
          std::shared_ptr<StreamLog> streamLog = std::make_shared<StreamLog>(std::cout, Log::DefaultMessageTypes);
          OpmLog::addBackend( "SimpleDefaultLog", streamLog);
-         streamLog->setMessageLimiter(std::make_shared<MessageLimiter>(message_limit));
+         streamLog->setMessageLimiter(std::make_shared<MessageLimiter>(10));
-         streamLog->setMessageFormatter(std::make_shared<SimpleMessageFormatter>(use_prefix, use_color_coding && stdoutIsTerminal()));
+         streamLog->setMessageFormatter(std::make_shared<SimpleMessageFormatter>(use_prefix, true));
     }
 /******************************************************************/
 

opm/common/OpmLog/OpmLog.hpp

@@ -64,16 +64,7 @@ public:
     static void removeAllBackends();
     static bool enabledMessageType( int64_t messageType );
     static void addMessageType( int64_t messageType , const std::string& prefix);
-
+    static void setupSimpleDefaultLogging(const bool use_prefix);
-    /// Create a basic logging setup that will send all log messages to standard output.
-    ///
-    /// By default category prefixes will be printed (i.e. Error: or
-    /// Warning:), color coding will be used, and a maximum of 10
-    /// messages with the same tag will be printed. These settings can
-    /// be controlled by the function parameters.
-    static void setupSimpleDefaultLogging(const bool use_prefix = true,
-                                          const bool use_color_coding = true,
-                                          const int message_limit = 10);
 
     template <class BackendType>
     static std::shared_ptr<BackendType> getBackend(const std::string& name) {

opm/common/ResetLocale.hpp

@@ -1,62 +0,0 @@
-// -*- mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
-// vi: set et ts=4 sw=4 sts=4:
-/*
-  This file is part of the Open Porous Media project (OPM).
-
-  OPM is free software: you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation, either version 3 of the License, or
-  (at your option) any later version.
-
-  OPM is distributed in the hope that it will be useful,
-  but WITHOUT ANY WARRANTY; without even the implied warranty of
-  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-  GNU General Public License for more details.
-
-  You should have received a copy of the GNU General Public License
-  along with OPM.  If not, see <http://www.gnu.org/licenses/>.
-*/
-/*!
- * \file
- * \brief Provides a free function to reset the localization settings
- *
- * Under some circumstances, some environments seem to set a locale which they do not
- * install. In turn this leads to std::runtime_errror being thrown by some parts of Boost
- * (for some versions) which causes unsolicited program aborts.
- *
- * This issue asside, it looks pretty weird if the e.g. the number format is inconsistent
- * with the language used by rest of the simulation.
- */
-#ifndef OPM_RESET_LOCALE_HH
-#define OPM_RESET_LOCALE_HH
-
-#include <stdlib.h>
-
-namespace Opm {
-
-inline void resetLocale()
-{
-#ifndef WIN32
-    // this probably only works for POSIX compatible operating systems. for all others,
-    // unsetting a few environment variables should not hurt, though.
-    unsetenv("LC_ALL");
-    unsetenv("LANG");
-    unsetenv("LANGUAGE");
-    unsetenv("LC_ADDRESS");
-    unsetenv("LC_COLLATE");
-    unsetenv("LC_CTYPE");
-    unsetenv("LC_IDENTIFICATION");
-    unsetenv("LC_MEASUREMENT");
-    unsetenv("LC_MESSAGES");
-    unsetenv("LC_MONETARY");
-    unsetenv("LC_NAME");
-    unsetenv("LC_NUMERIC");
-    unsetenv("LC_PAPER");
-    unsetenv("LC_TELEPHONE");
-    unsetenv("LC_TIME");
-#endif // !WIN32
-}
-
-} // namespace Opm
-
-#endif

opm/common/Unused.hpp

@@ -1,43 +0,0 @@
-// -*- mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
-// vi: set et ts=4 sw=4 sts=4:
-/*
-  This file is part of the Open Porous Media project (OPM).
-
-  OPM is free software: you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation, either version 3 of the License, or
-  (at your option) any later version.
-
-  OPM is distributed in the hope that it will be useful,
-  but WITHOUT ANY WARRANTY; without even the implied warranty of
-  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-  GNU General Public License for more details.
-
-  You should have received a copy of the GNU General Public License
-  along with OPM.  If not, see <http://www.gnu.org/licenses/>.
-*/
-/*!
- * \file
- * \brief Provides the OPM_UNUSED macro
- *
- * This macro can be used to mark variables as "potentially unused" which suppresses some
- * bogus compiler warnings. If the compiler does not support this, the macro is a no-op.
- */
-#ifndef OPM_UNUSED_HH
-#define OPM_UNUSED_HH
-
-#ifndef HAS_ATTRIBUTE_UNUSED
-#define OPM_UNUSED
-#else
-#define OPM_UNUSED __attribute__((unused))
-#endif
-
-#ifdef NDEBUG
-#define OPM_DEBUG_UNUSED
-#define OPM_OPTIM_UNUSED OPM_UNUSED
-#else
-#define OPM_DEBUG_UNUSED OPM_UNUSED
-#define OPM_OPTIM_UNUSED
-#endif
-
-#endif

opm/common/Valgrind.hpp

@@ -1,311 +0,0 @@
-// -*- mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
-// vi: set et ts=4 sw=4 sts=4:
-/*
-  This file is part of the Open Porous Media project (OPM).
-
-  OPM is free software: you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation, either version 2 of the License, or
-  (at your option) any later version.
-
-  OPM is distributed in the hope that it will be useful,
-  but WITHOUT ANY WARRANTY; without even the implied warranty of
-  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-  GNU General Public License for more details.
-
-  You should have received a copy of the GNU General Public License
-  along with OPM.  If not, see <http://www.gnu.org/licenses/>.
-
-  Consult the COPYING file in the top-level source directory of this
-  module for the precise wording of the license and the list of
-  copyright holders.
-*/
-/*!
- * \file
- * \brief Some templates to wrap the valgrind client request macros
- */
-#ifndef OPM_VALGRIND_HPP
-#define OPM_VALGRIND_HPP
-
-#include <opm/common/Unused.hpp>
-
-#if HAVE_VALGRIND
-#include <valgrind/memcheck.h>
-#endif
-
-#if HAVE_VALGRIND
-#define OPM_VALGRIND_OPTIM_UNUSED OPM_OPTIM_UNUSED
-#else
-#define OPM_VALGRIND_OPTIM_UNUSED OPM_UNUSED
-#endif
-
-namespace Opm {
-namespace Valgrind {
-/*!
- * \ingroup Valgrind
- * \brief Returns whether the program is running under Valgrind or not.
- */
-inline bool IsRunning()
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    return RUNNING_ON_VALGRIND;
-#else
-    return false;
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make valgrind complain if any of the memory occupied by an object
- *        is undefined.
- *
- * Please note that this does not check whether the destinations of an
- * object's pointers or references are defined. Also, for performance
- * reasons the compiler might insert "padding bytes" between within
- * the objects which leads to false positives.
- *
- * Example:
- *
- * \code
- * int i;
- * Valgrind::CheckDefined(i); // Valgrind complains!
- * \endcode
- *
- * \tparam T The type of the object which ought to be checked
- *
- * \param value the object which valgrind should check
- *
- * \return true iff there are no undefined bytes in the memory
- *         occupied by the object.
- */
-template <class T>
-inline bool CheckDefined(const T& value OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    auto tmp = VALGRIND_CHECK_MEM_IS_DEFINED(&value, sizeof(T));
-    return tmp == 0;
-#else
-    return true;
-#endif
-}
-
-
-
-/*!
- * \ingroup Valgrind
- * \brief Make valgrind complain if any of the memory occupied by an object
- *        is not addressable.
- *
- * Example:
- *
- * \code
- * int* i = NULL;
- * Valgrind::CheckAddressable(*i); // Valgrind complains!
- * \endcode
- *
- * \tparam T The type of the object which ought to be checked
- *
- * \param value the object which valgrind should check
- *
- * \return true iff there are no unadressable bytes in the memory
- *         occupied by the object.
- */
-template <class T>
-inline bool CheckAddressable(const T& value OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    auto tmp = VALGRIND_CHECK_MEM_IS_ADDRESSABLE(&value, sizeof(T));
-    return tmp == 0;
-#else
-    return true;
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make valgrind complain if any of the the memory occupied
- *        by a C-style array objects is undefined.
- *
- * Please note that this does not check whether the destinations of an
- * object's pointers or references are defined. Also, for performance
- * reasons the compiler might insert "padding bytes" between within
- * the objects which leads to false positives.
- *
- * Example:
- *
- * \code
- * int i[2];
- * Valgrind::CheckDefined(i, 2); // Valgrind complains!
- * \endcode
- *
- * \tparam T The type of the object which ought to be checked
- *
- * \param value Pointer to the first object of the array.
- * \param size The size of the array in number of objects
- *
- * \return true iff there are no undefined bytes in the memory
- *         occupied by the array.
- */
-template <class T>
-inline bool CheckDefined(const T* value OPM_VALGRIND_OPTIM_UNUSED, int size OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    auto tmp = VALGRIND_CHECK_MEM_IS_DEFINED(value, size*sizeof(T));
-    return tmp == 0;
-#else
-    return true;
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make the memory on which an object resides undefined in
- *        valgrind runs.
- *
- * Example:
- *
- * \code
- * int i = 0;
- * Valgrind::SetUndefined(i);
- * Valgrind::CheckDefined(i); // Valgrind complains!
- * \endcode
- *
- * \tparam T The type of the object which ought to be set to undefined
- *
- * \param value The object which's memory valgrind should be told is undefined
- */
-template <class T>
-inline void SetUndefined(const T &value OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    VALGRIND_MAKE_MEM_UNDEFINED(&value, sizeof(T));
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make the memory on which an array of object resides
- *        undefined in valgrind runs.
- *
- * Example:
- *
- * \code
- * int i[3] = {0, 1, 3};
- * Valgrind::SetUndefined(&i[1], 2);
- * Valgrind::CheckDefined(i, 3); // Valgrind complains!
- * \endcode
- *
- * \tparam T The type of the object which ought to be set to undefined
- *
- * \param value Pointer to the first object of the array.
- * \param size The size of the array in number of objects
- */
-template <class T>
-inline void SetUndefined(const T* value OPM_VALGRIND_OPTIM_UNUSED, int size OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    VALGRIND_MAKE_MEM_UNDEFINED(value, size*sizeof(T));
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make the memory on which an object resides defined.
- *
- * Example:
- *
- * \code
- * int i;
- * Valgrind::SetDefined(i);
- * Valgrind::CheckDefined(i); // Valgrind does not complain!
- * \endcode
- *
- * \tparam T The type of the object which valgrind should consider as defined
- *
- * \param value The object which's memory valgrind should consider as defined
- */
-template <class T>
-inline void SetDefined(const T& value OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    VALGRIND_MAKE_MEM_DEFINED(&value, sizeof(T));
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make the memory on which a C-style array of objects resides
- *        defined.
- *
- * Example:
- *
- * \code
- * int i[3];
- * Valgrind::SetDefined(i, 3);
- * Valgrind::CheckDefined(i, 3); // Valgrind does not complain!
- * \endcode
- *
- * \tparam T The type of the object which valgrind should consider as defined
- *
- * \param value Pointer to the first object of the array.
- * \param n The size of the array in number of objects
- */
-template <class T>
-inline void SetDefined(const T *value OPM_VALGRIND_OPTIM_UNUSED, int n OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    VALGRIND_MAKE_MEM_DEFINED(value, n*sizeof(T));
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make valgrind complain if an object's memory is accessed.
- *
- * Example:
- *
- * \code
- * int i = 1;
- * Valgrind::SetNoAccess(i);
- * int j = i; // Valgrind complains!
- * \endcode
- *
- * \tparam T The type of the object which valgrind should complain if accessed
- *
- * \param value The object which's memory valgrind should complain if accessed
- */
-template <class T>
-inline void SetNoAccess(const T &value OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    VALGRIND_MAKE_MEM_NOACCESS(&value, sizeof(T));
-#endif
-}
-
-/*!
- * \ingroup Valgrind
- * \brief Make valgrind complain if the memory of a C-style array of
- *        objects is accessed.
- *
- * Example:
- *
- * \code
- * int i[3] = {0, 1, 2};
- * Valgrind::SetNoAccess(i, 2);
- * int j = i[1]; // Valgrind complains!
- * \endcode
- *
- * \param value Pointer to the first object of the array.
- * \param size The size of the array in number of objects
- */
-template <class T>
-inline void SetNoAccess(const T *value OPM_VALGRIND_OPTIM_UNUSED, int size OPM_VALGRIND_OPTIM_UNUSED)
-{
-#if !defined NDEBUG && HAVE_VALGRIND
-    VALGRIND_MAKE_MEM_NOACCESS(value, size*sizeof(T));
-#endif
-}
-
-}} // namespace Valgrind, Opm
-
-#endif

opm/common/utility/platform_dependent/disable_warnings.h

@@ -73,9 +73,6 @@
 #pragma GCC diagnostic ignored "-Wunused-variable"
 #pragma GCC diagnostic ignored "-Wunused-local-typedefs"
 #pragma GCC diagnostic ignored "-Wcast-align"
-#pragma GCC diagnostic ignored "-Wcast-qual"
-#pragma GCC diagnostic ignored "-Wignored-attributes"
-#pragma GCC diagnostic ignored "-Wmisleading-indentation"
 #endif // COMPATIBLE_COMPILER
 
 #endif // SILENCE_EXTERNAL_WARNINGS

redhat/opm-common.spec

@@ -5,7 +5,7 @@
 %define tag final
 
 Name:           opm-common
-Version:        2017.04
+Version:        2016.10
 Release:        0
 Summary:        Open Porous Media - common helpers and buildsystem
 License:        GPL-3.0

travis/build-and-test-opm-common.sh

@@ -0,0 +1,8 @@
+#!/usr/bin/env bash
+set -e
+
+pushd . > /dev/null
+opm-common/travis/build-opm-common.sh
+cd opm-common/build
+ctest --output-on-failure
+popd > /dev/null

travis/build-and-test.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 set -e
 
-build_order=(opm-common opm-parser opm-material opm-output opm-grid opm-core ewoms opm-simulators)
+build_order=(opm-common opm-parser opm-material opm-output opm-core opm-grid ewoms opm-simulators opm-upscaling)
 
 # This shell script should be started with the name of a module as
 # only only command line argument. It will start by building all
@@ -30,7 +30,7 @@ function upstream_build {
     echo "Building: ${project}"
     mkdir -p ${project}/build
     pushd ${project}/build > /dev/null
-    cmake ../ -DENABLE_PYTHON=ON -DBUILD_TESTING=OFF -DSILENCE_EXTERNAL_WARNINGS=ON -DUSE_QUADMATH=OFF -DADD_DISABLED_CTESTS=OFF
+    cmake ../ -DENABLE_PYTHON=ON -DBUILD_TESTING=OFF -DSILENCE_EXTERNAL_WARNINGS=True -DUSE_QUADMATH=OFF -DADD_DISABLED_CTESTS=OFF
     make 
     popd > /dev/null
 }
@@ -44,14 +44,9 @@ function downstream_build_and_test {
     # The build commands cmake, make and ctest must be given as
     # separate commands and not chained with &&. If chaining with &&
     # is used the 'set -e' does not exit on first error.
-    if [ "$project" == "ewoms" ]; then
+    cmake ../ -DENABLE_PYTHON=ON -DBUILD_TESTING=ON -DSILENCE_EXTERNAL_WARNINGS=True -DUSE_QUADMATH=OFF -DADD_DISABLED_CTESTS=OFF
-       cmake ../ -DENABLE_PYTHON=ON -DBUILD_TESTING=OFF -DSILENCE_EXTERNAL_WARNINGS=ON -DUSE_QUADMATH=OFF -DADD_DISABLED_CTESTS=OFF
+    make
-       make
+    ctest --output-on-failure
-    else
-       cmake ../ -DENABLE_PYTHON=ON -DBUILD_TESTING=ON -DSILENCE_EXTERNAL_WARNINGS=ON -DUSE_QUADMATH=OFF -DADD_DISABLED_CTESTS=OFF
-       make
-       ctest --output-on-failure
-    fi
     popd > /dev/null
 }
 

travis/build-opm-common-shared.sh

@@ -0,0 +1,10 @@
+#!/usr/bin/env bash
+set -e
+
+pushd . > /dev/null
+cd opm-common
+mkdir build
+cd build
+cmake ../ -DBUILD_SHARED_LIBS=ON
+make
+popd > /dev/null

travis/build-opm-common.sh

@@ -0,0 +1,10 @@
+#!/usr/bin/env bash
+set -e
+
+pushd . > /dev/null
+cd opm-common
+mkdir build
+cd build
+cmake ../
+make
+popd > /dev/null