From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Subject: Re: [maintainer-tools PATCH v2 00/33] dim: function based subcommand handling
Date: Thu, 28 Jan 2016 17:12:17 +0200 [thread overview]
Message-ID: <878u39u4a6.fsf@intel.com> (raw)
In-Reply-To: <cover.1453985388.git.jani.nikula@intel.com>
On Thu, 28 Jan 2016, Jani Nikula <jani.nikula@intel.com> wrote:
> This is v2 of [1], with Daniel's concern regarding aliases
> addressed. It's much better now.
>
> Patches 1-11 and 15-31 are mostly just mechanical renames or code
> movement. They explode the series so big, but I think they're easier to
> look at this way.
>
> The real changes are in patches 12-14 and 32-33.
>
> All in all this should make dim more maintainable. It's getting pretty
> big for a shell script.
Pushed the lot with Daniel's IRC ack. Fixed one small typo on the way.
Please report any odd behaviour; it's possible I've screwed up
something...
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2016-01-28 15:12 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 ` [maintainer-tools PATCH v2 01/33] dim: rename dim_push to dim_push_branch Jani Nikula
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 ` Jani Nikula [this message]
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=878u39u4a6.fsf@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox