Can't change lime-packages branch when old branch has been deleted

[libremesh]

Can't change lime-packages branch when old branch has been deleted

[G10h4ck]

as a temporary workaround you can run
make clean

[ilario]

I already solved removing the git pull in the pointed line.
If you want I can make a pull request with this.

[G10h4ck]

Seems a nice idea to have a pull request fixing this, although I am not lime-build expert Pau and Gui should review your PR

Cheers!

[ilario]

Today I had somewhat the opposite problem, I moved from develop to testing/16.07 lime-build branch without running a make clean.
So:

cd lime-build
git pull
git checkout testing/16.07
make T=ar71xx P=generic J=3 UPDATE=1

and the output was:

Using LiMe Git repository git://github.com/libre-mesh/lime-packages.git
cd build/lime-packages && git checkout testing/16.07 && git pull
error: pathspec 'testing/16.07' did not match any file(s) known to git.
Makefile:166: recipe for target 'checkout' failed
make: *** [checkout] Error 1

This happened because a git checkout to the new branch was issued before the pull, so the branch didn't exist yet.
A solution (that doesn't trigger the old problem object of this issue) would be to replace the two occurrences of
git checkout $(LIME_GIT_BRANCH)
in Makefile with
git checkout develop && git pull && git checkout $(LIME_GIT_BRANCH)
as the develop branch will always exist (I believe).

[p4u]

|I think

cd build/lime-packages && git pull origin ||testing/16.07| && |git
checkout testing/16.07|

Should solve the issue, but I have not tested it.
If you are able to test it and works, please send the PR to lime-build
develop branch.

Thanks.

On 25/07/16 12:25, Ilario Gelmetti wrote:

|cd build/lime-packages && git checkout testing/16.07 && git pull|

[ilario]

Seems ok, but I suspect we will find more corner cases breaking this in the future.

[altergui]

On 25/07/16 16:06, Ilario Gelmetti wrote:

Seems ok, but I suspect we will find more corner cases breaking this in
the future.

git remote update && git checkout whatever-new-branch

should solve it in a "proper" way

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACr1I9UUXuRUGpQJuPjoAXFA0-ptl9iqks5qZMLMgaJpZM4IW_3h.

[ilario]

Should have been fixed in fb64a79.