[ISSUE] Suggestion - Split `helix-grammars` from `helix` package

[ISSUE] Suggestion - Split `helix-grammars` from `helix` package

[Zapeth]

[-- Attachment #1: Type: text/plain, Size: 744 bytes --]

New issue by Zapeth on void-packages repository

https://github.com/void-linux/void-packages/issues/45828

Description:
The `helix` package by default includes all its compiled treesitter grammars, which take up the majority of the package size (~110MB or  86% of the total size in the latest release, see also https://github.com/helix-editor/helix/issues/6187).

While it is useful to have LSP support for all these languages out of the box, I think it should be up to the user to decide if its worth the extra space, especially since its likely to increase as more languages get added.

So my suggestion would be to split the grammar *.so at `/usr/lib/helix/runtime/grammars` in an extra package `helix-grammars` that depends on `helix`.

Re: Suggestion - Split `helix-grammars` from `helix` package

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 263 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1699679097

Comment:
you could also [`noextract=/usr/lib/helix/runtime/grammars/*`](https://man.voidlinux.org/xbps.d#noextract=pattern)

Re: Suggestion - Split `helix-grammars` from `helix` package

[Zapeth]

[-- Attachment #1: Type: text/plain, Size: 467 bytes --]

New comment by Zapeth on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1699732408

Comment:
Thanks, I wasn't aware of that option. That should also allow a custom selection of grammars that are relevant to the user, so I'll use that for the time being.

I still think a package separation would be more convenient, but if the consensus is that a user configuration is sufficient then feel free to close this issue.

Re: Suggestion - Split `helix-grammars` from `helix` package

[Bnyro]

[-- Attachment #1: Type: text/plain, Size: 458 bytes --]

New comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1699759201

Comment:
I'd assume that most users expect Helix to support all the treesitters out of the box as described in the official Helix docs.

If this will be done, we should also add a note about it that shows up when installing the package, or instead make helix depend on helix-grammars, and not vice-versa as described above.

Re: Suggestion - Split `helix-grammars` from `helix` package

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 207 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1699679097

Comment:
you could also `noextract=/usr/lib/helix/runtime/grammars`

Re: Suggestion - Split `helix-grammars` from `helix` package

[mhmdanas]

[-- Attachment #1: Type: text/plain, Size: 405 bytes --]

New comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1701406236

Comment:
> I'd assume that most users expect Helix to support all the treesitters out of the box as described in the official Helix docs.

We could instead have `helix-minimal`, the core editor, and `helix`, which depends on both `helix-minimal` and `helix-grammars`.

Re: Suggestion - Split `helix-grammars` from `helix` package

[Bnyro]

[-- Attachment #1: Type: text/plain, Size: 173 bytes --]

New comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1701409746

Comment:
Cc maintainer @tranzystorek-io 

Re: Suggestion - Split `helix-grammars` from `helix` package

[tranzystorek-io]

[-- Attachment #1: Type: text/plain, Size: 279 bytes --]

New comment by tranzystorek-io on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1701432271

Comment:
I'd rather avoid making this package more complex than it needs to be, unless it achieves sth more than just size optimization.

Re: Suggestion - Split `helix-grammars` from `helix` package

[mhmdanas]

[-- Attachment #1: Type: text/plain, Size: 237 bytes --]

New comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1701436485

Comment:
That's fine with me, but a `vmove` in a new subpackage isn't really that complicated, is it?

Re: Suggestion - Split `helix-grammars` from `helix` package

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 188 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1701453336

Comment:
we don't need to subpackage everything 

Re: Suggestion - Split `helix-grammars` from `helix` package

[mhmdanas]

[-- Attachment #1: Type: text/plain, Size: 224 bytes --]

New comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1701604711

Comment:
Again, I don't object, I just disagree with the point that this is complicated.

Re: Suggestion - Split `helix-grammars` from `helix` package

[Zapeth]

[-- Attachment #1: Type: text/plain, Size: 208 bytes --]

New comment by Zapeth on void-packages repository

https://github.com/void-linux/void-packages/issues/45828#issuecomment-1702731779

Comment:
Then I'll close this, `noextract` is a good enough workaround me.

Re: [ISSUE] [CLOSED] Suggestion - Split `helix-grammars` from `helix` package

[Zapeth]

[-- Attachment #1: Type: text/plain, Size: 747 bytes --]

Closed issue by Zapeth on void-packages repository

https://github.com/void-linux/void-packages/issues/45828

Description:
The `helix` package by default includes all its compiled treesitter grammars, which take up the majority of the package size (~110MB or  86% of the total size in the latest release, see also https://github.com/helix-editor/helix/issues/6187).

While it is useful to have LSP support for all these languages out of the box, I think it should be up to the user to decide if its worth the extra space, especially since its likely to increase as more languages get added.

So my suggestion would be to split the grammar *.so at `/usr/lib/helix/runtime/grammars` in an extra package `helix-grammars` that depends on `helix`.