* Always copy proj data dir from the same pkg-config datadir it used during installation

Author: caiohamamura

R/zzz.R

@@ -1,19 +1,16 @@
+OLD_PROJ <- Sys.getenv("PROJ_LIB")
+
 .onUnload <- function(libpath) {
   library.dynam.unload("gdalBindings", libpath)
+  Sys.setenv("PROJ_LIB" =  OLD_PROJ)
   invisible()
 }
 
 #' @useDynLib gdalBindings, .registration=TRUE
 .onLoad <- function(libname, pkgname) {
-  Rcpp::loadModule("gdal_module", TRUE, TRUE)
-  InitializeGDAL()
-}
-
-.onAttach <- function(libname, pkgname) {
-  if (Sys.getenv("PROJ_LIB") == "") {
+  if (system.file("proj", package = pkgname)[1] != "") {
     Sys.setenv("PROJ_LIB" = system.file("proj", package = pkgname)[1])
   }
-  if (Sys.getenv("GDAL_DATA") == "") {
-    Sys.setenv("GDAL_DATA" = system.file("gdal", package = pkgname)[1])
-  }
-}
+  Rcpp::loadModule("gdal_module", TRUE, TRUE)
+  InitializeGDAL()
+}

configure

@@ -646,6 +646,7 @@ ac_includes_default="\
 ac_header_c_list=
 ac_subst_vars='LTLIBOBJS
 LIBOBJS
+PROJ_DATADIR
 PROJ_LIBS
 PROJ_CFLAGS
 GDAL_DEP_LIBS
@@ -725,7 +726,8 @@ GDAL_LIBS
 CC
 CFLAGS
 PROJ_CFLAGS
-PROJ_LIBS'
+PROJ_LIBS
+PROJ_DATADIR'
 
 
 # Initialize some variables set by options.
@@ -1359,6 +1361,8 @@ Some influential environment variables:
   CFLAGS      C compiler flags
   PROJ_CFLAGS C compiler flags for PROJ, overriding pkg-config
   PROJ_LIBS   linker flags for PROJ, overriding pkg-config
+  PROJ_DATADIR
+              value of datadir for proj, overriding pkg-config
 
 Use these variables to override the choices made by `configure' or to help
 it to find libraries and programs with nonstandard names/locations.
@@ -4429,6 +4433,39 @@ printf "%s\n" "yes" >&6; }
 
 fi
 
+# Copy PROJ datadir
+
+if test -n "$PROJ_DATADIR"; then
+    pkg_cv_PROJ_DATADIR="$PROJ_DATADIR"
+ elif test -n "$PKG_CONFIG"; then
+    if test -n "$PKG_CONFIG" && \
+    { { printf "%s\n" "$as_me:${as_lineno-$LINENO}: \$PKG_CONFIG --exists --print-errors \"proj\""; } >&5
+  ($PKG_CONFIG --exists --print-errors "proj") 2>&5
+  ac_status=$?
+  printf "%s\n" "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
+  test $ac_status = 0; }; then
+  pkg_cv_PROJ_DATADIR=`$PKG_CONFIG --variable="datadir" "proj" 2>/dev/null`
+		      test "x$?" != "x0" && pkg_failed=yes
+else
+  pkg_failed=yes
+fi
+ else
+    pkg_failed=untried
+fi
+PROJ_DATADIR=$pkg_cv_PROJ_DATADIR
+
+if test "x$PROJ_DATADIR" = x""
+then :
+
+fi
+
+{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking proj datadir path" >&5
+printf %s "checking proj datadir path... " >&6; }
+{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: result: $PROJ_DATADIR" >&5
+printf "%s\n" "$PROJ_DATADIR" >&6; }
+
+
+
 ##################
 ## END
 ##################

configure.ac

@@ -105,6 +105,14 @@ AC_SUBST([GDAL_DEP_LIBS])
 ################
 PKG_CHECK_MODULES([PROJ], [proj])
 
+# Copy PROJ datadir
+PKG_CHECK_VAR([PROJ_DATADIR], [proj], [datadir])
+
+AC_MSG_CHECKING([proj datadir path])
+AC_MSG_RESULT([$PROJ_DATADIR])
+
+AC_SUBST([PROJ_DATADIR])
+
 ##################
 ## END
 ##################

src/Makevars.in

@@ -1,2 +1,15 @@
 PKG_CPPFLAGS = @GDAL_CFLAGS@ @PROJ_CFLAGS@
 PKG_LIBS = @GDAL_LIBS@ @GDAL_DEP_LIBS@ @PROJ_LIBS@
+
+# Copy PROJ data directory to inst/proj
+PROJ_DATADIR = @PROJ_DATADIR@
+
+copy_proj_data:
+	if [ -n "$(PROJ_DATADIR)" ]; then \
+		cp -r $PROJ_DATADIR ./inst/proj/;
+	else \
+		echo "PROJ_DATADIR is empty. Skipping file copy."; \
+	fi
+
+# Override the default build rule to include the custom file copying rule
+all: copy_proj_data