dump deprecated entries

4 files changed, 90 insertions, 49 deletions

diff --git a/xmake/core/base/deprecated.lua b/xmake/core/base/deprecated.lua
new file mode 100644
index 000000000..b676b0bfa
--- /dev/null
+++ b/xmake/core/base/deprecated.lua
@@ -0,0 +1,59 @@
+--!The Automatic Cross-platform Build Tool
+-- 
+-- XMake is free software; you can redistribute it and/or modify
+-- it under the terms of the GNU Lesser General Public License as published by
+-- the Free Software Foundation; either version 2.1 of the License, or
+-- (at your option) any later version.
+-- 
+-- XMake is distributed in the hope that it will be useful,
+-- but WITHOUT ANY WARRANTY; without even the implied warranty of
+-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+-- GNU Lesser General Public License for more details.
+-- 
+-- You should have received a copy of the GNU Lesser General Public License
+-- along with XMake; 
+-- If not, see <a href="http://www.gnu.org/licenses/"> http://www.gnu.org/licenses/</a>
+-- 
+-- Copyright (C) 2015 - 2016, ruki All rights reserved.
+--
+-- @author      ruki
+-- @file        deprecated.lua
+--
+
+-- define module
+local deprecated = deprecated or {}
+
+-- load modules
+local utils     = require("base/utils")
+local table     = require("base/table")
+local string    = require("base/string")
+
+-- add deprecated entry
+function deprecated.add(newformat, oldformat, ...)
+
+    -- the entries
+    deprecated._ENTRIES = deprecated._ENTRIES or {}
+
+    -- the old and new entries
+    local old = string.format(oldformat, ...)
+    local new = string.format(newformat, ...)
+
+    -- add it
+    deprecated._ENTRIES[old] = new
+end
+
+-- dump all deprecated entries
+function deprecated.dump()
+
+    -- the entries
+    deprecated._ENTRIES = deprecated._ENTRIES or {}
+
+    -- dump all
+    print("")
+    for old, new in pairs(deprecated._ENTRIES) do
+        utils.printf("deprecated: please uses %s instead of %s", new, old)
+    end
+end
+
+-- return module
+return deprecated
diff --git a/xmake/core/base/deprecated/interpreter.lua b/xmake/core/base/deprecated/interpreter.lua
index ec06d8866..1c4f8e18e 100644
--- a/xmake/core/base/deprecated/interpreter.lua
+++ b/xmake/core/base/deprecated/interpreter.lua
@@ -24,12 +24,13 @@
 local deprecated_interpreter = deprecated_interpreter or {}
 
 -- load modules
-local os        = require("base/os")
-local path      = require("base/path")
-local table     = require("base/table")
-local utils     = require("base/utils")
-local string    = require("base/string")
-local sandbox   = require("sandbox/sandbox")
+local os            = require("base/os")
+local path          = require("base/path")
+local table         = require("base/table")
+local utils         = require("base/utils")
+local string        = require("base/string")
+local deprecated    = require("base/deprecated")
+local sandbox       = require("sandbox/sandbox")
 
 -- register api for set_scope()
 function deprecated_interpreter:api_register_set_scope(...)
@@ -44,9 +45,9 @@ function deprecated_interpreter:api_register_set_scope(...)
         local scope_for_kind = scopes[scope_kind] or {}
         scopes[scope_kind] = scope_for_kind
 
-        -- warning
+        -- deprecated
         if not scope_name:startswith("__") then
-            utils.warning("please uses %s(\"%s\"), \"set_%s\" has been deprecated!", scope_kind, scope_name, scope_kind)
+            deprecated.add("%s(\"%s\")", "set_%s(\"%s\")", scope_kind, scope_name)
         end
 
         -- check 
@@ -83,9 +84,9 @@ function deprecated_interpreter:api_register_add_scope(...)
         local scope_for_kind = scopes[scope_kind] or {}
         scopes[scope_kind] = scope_for_kind
 
-        -- warning
+        -- deprecated
         if not scope_name:startswith("__") then
-            utils.warning("please uses %s(\"%s\"), \"add_%s\" has been deprecated!", scope_kind, scope_name, scope_kind)
+            deprecated.add("%s(\"%s\")", "add_%s(\"%s\")", scope_kind, scope_name)
         end
 
         -- check 
@@ -118,8 +119,8 @@ function deprecated_interpreter:api_register_set_script(scope_kind, prefix, ...)
     -- define implementation
     local implementation = function (self, scope, name, script)
 
-        -- warning
-        utils.warning("please uses on_%s(), \"set_%s()\" has been deprecated!", name, name, scope)
+        -- deprecated
+        deprecated.add("on_%s()", "set_%s()", name)
 
         -- make sandbox instance with the given script
         local instance, errors = sandbox.new(script, self:filter(), self:rootdir())
diff --git a/xmake/core/base/option.lua b/xmake/core/base/option.lua
index eedeec158..9acf1b48b 100644
--- a/xmake/core/base/option.lua
+++ b/xmake/core/base/option.lua
@@ -175,12 +175,12 @@ function option.init(argv, menu)
         -- check key
         if prefix and #key == 0 then
 
-            -- invalid option
-            print("invalid option: " .. arg)
-
             -- print menu
             option.show_menu(context.taskname)
 
+            -- invalid option
+            print("\nerror: invalid option: " .. arg)
+
             -- failed
             return false
         end
@@ -209,12 +209,12 @@ function option.init(argv, menu)
             -- not found?
             if not opt then
 
-                -- invalid option
-                print("invalid option: " .. arg)
-
                 -- print menu
                 option.show_menu(context.taskname)
 
+                -- invalid option
+                print("\nerror: invalid option: " .. arg)
+
                 -- failed
                 return false
             end
@@ -229,12 +229,12 @@ function option.init(argv, menu)
                 _k = idx
                 if idx == nil or arg:startswith("-") then 
 
-                    -- invalid option
-                    print("invalid option: " .. option._ifelse(idx, arg, key))
-
                     -- print menu
                     option.show_menu(context.taskname)
 
+                    -- invalid option
+                    print("\nerror: invalid option: " .. option._ifelse(idx, arg, key))
+
                     -- failed
                     return false
                 end
@@ -246,12 +246,12 @@ function option.init(argv, menu)
             -- check mode
             if (opt[3] == "k" and type(value) ~= "boolean") or (opt[3] == "kv" and type(value) ~= "string") then
 
-                -- invalid option
-                print("invalid option: " .. arg)
-            
                 -- print menu
                 option.show_menu(context.taskname)
 
+                -- invalid option
+                print("\nerror: invalid option: " .. arg)
+            
                 -- failed
                 return false
             end
@@ -283,12 +283,12 @@ function option.init(argv, menu)
             -- not found?
             if not context.taskname or not menu[context.taskname] then
 
-                -- invalid task
-                print("invalid task: " .. key)
-
                 -- print the main menu
                 option.show_main()
 
+                -- invalid task
+                print("\nerror: invalid task: " .. key)
+
                 -- failed
                 return false
                 
@@ -345,12 +345,13 @@ function option.init(argv, menu)
                     table.insert(o, key)
                 end
             else
-                -- invalid option
-                print("invalid option: " .. arg)
             
                 -- print menu
                 option.show_menu(context.taskname)
 
+                -- invalid option
+                print("\nerror: invalid option: " .. arg)
+
                 -- failed
                 return false
             end
diff --git a/xmake/core/base/utils.lua b/xmake/core/base/utils.lua
index 5d69bdcd0..f5c863c97 100644
--- a/xmake/core/base/utils.lua
+++ b/xmake/core/base/utils.lua
@@ -68,31 +68,11 @@ function utils.warning(msg, ...)
     utils._WARNINGS = utils._WARNINGS or {}
     local warnings = utils._WARNINGS
 
-    -- the cached file path
-    local cachedpath = path.translate(xmake._PROJECT_DIR .. "/.xmake/warnings")
-
-    -- load warnings from the cached file
-    local cachedfile = io.open(cachedpath, "r")
-    if cachedfile then
-        for line in cachedfile:lines() do
-            warnings[line] = true
-        end
-    end
-
     -- trace only once
     if not warnings[msg] then
         print(msg)
         warnings[msg] = true
     end
-
-    -- cache warnings
-    cachedfile = io.open(cachedpath, "w")
-    if cachedfile then
-        for line, _ in pairs(warnings) do
-            cachedfile:write(line .. "\n")
-        end
-        cachedfile:close()
-    end
 end
 
 -- ifelse, a? b : c