[gentoo-commits] repo/gentoo:master commit in: sci-astronomy/celestia/, sci-astronomy/celestia/files/

["Lars Wendler" <polynomial-c@gentoo.org>]

commit:     95e61d21a335b5ac9d608c50a6844c5b1943604a
Author:     Lars Wendler <polynomial-c <AT> gentoo <DOT> org>
AuthorDate: Tue May 15 08:28:54 2018 +0000
Commit:     Lars Wendler <polynomial-c <AT> gentoo <DOT> org>
CommitDate: Tue May 15 08:32:47 2018 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=95e61d21

sci-astronomy/celestia: Fixed live ebuild.

Package-Manager: Portage-2.3.36, Repoman-2.3.9

 sci-astronomy/celestia/celestia-9999.ebuild        |  18 +-
 .../files/celestia-1.6.99-compiler_warnings.patch  | 226 ---------------------
 .../files/celestia-1.6.99-default_source.patch     |  40 ----
 .../celestia/files/celestia-1.6.99-linking.patch   |  16 +-
 .../files/celestia-1.6.99-models_makefile.patch    |  21 --
 .../celestia/files/celestia-1.6.99-symlink.patch   |  19 --
 6 files changed, 10 insertions(+), 330 deletions(-)

diff --git a/sci-astronomy/celestia/celestia-9999.ebuild b/sci-astronomy/celestia/celestia-9999.ebuild
index afec84fc229..48a0e3acd11 100644
--- a/sci-astronomy/celestia/celestia-9999.ebuild
+++ b/sci-astronomy/celestia/celestia-9999.ebuild
@@ -57,17 +57,6 @@ PATCHES=(
 
 	# libpng16 #464764
 	"${FILESDIR}"/${PN}-1.6.1-libpng16.patch
-
-	# Patches from upstream PRs
-
-	# https://github.com/CelestiaProject/Celestia/pull/35
-	#"${FILESDIR}/${PN}-1.6.99-automake.patch"
-	"${FILESDIR}/${PN}-1.6.99-models_makefile.patch"
-	"${FILESDIR}/${PN}-1.6.99-default_source.patch"
-	"${FILESDIR}/${PN}-1.6.99-symlink.patch"
-
-	# https://github.com/CelestiaProject/Celestia/pull/37
-	"${FILESDIR}/${PN}-1.6.99-compiler_warnings.patch"
 )
 
 pkg_setup() {
@@ -89,11 +78,8 @@ pkg_setup() {
 src_prepare() {
 	default
 
-	if [[ -f configure.in ]] ; then
-		mv configure.{in,ac} || die
-	else
-		elog "configure.in file is gone. Clean up the ebuild!"
-	fi
+	# This commit introduced lots of glut related undefined reference errors
+	eapply -R "${FILESDIR}/${PN}-1.6.99-automake.patch"
 
 	# remove flags to let the user decide
 	local

diff --git a/sci-astronomy/celestia/files/celestia-1.6.99-compiler_warnings.patch b/sci-astronomy/celestia/files/celestia-1.6.99-compiler_warnings.patch
deleted file mode 100644
index d1ed92e212a..00000000000
--- a/sci-astronomy/celestia/files/celestia-1.6.99-compiler_warnings.patch
+++ /dev/null
@@ -1,226 +0,0 @@
-From dba25141a60ba7a09eac8f64eac1c18c48acc3a3 Mon Sep 17 00:00:00 2001
-From: Hleb Valoshka <375gnu@gmail.com>
-Date: Mon, 12 Mar 2018 20:18:51 +0300
-Subject: [PATCH] fix (some) compiler warnings
-
----
- src/cel3ds/3dsread.cpp       |  5 +++--
- src/celengine/command.cpp    | 15 ++++++---------
- src/celengine/observer.cpp   |  8 ++++----
- src/celengine/render.cpp     |  2 ++
- src/celengine/renderglsl.cpp |  2 +-
- src/celengine/star.cpp       |  2 +-
- src/celestia/glutmain.cpp    |  9 ++++-----
- src/celestia/qt/qtmain.cpp   |  2 +-
- src/celutil/bigfix.cpp       |  3 +--
- 9 files changed, 23 insertions(+), 25 deletions(-)
-
-diff --git a/src/cel3ds/3dsread.cpp b/src/cel3ds/3dsread.cpp
-index 3b78847f..3ee3cf3d 100644
---- a/src/cel3ds/3dsread.cpp
-+++ b/src/cel3ds/3dsread.cpp
-@@ -116,8 +116,9 @@ void indent()
-         cout << "  ";
- }
- 
--void logChunk(uint16 chunkType/*, int chunkSize*/)
-+void logChunk(uint16 /*chunkType*/ /*, int chunkSize*/)
- {
-+#if 0
-     const char* name = NULL;
- 
-     switch (chunkType)
-@@ -193,7 +194,7 @@ void logChunk(uint16 chunkType/*, int chunkSize*/)
-     default:
-         break;
-     }
--#if 0
-+
-     indent();
- 
-     if (name == NULL)
-diff --git a/src/celengine/command.cpp b/src/celengine/command.cpp
-index e190fa5b..a258f4c0 100644
---- a/src/celengine/command.cpp
-+++ b/src/celengine/command.cpp
-@@ -703,24 +703,21 @@ CommandCapture::CommandCapture(const std::string& _type,
- void CommandCapture::process(ExecutionEnvironment&)
- {
- #ifndef TARGET_OS_MAC
--    bool success = false;
--
--
-     // Get the dimensions of the current viewport
-     int viewport[4];
-     glGetIntegerv(GL_VIEWPORT, viewport);
- 
-     if (compareIgnoringCase(type, "jpeg") == 0)
-     {
--        success = CaptureGLBufferToJPEG(filename,
--                                        viewport[0], viewport[1],
--                                        viewport[2], viewport[3]);
-+        CaptureGLBufferToJPEG(filename,
-+                              viewport[0], viewport[1],
-+                              viewport[2], viewport[3]);
-     }
-     if (compareIgnoringCase(type, "png") == 0)
-     {
--        success = CaptureGLBufferToPNG(filename,
--                                       viewport[0], viewport[1],
--                                       viewport[2], viewport[3]);
-+        CaptureGLBufferToPNG(filename,
-+                             viewport[0], viewport[1],
-+                             viewport[2], viewport[3]);
-     }
- #endif
- }
-diff --git a/src/celengine/observer.cpp b/src/celengine/observer.cpp
-index 3b5af97a..60b818b3 100644
---- a/src/celengine/observer.cpp
-+++ b/src/celengine/observer.cpp
-@@ -426,7 +426,7 @@ void Observer::update(double dt, double timeScale)
-                 UniversalCoord origin = centerObj.getPosition(simTime);
- 
-                 Vector3d v0 = ufrom.offsetFromKm(origin);
--                Vector3d v1 = uto.offsetFromKm(origin);
-+                //Vector3d v1 = uto.offsetFromKm(origin);
- 
-                 if (jv.norm() == 0.0)
-                 {
-@@ -629,7 +629,7 @@ void Observer::computeGotoParameters(const Selection& destination,
-     }
-     
-     UniversalCoord targetPosition = destination.getPosition(getTime());
--    Vector3d v = targetPosition.offsetFromKm(getPosition()).normalized();
-+    //Vector3d v = targetPosition.offsetFromKm(getPosition()).normalized();
-     
-     jparams.traj = Linear;
-     jparams.duration = gotoTime;
-@@ -694,7 +694,7 @@ void Observer::computeGotoParametersGC(const Selection& destination,
-     setFrame(frame->getCoordinateSystem(), destination);
- 
-     UniversalCoord targetPosition = destination.getPosition(getTime());
--    Vector3d v = targetPosition.offsetFromKm(getPosition()).normalized();
-+    //Vector3d v = targetPosition.offsetFromKm(getPosition()).normalized();
- 
-     jparams.traj = GreatCircle;
-     jparams.duration = gotoTime;
-@@ -791,7 +791,7 @@ void Observer::computeCenterCOParameters(const Selection& destination,
- 
-     Selection centerObj = frame->getRefObject();
-     UniversalCoord centerPos = centerObj.getPosition(getTime());
--    UniversalCoord targetPosition = destination.getPosition(getTime());
-+    //UniversalCoord targetPosition = destination.getPosition(getTime());
- 
-     Quaterniond q;
-     q.setFromTwoVectors(v, w);
-diff --git a/src/celengine/render.cpp b/src/celengine/render.cpp
-index 018153d5..bfcefd39 100644
---- a/src/celengine/render.cpp
-+++ b/src/celengine/render.cpp
-@@ -8033,8 +8033,10 @@ void Renderer::renderCometTail(const Body& body,
- {
-     Vector3f cometPoints[MaxCometTailPoints];
-     Vector3d pos0 = body.getOrbit(now)->positionAtTime(now);
-+#if 0
-     Vector3d pos1 = body.getOrbit(now)->positionAtTime(now - 0.01);
-     Vector3d vd = pos1 - pos0;
-+#endif
-     double t = now;
- 
-     float distanceFromSun, irradiance_max = 0.0f;
-diff --git a/src/celengine/renderglsl.cpp b/src/celengine/renderglsl.cpp
-index 15644440..14774239 100644
---- a/src/celengine/renderglsl.cpp
-+++ b/src/celengine/renderglsl.cpp
-@@ -773,7 +773,7 @@ void renderGeometryShadow_GLSL(Geometry* geometry,
-     int lightIndex = 0;
-     Vector3f viewDir = -ls.lights[lightIndex].direction_obj;
-     Vector3f upDir = viewDir.unitOrthogonal();
--    Vector3f rightDir = upDir.cross(viewDir);
-+    /*Vector3f rightDir = */upDir.cross(viewDir);
- 
- 
-     glUseProgramObjectARB(0);
-diff --git a/src/celengine/star.cpp b/src/celengine/star.cpp
-index a5119cd6..662db92c 100644
---- a/src/celengine/star.cpp
-+++ b/src/celengine/star.cpp
-@@ -761,7 +761,7 @@ StarDetails::StarDetails() :
-     bolometricCorrection(0.0f),
-     knowledge(0u),
-     visible(true),
--    texture(texture),
-+    texture(texture), // warning: ‘StarDetails::texture’ is initialized with itself [-Winit-self]
-     geometry(InvalidResource),
-     orbit(NULL),
-     orbitalRadius(0.0f),
-diff --git a/src/celestia/glutmain.cpp b/src/celestia/glutmain.cpp
-index b1fb416c..36ae331b 100644
---- a/src/celestia/glutmain.cpp
-+++ b/src/celestia/glutmain.cpp
-@@ -155,8 +155,7 @@ static void MouseButton(int button, int state, int x, int y)
-     lastY = y;
- }
- 
--
--static void KeyPress(unsigned char c, int x, int y)
-+static void KeyPress(unsigned char c, int /*x*/, int /*y*/)
- {
-     // Ctrl-Q exits
-     if (c == '\021')
-@@ -167,7 +166,7 @@ static void KeyPress(unsigned char c, int x, int y)
- }
- 
- 
--static void KeyUp(unsigned char c, int x, int y)
-+static void KeyUp(unsigned char c, int /*x*/, int /*y*/)
- {
-     appCore->keyUp((int) c);
- }
-@@ -237,13 +236,13 @@ static void HandleSpecialKey(int key, bool down)
- }
- 
- 
--static void SpecialKeyPress(int key, int x, int y)
-+static void SpecialKeyPress(int key, int /*x*/, int /*y*/)
- {
-     HandleSpecialKey(key, true);
- }
- 
- 
--static void SpecialKeyUp(int key, int x, int y)
-+static void SpecialKeyUp(int key, int /*x*/, int /*y*/)
- {
-     HandleSpecialKey(key, false);
- }
-diff --git a/src/celestia/qt/qtmain.cpp b/src/celestia/qt/qtmain.cpp
-index f8dc5080..04b666d5 100644
---- a/src/celestia/qt/qtmain.cpp
-+++ b/src/celestia/qt/qtmain.cpp
-@@ -29,7 +29,7 @@
- using namespace std;
- 
- extern "C" { FILE __iob_func[3] = { *stdin,*stdout,*stderr }; }
--static const char *description = "Celestia";
-+//static const char *description = "Celestia";
- 
- // Command line options
- static bool startFullscreen = false;
-diff --git a/src/celutil/bigfix.cpp b/src/celutil/bigfix.cpp
-index 1eacc5e2..a4ff2f4b 100644
---- a/src/celutil/bigfix.cpp
-+++ b/src/celutil/bigfix.cpp
-@@ -416,11 +416,10 @@ std::string BigFix::toString()
- 
-     // Conversion using code from the original BigFix class.
-     std::string encoded("");
--    int bits, c, char_count, started, i, j;
-+    int bits, c, char_count, i, j;
- 
-     char_count = 0;
-     bits = 0;
--    started = 0;
- 
-     // Find first significant (non null) byte
-     i = 16;

diff --git a/sci-astronomy/celestia/files/celestia-1.6.99-default_source.patch b/sci-astronomy/celestia/files/celestia-1.6.99-default_source.patch
deleted file mode 100644
index c896896ef1d..00000000000
--- a/sci-astronomy/celestia/files/celestia-1.6.99-default_source.patch
+++ /dev/null
@@ -1,40 +0,0 @@
-From 1fe7a493eae81a4f32f65be03f594a7553429268 Mon Sep 17 00:00:00 2001
-From: Hleb Valoshka <375gnu@gmail.com>
-Date: Sat, 24 Feb 2018 16:45:36 +0300
-Subject: [PATCH] replace _BSD_SOURCE with _DEFAULT_SOURCE
-
----
- acinclude.m4          | 4 ++--
- admin/acinclude.m4.in | 4 ++--
- 2 files changed, 4 insertions(+), 4 deletions(-)
-
-diff --git a/acinclude.m4 b/acinclude.m4
-index e2d5da23..49a086f8 100644
---- a/acinclude.m4
-+++ b/acinclude.m4
-@@ -3223,8 +3223,8 @@ AC_DEFUN([AC_CHECK_COMPILERS],
-         CXXFLAGS="-Wall -W -Wpointer-arith $CXXFLAGS"
-         case $host in
-           *-*-linux-gnu)	
--            CFLAGS="-std=iso9899:1990 -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_XOPEN_SOURCE=500 -D_BSD_SOURCE $CFLAGS"
--            CXXFLAGS="-ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wchar-subscripts $CXXFLAGS"
-+            CFLAGS="-std=iso9899:1990 -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_XOPEN_SOURCE=500 -D_DEFAULT_SOURCE $CFLAGS"
-+            CXXFLAGS="-ansi -D_XOPEN_SOURCE=500 -D_DEFAULT_SOURCE -Wcast-align -Wchar-subscripts $CXXFLAGS"
-             KDE_CHECK_COMPILER_FLAG(Wmissing-format-attribute, [CXXFLAGS="$CXXFLAGS -Wformat-security -Wmissing-format-attribute"])
-             KDE_CHECK_C_COMPILER_FLAG(Wmissing-format-attribute, [CFLAGS="$CFLAGS -Wformat-security -Wmissing-format-attribute"])
-           ;;
-diff --git a/admin/acinclude.m4.in b/admin/acinclude.m4.in
-index e2d5da23..49a086f8 100644
---- a/admin/acinclude.m4.in
-+++ b/admin/acinclude.m4.in
-@@ -3223,8 +3223,8 @@ AC_DEFUN([AC_CHECK_COMPILERS],
-         CXXFLAGS="-Wall -W -Wpointer-arith $CXXFLAGS"
-         case $host in
-           *-*-linux-gnu)	
--            CFLAGS="-std=iso9899:1990 -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_XOPEN_SOURCE=500 -D_BSD_SOURCE $CFLAGS"
--            CXXFLAGS="-ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wchar-subscripts $CXXFLAGS"
-+            CFLAGS="-std=iso9899:1990 -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_XOPEN_SOURCE=500 -D_DEFAULT_SOURCE $CFLAGS"
-+            CXXFLAGS="-ansi -D_XOPEN_SOURCE=500 -D_DEFAULT_SOURCE -Wcast-align -Wchar-subscripts $CXXFLAGS"
-             KDE_CHECK_COMPILER_FLAG(Wmissing-format-attribute, [CXXFLAGS="$CXXFLAGS -Wformat-security -Wmissing-format-attribute"])
-             KDE_CHECK_C_COMPILER_FLAG(Wmissing-format-attribute, [CFLAGS="$CFLAGS -Wformat-security -Wmissing-format-attribute"])
-           ;;

diff --git a/sci-astronomy/celestia/files/celestia-1.6.99-linking.patch b/sci-astronomy/celestia/files/celestia-1.6.99-linking.patch
index 809105bbda5..18b1d10a392 100644
--- a/sci-astronomy/celestia/files/celestia-1.6.99-linking.patch
+++ b/sci-astronomy/celestia/files/celestia-1.6.99-linking.patch
@@ -1,4 +1,4 @@
-From 803f4652895d40acc698901a87114a8582c050e6 Mon Sep 17 00:00:00 2001
+From 60dffe49ff32cbee0144c0dc9c6ae4f954aec784 Mon Sep 17 00:00:00 2001
 From: Lars Wendler <polynomial-c@gentoo.org>
 Date: Wed, 4 Apr 2018 15:40:08 +0200
 Subject: [PATCH] Fix linking issues:
@@ -7,15 +7,15 @@ Subject: [PATCH] Fix linking issues:
  * as-needed (https://bugs.gentoo.org/281258)
  * theora with glut (https://bugs.gentoo.org/399237)
 ---
- configure.in             |  1 +
+ configure.ac             |  1 +
  src/celestia/Makefile.am | 13 ++++++++-----
  2 files changed, 9 insertions(+), 5 deletions(-)
 
-diff --git a/configure.in b/configure.in
-index 0b70581e..3ea150b9 100644
---- a/configure.in
-+++ b/configure.in
-@@ -242,6 +242,7 @@ if (test "$ui_glut" = "yes"); then
+diff --git a/configure.ac b/configure.ac
+index c9e722a1..7efce6ef 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -243,6 +243,7 @@ if (test "$ui_glut" = "yes"); then
  	dnl Check for GLUT.
  	AC_CHECK_LIB(glut, glutKeyboardUpFunc, ,
  	             [AC_MSG_ERROR([GLUT library version >= 3.7 not found])])
@@ -24,7 +24,7 @@ index 0b70581e..3ea150b9 100644
  AM_CONDITIONAL(ENABLE_GLUT, test "$ui_glut" = "yes")
  
 diff --git a/src/celestia/Makefile.am b/src/celestia/Makefile.am
-index 95ba9ab6..8402256b 100644
+index 60609fd3..394af68e 100644
 --- a/src/celestia/Makefile.am
 +++ b/src/celestia/Makefile.am
 @@ -7,14 +7,15 @@ DEFS = -DCONFIG_DATA_DIR='"$(PKGDATADIR)"' -DLOCALEDIR='"$(datadir)/locale"' @DE

diff --git a/sci-astronomy/celestia/files/celestia-1.6.99-models_makefile.patch b/sci-astronomy/celestia/files/celestia-1.6.99-models_makefile.patch
deleted file mode 100644
index 53d6307ac19..00000000000
--- a/sci-astronomy/celestia/files/celestia-1.6.99-models_makefile.patch
+++ /dev/null
@@ -1,21 +0,0 @@
-From abe1b03873af0e33d6cc23dec22755564794d41c Mon Sep 17 00:00:00 2001
-From: Hleb Valoshka <375gnu@gmail.com>
-Date: Sat, 24 Feb 2018 15:58:35 +0300
-Subject: [PATCH] add Makefile for models/ subdir
-
----
- models/Makefile.am | 5 +++++
- 1 file changed, 5 insertions(+)
- create mode 100644 models/Makefile.am
-
-diff --git a/models/Makefile.am b/models/Makefile.am
-new file mode 100644
-index 00000000..09d37338
---- /dev/null
-+++ b/models/Makefile.am
-@@ -0,0 +1,5 @@
-+pkgdatadir = @datadir@/@PACKAGE@/models
-+
-+pkgdata_DATA = $(wildcard *.cmod) $(wildcard *.png)
-+
-+EXTRA_DIST = $(pkgdata_DATA)

diff --git a/sci-astronomy/celestia/files/celestia-1.6.99-symlink.patch b/sci-astronomy/celestia/files/celestia-1.6.99-symlink.patch
deleted file mode 100644
index 4d0cd56cac3..00000000000
--- a/sci-astronomy/celestia/files/celestia-1.6.99-symlink.patch
+++ /dev/null
@@ -1,19 +0,0 @@
-From 20fb86991babb1aa5aa278b2cd4dc454c78d5cd4 Mon Sep 17 00:00:00 2001
-From: Hleb Valoshka <375gnu@gmail.com>
-Date: Sat, 24 Feb 2018 18:36:20 +0300
-Subject: [PATCH] fix symlink creation
-
----
- src/celestia/Makefile.am | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/src/celestia/Makefile.am b/src/celestia/Makefile.am
-index 25eb7d64..60609fd3 100644
---- a/src/celestia/Makefile.am
-+++ b/src/celestia/Makefile.am
-@@ -79,4 +79,4 @@ noinst_DATA = ../../celestia
- CLEANFILES = ../../celestia
- 
- ../../celestia: celestia
--	(cd ../..; ln -s src/celestia/celestia)
-+	(cd ../..; ln -sf src/celestia/celestia)