> > >> So however you got a merge conflict. Your tree wasn't up to date.
> > >>
> > > I do not encounter any merge conflicts, is just that the branches is
> > > not
> > merging. By checking my build, I notice that the valleyisland-io-1.0
> > branch has the same HEAD of standard/base (which means I don't see any
> > I/O patches there in the feature branch that fetched by bitbake). This
> > means that source tree I'm fetching during build time is not updated
> > yet. Is there any occurrence that the new check-ins need some time to
> reflect to end users?
> >
> > It could be a fetcher problem, since the branches are definitely not the
> same.
> >
> > yow-bashfiel-d3 [/home/bruc....git-merge]> git describe standard/base
> > v3.10.38-2595-g02f7e63e56c0
> > yow-bashfiel-d3 [/home/bruc....git-merge]> git describe
> > valleyisland-io-1.0
> > v3.10.38-2618-g8ea4fb625f26

Hi both,
        I did some research onto the do_validate_branches logic and realize
the reason why we are not seeing the correct commit that we want to have
at valleyisland-io feature branch in the build folder. 

Basically, the logic below is force any branch not having the HEAD as the 
machine branch
to be downgraded, i.e. valleyisland-io branch HEAD which is ahead of 
standard/base, is 
forced to be the same as the HEAD of standard/base after the following logic:

 ==========================================================================

**** target_branch_head is the standard/base SCRREV (set as HEAD) commit  *****

        containing_branches=`git branch --contains $target_branch_head | sed 
's/^..//'`
        if [ -z "$containing_branches" ]; then
                echo "ERROR: SRCREV was set to \"$target_branch_head\", but no 
branches"
                echo "       contain this commit"
                exit 1
        fi

****  after this logic, valleyisland-io will be listed under containing_branch 
because
          the patchset is applied ontop of standard/base HEAD ******
 
        # force the SRCREV in each branch that contains the specified
        # SRCREV (if it isn't the current HEAD of that branch)
        git checkout -q master
        for b in $containing_branches; do
                branch_head=`git show-ref -s --heads ${b}`
                if [ "$branch_head" != "$target_branch_head" ]; then
                        echo "[INFO] Setting branch $b to ${target_branch_head}"
                        if [ "$b" = "master" ]; then
                                git reset --hard $target_branch_head > /dev/null
                        else
                                git branch -D $b > /dev/null
                                git branch $b $target_branch_head > /dev/null
                        fi
                fi
        done

**** The above logic force any branch listed in containing_branches (which 
valleyisland-io branch is in)
         not having the same commit as KBRANCH to be downgraded to the be the 
same as
         KBRANCH SRCREV commit  ****
==========================================================================

The output of the do_validate_branch is as follow and confirmed the behavior.

DEBUG: Executing shell function do_validate_branches
[INFO] Setting branch standard/arm-versatile-926ejs to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/axxia/base to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/beagleboard to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/ck to 02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/edf to 02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/fsl-mpc8315e-rdb to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/mti-malta32 to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/mti-malta64 to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/ocf to 02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/preempt-rt/base to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/preempt-rt/fri2 to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/preempt-rt/minnow to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/preempt-rt/qemuppc to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/preempt-rt/routerstationpro to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/qemuppc to 
02f7e63e56c061617957388c23bd5cf9b05c5388
[INFO] Setting branch standard/routerstationpro to 
02f7e63e56c061617957388c23bd5cf9b05c5388
****[INFO] Setting branch valleyisland-io-1.0 to 
02f7e63e56c061617957388c23bd5cf9b05c5388****
DEBUG: Shell function do_validate_branches finished
 
Bruce, how do we resolve this? What is the practice in linux-yocto on feature 
branch?
Do you know think that having all patches on standard/base is a more correct 
approach?

We plan to release meta-valleyisland out together with other YP1.6 meta-intel 
release targeting
next week. 
Appreciate your timely response here, so that we can adjust accordingly based 
on your merge strategy and
eventually send in the last commit update for meta-valleyisland 
linux-yocto_3.10.bbapend. 

Many thanks! 


-- 
_______________________________________________
linux-yocto mailing list
linux-yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/linux-yocto

Reply via email to