On Fri, Aug 24, 2018 at 02:43:34PM -0700, Rodrigo Vivi wrote: > I don't understand exactly why we need this special handling only > for drm-intel-next. But let's at least consolidate this into > branch_to_repo instead of spreading it around. > > Cc: Jani Nikula <[email protected]> > Cc: Daniel Vetter <[email protected]> > Signed-off-by: Rodrigo Vivi <[email protected]>
drm-intel-next is not a dim-managed branch (i.e. it's not listed in nightly.conf). I think that would be good to add in a comment. With that added: Reviewed-by: Daniel Vetter <[email protected]> Aside, I recommend you just remove drm-intel-next and all the special casing. With the uncoupling of tagging from sending pull requests there's really no need for this anymore. The only other thing coupled with drm-intel-next is drm-intel-testing. This was used by Gordon's QA team for the 1-week long manual test cycle. Gordon's gfx QA team is no more (since years!), so there's really no point in this. But there's also not much real harm (except the confusion drm-intel-next/testing cause). -Daniel > --- > dim | 13 +++++-------- > 1 file changed, 5 insertions(+), 8 deletions(-) > > diff --git a/dim b/dim > index 5040d6b046b2..fec51f766e55 100755 > --- a/dim > +++ b/dim > @@ -290,6 +290,10 @@ function branch_to_repo # branch > fi > done > > + if [[ "$1" == "drm-intel-next" ]] ; then > + echo "drm-intel" > + fi > + > echo "" > } > > @@ -1402,10 +1406,6 @@ function dim_checkout > if ! git_branch_exists $branch ; then > repo=$(branch_to_repo $branch) > > - if [[ $branch == "drm-intel-next" ]] ; then > - repo="drm-intel" > - fi > - > if [[ $repo == "" ]] ; then > echoerr "$branch not found in $dim_integration_config" > exit 1 > @@ -1890,6 +1890,7 @@ function dim_pull_request > branch=${1:?$usage} > upstream=${2:?$usage} > remote=$(branch_to_remote $branch) > + repo=$(branch_to_repo $branch) > req_file=$(mktemp) > > if [ "$branch" != "drm-intel-next" ]; then > @@ -1906,16 +1907,12 @@ function dim_pull_request > drm_intel_next_tags=$(git log "$branch@{upstream}" ^$upstream > --decorate | grep "(.*tag: drm-intel-next-" | sed -e "s/^.*(.*tag: > \(drm-intel-next-[^ ,]*\).*)$/\1/") > prep_pull_mail $req_file $drm_intel_next_tags > tag=$(git describe --all --exact "$branch@{upstream}") > - > - repo="drm-intel" > else > tag=$(tag_name "$branch") > gitk --first-parent "$branch" ^$upstream & > tag_branch $tag $branch > $DRY git push $remote $tag > prep_pull_mail $req_file $tag > - > - repo=$(branch_to_repo $branch) > fi > > url_list=${drm_tip_repos[$repo]} > -- > 2.17.1 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dim-tools mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dim-tools
