All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [maintainer-tools PATCH v2 01/33] dim: rename dim_push to dim_push_branch
Date: Thu, 28 Jan 2016 15:26:55 +0200	[thread overview]
Message-ID: <f4f1176b700aa678dbb20e6e05ad1bb70e178d6f.1453985388.git.jani.nikula@intel.com> (raw)
In-Reply-To: <cover.1453985388.git.jani.nikula@intel.com>
In-Reply-To: <cover.1453985388.git.jani.nikula@intel.com>

Rename to match the subcommand name. Move error checking to the function
too.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 dim | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/dim b/dim
index e08fdffd3d86..faa98114a027 100755
--- a/dim
+++ b/dim
@@ -349,8 +349,13 @@ function update_nightly
 
 # push branch $1, rebuild nightly. the rest of the arguments are passed to git
 # push.
-function dim_push
+function dim_push_branch
 {
+	if [[ "x$1" = "x" ]]; then
+		echo "usage: $0 $subcommand branch"
+		exit 1
+	fi
+
 	branch=$1
 	shift
 
@@ -773,20 +778,16 @@ case "$subcommand" in
 		cat > `cat .fixup_file_path`
 		;;
 	push-branch)
-		if [[ "x$1" = "x" ]]; then
-			echo "usage: $0 $subcommand branch"
-			exit 1
-		fi
-		dim_push "$@"
+		dim_push_branch "$@"
 		;;
 	push-queued|pq)
-		dim_push drm-intel-next-queued "$@"
+		dim_push_branch drm-intel-next-queued "$@"
 		;;
 	push-next-fixes|pnf)
-		dim_push drm-intel-next-fixes "$@"
+		dim_push_branch drm-intel-next-fixes "$@"
 		;;
 	push-fixes|pf)
-		dim_push drm-intel-fixes "$@"
+		dim_push_branch drm-intel-fixes "$@"
 		;;
 	checkout|co)
 		if [[ "x$1" = "x" ]]; then
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2016-01-28 13:27 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-28 13:26 [maintainer-tools PATCH v2 00/33] dim: function based subcommand handling Jani Nikula
2016-01-28 13:26 ` Jani Nikula [this message]
2016-01-28 13:26 ` [maintainer-tools PATCH v2 02/33] dim: rename dim_apply to dim_apply_branch Jani Nikula
2016-01-28 13:26 ` [maintainer-tools PATCH v2 03/33] dim: rename dim_co to dim_checkout Jani Nikula
2016-01-28 13:26 ` [maintainer-tools PATCH v2 04/33] dim: rename dim_shell_checkpatch to shell_checkpatch Jani Nikula
2016-01-28 13:26 ` [maintainer-tools PATCH v2 05/33] dim: rename dim_checkpatch to checkpatch_commit Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 06/33] dim: rename dim_checkrange to dim_checkpatch Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 07/33] dim: rename setup_dim to dim_setup Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 08/33] dim: rename create_workdir to dim_create_workdir Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 09/33] dim: rename update_nightly to dim_rebuild_nightly Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 10/33] dim: rename magic_patch to dim_magic_patch Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 11/33] dim: move dim_pull_request parameter check to the function Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 12/33] dim: separate handling of aliases Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 13/33] dim: handle subcommands through functions Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 14/33] dim: add alias command to list aliases Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 15/33] dim: abstract dim_nightly_forget Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 16/33] dim: abstract dim_update_branches Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 17/33] dim: abstract dim_for_each_workdirs Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 18/33] dim: abstract dim_push_* Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 19/33] dim: abstract dim_checkout callers Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 20/33] dim: abstract dim apply* subcommands Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 21/33] dim: abstract dim_magic_rebase_resolve Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 22/33] dim: abstract dim_cat_to_fixup Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 23/33] dim: abstract dim_tc Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 24/33] dim: abstract dim_fixes Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 25/33] dim: abstract dim_cherry_pick Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 26/33] dim: abstract dim_pull_request_* Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 27/33] dim: abstract dim_update_next Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 28/33] dim: abstract dim_tag_next Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 29/33] dim: abstract dim_{create, remove}_branch Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 30/33] dim: abstract dim_checker Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 31/33] dim: abstract dim_help Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 32/33] dim: show usage instead of the man page on unknown subcommands Jani Nikula
2016-01-28 13:27 ` [maintainer-tools PATCH v2 33/33] dim: add -h option for usage, imrove output on unknown options Jani Nikula
2016-01-28 15:12 ` [maintainer-tools PATCH v2 00/33] dim: function based subcommand handling Jani Nikula

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f4f1176b700aa678dbb20e6e05ad1bb70e178d6f.1453985388.git.jani.nikula@intel.com \
    --to=jani.nikula@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.