From 508147d24443498a2c12a7c55240e70941c49dd6 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 3 Apr 2017 12:50:27 -0300 Subject: [PATCH] Bug 18373: Re-add UpgradeBackup.pm Bug 18028 removed the install_misc directory but install_misc/UpgradeBackup.pm was still used by the 'upgrade' rule of make. Other files from install_misc were useless to it may be better not to reintroduce this directory with only 1 file. Test plan: `make` `sudo make install` `make upgrade` Signed-off-by: Josef Moravec Signed-off-by: Marcel de Rooy Signed-off-by: Kyle M Hall --- C4/Installer/UpgradeBackup.pm | 94 +++++++++++++++++++++++++++++++++++ Makefile.PL | 2 +- 2 files changed, 95 insertions(+), 1 deletion(-) create mode 100644 C4/Installer/UpgradeBackup.pm diff --git a/C4/Installer/UpgradeBackup.pm b/C4/Installer/UpgradeBackup.pm new file mode 100644 index 0000000000..2e30c53676 --- /dev/null +++ b/C4/Installer/UpgradeBackup.pm @@ -0,0 +1,94 @@ +package C4::Installer::UpgradeBackup; + +# Copyright (C) 2008 LibLime +# +# This file is part of Koha. +# +# Koha is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# Koha is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with Koha; if not, see . + +use strict; +#use warnings; FIXME - Bug 2505 +use File::Compare qw(compare); +use Cwd qw(cwd); +use File::Copy; +use File::Find; +use File::Spec; +use Exporter; + +use vars qw(@ISA @EXPORT ); + +@ISA = ('Exporter'); +@EXPORT = ('backup_changed_files'); + +=head1 NAME + +C4::Installer::UpgradeBackup + +=head1 DESCRIPTION + +This is a helper module used during a 'make upgrade' that +creates backups of files updated during an upgrade. + +=cut + +sub backup_changed_files { + my $from_to = shift; + my $suffix = shift; + my $verbose = shift; + my $inc_uninstall = shift; + + my $cwd = cwd(); + foreach my $sourceroot (sort keys %$from_to) { + my $targetroot = $from_to->{$sourceroot}; + my $currdir = File::Spec->catdir($cwd, $sourceroot); + + next unless -d $currdir; + + chdir $currdir or die "could not change to $currdir: $!"; + + # expand path + find(sub { + return unless -f $_; + my $filename = $_; + + my $targetdir = File::Spec->catdir($targetroot, $File::Find::dir); + my $targetfile = File::Spec->catfile($targetdir, $filename); + my $sourcedir = File::Spec->catdir($currdir, $File::Find::dir); + my $sourcefile = File::Spec->catfile($sourcedir, $filename); + + if (-f $targetfile) { + my ($size) = (stat $sourcefile)[7]; + my $backup = $targetfile . $suffix; + unless (-s $targetfile == $size and not compare($sourcefile, $targetfile)) { + print "Backed up $targetfile to $backup\n"; + File::Copy::copy($targetfile, $backup); + } + } + }, "."); + } +} + +=head1 AUTHOR + +Code based on parts of ExtUtils::Install in order to +approximately track how it identifies files to +install. + +Koha Development Team + +Galen Charlton + +=cut + +1; diff --git a/Makefile.PL b/Makefile.PL index b9b4918689..1cd5b89f3b 100644 --- a/Makefile.PL +++ b/Makefile.PL @@ -1687,7 +1687,7 @@ sub upgrade { } $upgrade .= qq/ -MOD_BACKUP = \$(ABSPERLRUN) -Minstall_misc::UpgradeBackup -e 'backup_changed_files({\@ARGV}, '$backup_suffix', '\''\$(VERBINST)'\'', '\''\$(UNINST)'\'');' -- +MOD_BACKUP = \$(ABSPERLRUN) -MC4::Installer::UpgradeBackup -e 'backup_changed_files({\@ARGV}, '$backup_suffix', '\''\$(VERBINST)'\'', '\''\$(UNINST)'\'');' -- upgrade :: make_upgrade_backup install \t\$(NOECHO) \$(NOOP) -- 2.39.5