Skip to content

Conversation

@IntinteDAO
Copy link
Contributor

Known issues:

  • The game (probably) cannot localize a sound / music data. The sound works if you delete CMAKE Install Dirs, but creating mess

@robertkirkman

@IntinteDAO IntinteDAO changed the title [Draft] Lugaru [Draft] Newpkg: Lugaru Jun 10, 2025
@IntinteDAO IntinteDAO marked this pull request as draft June 24, 2025 14:55
@IntinteDAO
Copy link
Contributor Author

@robertkirkman @hansm629 All known bugs fixed, please check :)

@IntinteDAO IntinteDAO changed the title [Draft] Newpkg: Lugaru Newpkg: Lugaru Jul 6, 2025
@IntinteDAO IntinteDAO marked this pull request as ready for review July 6, 2025 16:03
TERMUX_PKG_HOMEPAGE=https://osslugaru.gitlab.io
TERMUX_PKG_DESCRIPTION="Lugaru HD, free and open source ninja rabbit fighting game"
TERMUX_PKG_LICENSE="GPL-2.0"
TERMUX_PKG_MAINTAINER="@termux-user-repository"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is better to set yourself as maintainer.

Copy link
Contributor Author

@IntinteDAO IntinteDAO Jul 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@licy183
Thanks for reply. Should that be the Github nickname or the actual nickname? Probably the github one due to the fact that the bot will call me if the package doesn't work, but I prefer to inquire

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be the nickname on GitHub.

Copy link
Member

@robertkirkman robertkirkman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

The game is working very very well, and the framerate is really great with Zink + Turnip on Adreno 612. The audio and music are also both working!

It's disappointing that this game doesn't have any touchscreen buttons, but since it is open source, some day I could make some. I am trying to make some other projects other than touchscreen buttons currently.

Could you change the PR description and commit history to explain that the audio issue is fixed by the patch?

@robertkirkman
Copy link
Member

This game does capture the Termux:X11 cursor inescapably and if you don't have a physical, hardware keyboard, it will unfortunately trap the Termux:X11 cursor inescapably. however, the Android OS cursor will still work and it will allow you to go to Termux App and type pkill lugaru which will escape this by force quitting the game.

Unfortunately, this problem is just the kind of thing that is common to all games generally that don't have any touchscreen buttons implemented. On a touchscreen-only device, including GNU/Linux as well, there are a lot of unreachable inputs and the game is unplayable.

@IntinteDAO
Copy link
Contributor Author

This game does capture the Termux:X11 cursor inescapably and if you don't have a physical, hardware keyboard, it will unfortunately trap the Termux:X11 cursor inescapably. however, the Android OS cursor will still work and it will allow you to go to Termux App and type pkill lugaru which will escape this by force quitting the game.

Unfortunately, this problem is just the kind of thing that is common to all games generally that don't have any touchscreen buttons implemented. On a touchscreen-only device, including GNU/Linux as well, there are a lot of unreachable inputs and the game is unplayable.

Well, I use lapdock. However the other way is to add touchscreen on top of screen. Termux-x11-extra can do that, but a code have some Bugs to upstream to termux-x11

TERMUX_PKG_SHA256=f3ea477caf78911c69939fbdc163f9f6517c7ef2267e716a0e050be1a166ef97
TERMUX_PKG_DEPENDS="sdl2, glu, openal-soft, lugaru-data"
TERMUX_PKG_EXTRA_CONFIGURE_ARGS="
-DCMAKE_INSTALL_PREFIX=$TERMUX_PREFIX
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is automatically applied by termux_step_configure_cmake() and is not necessary to specify explicitly here.

-DSYSTEM_INSTALL=ON
-DCMAKE_INSTALL_BINDIR=bin
-DCMAKE_INSTALL_DATADIR=share/games
-DCMAKE_POLICY_VERSION_MINIMUM=3.5
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This additional patch is also necessary in order to fix the second error that occurs with CMake 4:

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1,7 +1,6 @@
 project(lugaru)
 
 cmake_minimum_required(VERSION 3.0)
-cmake_policy(SET CMP0004 OLD)
 
 include(FindPkgConfig)
 include(GNUInstallDirs)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants