[Git][qa/jenkins.debian.net][master] Revert "reproducible debian: breakages: comment away the file cleanup code as...

[Holger Levsen (@holger)]

Holger Levsen pushed to branch master at Debian QA / jenkins.debian.net


Commits:
49073ad1 by Holger Levsen at 2024-03-19T12:41:16+01:00
Revert "reproducible debian: breakages: comment away the file cleanup code as I think it's removing too much"

This reverts commit 222b5302e884a5b5dd35d9776d5dd53740494e21.

- - - - -


1 changed file:

- bin/reproducible_html_breakages.py


Changes:

=====================================
bin/reproducible_html_breakages.py
=====================================
@@ -181,10 +181,8 @@ def alien_log(directory=None):
             if strip_epoch(rversion) != version:
                 try:
                     if os.path.getmtime('/'.join([root, file])) < time.time()-86400:
-                        #os.remove('/'.join([root, file]))
-                        #log.warning('%s should not be there and was older than a day so it was removed',
-                        bad_files.append('/'.join([root, file]))
-                        log.warning('%s should not be there but was not removed',
+                        os.remove('/'.join([root, file]))
+                        log.warning('%s should not be there and was older than a day so it was removed',
                                     '/'.join([root, file]))
                     else:
                         bad_files.append('/'.join([root, file]))
@@ -227,10 +225,8 @@ def alien_buildinfo():
             if strip_epoch(rversion) != version:
                 try:
                     if os.path.getmtime('/'.join([root, file])) < time.time()-86400:
-                        #os.remove('/'.join([root, file]))
-                        #log.warning('%s should not be there and and was older than a day so it was removed.',
-                        bad_files.append('/'.join([root, file]))
-                        log.warning('%s should not be there, but was not removed',
+                        os.remove('/'.join([root, file]))
+                        log.warning('%s should not be there and and was older than a day so it was removed.',
                                     '/'.join([root, file]))
                     else:
                         bad_files.append('/'.join([root, file]))



View it on GitLab: https://salsa.debian.org/qa/jenkins.debian.net/-/commit/49073ad1fa693bcbc43140a7e83605725889b393

-- 
View it on GitLab: https://salsa.debian.org/qa/jenkins.debian.net/-/commit/49073ad1fa693bcbc43140a7e83605725889b393
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/qa-jenkins-scm/attachments/20240319/edcf7760/attachment-0001.htm>