summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--usr.sbin/pkg_add/OpenBSD/Add.pm6
-rw-r--r--usr.sbin/pkg_add/OpenBSD/ArcCheck.pm6
-rw-r--r--usr.sbin/pkg_add/OpenBSD/PackageRepository.pm4
-rw-r--r--usr.sbin/pkg_add/OpenBSD/PackingElement.pm17
-rw-r--r--usr.sbin/pkg_add/OpenBSD/PackingList.pm4
-rw-r--r--usr.sbin/pkg_add/OpenBSD/PkgCfl.pm4
-rw-r--r--usr.sbin/pkg_add/OpenBSD/Ustar.pm20
-rw-r--r--usr.sbin/pkg_add/pkg6
8 files changed, 36 insertions, 31 deletions
diff --git a/usr.sbin/pkg_add/OpenBSD/Add.pm b/usr.sbin/pkg_add/OpenBSD/Add.pm
index 99ca49ef26b..a2885f3e54b 100644
--- a/usr.sbin/pkg_add/OpenBSD/Add.pm
+++ b/usr.sbin/pkg_add/OpenBSD/Add.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: Add.pm,v 1.76 2007/06/09 13:39:31 espie Exp $
+# $OpenBSD: Add.pm,v 1.77 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org>
#
@@ -662,7 +662,9 @@ sub copy_info
my ($self, $dest) = @_;
require File::Copy;
- File::Copy::move($self->fullname, $dest);
+ File::Copy::move($self->fullname, $dest) or
+ print STDERR "Problem while moving ", $self->fullname,
+ " into $dest: $!\n";
}
package OpenBSD::PackingElement::FINSTALL;
diff --git a/usr.sbin/pkg_add/OpenBSD/ArcCheck.pm b/usr.sbin/pkg_add/OpenBSD/ArcCheck.pm
index eddf5c28f11..cf73f6dca00 100644
--- a/usr.sbin/pkg_add/OpenBSD/ArcCheck.pm
+++ b/usr.sbin/pkg_add/OpenBSD/ArcCheck.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: ArcCheck.pm,v 1.9 2007/06/04 14:40:39 espie Exp $
+# $OpenBSD: ArcCheck.pm,v 1.10 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2005-2006 Marc Espie <espie@openbsd.org>
#
@@ -117,10 +117,10 @@ sub prepare_long
my $filename = $item->{name};
my $entry = $self->prepare($filename);
if (!defined $entry->{uname}) {
- die "No user name for ", $entry->{name}, " (uid ", $entry->{uid}, ")\n";
+ die "No user name for ", $entry->{name}, " (uid ", $entry->{uid}, ")";
}
if (!defined $entry->{gname}) {
- die "No group name for ", $entry->{name}, " (gid ", $entry->{gid}. "\n";
+ die "No group name for ", $entry->{name}, " (gid ", $entry->{gid}. ")";
}
my ($prefix, $name) = split_name($entry->{name});
if (length($name) > MAXFILENAME || length($prefix) > MAXPREFIX) {
diff --git a/usr.sbin/pkg_add/OpenBSD/PackageRepository.pm b/usr.sbin/pkg_add/OpenBSD/PackageRepository.pm
index 1bb91809786..9aea21681a2 100644
--- a/usr.sbin/pkg_add/OpenBSD/PackageRepository.pm
+++ b/usr.sbin/pkg_add/OpenBSD/PackageRepository.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: PackageRepository.pm,v 1.44 2007/06/10 14:25:18 espie Exp $
+# $OpenBSD: PackageRepository.pm,v 1.45 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org>
#
@@ -326,7 +326,7 @@ sub pkg_copy
do {
$n = sysread($in, $buffer, $buffsize);
if (!defined $n) {
- die "Error reading\n";
+ die "Error reading: $!";
}
if ($n > 0) {
$nonempty = 1;
diff --git a/usr.sbin/pkg_add/OpenBSD/PackingElement.pm b/usr.sbin/pkg_add/OpenBSD/PackingElement.pm
index fdf55b3e006..c09de9eb3f5 100644
--- a/usr.sbin/pkg_add/OpenBSD/PackingElement.pm
+++ b/usr.sbin/pkg_add/OpenBSD/PackingElement.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: PackingElement.pm,v 1.133 2007/06/06 12:32:09 espie Exp $
+# $OpenBSD: PackingElement.pm,v 1.134 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org>
#
@@ -1327,12 +1327,15 @@ sub prepare
{
my ($self, $state) = @_;
my $fname = $self->fullname;
- open(my $src, '<', $fname) or Warn "Can't open $fname: $!";
- while (<$src>) {
- next if m/^\+\-+\s*$/o;
- s/^[+-] //o;
- $state->print($_);
- }
+ if (open(my $src, '<', $fname)) {
+ while (<$src>) {
+ next if m/^\+\-+\s*$/o;
+ s/^[+-] //o;
+ $state->print($_);
+ }
+ } else {
+ Warn "Can't open $fname: $!\n";
+ }
}
package OpenBSD::PackingElement::FDISPLAY;
diff --git a/usr.sbin/pkg_add/OpenBSD/PackingList.pm b/usr.sbin/pkg_add/OpenBSD/PackingList.pm
index cde7c0be3db..e4d29a8ce06 100644
--- a/usr.sbin/pkg_add/OpenBSD/PackingList.pm
+++ b/usr.sbin/pkg_add/OpenBSD/PackingList.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: PackingList.pm,v 1.77 2007/06/06 12:32:09 espie Exp $
+# $OpenBSD: PackingList.pm,v 1.78 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org>
#
@@ -320,7 +320,7 @@ sub addunique
my ($plist, $object) = @_;
my $category = $object->category;
if (defined $plist->{$category}) {
- die "Duplicate $category in plist\n";
+ die "Duplicate $category in plist";
}
$plist->{$category} = $object;
}
diff --git a/usr.sbin/pkg_add/OpenBSD/PkgCfl.pm b/usr.sbin/pkg_add/OpenBSD/PkgCfl.pm
index 36cf357524a..c60762f2fd0 100644
--- a/usr.sbin/pkg_add/OpenBSD/PkgCfl.pm
+++ b/usr.sbin/pkg_add/OpenBSD/PkgCfl.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: PkgCfl.pm,v 1.23 2007/05/29 10:40:20 espie Exp $
+# $OpenBSD: PkgCfl.pm,v 1.24 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2003-2005 Marc Espie <espie@openbsd.org>
#
@@ -100,7 +100,7 @@ sub find($$)
while (my ($name, $l) = each %{$state->{conflict_list}}) {
next if $name eq $pkgname;
if (!defined $l) {
- die "Error: $name has no definition\n";
+ die "Error: $name has no definition";
}
if ($l->conflicts_with($pkgname)) {
push(@bad, $name);
diff --git a/usr.sbin/pkg_add/OpenBSD/Ustar.pm b/usr.sbin/pkg_add/OpenBSD/Ustar.pm
index a11c58399b1..5f7e8167815 100644
--- a/usr.sbin/pkg_add/OpenBSD/Ustar.pm
+++ b/usr.sbin/pkg_add/OpenBSD/Ustar.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: Ustar.pm,v 1.48 2007/06/04 14:40:39 espie Exp $
+# $OpenBSD: Ustar.pm,v 1.49 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2002-2007 Marc Espie <espie@openbsd.org>
#
@@ -73,10 +73,10 @@ sub skip
}
my $actual = read($self->{fh}, $temp, $toread);
if (!defined $actual) {
- die "Error while skipping archive: $!\n";
+ die "Error while skipping archive: $!";
}
if ($actual == 0) {
- die "Premature end of archive in header: $!\n";
+ die "Premature end of archive in header: $!";
}
$self->{swallow} -= $actual;
}
@@ -222,19 +222,19 @@ sub mkheader
$linkname = '';
}
if (length $prefix > MAXPREFIX) {
- die "Prefix too long $prefix\n";
+ die "Prefix too long $prefix";
}
if (length $name > MAXFILENAME) {
- die "Name too long $name\n";
+ die "Name too long $name";
}
if (length $linkname > MAXLINKNAME) {
- die "Linkname too long $linkname\n";
+ die "Linkname too long $linkname";
}
if (length $uname > MAXUSERNAME) {
- die "Username too long $uname\n";
+ die "Username too long $uname";
}
if (length $gname > MAXGROUPNAME) {
- die "Groupname too long $gname\n";
+ die "Groupname too long $gname";
}
my $header;
my $cksum = ' 'x8;
@@ -343,7 +343,7 @@ sub new
my ($class, $object) = @_;
if ($object->{size} != 0) {
- die "Bad archive: non null size for arbitrary entry\n";
+ die "Bad archive: non null size for arbitrary entry";
}
bless $object, $class;
}
@@ -462,7 +462,7 @@ sub resolve_links
$self->{linkname} = $arc->{key}->{$k};
} else {
print join("\n", keys(%{$arc->{key}})), "\n";
- die "Can't copy link over: original for $k NOT available\n";
+ die "Can't copy link over: original for $k NOT available";
}
}
diff --git a/usr.sbin/pkg_add/pkg b/usr.sbin/pkg_add/pkg
index 74efca62cf1..82e21372f91 100644
--- a/usr.sbin/pkg_add/pkg
+++ b/usr.sbin/pkg_add/pkg
@@ -1,6 +1,6 @@
#! /usr/bin/perl
# ex:ts=8 sw=4:
-# $OpenBSD: pkg,v 1.8 2007/05/14 11:02:14 espie Exp $
+# $OpenBSD: pkg,v 1.9 2007/06/12 09:53:36 espie Exp $
#
# Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org>
#
@@ -34,7 +34,7 @@ sub check_dependencies($)
my %opts;
my %legacy = map +($_, 1), qw{add info delete create};
if (@ARGV == 0) {
- die "needs arguments\n";
+ die "needs arguments";
}
getopts('v', \%opts);
@@ -56,4 +56,4 @@ if (defined $legacy{$cmd}) {
}
}
}
-die "Bad command $cmd\n";
+die "Bad command $cmd";