-
Notifications
You must be signed in to change notification settings - Fork 208
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR for Open-Typer app request #2421
Conversation
apps/Open-Typer/install-32
Outdated
echo "[Desktop Entry] | ||
Name=Open-Typer | ||
Comment=An Open Source spiritual successor to the flash game Line Rider | ||
Icon=$(dirname "$0")/icon-64.png | ||
Exec=/opt/Open-Typer.AppImage %f | ||
Type=Application | ||
Terminal=false | ||
Categories=Utility;" | sudo tee /usr/local/share/applications/Open-Typer.desktop >/dev/null || error "Failed to create menu button!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Are you sure Utility is the best category for this? I was thinking Education would be the best fit.
- Please update the comment.
- The
%f
argument in the Exec line denotes the ability to open files. Does Open-Typer actually have the ability to open any kind of files?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Meant to change from Utility to Education, sorry. Did not know how to execute an appimage from a menu entry, and saw both Cura and OpenSCAD use it, so I put it in. adazem009 says that there might be opening files in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any idea what %U does? adazem009 thought it was also files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any suggestions for which category this would fit best in? |
I'm the developer of Open-Typer and I use the |
Accidentally kept the wrong comment in the menu entry, so just fixed that. All seems well and mergeable now. |
Sorry, I meant pi-apps category. We don't have an education category, because until now everything educational has been about coding, so we've put them all in the Programming category. I will give this a quick test soon and afterwards will probably have a better idea where it best fits. |
Tools possibly? I would still try it though, so you can get a better opinion. |
please stop doing this f65b36d (merge commit). You should use the rebase tool (via git or your favorite git GUI client such as Github Desktop) it dirtys the git history and makes it impossible for our CI to run. |
Will remember. |
Are there any other issues? I have tested it, and it runs perfectly. |
sorry going to have to close this. it can't be included with such high GLIBC requirements. that means it can't run on buster and bionic.
@adazem009 we can re-open if you build your appimages on an older distro for compatabiilty. I suggest setting up build chroots or docker containers on ubuntu bionic. keep in mind that github actions (since I see you use it at least partially for CI purposes) can build in a docker container easily with edit: also, I suggest simply using the following QT6 ppa rather than bulding it from source unless you need something newer than 6.2 or have custom patches https://launchpad.net/~okirby/+archive/ubuntu/qt6-backports edit2: I'm also not opposed to a flatpak but we can't/won't do snaps |
@theofficialgman I checked that PPA before, but I need at least Qt 6.4, so it's better to cross-compile it on x86 (building it in inside an emulator is definitely not a choice because of GitHub Actions limit). I'll try to build Open-Typer 5.3 on an older distro (I'll release it in October). |
@theofficialgman I think I'll keep the builds on Ubuntu 20.04. Bionic has a too old toolchain and I need at least C++17 for Qt 6. I've been trying to build a gcc cross compiler myself for a few weeks, but even after I'd managed to do it, it didn't pass the "compiler tests" Qt runs before building. I'll move this to the next minor version and try to build Qt 6 in a PPA instead. It should be easier to build it directly on the target architecture. |
Reopening now that we have dropped Buster and Bionic support. |
at some point I will stop caring
@Sussy-OS thanks for the PR, it has been merged |
The application is designed specifically for the desktop to help people learn to type on a physical keyboard. However, I could show the native keyboard on touch screen devices instead. |
This PR is for adding Open-Typer to pi-apps.