Add unlink_files method to TestCmd, replace file deletion loops with method calls in msvs executable test scripts, re-order CHANGES.txt.

Author: jcbrill

CHANGES.txt

@@ -9,6 +9,10 @@ NOTE: 4.3.0 now requires Python 3.6.0 and above. Python 3.5.x is no longer suppo
 
 RELEASE  VERSION/DATE TO BE FILLED IN LATER
 
+  From Ataf Fazledin Ahamed:
+    - Use of NotImplemented instead of NotImplementedError for special methods
+      of _ListVariable class
+
   From Joseph Brill:
     - Fix issue #2755: the msvs tool no longer writes the OS environment SCONS_HOME
       value into the SCons environment when the SCONS_HOME variable already exists
@@ -32,6 +36,9 @@ RELEASE  VERSION/DATE TO BE FILLED IN LATER
       build.  The tests were modified to delete the test executable, object file,
       and sconsign file prior to the command-line invocation of the VS development
       binary.
+    - A method, unlink_files, was added to the TestCmd class that unlinks a list of
+      files from a specified directory.  An attempt to unlink a file is made only when
+      the file exists, otherwise, the file is ignored.
 
   From Michał Górny:
     - Remove unecessary dependencies on pypi packages from setup.cfg
@@ -40,10 +47,6 @@ RELEASE  VERSION/DATE TO BE FILLED IN LATER
     - Fix of the --debug=sconscript option to return exist statements when using return
       statement with stop flag enabled
 
-  From Ataf Fazledin Ahamed:
-    - Use of NotImplemented instead of NotImplementedError for special methods
-      of _ListVariable class
-
 RELEASE 4.6.0 -  Sun, 19 Nov 2023 17:22:20 -0700
 
   From Max Bachmann:

RELEASE.txt

@@ -16,7 +16,10 @@ Here is a summary of the changes since 4.6.0:
 NEW FUNCTIONALITY
 -----------------
 
-- List new features (presumably why a checkpoint is being released)
+- A method, unlink_files, was added to the TestCmd class that unlinks a list of
+  files from a specified directory.  An attempt to unlink a file is made only when
+  the file exists, otherwise, the file is ignored.
+
 
 DEPRECATED FUNCTIONALITY
 ------------------------

test/MSVS/vs-10.0-exec.py

@@ -90,10 +90,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-10.0Exp-exec.py

@@ -91,10 +91,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-11.0-exec.py

@@ -91,10 +91,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-11.0Exp-exec.py

@@ -91,10 +91,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-14.0-exec.py

@@ -92,10 +92,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-14.0Exp-exec.py

@@ -92,10 +92,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-14.1-exec.py

@@ -92,10 +92,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))
 

test/MSVS/vs-14.2-exec.py

@@ -92,10 +92,7 @@
 
 test.run(chdir='sub dir', arguments='.')
 
-for filename in ('foo.exe', 'foo.obj', '.sconsign.dblite'):
-    filepath = test.workpath('sub dir', filename)
-    if os.path.exists(filepath):
-        test.unlink(filepath)
+test.unlink_files('sub dir', ['foo.exe', 'foo.obj', '.sconsign.dblite'])
 
 test.vcproj_sys_path(test.workpath('sub dir', 'foo.vcxproj'))