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

Proposal to add wget to our included packages #133

Closed
phillxnet opened this issue Jan 30, 2023 · 2 comments · Fixed by #136
Closed

Proposal to add wget to our included packages #133

phillxnet opened this issue Jan 30, 2023 · 2 comments · Fixed by #136

Comments

@phillxnet
Copy link
Member

Currently, for development purposes, the only package not pre-installed in the resulting image/install, either via explicit kiwi-ng config or via the 'rockstor' package's dependencies is wget. Given we currently already pre-install such developer focused nice-to-haves as nano, it is proposed that we install this final package 'hurdle' by way of a further additional explicit inclusion in the installer config. It does not represent a package dependency as such given there is no non-developer orientated requirement function wise within the package. Hence the suggestion we add it as a convenience as per our nano inclusion.

This is in-keeping with easing user on-boarding regarding development involvement.

@FroggyFlox
Copy link
Member

Thanks for that one, @phillxnet !
I actually started creating that very issue the other day but changed my mind. I'm still undecided but we might as well include it indeed.

@phillxnet
Copy link
Member Author

@FroggyFlox Yes, I was back and forth on this when last cleaning up the installer package list, and updating the package dependencies. But it's now one straggler and nano + friends (i.e. htop) are already included in the installer.

I think the numb of it is that we are predominantly DIY friendly, and thus have system tools a-plenty. This way at least folks can move to a development setup, from a pre-build installer, with zero additional packages which is neat.

phillxnet added a commit to phillxnet/rockstor-installer that referenced this issue Mar 9, 2023
- Add wget rockstor#133.
- Additional kernel-firmware packages rockstor#134.
- Add discreet gzip, gawk, grep, xz to bootstrap.
This way we avoid the busybox variants being
favoured, via dependency, and causing complexities
with install of such things as rpm-build for
self-hosting our own rpm build.

## Indicentally includes:
- Minor typo on a Tumbleweed Pi4 profile.
- Bump to lastest rockstor package version.
phillxnet added a commit that referenced this issue Mar 9, 2023
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 a pull request may close this issue.

2 participants