Skip to content

Commit

Permalink
Feature/visibility (#783)
Browse files Browse the repository at this point in the history
* start working on #772
first check what happens

* up #772 and #773
 - add FEELPP_{NO_}EXPORT to some classes and blocks
 - remove some obsolete code (feel/feelcore/application*.*)

* NO_EXPORT in private
should add EXPORT in public

* revert

* update to latest glog #642 and enable cmake

* update to latest gflags #642 and enable cmake

* enable gflags/glog from cmake now #642, no more using autoconf/automake
it simplifies the process

* add CXXFLAGS to reduce compilation cost

* add MESH option to feelpp_add_test, use it for med test #735

* set number of jobs to 1 in docker, instead of default value

* more build with travis

 . 14.04 trusty, add cmake >= 3. for gflags/glog support in cmake
 . 16.04 add support for xenial, clang 3.8
 . remove cxx flags

* allow failures for now

* pass CXXFLAGS to reduce compilation cost

* up travis

* use 16.10 for minimal

* enhance MeshBase interface

 . hasAnyMarker checks for existence of a mesh marker passed in the list
 . hasAllMarkers checks all strings are markers of the mesh

* disable mesh_partitioner app in minimal build

* use Dockerfile-16.10 for minimal

* Feel++ is no more strongly dependent on Gmsh #790

We can now compile and run Feel++ applications without having Gmsh
installed. Of course the features associated to Gmsh are no longer
available if Gmsh support is not available

* --warnings in minimal version of Feel++ if petsc is not available #790

We need to add support for eigen non linear solvers see #827

* fixes glog include_directories #642

* do not depend on feelpp_mesh_partitioner #790

do not depend on feelpp_mesh_partitioner in minimal build, we do not
have access to a mesh partitioner

* fixes macro name that handles Gmsh support #790
  • Loading branch information
prudhomm committed Dec 28, 2016
1 parent 042b1b3 commit 098b682
Show file tree
Hide file tree
Showing 48 changed files with 645 additions and 1,463 deletions.
2 changes: 1 addition & 1 deletion .clang-format
@@ -1,6 +1,7 @@
BasedOnStyle: LLVM

BreakBeforeBraces: Allman
AlignTrailingComments: true
IndentWidth: 4
UseTab: Never
AlignAfterOpenBracket: Align
Expand All @@ -12,6 +13,5 @@ AllowShortIfStatementsOnASingleLine: true
NamespaceIndentation: None
ColumnLimit: 0
ReflowComments: true
AlignTrailingComments: true
SpacesInParentheses: true

2 changes: 2 additions & 0 deletions cmake/modules/feelpp.dependencies.cmake
Expand Up @@ -80,6 +80,8 @@ IF( ("${CMAKE_CXX_COMPILER_ID}" MATCHES "XL") )
set( CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -qlanglvl=extc1x" )
endif()

set( CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fvisibility-inlines-hidden") # -fvisibility=hidden" )

# Sometimes relinking libraries in contrib does not work becaus of bad paths
# A discussion has been opened on the bugtracker of CMake: https://cmake.org/Bug/print_bug_page.php?bug_id=13934
# To fix this: if the executable format has not been specified, which seems to happen with Clang, we force it to ELF for Linux
Expand Down
4 changes: 2 additions & 2 deletions feel/feelalg/datamap.hpp
Expand Up @@ -44,7 +44,7 @@ class DataMap;
/**
*
*/
class IndexSplit : public std::vector<std::vector<size_type> >
class FEELPP_EXPORT IndexSplit : public std::vector<std::vector<size_type> >
{
typedef IndexSplit self_type;
typedef boost::shared_ptr<self_type> self_ptrtype;
Expand Down Expand Up @@ -130,7 +130,7 @@ private :
* @author Christophe Prud'homme
* @see
*/
class DataMap
class FEELPP_EXPORT DataMap
{

public:
Expand Down
138 changes: 0 additions & 138 deletions feel/feelcore/applicationepetra.cpp

This file was deleted.

182 changes: 0 additions & 182 deletions feel/feelcore/applicationepetra.hpp

This file was deleted.

0 comments on commit 098b682

Please sign in to comment.