Skip to content

Commit

Permalink
Bug 18373: Re-add UpgradeBackup.pm
Browse files Browse the repository at this point in the history
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 <josef.moravec@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
  • Loading branch information
joubu authored and kylemhall committed Apr 14, 2017
1 parent d26b066 commit 508147d
Show file tree
Hide file tree
Showing 2 changed files with 95 additions and 1 deletion.
94 changes: 94 additions & 0 deletions 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 <http://www.gnu.org/licenses>.

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 <http://koha-community.org/>
Galen Charlton <galen.charlton@liblime.com>
=cut

1;
2 changes: 1 addition & 1 deletion Makefile.PL
Expand Up @@ -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)
Expand Down

0 comments on commit 508147d

Please sign in to comment.