Add warning for --no-* target specifiers (#245)

Closes #236

Author: slonopotamus

ue4docker/build.py

@@ -48,7 +48,7 @@ def build():
 
     # Parse the supplied command-line arguments
     try:
-        config = BuildConfiguration(parser, sys.argv[1:])
+        config = BuildConfiguration(parser, sys.argv[1:], logger)
     except RuntimeError as e:
         logger.error("Error: {}".format(e))
         sys.exit(1)

ue4docker/infrastructure/BuildConfiguration.py

@@ -261,7 +261,7 @@ def addArguments(parser):
             help="Set a specific changelist number in the Unreal Engine's Build.version file",
         )
 
-    def __init__(self, parser, argv):
+    def __init__(self, parser, argv, logger):
         """
         Creates a new build configuration based on the supplied arguments object
         """
@@ -284,6 +284,10 @@ def __init__(self, parser, argv):
             self.args.target = ["all"]
         elif using_target_specifier_old and not using_target_specifier_new:
             # Convert these to the new style
+            logger.warning(
+                "Using deprecated `--no-*` target specifiers; recommend changing to `--target`",
+                False,
+            )
 
             # no-minimal implies no-full
             if self.args.no_minimal:

ue4docker/infrastructure/Logger.py

@@ -26,6 +26,12 @@ def info(self, output, newline=True):
         """
         Prints information that does not pertain to an action or an error
         """
+        self._print("green", output, newline)
+
+    def warning(self, output, newline=True):
+        """
+        Prints a warning (something that is not an error as of today, but might break in future releases)
+        """
         self._print("yellow", output, newline)
 
     def _print(self, colour, output, newline):