From: Don Armstrong <don@donarmstrong.com>
Date: Sun, 2 Jul 2017 18:43:24 +0000 (-0700)
Subject: fix failure to import locking funcs and don't save empty spamlogs
X-Git-Tag: release/2.6.0~112
X-Git-Url: https://git.donarmstrong.com/?p=debbugs.git;a=commitdiff_plain;h=fdec16f10fb4c0cb7e508e2eb709c8869172aadf

fix failure to import locking funcs and don't save empty spamlogs
---

diff --git a/Debbugs/Log/Spam.pm b/Debbugs/Log/Spam.pm
index cca94de..df2d5e0 100644
--- a/Debbugs/Log/Spam.pm
+++ b/Debbugs/Log/Spam.pm
@@ -45,7 +45,7 @@ BEGIN{
 use Carp;
 use feature 'state';
 use Params::Validate qw(:types validate_with);
-use Debbugs::Common qw(getbuglocation getbugcomponent);
+use Debbugs::Common qw(getbuglocation getbugcomponent filelock unfilelock);
 
 =head1 FUNCTIONS
 
@@ -135,6 +135,7 @@ Saves changes to the bug log spam file.
 
 sub save {
     my $self = shift;
+    return unless keys %{$self->{spam}};
     filelock($self->{name});
     open(my $fh,'>',$self->{name}.'.tmp') or
         croak "Unable to open bug log spam '$self->{name}.tmp' for writing: $!";