You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by ma...@apache.org on 2017/03/06 21:44:19 UTC
[19/41] incubator-mynewt-newt git commit: MYNEWT-639 Add help text
and check for pkg name 1) Update help text for "newt pkg new" to have package
name input. 2) Error check that a package name is provided for the command.
3) Changed "libs/mylib" to "sys/m
MYNEWT-639 Add help text and check for pkg name
1) Update help text for "newt pkg new" to have package name input.
2) Error check that a package name is provided for the command.
3) Changed "libs/mylib" to "sys/mylib" in "newt pkg" help text to reflect latest source structure.
(this change is not related to MYNEWT-639)
4) Added <package-name> argument to "newt pkg remove" Usage help text
Project: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/commit/f7f8ad81
Tree: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/tree/f7f8ad81
Diff: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/diff/f7f8ad81
Branch: refs/heads/mynewt_1_0_0
Commit: f7f8ad814e39ccad7169acbdde999fc421f7f9b8
Parents: 6493620
Author: cwanda <wa...@happycity.com>
Authored: Fri Feb 24 22:15:46 2017 -0800
Committer: Marko Kiiskila <ma...@runtime.io>
Committed: Mon Mar 6 13:36:11 2017 -0800
----------------------------------------------------------------------
newt/cli/pkg_cmds.go | 21 +++++++++++++++------
1 file changed, 15 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/blob/f7f8ad81/newt/cli/pkg_cmds.go
----------------------------------------------------------------------
diff --git a/newt/cli/pkg_cmds.go b/newt/cli/pkg_cmds.go
index 0ddde3d..d17e81f 100644
--- a/newt/cli/pkg_cmds.go
+++ b/newt/cli/pkg_cmds.go
@@ -37,6 +37,15 @@ import (
var NewTypeStr = "pkg"
func pkgNewCmd(cmd *cobra.Command, args []string) {
+
+ if len(args) == 0 {
+ NewtUsage(cmd, util.NewNewtError("Must specify a package name"))
+ }
+
+ if len(args) != 1 {
+ NewtUsage(cmd, util.NewNewtError("Exactly one argument required"))
+ }
+
NewTypeStr = strings.ToUpper(NewTypeStr)
pw := project.NewPackageWriter()
@@ -51,11 +60,11 @@ func pkgNewCmd(cmd *cobra.Command, args []string) {
type dirOperation func(string, string) error
func pkgCloneCmd(cmd *cobra.Command, args []string) {
- pkgCloneOrMoveCmd(cmd, args, util.CopyDir, "Cloning");
+ pkgCloneOrMoveCmd(cmd, args, util.CopyDir, "Cloning")
}
func pkgMoveCmd(cmd *cobra.Command, args []string) {
- pkgCloneOrMoveCmd(cmd, args, util.MoveDir, "Moving");
+ pkgCloneOrMoveCmd(cmd, args, util.MoveDir, "Moving")
}
func pkgCloneOrMoveCmd(cmd *cobra.Command, args []string, dirOpFn dirOperation, opStr string) {
@@ -207,7 +216,7 @@ func AddPackageCommands(cmd *cobra.Command) {
* keyed
*/
pkgHelpText := "Commands for creating and manipulating packages"
- pkgHelpEx := " newt pkg new --type=pkg libs/mylib"
+ pkgHelpEx := " newt pkg new --type=pkg sys/mylib"
pkgCmd := &cobra.Command{
Use: "pkg",
@@ -226,8 +235,8 @@ func AddPackageCommands(cmd *cobra.Command) {
newCmdHelpEx := ""
newCmd := &cobra.Command{
- Use: "new",
- Short: "Create a new package, from a template",
+ Use: "new <package-name>",
+ Short: "Create a new package in the current directory, from a template",
Long: newCmdHelpText,
Example: newCmdHelpEx,
Run: pkgNewCmd,
@@ -268,7 +277,7 @@ func AddPackageCommands(cmd *cobra.Command) {
removeCmdHelpEx := ""
removeCmd := &cobra.Command{
- Use: "remove",
+ Use: "remove <package-name>",
Short: "Remove a package",
Long: removeCmdHelpText,
Example: removeCmdHelpEx,