From fc369e7df4969f0aaa521d290725f9511df387b4 Mon Sep 17 00:00:00 2001
From: Jose Luis Blanco Claraco <joseluisblancoc@gmail.com>
Date: Mon, 19 Oct 2020 23:22:08 +0200
Subject: [PATCH] Fix exporting simpleini sources

---
 doc/source/doxygen-docs/changelog.md   |  3 ++-
 scripts/prepare_debian.sh              | 14 +++++++++-----
 scripts/prepare_release.sh             |  3 +--
 scripts/prepare_ubuntu_pkgs_for_ppa.sh |  7 +++++++
 4 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/doc/source/doxygen-docs/changelog.md b/doc/source/doxygen-docs/changelog.md
index a913a1e766..98c21e0ca1 100644
--- a/doc/source/doxygen-docs/changelog.md
+++ b/doc/source/doxygen-docs/changelog.md
@@ -1,8 +1,9 @@
 \page changelog Change Log
 
-# Version 2.1.1: (Under development)
+# Version 2.1.2: (Under development)
 - BUG FIXES:
   - Fix wrong coloring of graph edges in mrpt::opengl::graph_visualize() (Closes [#1111](https://github.com/MRPT/mrpt/issues/1111)).
+  - Fix Debian Lintian error: exporting copyrighted sources as part of simpleini submodule.
 
 ------
 # Version 2.1.1: Released Oct 19th, 2020
diff --git a/scripts/prepare_debian.sh b/scripts/prepare_debian.sh
index e760b7a3e7..9f4ff71026 100755
--- a/scripts/prepare_debian.sh
+++ b/scripts/prepare_debian.sh
@@ -64,14 +64,18 @@ if [ -f version_prefix.txt ];
 then
 	MRPTSRC=`pwd`
 
-  if [ -f $HOME/mrpt_release/mrpt*.tar.gz ];
-  then
-    echo "## release file already exists. Reusing it."
-  else
+  # Do not reuse the sources since each Ubuntu version may require different sources...
+  # Re-add after Bionic 18.04 EOL, which right now (2020) is a special case since
+  # it misses libsimpleini-dev
+  #
+  #if [ -f $HOME/mrpt_release/mrpt*.tar.gz ];
+  #then
+  #  echo "## release file already exists. Reusing it."
+  #else
     source scripts/prepare_release.sh
     echo
     echo "## Done prepare_release.sh"
-  fi
+  #fi
 else
 	echo "ERROR: Run this script from the MRPT root directory."
 	exit 1
diff --git a/scripts/prepare_release.sh b/scripts/prepare_release.sh
index 66feeb04ee..6388328ee4 100755
--- a/scripts/prepare_release.sh
+++ b/scripts/prepare_release.sh
@@ -45,8 +45,7 @@ then
 	git archive --format=tar HEAD | tar -x -C ${OUT_DIR}
 
 	# Include external submodules:
-	# TODO: Remove "simpleini" when bionic 18.04 is EOL.
-	EXTERNAL_MODS="nanogui nanogui/ext/nanovg googletest libfyaml rplidar_sdk simpleini"
+	EXTERNAL_MODS="nanogui nanogui/ext/nanovg googletest libfyaml rplidar_sdk ${MRPT_PKG_EXPORTED_SUBMODULES}"
 	for MOD in $EXTERNAL_MODS;
 	do
 		cd ${MRPTSRC}/3rdparty/$MOD
diff --git a/scripts/prepare_ubuntu_pkgs_for_ppa.sh b/scripts/prepare_ubuntu_pkgs_for_ppa.sh
index 37f8bb180b..a011f1c9cc 100755
--- a/scripts/prepare_ubuntu_pkgs_for_ppa.sh
+++ b/scripts/prepare_ubuntu_pkgs_for_ppa.sh
@@ -19,6 +19,9 @@ LST_DISTROS=(xenial bionic eoan focal groovy)
 # Special case for Xenial: enforce g++7
 export MRPT_PKG_CUSTOM_CMAKE_PARAMS_xenial="-DCMAKE_C_COMPILER=/usr/bin/gcc-7 -DCMAKE_CXX_COMPILER=/usr/bin/g++-7"
 
+# Special case for Bionic: use embedded version of simpleini
+export MRPT_PKG_EXPORTED_SUBMODULES_bionic="simpleini"
+
 
 # Checks
 # --------------------------------
@@ -72,6 +75,10 @@ do
 	cd ${MRPTSRC}
 	auxVarName=MRPT_PKG_CUSTOM_CMAKE_PARAMS_${DEBIAN_DIST}
 	auxVarName=${!auxVarName} # Replace by variable contents
+
+	auxVarName2=MRPT_PKG_EXPORTED_SUBMODULES_${DEBIAN_DIST}
+	export MRPT_PKG_EXPORTED_SUBMODULES=${!auxVarName2} # Replace by variable contents
+
 	bash scripts/prepare_debian.sh -s -u -h -d ${DEBIAN_DIST} ${EMBED_EIGEN_FLAG}  -c "${MRPT_PKG_CUSTOM_CMAKE_PARAMS}${auxVarName}"
 
 	CUR_SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"