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

New package: abaddon-0.2.1 #46753

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

Conversation

rookiejet
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

New package

@classabbyamp classabbyamp added the new-package This PR adds a new package label Oct 17, 2023
@rookiejet rookiejet marked this pull request as draft October 17, 2023 16:25
@rookiejet rookiejet marked this pull request as ready for review October 17, 2023 16:55
@rookiejet rookiejet changed the title New package: abaddon-0.1.12 New package: abaddon-0.1.13 Nov 13, 2023
@rookiejet rookiejet changed the title New package: abaddon-0.1.13 New package: abaddon-0.1.14 Dec 8, 2023
@MIvanchev
Copy link
Contributor

I tested it as well, seems to work fine (glibc x86_64)

@rookiejet rookiejet changed the title New package: abaddon-0.1.14 New package: abaddon-0.2.0 Mar 28, 2024
@MIvanchev
Copy link
Contributor

MIvanchev commented Apr 9, 2024

Thank you for maintaining that package! Just 1 thing, abaddon doesn't currently install an icon so the entry in the .desktop file is wrong. Maybe it's better to remove the line.

Also you need a depends="desktop-file-utils" because of the desktop file and later on depends="desktop-file-utils hicolor-icon-theme" when there is an icon.

@rookiejet
Copy link
Contributor Author

rookiejet commented Apr 11, 2024

Thanks for the suggestions. Do you know if there is a failure case if the icon entry is included without an icon? It seems to validate fine. I'd prefer to leave it because I'd likely forget to add it later.

Can you also clarify why hicolor-icon-theme is required? My understanding is it is depended on if the app uses icons from the desktop icon theme for e.g. in its buttons, which abaddon does not do.

@rookiejet rookiejet changed the title New package: abaddon-0.2.0 New package: abaddon-0.2.1 Apr 11, 2024
@MIvanchev
Copy link
Contributor

Thanks for the suggestions. Do you know if there is a failure case if the icon entry is included without an icon? It seems to validate fine. I'd prefer to leave it because I'd likely forget to add it later.

Void doesn't validate the file at all. With my launcher (xlunch) there isn't any icon in this case (icon specified but not found) whereas xlunch puts a generic icon if there is no icon specified at all. So for me nothing is slightly better but different launchers likely handle this in their own manner.

Can you also clarify why hicolor-icon-theme is required? My understanding is it is depended on if the app uses icons from the desktop icon theme for e.g. in its buttons, which abaddon does not do.

IMO the package is required if you use something from /usr/share/icons/hicolor even if you put it there but I'm not 100% certain.

@rookiejet
Copy link
Contributor Author

I should clarify, I meant validate using desktop-file-validate, but I assume that is only checking syntax. Letting menus display a generic icon if they can is probably better, I will remove the icon line then. Thanks.

Copy link

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label Jul 12, 2024
Copy link

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants