https://bugzilla.redhat.com/show_bug.cgi?id=2246561
--- Comment #18 from blinxen h-k-81@hotmail.com ---
PS: I might ask somebody else for a second opinion later since this is a rather complicated package with many components, and I'm not sure I'm getting 100% of the things right here.
Sure, no problem :D
It's OK in the short-term, but please switch back to upstream sources as soon as the projects merge your PRs.
I am already doing that :D I am also creating PRs [1] for upstream to use the newest version that includes the license commits.
- Looks like there's a file "/usr/share/licenses/helix/LICENSE-tree-sitter-." getting installed, this looks like a mistake?
Should be fixed now.
- It seems that the grammar plugins that are getting compiled with "cc" link libm and libgcc_s even if they don't have to:
Will report this upstream.
- There appear to be some files that have executable permissions that should not have them:
Should be fixed now
- These shell completion scripts have shebangs but don't need them (I think)?
I can report this upstream but it seems that this is not required but does not really do any damage if kept.
- Inconsistent indentation: The tags from Source100 up to the Provides for bundled tree-sitter grammars are indented inconsistently from the rest of the spec file, which uses 16 columns of indentation. For the bundled Provides, you can just tweak the command that you used to generate them (i.e. add the appropriate amount of spaces in the command at the right position).
Should be fixed now
- You install a .desktop file for Helix, but it is not validated, which is a mandatory requirement.
Desktop file validation has been added
- The other steps for preparing license files for installation all happen in %prep, but this one happens in %install, even though it does not *install* anything, just moves things around in the build directory:
Makes sense
https://github.com/helix-editor/helix/pull/8691