[prev in list] [next in list] [prev in thread] [next in thread] 

List:       fedora-buildsys-list
Subject:    [PATCH] cli: Set the mtime from urlgrabber/pycurl, make users happy. BZ 872631.
From:       James Antill <james () and ! org>
Date:       2013-10-29 22:30:14
Message-ID: 1383085814-32557-1-git-send-email-james () and ! org
[Download RAW message or body]

---
 cli/koji |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/cli/koji b/cli/koji
index a870342..04d00bc 100755
--- a/cli/koji
+++ b/cli/koji
@@ -5726,6 +5726,15 @@ def anon_handle_download_build(options, session, args):
                 os.write(out, buf)
         finally:
             os.close(out)
+
+            # Set the mtime to the remote mtime, people like this. BZ 87263.
+            try:
+                import pycurl
+                mod_time = file.curl_obj.getinfo(pycurl.INFO_FILETIME)
+                os.utime(relpath, (mod_time, mod_time))
+            except:
+                pass # Ignore errors here.
+
             file.close()
 
 def anon_handle_wait_repo(options, session, args):
-- 
1.7.7.6

--
buildsys mailing list
buildsys@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/buildsys
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic