<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Mar 29, 2018 at 9:54 AM Jani Nikula <<a href="mailto:jani.nikula@linux.intel.com">jani.nikula@linux.intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 29 Mar 2018, Sean Paul <<a href="mailto:seanpaul@chromium.org" target="_blank">seanpaul@chromium.org</a>> wrote:<br>
> Use the local branch when launching gitk for the purpose of pull<br>
> requests. Since the pull is generated from the local branch, this gives<br>
> a view of exactly what will be sent, thus avoiding embarrassing empty<br>
> pulls (*cough* [1] *cough*) or invalid tag summaries.<br>
<br>
What's the scenario where these would be out of sync?<br></blockquote><div><br></div><div>If the local branch is behind the upstream branch. This could be intentional or unintentional. </div><div><br></div><div>Intentional: I sometimes will want to cut a tag at a certain point if I'm sending -misc-next pulls</div><div>Unintentional: As was the case last night, I sometimes forget to run dim update-branches</div><div><br></div><div>In both of these cases, I want the gitk output to show me what I'm tagging instead of what is in the upstream branch.</div><div><br></div><div>Sean</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
BR,<br>
Jani.<br>
<br>
><br>
> Cc: Dave Airlie <<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>><br>
> Signed-off-by: Sean Paul <<a href="mailto:seanpaul@chromium.org" target="_blank">seanpaul@chromium.org</a>><br>
><br>
> [1]- <a href="https://lists.freedesktop.org/archives/dri-devel/2018-March/171055.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/archives/dri-devel/2018-March/171055.html</a><br>
> ---<br>
>  dim | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/dim b/dim<br>
> index 29647f87381e..068efb674cd3 100755<br>
> --- a/dim<br>
> +++ b/dim<br>
> @@ -1818,7 +1818,7 @@ function dim_pull_request<br>
>               repo="drm-intel"<br>
>       else<br>
>               tag=$(tag_name "$branch")<br>
> -             gitk "$branch@{upstream}" ^$upstream &<br>
> +             gitk "$branch" ^$upstream &<br>
>               tag_branch $tag $branch<br>
>               $DRY git push $remote $tag<br>
>               prep_pull_mail $req_file $tag<br>
<br>
--<br>
Jani Nikula, Intel Open Source Technology Center<br>
</blockquote></div></div>