Skip to content

Conversation

@KwadFan
Copy link
Contributor

@KwadFan KwadFan commented Oct 19, 2021

  • PreInstall Module provides a 'apt full-upgrade' to bring latest updates to build-Image
  • Also, it can be configured to install all dependent packages for your Image.
    See preinstall/config for configuration
    by default only upgrade and cleanup are performed.

This PR also includes PR #146 .
You have the choice, this PR is not independent because it reuses "apt_update_skip" function from PR #146.

I would be happy if you merge this one.

Regards

* check_install_pkgs - check if pkgs is installed if not installs it.
* apt_update_skip - Skips 'apt update' if cache not older than one Hour
* PreInstall Module provides a 'apt full-upgrade' to bring latest updates to build-Image
* Also, it can be configured to install all dependent packages for your Image.
See preinstall/config for configuration
by default only upgrade and cleanup are performed.
Forgot to set missing_pkgs Variable to an Array
@guysoft
Copy link
Owner

guysoft commented Oct 20, 2021

I would call it apt-upgrade or pkg-upgrade even apt-install module or something like that.
The name pre-install is too general.

@KwadFan
Copy link
Contributor Author

KwadFan commented Oct 20, 2021

I think thats really a better option. So I called it 'apt-install', what do you think?

@KwadFan KwadFan changed the title Pre install Apt-Install Oct 22, 2021
@KwadFan
Copy link
Contributor Author

KwadFan commented Oct 23, 2021

after renaming it doesnt work anymore, so closed for now.

@KwadFan KwadFan closed this Oct 23, 2021
@guysoft
Copy link
Owner

guysoft commented Oct 23, 2021

Ok, any idea why its not working? It should work after such a change.

@KwadFan
Copy link
Contributor Author

KwadFan commented Oct 23, 2021

Indeed, it seems that even trough globbing prevention ${foo}, the APT in that Var gets lost and then all according Vars arent complete. So, I fixed that by renaming to pkgupgrade. Also, however the - leads to Errors what it shouldn't do.

I will open a new PR, including a small improvment to #146, the Error Message if found and installed is misleading.
Hope I get that done ASAP. Maybe tommorow morning.

Regards

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.

2 participants