On Tue, 02 May 2017, Sean Paul <[email protected]> wrote: > Add some standard headers to the pull request tag annotation.
I've read the subject, commit message, and code twice, and I have no clue what you want to achieve here. Please elaborate. BR, Jani. > > Changes in v2: > - Tweaked the template var name s/PULL/TAG/ (Daniel) > > Signed-off-by: Sean Paul <[email protected]> > --- > dim | 25 ++++++++++++++++++++++++- > dim.rst | 4 ++++ > 2 files changed, 28 insertions(+), 1 deletion(-) > > diff --git a/dim b/dim > index 8937803..e94eb0c 100755 > --- a/dim > +++ b/dim > @@ -67,6 +67,9 @@ > DIM_TEMPLATE_HELLO=${DIM_TEMPLATE_HELLO:-$HOME/.dim.template.hello} > # signature pull request template > > DIM_TEMPLATE_SIGNATURE=${DIM_TEMPLATE_SIGNATURE:-$HOME/.dim.template.signature} > > +# tag headers for pull requests > +DIM_TEMPLATE_TAG_HEADERS=${DIM_TEMPLATE_TAG_HEADERS:-$HOME/.dim.template.tagheaders} > + > # > # Internal configuration. > # > @@ -1501,10 +1504,28 @@ function dim_tag_next > > } > > +function prep_pull_tag_headers > +{ > + local template > + > + if [ -r $DIM_TEMPLATE_TAG_HEADERS ]; then > + while read -r m || [[ -n "$line" ]]; do > + template="$template -m '$m'" > + done < "$DIM_TEMPLATE_TAG_HEADERS" > + else > + template="-m 'UABI Changes:'" > + template="$template -m 'Cross-subsystem Changes:'" > + template="$template -m 'Core Changes:'" > + template="$template -m 'Driver Changes:'" > + fi > + echo $template > +} > + > # dim_pull_request branch upstream > function dim_pull_request > { > local branch upstream remote repo req_file url git_url suffix tag > + local tag_headers > > branch=${1:?$usage} > upstream=${2:?$usage} > @@ -1535,7 +1556,9 @@ function dim_pull_request > done > > gitk "$branch@{upstream}" ^$upstream & > - $DRY git tag -a $tag "$branch@{upstream}" > + tag_headers=$(prep_pull_tag_headers) > + $DRY git tag $tag_headers $tag "$branch@{upstream}" > + $DRY git tag -a -f $tag > $DRY git push $remote $tag > prep_pull_mail $req_file $tag > > diff --git a/dim.rst b/dim.rst > index 3dd19f9..d6438a3 100644 > --- a/dim.rst > +++ b/dim.rst > @@ -464,6 +464,10 @@ DIM_TEMPLATE_SIGNATURE > ---------------------- > Path to a file containing a signature template for pull request mails. > > +DIM_TEMPLATE_TAG_HEADERS > +------------------------- > +Path to a file containing tag headers for pull requests, each on their own > line. > + > dim_alias_<alias> > ----------------- > Make **<alias>** an alias for the subcommand defined as the value. For > example, -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
