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

Add a sysext for vim-huge #78

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

Add a sysext for vim-huge #78

wants to merge 2 commits into from

Conversation

jsalatiel
Copy link
Contributor

There are several cases when the user needs to edit k8s yamls inplace directly on the master nodes and having syntax highlighting and advanced formatting of vim available is a nice feature since the default vim of flatcar have those disabled.

There are several cases when the user needs to edit k8s yamls inplace directly on the master nodes and having syntax highlighting and advanced formatting of vim available is a nice feature since the default vim of flatcar have those disabled.
@chewi
Copy link

chewi commented Jun 10, 2024

We have just discussed whether we might want to leverage existing Gentoo packages for this instead. We could potentially do this in such a way that avoids dependence on system libraries (besides glibc) either by still statically linking or by placing the dynamic libraries within the sysext. I could look into this soon, but I don't mind accepting this in the interim.

@jsalatiel
Copy link
Contributor Author

Hi @chewi , will this be accepted?

@chewi
Copy link

chewi commented Aug 1, 2024

I have no objections, but I'd want someone else to look over it. I'm not familiar with sysext building yet.

@pothos
Copy link
Member

pothos commented Aug 25, 2024

FYI, the build process of the extensions in this repo is supposed to be fast, avoiding compilation if we can (that's the reason one build recipe was not included in the official repo releases so far).
One can bundle existing distro binaries (e.g., from a Docker image) with one of these helper scripts:
https://github.com/flatcar/sysext-bakery?tab=readme-ov-file#flix-and-flatwrap - for choosing one way and making it work one needs to know what paths vim relies on at runtime.

That said, I'm not reviewing things here now and someone else has to decide what to do with this PR.

@xinity
Copy link

xinity commented Sep 5, 2024

IMHO sysext should not be considered as the next package manager, so perhaps vim as an editor that exists in every linux distribution might not fit here.

just sharing my point of view, no harm :)

@mnbro
Copy link

mnbro commented Nov 26, 2024

@pothos @xinity Do you have a resolution for this? Would @jsalatiel PR be accepted?

@xinity
Copy link

xinity commented Nov 26, 2024

@pothos @xinity Do you have a resolution for this? Would @jsalatiel PR be accepted?

well i'm not a sysext-bakery maintainer so my opinion worth almost nothing :)
if you ask me, sysext-bakery should be targeting complex setup and have a functional purpose, or else it will just end up as another packager manager, which i'm sure wasn't why it has been built for.

TLDR: before accepting or rejecting any PR , perhaps maintainers should think about guidelines, clear targets and expections.

Again i'm nobody to tell anyone what they should do, i'm just sharing my very personal point of view.
@pothos your thoughts ?

@chewi
Copy link

chewi commented Nov 27, 2024

@pothos is away for an extended period. Unfortunately, I still don't know any more about this than I did in August. @tormath1, does this look okay to you?

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.

5 participants