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

Drop support for package_errors.log + EXIT_ON_MISSING_PACKAGES #304

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

mika
Copy link
Member

@mika mika commented Jan 30, 2025

This was unreliable already within FAI, esp. when aborting and rerunning a build.

Now with the switch from FAI to our minifai, we no longer support EXIT_ON_MISSING_PACKAGES. Minifai operates per class and aborts on any package installation that fails. There's no point in generating package_errors.log and TEST-MissingPackages.xml (meant for inclusion in e.g. Jenkins jobs), so drop config/scripts/GRMLBASE/01-packages overall, and related code from grml-live and setting in etc/grml/grml-live.conf.

Closes: #295

@mika mika requested a review from zeha January 30, 2025 16:47
This was unreliable already within FAI, esp. when aborting and rerunning
a build.

Now with the switch from FAI to our minifai, we no longer support
EXIT_ON_MISSING_PACKAGES. Minifai operates per class and aborts on any
package installation that fails. There's no point in generating
package_errors.log and TEST-MissingPackages.xml (meant for inclusion in
e.g. Jenkins jobs), so drop config/scripts/GRMLBASE/01-packages overall,
and related code from grml-live and setting in etc/grml/grml-live.conf.

While at it, drop unused $VERBOSE, as complained by shellcheck.

Closes: #295
@mika mika force-pushed the mika/package_errors branch from 1d7072f to ca86de6 Compare January 30, 2025 16:51
@mika mika merged commit a9a497d into master Jan 30, 2025
12 checks passed
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.

Consider removing package_errors.log
2 participants