Modified environment so that no preprocessor symbols are in header. T…

…hese may

confuse builds outside of gridpack.

src/environment/environment.cpp

@@ -92,19 +92,34 @@ Environment::Environment(int argc, char **argv,const char* help,const long int&
 
 }
 
-#ifdef ENABLE_ENVIRONMENT_FROM_COMM
 Environment::Environment(int argc, char **argv, MPI_Comm &comm): p_boostEnv(argc,argv),clparser(argc,argv),p_from_comm(false)
 {
+#ifdef ENABLE_ENVIRONMENT_FROM_COMM
   PrintStatus();
   pma_stack = 200000;
   pma_heap  = 200000;
   if (GA_Initialize_comm(comm)) {
     p_from_comm = true;
     MA_init(C_DBL,pma_stack,pma_heap);
     gridpack::math::Initialize(&argc,&argv);
+  } else {
+    int rank;
+    MPI_Comm_rank(comm,&rank);
+    if (rank == 0) {
+      std::cout<<"Initialize GA from communicator failed" << std::endl;
+    }
+    MPI_Abort(comm, 1);
   }
-}
+#else
+  int rank;
+  MPI_Comm_rank(comm,&rank);
+  if (rank == 0) {
+    std::cout<<"Initialize GridPACK Environment from communicator not implemented"
+      << std::endl;
+  }
+  MPI_Abort(comm, 1);
 #endif
+}
 
 Environment::~Environment(void)
 {

src/environment/environment.hpp

@@ -80,7 +80,6 @@ class Environment : private utility::Uncopyable
               const long int& ma_heap);
   Environment(int argc, char **argv);
 
-#ifdef ENABLE_ENVIRONMENT_FROM_COMM
   /**
    * Initialize environment from existing communicator
    * @param argc        number of program command line arguments
@@ -90,7 +89,6 @@ class Environment : private utility::Uncopyable
    * @return 
    */
   Environment(int argc, char **argv, MPI_Comm &comm);
-#endif
 
   /**
    * Return true if processor is active in this environment, false otherwise.