Merge pull request #559 from cpanel/RE-980

Prefer using apt-get over apt in Elevate::PkgMgr
cpanel · Nov 25, 2024 · f2db87d · f2db87d
2 parents 3e6d355 + 1389f1a
commit f2db87d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/elevate-cpanel b/elevate-cpanel
@@ -7416,7 +7416,6 @@ deb https://wp-toolkit.plesk.com/cPanel/Ubuntu-22.04-x86_64/latest/thirdparty/ .
       -o Dpkg::Options::=--force-confold
     };
 
-    our $apt        = '/usr/bin/apt';
     our $apt_get    = '/usr/bin/apt-get';
     our $apt_mark   = '/usr/bin/apt-mark';
     our $dpkg       = '/usr/bin/dpkg';
@@ -7538,10 +7537,11 @@ deb https://wp-toolkit.plesk.com/cPanel/Ubuntu-22.04-x86_64/latest/thirdparty/ .
 
         my @apt_args = (
             '-y',
+            '--with-new-pkgs',
             APT_NON_INTERACTIVE_ARGS,
         );
 
-        $self->ssystem_and_die( $apt, @apt_args, 'upgrade' );
+        $self->ssystem_and_die( $apt_get, @apt_args, 'upgrade' );
 
         return;
     }
@@ -7551,13 +7551,13 @@ deb https://wp-toolkit.plesk.com/cPanel/Ubuntu-22.04-x86_64/latest/thirdparty/ .
     }
 
     sub update_allow_erasing ( $self, @additional_args ) {
-        $self->ssystem_and_die( $apt, '-y', 'autoremove', '--purge' );
+        $self->ssystem_and_die( $apt_get, '-y', 'autoremove', '--purge' );
         $self->update();
         return;
     }
 
     sub makecache ($self) {
-        my $out    = $self->ssystem_capture_output( $apt, 'update' );
+        my $out    = $self->ssystem_capture_output( $apt_get, 'update' );
         my @errors = grep { $_ !~ m/apt does not have a stable CLI interface/ } @{ $out->{stderr} };
         my $stderr = join "\n", @errors;
         return $stderr;

diff --git a/lib/Elevate/PkgMgr/APT.pm b/lib/Elevate/PkgMgr/APT.pm
@@ -25,7 +25,6 @@ use constant APT_NON_INTERACTIVE_ARGS => qw{
   -o Dpkg::Options::=--force-confold
 };
 
-our $apt        = '/usr/bin/apt';
 our $apt_get    = '/usr/bin/apt-get';
 our $apt_mark   = '/usr/bin/apt-mark';
 our $dpkg       = '/usr/bin/dpkg';
@@ -197,10 +196,11 @@ sub update ($self) {
 
     my @apt_args = (
         '-y',
+        '--with-new-pkgs',
         APT_NON_INTERACTIVE_ARGS,
     );
 
-    $self->ssystem_and_die( $apt, @apt_args, 'upgrade' );
+    $self->ssystem_and_die( $apt_get, @apt_args, 'upgrade' );
 
     return;
 }
@@ -218,13 +218,13 @@ sub update_with_options ( $self, $options, $pkgs ) {
 }
 
 sub update_allow_erasing ( $self, @additional_args ) {
-    $self->ssystem_and_die( $apt, '-y', 'autoremove', '--purge' );
+    $self->ssystem_and_die( $apt_get, '-y', 'autoremove', '--purge' );
     $self->update();
     return;
 }
 
 sub makecache ($self) {
-    my $out    = $self->ssystem_capture_output( $apt, 'update' );
+    my $out    = $self->ssystem_capture_output( $apt_get, 'update' );
     my @errors = grep { $_ !~ m/apt does not have a stable CLI interface/ } @{ $out->{stderr} };
     my $stderr = join "\n", @errors;
     return $stderr;