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 missing deps for armbian-config #7562

Merged
merged 2 commits into from
Dec 11, 2024

Conversation

dimitry-ishenko
Copy link
Contributor

Description

armbian-config uses debconf-apt-progress which is part of debconf, so it should be listed as a dependency, and missing libtext-iconv-perl causes an error as described in armbian/configng#272 (comment).

How Has This Been Tested?

  • debconf is obvious. It's also pulled in by a number of other packages, so it hasn't come up as a problem. But, since armbian-config uses it directly, it should specify it as a direct dependency.

  • libtext-iconv-perl is a bit less obvious. It's probably also pulled in by other packages on most systems, so it hasn't been caught before. But, on a minimal Armbian image where it's not present, I was getting the error mentioned above. Manually installing this package handled the error.

Checklist:

Please delete options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

armbian-config uses debconf-apt-progress which is part of debconf.

libtext-iconv-perl is needed to prevent the following error, when
uninstalling a package through debconf-apt-progress:

Can't locate object method "new" via package "Text::Iconv" (perhaps you forgot to load "Text::Iconv"?) at /usr/share/perl5/Debconf/Encoding.pm line 65, <GEN2> line 2.
dpkg: error processing package nfs-common (--configure):
@dimitry-ishenko dimitry-ishenko requested a review from a team as a code owner December 11, 2024 04:07
@github-actions github-actions bot added size/small PR with less then 50 lines Needs review Seeking for review Framework Framework components labels Dec 11, 2024
Copy link
Member

@igorpecovnik igorpecovnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree. Made a new task related to this https://armbian.atlassian.net/browse/AR-2562

@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label Dec 11, 2024
@igorpecovnik igorpecovnik merged commit 6d63fbd into armbian:main Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Framework components Needs review Seeking for review Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants