From 82ce89e31583946651a597b8acf6a07ea7fe96a5 Mon Sep 17 00:00:00 2001
From: Sylvain Munaut <tnt@246tNt.com>
Date: Sat, 10 Aug 2019 16:22:04 +0200
Subject: [PATCH] ice40: Only warn about default package if there is no package
 argument

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
---
 ice40/main.cc | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/ice40/main.cc b/ice40/main.cc
index 5e5bf6ad..5a5fa0c7 100644
--- a/ice40/main.cc
+++ b/ice40/main.cc
@@ -157,6 +157,10 @@ std::unique_ptr<Context> Ice40CommandHandler::createContext(std::unordered_map<s
 
     if (vm.count("package"))
         chipArgs.package = vm["package"].as<std::string>();
+    else
+        log_warning("Use of default value for --package is deprecated. Please add '--package %s' to arguments.\n",
+                    chipArgs.package.c_str());
+
     if (values.find("arch.name") != values.end()) {
         std::string arch_name = values["arch.name"].as_string();
         if (arch_name != "ice40")
@@ -207,9 +211,6 @@ std::unique_ptr<Context> Ice40CommandHandler::createContext(std::unordered_map<s
     }
 #endif
 
-    log_warning("Use of default value for --package is deprecated. Please add '--package %s' to arguments.\n",
-                chipArgs.package.c_str());
-
     auto ctx = std::unique_ptr<Context>(new Context(chipArgs));
     for (auto &val : values)
         ctx->settings[ctx->id(val.first)] = val.second;