From 3e0d22b53134333bef81139e8eb1265169de879a Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Wed, 30 Jan 2008 07:55:16 +0000 Subject: [PATCH] =?utf8?q?Bug=20414726:=20Bugzilla::Update=20never=20updat?= =?utf8?q?es=20bugzilla-update.xml=20if=20the=20file=20already=20exists=20?= =?utf8?q?locally=20-=20Patch=20by=20Fr=C3=83=C2=A9d=C3=83=C2=A9ric=20Bucl?= =?utf8?q?in=20=20r=3Dmkanat=20a=3DLpSolit?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- Bugzilla/Update.pm | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/Bugzilla/Update.pm b/Bugzilla/Update.pm index f2b17a6b00..e9da91afb3 100644 --- a/Bugzilla/Update.pm +++ b/Bugzilla/Update.pm @@ -42,8 +42,16 @@ sub get_notifications { # but we failed because we cannot modify its timestamp? my $can_alter = 1; if (-e $local_file) { - # Try to alter its last modification time. + # Try to alter its last modification time. We first save the + # access and modification times of the file to restore them + # right after our test. + my $atime = (stat($local_file))[8]; + my $mtime = (stat($local_file))[9]; $can_alter = utime(undef, undef, $local_file); + # Restore the access and modification times to their + # original values, else LWP::UserAgent will never see the + # updated file on the server as newer than our local one. + utime($atime, $mtime, $local_file); } if ($can_alter) { my $error = _synchronize_data(); @@ -148,6 +156,8 @@ sub _synchronize_data { else { $ua->env_proxy; } + # Download the file from the server if its modification time is newer + # than the local one. $ua->mirror(REMOTE_FILE, $local_file); # $ua->mirror() forces the modification time of the local XML file -- 2.47.3