Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Somehow fell apart again in testing, let's do the same thing as with
apply-branch, where we also apply the Link: after the sob is added.

Cc: Jani Nikula <jani.nikula@linux.intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Don't fire up an editor by default. Requested by Jani.

Cc: Jani Nikula <jani.nikula@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Sean Paul authored 7 years ago and Simona Vetter committed 7 years ago

Instead of hardcoding ~/tmp in dim (and failing when it doesn't
exist), use mktemp to create the pull-request mail file.

Signed-off-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Does a few sanity checks to avoid common gotchas:
- make sure the backmerge is in drm-tip already
- check that git rerere resolves all conflict, and cuation if not
- merge commit template.

v2: Spelling fix (Sean)

Cc: Sean Paul <seanpaul@chromium.org>
Acked-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

I think the initial experiment worked fairly well, and the
kickstarting of a review (well, ack/best practices really) market
seems to work ok-ish too. I think we can try to make it (slowly) scale
to more drivers, with driver teams interested in being part of
drm-misc. I do think we need to be careful though, to make sure we
don't overstretch the review/mentoring bandwidth and can improve
process&tooling as we go.

v2: Spelling fix (Jani).

Cc: Sumit Semwal <sumit.semwal@linaro.org>
Cc: Gustavo Padovan <gustavo@padovan.org>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Acked-by: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Archit Taneja <architt@codeaurora.org>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
Acked-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Mark Yao <mark.yao@rock-chips.com>
Cc: Eric Anholt <eric@anholt.net>
Cc: Shawn Guo <shawnguo@kernel.org>
Acked-by: Shawn Guo <shawnguo@kernel.org>
Cc: Dave Airlie <airlied@redhat.com>
Acked-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

I'm getting real lazy, let's start scripting this. Very rough draft,
but adds a Link: (patchwork tracks pull requests too, maybe we'll
start CI-ing them too), and sob line. In the future we might add more
checks here ...

v2: Review from Jani:
- Move abbrevation into my own .dimrc
- Drop shift, we don't use $@

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

  authored 8 years ago and Simona Vetter committed 8 years ago

Just a bunch of trivial typos that caught my eye while perusing the
documentation.

Signed-off-by: Lukas Wunner <lukas@wunner.de>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

This repo stopped being my own personal turf a long time ago ...

Very-much-acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

This will print a generic usage error message and bail out on required
arguments missing. This is in preparation for enabling 'set -u'.

This is not an exhaustive conversion, but a good start.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Use -s for "strict" to 'set -u'. Fix the variable references to not fail
before subcommand call. The goal is to unconditionally 'set -u' at the
top, but it can't be done in one go, so add a way to test drive first.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Inspired by Jani's efforts to clean this up and structure it better.

v2: Move the FIXME to TODO and polish the documentation per Jani's
feedback.

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

It's kinda beyond just drm-intel nowadays ...

Idea from Jani on irc.

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

I wanted to get Sean Paul to run the drm-misc show for a bit, for
training reasons and to increase the bus factor. And then realized
there's no docs about what maintainers are doing.

Fix that.

v2: Add backmerges and taking the blame.

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Grammatically better, already documented that way too. While at it,
remove the leftover alias documentation for for-each-workdir.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Group the man page by:

- commands for developers
- commands for committers and maintainers
- advanced commands for committers and maintainers
- commands for maintainers
- dim help commands.

We have so many commands by now that it should be helpful to group them
a little.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Hopefully increases clarity for people not working on drm-intel.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Out of scope for dim, and relies on hard coded paths. People can add the
function to their .dimrc if needed.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Increase clarity. People can list the aliases using 'dim list-aliases'
and add their own.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Just a simple grep.

v2: also add 'make check' to run all checks (Daniel)

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Add bash_completion to 'make shellcheck' target, and fix the failures.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

For example, piping from emacs only pops up the stdout/stderr buffer
when the exit code is non-zero.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Return the exit code to let callers make better decisions on what to do.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

We apply the patch regardless of the warn_or_fail result, so just switch
to using an error message.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Separate maintainer scope check from checkpatch, and only do the check
on applying patches. Also, fix the failures on grep not matching.

Fixes: 56e53a49 ("dim: declare and assign separately")
Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Oops, the comment told us to watch out for this.

Fixes: 56e53a49 ("dim: declare and assign separately")
Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

The shift did eat fail (or eat the arguments), resulting in the script
silently aborting.

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

It's evil. Instead generate a new tag.

v2: Autogenerate a new tag with increasing suffice, to make Jani
happy.

v3: Don't forget to fix the -testing tag too (Jani).

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Makes it easier to not fumble this and write the changelog for the
wrong thing ...

v2: use @{upstream}

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

The integration branch moved!

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Silence shellcheck SC2115: Use "${var:?}" to ensure this never expands
to /. We check the variables and their sources near the start of the
script.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Fix shellcheck SC2126: Consider using grep -c instead of grep|wc.

We can use -q as suggested by Daniel because we don't care about the
exact count.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Fixes shellcheck SC2089: Quotes/backslashes will be treated
literally. Use an array.

Fixes shellcheck SC2090: Quotes/backslashes in this variable will not be
respected.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Fix shellcheck SC2005: Useless echo? Instead of 'echo $(cmd)', just use
'cmd'.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Silence shellcheck SC1090: Can't follow non-constant source. Use a
directive to specify location.

Shellcheck is capable of checking the sourced files too, but only if the
source is at a constant location.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Fix shellcheck SC2162: read without -r will mangle backslashes.

Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>