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

feat(ci): Add RPM package; bump actions; clean unused step; add manual trigger #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xlionjuan
Copy link

Summary

  • Add RPM package
  • Add workflow_dispatch: for manually trigger
  • Bump actions/checkout to v4
  • Bump actions/setup-node to v4

Test

  • Able to start the program on Distrobox with Fedora 41 container

Note: Install software to /opt is a bad ideas, this place should install softwares that are not managed by package managers, though it won't cause problems most of times, but will make this software not installable on Fedora Atomic based systems(It should works, but installing to /opt continuously causing problems), though the more recommended way to install GUI softwares is using Flatpak, but having more options are always great.

But....I can't find any documentation about changing install path for the electron-builder, so maybe just focused on Flatpak.

Related:
https://discussion.fedoraproject.org/t/will-semanage-fcontext-and-restorecon-safe-to-run-when-selinux-disabled-during-atomic-custom-image-build-and-others/139943

Suggestion for Auto startup on Linux

Check what 地牛 Wake Up! did

cat ~/.config/autostart/wake_up.appimage.desktop

[Desktop Entry]
Type=Application
Version=1.0
Name=wake_up.appimage
Comment=wake_up.appimagestartup script
Exec=/var/home/xlion/AppImages/wake_up.appimage --hidden
StartupNotify=false
Terminal=false% 

Same method will also works on both flatpak packages or installing to host systems, just ensure the path is right.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant