Package Details: nestopia 1.53.0-1

Git Clone URL: https://aur.archlinux.org/nestopia.git (read-only, click to copy)
Package Base: nestopia
Description: High-accuracy NES/Famicom emulator
Upstream URL: https://github.com/0ldsk00l/nestopia
Licenses: GPL-2.0-only
Submitter: Barthalion
Maintainer: xiota
Last Packager: xiota
Votes: 15
Popularity: 0.48
First Submitted: 2018-01-07 17:35 (UTC)
Last Updated: 2024-12-24 05:46 (UTC)

Latest Comments

1 2 Next › Last »

Polizei commented on 2023-11-21 01:07 (UTC) (edited on 2023-11-21 01:07 (UTC) by Polizei)

You are clearly mistaking glu for libgl (libglvnd in current case.)

Having glu installed and building with -lGLU does not affect the produced binary in any way.

libglvnd is a direct dependency of fltk and is installed no matter what.

The sed patch is simply removing the unnecessary dependency glu is.

Anyway, enjoy.

Polizei commented on 2023-11-19 10:33 (UTC)

@xiota,

Building with or without GLU makes no difference to the produced binary. Check with ldd yourself.

-lGLU only comes from fltk-config --ldflags and is not actually used anywhere in the source.

Can you share more details on how you produced these benchmarks and how did you build nestopia with GLU?

carstene1ns commented on 2023-11-19 10:22 (UTC)

@xiota: Again, you do not understand what is an issue in fltk and what is an issue in nestopia. However, I am not against including glu now, since it does so much.

When that person refuses to maintain the package, intentionally keeping broken, then intentionally breaking it further, the package should be orphaned. The current maintainer has had ample opportunity to fix this package the last 2.5 years. It is long past time to allow someone else to fix it.

This hurts, and I imply every package maintainer does their best to keep their stuff in a good state. However, it seems you take this way more personal than I am, so I will disown this package and a couple others now, giving you the opportunity to maintain them better than I ever could. Have fun, enjoy your win, you make the community a better place! :)

carstene1ns commented on 2023-11-18 19:22 (UTC)

Sorry for the hassle, looks like my testing was flawed. Thanks @Polizei :)

Polizei commented on 2023-11-17 09:41 (UTC)

Apparently m4 doesn't play nice with when substituting shell pipes. Fix:

diff --git a/PKGBUILD b/PKGBUILD
index 00cf82d..4af17d9 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -14,18 +14,17 @@ source=($pkgname-$pkgver.tar.gz::"https://github.com/0ldsk00l/nestopia/archive/$
 sha256sums=('eae1d2f536ae8585edb8d723caf905f4ae65349edee4ffbee45f9f52b5e3b06c')
 install=$pkgname.install

-prepare() {
-  # remove GLU dependency
-  sed -i "s/fltk-config.*--ldflags/&|sed 's#-lGLU ##g'/g" $pkgname-$pkgver/configure.ac
-}
-
 build() {
   cd $pkgname-$pkgver

+  autoreconf -fi
+
+  # remove GLU dependency
+  sed -i "s/fltk-config.*--ldflags/&|sed 's#-lGLU ##g'/g" configure
+
   # build system normally disables this warning
   export CXXFLAGS="$CXXFLAGS -Wno-narrowing"

-  autoreconf -fi
   ./configure --prefix=/usr
   make
 }

Pillgar commented on 2023-11-17 00:57 (UTC) (edited on 2023-11-17 01:00 (UTC) by Pillgar)

I'm seeing this error when updating:

./configure: line 5715: syntax error near unexpected token `)'

./configure: line 5715: ` FLTK_LIBS="$(fltk-config --use-gl --use-images --ldflags|sed 's#-lGLU ##g')") ;;[]_AS_CASE([linux*], [AC_SUBST(FLTK_LIBS,"-lGL $(fltk-config --use-gl --use-images --ldflags|sed 's#-lGLU ##g')")'

==> ERROR: A failure occurred in build().

Aborting...