Skip to content
This repository has been archived by the owner on Oct 10, 2019. It is now read-only.

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

Closed
ilario opened this issue May 4, 2016 · 8 comments
Closed

Comments

@ilario
Copy link
Member

ilario commented May 4, 2016

Yesterday I compiled from lime-packages branch sandbox/vlanzero.
Then today I tried to compile from branch develop but I got an error.
The problem is that the sandbox/vlanzero has been deleted (branches are deleted quite often from lime-packages) but lime-build tries to fetch the old branch even if a new one is specified.

cd lime-build
git checkout develop
git pull
make T=tl-wdr3600 LIME_GIT_BRANCH=develop UPDATE=1 build

Updating LiMe repository (cd build/lime-packages && git pull && git checkout develop && git pull origin develop) Your configuration specifies to merge with the ref 'refs/heads/sandbox/vlanzero' from the remote, but no such ref was fetched. Makefile:194: recipe for target 'checkout' failed make: *** [checkout] Error 1

Seems that this is caused by the first git pull on this line.

@G10h4ck
Copy link
Member

G10h4ck commented May 4, 2016

as a temporary workaround you can run
make clean

@ilario
Copy link
Member Author

ilario commented May 4, 2016

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

@G10h4ck
Copy link
Member

G10h4ck commented Jul 5, 2016

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!

@p4u p4u closed this as completed Jul 13, 2016
@ilario
Copy link
Member Author

ilario commented Jul 25, 2016

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
Copy link
Member

p4u commented Jul 25, 2016

|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
Copy link
Member Author

ilario commented Jul 25, 2016

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

@altergui
Copy link
Member

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.

@G10h4ck G10h4ck reopened this Jul 31, 2016
@ilario
Copy link
Member Author

ilario commented Aug 8, 2016

Should have been fixed in fb64a79.

@ilario ilario closed this as completed Aug 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants