Skip to content
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

Merged
merged 13 commits into from
Oct 22, 2023
Merged

PR for Open-Typer app request #2421

merged 13 commits into from
Oct 22, 2023

Conversation

Sussy-OS
Copy link
Contributor

@Sussy-OS Sussy-OS commented Aug 5, 2023

This PR is for adding Open-Typer to pi-apps.

Comment on lines 20 to 27
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!"
Copy link
Owner

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?

Copy link
Contributor Author

@Sussy-OS Sussy-OS Aug 5, 2023

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.

Copy link
Contributor Author

@Sussy-OS Sussy-OS Aug 5, 2023

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

A quick search told me it was for accessing urls.
image

@Botspot
Copy link
Owner

Botspot commented Aug 5, 2023

Any suggestions for which category this would fit best in?

@adazem009
Copy link

Any suggestions for which category this would fit best in?

I'm the developer of Open-Typer and I use the Education category in all .desktop files. For example: https://github.com/Open-Typer/Open-Typer/blob/master/res/linux-release/usr/share/applications/open-typer-appimage.desktop

@Sussy-OS
Copy link
Contributor Author

Sussy-OS commented Aug 5, 2023

Accidentally kept the wrong comment in the menu entry, so just fixed that. All seems well and mergeable now.

@Botspot
Copy link
Owner

Botspot commented Aug 5, 2023

Any suggestions for which category this would fit best in?

I'm the developer of Open-Typer and I use the Education category in all .desktop files. For example: https://github.com/Open-Typer/Open-Typer/blob/master/res/linux-release/usr/share/applications/open-typer-appimage.desktop

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.

@Sussy-OS
Copy link
Contributor Author

Sussy-OS commented Aug 5, 2023

Any suggestions for which category this would fit best in?

I'm the developer of Open-Typer and I use the Education category in all .desktop files. For example: https://github.com/Open-Typer/Open-Typer/blob/master/res/linux-release/usr/share/applications/open-typer-appimage.desktop

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.

@theofficialgman
Copy link
Collaborator

theofficialgman commented Aug 5, 2023

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.
you have multiple files that have incorrect permissions btw (need executable bit set)

@Sussy-OS
Copy link
Contributor Author

Sussy-OS commented Aug 6, 2023

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. you have multiple files that have incorrect permissions btw (need executable bit set)

Will remember.

@Sussy-OS
Copy link
Contributor Author

Sussy-OS commented Aug 6, 2023

Are there any other issues? I have tested it, and it runs perfectly.

@theofficialgman
Copy link
Collaborator

theofficialgman commented Aug 6, 2023

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.

/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6Widgets.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6Quick.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6Gui.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6Qml.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6Core.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6Core.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libQt6OpenGL.so.6)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libglib-2.0.so.0)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libpng16.so.16)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libicui18n.so.66)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libicuuc.so.66)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libsystemd.so.0)
/tmp/.mount_Open-T8ijOB6/AppRun.wrapped: /lib/aarch64-linux-gnu/libc.so.6: version `GLIBC_2.30' not found (required by /tmp/.mount_Open-T8ijOB6/usr/bin/../lib/libsystemd.so.0)

@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 container: nameofcontainer at the same level as the runs-on: version. eg: container: ubuntu:18.04

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

@adazem009
Copy link

@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).

@adazem009
Copy link

@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.

@Botspot
Copy link
Owner

Botspot commented Oct 14, 2023

Reopening now that we have dropped Buster and Bionic support.

@Botspot Botspot reopened this Oct 14, 2023
@theofficialgman theofficialgman merged commit f24028d into Botspot:master Oct 22, 2023
2 checks passed
@theofficialgman
Copy link
Collaborator

theofficialgman commented Oct 22, 2023

@Sussy-OS thanks for the PR, it has been merged
@adazem009 thanks for the development of this application. One thing I will note about it is the displayed keyboard seems useless to me since it can't be interacted with (ubuntu gnome 22.04 x11 session with a device with a builtin touchscreen). I am sure that is intentional, it just looks like on on-screen intractable keyboard to me

@adazem009
Copy link

@Sussy-OS thanks for the PR, it has been merged
@adazem009 thanks for the development of this application. One thing I will note about it is the displayed keyboard seems useless to me since it can't be interacted with (ubuntu gnome 22.04 x11 session with a device with a builtin touchscreen). I am sure that is intentional, it just looks like on on-screen intractable keyboard to me

The application is designed specifically for the desktop to help people learn to type on a physical keyboard.
I don't think it would make sense to make it interactable because there are many different on-screen keyboards people use.

However, I could show the native keyboard on touch screen devices instead.

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.

4 participants