Skip to content

Added workflow for AUR, DEB, and RPM packages #7

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

SimplyCEO
Copy link

Changed a bunch of directory structure, may need to tweak things.

Changed a bunch of directory structure, may need to tweak things.
@SimplyCEO
Copy link
Author

It is not ready for merge yet.

AUR build is complete.
DEB build is half complete, it looks different from the one provided by the repository.
RPM is broken.

There are inconsistencies related to file structure. Do I need to keep /opt for both Shell and PNG files or use the hicolor directory? It differs from RPM to the others.

@SimplyCEO SimplyCEO changed the title Added workflow for DEB and RPM packages Added workflow for AUR, DEB, and RPM packages Jun 5, 2025
@psygreg
Copy link
Owner

psygreg commented Jun 5, 2025

The confusion seems to stem from the fact each package is installed in different places on each distro. That happens because of the RPM package being more restrictive about where to put files.

  • DEB version: places linuxtoys.sh and linuxtoys.png in /opt/linuxtoys
  • RPM version: places linuxtoys.sh in /usr/bin and linuxtoys.png in the hicolor directory
  • Arch version: places both in /usr/bin.

linuxtoys.desktop is placed always in /usr/share/applications.

That's a bit of an unfortunate inheritance from the early versions - from when I wasn't that familiar with packaging - that we will have to live with.

@SimplyCEO
Copy link
Author

Is the packages folder really needed? After this workflow they may generate automatically.

@psygreg
Copy link
Owner

psygreg commented Jun 7, 2025

Not really, you may delete it with the PR if you want.

@psygreg
Copy link
Owner

psygreg commented Jun 7, 2025

I have been adding quite a bit of stuff, but nothing related to package building or the main linuxtoys.sh module so I think I shouldn't be getting on your way much. The only change I did was adding a line to linuxtoys.spec to get it to build on COPR.

@SimplyCEO
Copy link
Author

No problem. I have been busy for some days, but I will keep the work.

@SimplyCEO
Copy link
Author

I have tested and it works. There is a problem though.

BuildRequires: desktop-file-utils
This section only works if it is commented out, even having desktop-file-utils installed. Do you have any idea on how to manipulate this bug?

@psygreg
Copy link
Owner

psygreg commented Jun 16, 2025

Never seen that problem before. Does it build the package correctly without that line?
What if you leave BuildRequires empty?

@SimplyCEO
Copy link
Author

Does it build the package correctly without that line?

It does, strangely.

@SimplyCEO
Copy link
Author

There are packages available at https://github.com/SimplyCEO/linuxtoys/actions/runs/15656299420 action, where I was testing the best way to improve the workflow. Please check if they suit your needs.

@psygreg
Copy link
Owner

psygreg commented Jun 16, 2025

Looks almost perfect, just gotta sync to current version just for safety I think. Also, would it be possible for it to make SRPMS too? If it isn't I can build those manually, but would be nice if it could. On terminal, it can be achieved by sending the -ba flag instead of -bb to rpmbuild if that helps

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.

2 participants