summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarc Espie <espie@cvs.openbsd.org>2019-11-16 11:07:44 +0000
committerMarc Espie <espie@cvs.openbsd.org>2019-11-16 11:07:44 +0000
commit91b8623346031b768979a8e46b66c8a0f614ea9f (patch)
treefe587313fed290cbd741ca2612a1a1d59afe1ab7
parente49aa00647fe3184f020ccf9eff596c7741afc9f (diff)
preliminary addition of -d option (semantics: add debug packages if
they're present along the packages we want to add/update) probably not perfect yet, but 100% a no-op if it's not used, so won't break existing usage.
-rw-r--r--usr.sbin/pkg_add/OpenBSD/PkgAdd.pm7
-rw-r--r--usr.sbin/pkg_add/OpenBSD/Update.pm34
2 files changed, 33 insertions, 8 deletions
diff --git a/usr.sbin/pkg_add/OpenBSD/PkgAdd.pm b/usr.sbin/pkg_add/OpenBSD/PkgAdd.pm
index 7a0e78db4c0..8959a49f821 100644
--- a/usr.sbin/pkg_add/OpenBSD/PkgAdd.pm
+++ b/usr.sbin/pkg_add/OpenBSD/PkgAdd.pm
@@ -1,7 +1,7 @@
#! /usr/bin/perl
# ex:ts=8 sw=4:
-# $OpenBSD: PkgAdd.pm,v 1.115 2019/08/19 12:25:40 espie Exp $
+# $OpenBSD: PkgAdd.pm,v 1.116 2019/11/16 11:07:43 espie Exp $
#
# Copyright (c) 2003-2014 Marc Espie <espie@openbsd.org>
#
@@ -112,8 +112,8 @@ our @ISA = qw(OpenBSD::AddDelete::State);
sub handle_options
{
my $state = shift;
- $state->SUPER::handle_options('ruUzl:A:P:',
- '[-acinqrsUuVvxz] [-A arch] [-B pkg-destdir] [-D name[=value]]',
+ $state->SUPER::handle_options('druUzl:A:P:',
+ '[-adcinqrsUuVvxz] [-A arch] [-B pkg-destdir] [-D name[=value]]',
'[-L localbase] [-l file] [-P type] pkg-name ...');
$state->{arch} = $state->opt('A');
@@ -133,6 +133,7 @@ sub handle_options
$state->{pkglist} = $state->opt('l');
$state->{update} = $state->opt('u');
$state->{fuzzy} = $state->opt('z');
+ $state->{debug_packages} = $state->opt('d');
if ($state->defines('snapshot')) {
$state->{subst}->add('snap', 1);
}
diff --git a/usr.sbin/pkg_add/OpenBSD/Update.pm b/usr.sbin/pkg_add/OpenBSD/Update.pm
index 00ff61dd009..baa661a92cd 100644
--- a/usr.sbin/pkg_add/OpenBSD/Update.pm
+++ b/usr.sbin/pkg_add/OpenBSD/Update.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: Update.pm,v 1.164 2019/05/08 13:04:27 espie Exp $
+# $OpenBSD: Update.pm,v 1.165 2019/11/16 11:07:43 espie Exp $
#
# Copyright (c) 2004-2014 Marc Espie <espie@openbsd.org>
#
@@ -68,6 +68,30 @@ sub add_location
OpenBSD::Handle->from_location($location));
}
+sub look_for_debug
+{
+ my ($self, $set, $oldname, $newname, $state) = @_;
+
+ # hurdles to pass before adding debug packages
+ return unless $state->{debug_packages};
+
+ return if $state->tracker->is_to_update("debug-".$oldname);
+ my $dbg = "debug-".$newname;
+ my $l = $set->match_locations(OpenBSD::Search::Exact->new($dbg));
+ # TODO if @$l == 0, I should look for other packages with similar names
+ # just so I can warn for out-of-date/shearing in the mirrors.
+ return if @$l != 1;
+ $set->add_newer(OpenBSD::Handle->from_location($l->[0]));
+}
+
+sub found_update
+{
+ my ($self, $set, $old, $location, $state) = @_;
+
+ $self->add_location($set, $old, $location);
+ $self->look_for_debug($set, $old->pkgname, $location->name, $state);
+}
+
sub progress_message
{
my ($self, $state, @r) = @_;
@@ -207,7 +231,7 @@ sub process_handle
$set->move_kept($h);
$self->progress_message($state,
"No need to update #1", $pkgname);
-
+ $self->look_for_debug($set, $pkgname, $pkgname, $state);
return 0;
}
return undef;
@@ -218,7 +242,7 @@ sub process_handle
my $r = $state->choose_location($pkgname, $l);
if (defined $r) {
- $self->add_location($set, $h, $r);
+ $self->found_update($set, $h, $r, $state);
return 1;
} else {
$state->{issues} = 1;
@@ -285,13 +309,13 @@ sub process_hint
if (@$l > 1) {
my $r = find_nearest($hint_name, $l);
if (defined $r) {
- $self->add_location($set, $hint, $r);
+ $self->found_update($set, $hint, $r, $state);
return 1;
}
}
my $r = $state->choose_location($hint_name, $l);
if (defined $r) {
- $self->add_location($set, $hint, $r);
+ $self->found_update($set, $hint, $r, $state);
OpenBSD::Add::tag_user_packages($set);
return 1;
} else {