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

Require awk for use in Startup Scripts #1096

Merged
merged 1 commit into from Aug 15, 2023
Merged

Require awk for use in Startup Scripts #1096

merged 1 commit into from Aug 15, 2023

Conversation

shundhammer
Copy link
Contributor

Bugzilla

https://bugzilla.suse.com/show_bug.cgi?id=1214277

Trello

https://trello.com/c/bgfX60e4/

Problem

If awk is not installed, there are error messages on the console during a firstboot workflow.

Fix

Require awk in the spec file.

Explicitly using awk and not gawk (the GNU version of awk) so lighterweight alternatives like busybox can be used if they provide the awk symbol (which busybox currently does not).

What about other Common Linux Tools?

Many are part of the coreutils package which we already require. Others are so common that a Linux system has little chance of working when they are missing, such as grep or sed: A lot of other packages depend on them.

See also the discussion in the bug.

@coveralls
Copy link

Coverage Status

coverage: 40.807%. remained the same when pulling 256c69b on huha-awk into ea39984 on master.

package/yast2-installation.spec Show resolved Hide resolved
@shundhammer shundhammer merged commit d49fbda into master Aug 15, 2023
16 checks passed
@shundhammer shundhammer deleted the huha-awk branch August 15, 2023 09:14
@yast-bot
Copy link
Contributor

✔️ Public Jenkins job #257 successfully finished
✔️ Created OBS submit request #1104012

@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #147 successfully finished
✔️ Created IBS submit request #305523

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.

None yet

4 participants