Skip to content
This repository has been archived by the owner on Jan 6, 2022. It is now read-only.

build failure when hero-z-7045-builder: asking for password #28

Closed
speedzheng opened this issue Sep 18, 2018 · 10 comments · Fixed by #29
Closed

build failure when hero-z-7045-builder: asking for password #28

speedzheng opened this issue Sep 18, 2018 · 10 comments · Fixed by #29
Assignees
Labels
bug Something isn't working

Comments

@speedzheng
Copy link

Hi,

I am trying to build HERO SDK but met the following error when run ./hero-z-7045-builder -A command. Do you miss something? Thanks!

sdk:hal:prepare (hero-z-7045@config_file=/home/speedzheng/hero/hero-sdk/pulp-sdk/configs/json/hero-z-7045.json): make header
make: Nothing to be done for 'header'.
Reached EOF with exit status 0

MODULE jtag-bridge:jtag-bridge command checkout
Cloning into '/home/speedzheng/hero/hero-sdk/pulp-sdk/tools/jtag-bridge'...
git@iis-git.ee.ethz.ch's password:
Permission denied, please try again.
git@iis-git.ee.ethz.ch's password:
Permission denied, please try again.
git@iis-git.ee.ethz.ch's password:

@vogelpi
Copy link
Contributor

vogelpi commented Sep 18, 2018

Hi,
thanks for reporting the issue. It seems that the builder tries to checkout three repositories from our internal server. These should not be needed anymore. But I am currently doing a complete rebuild without these repositories to be sure.
We let you know asap.

Best regards,
Pirmin

vogelpi added a commit that referenced this issue Sep 18, 2018
@vogelpi
Copy link
Contributor

vogelpi commented Sep 18, 2018

We could fix the issue and successfully tested it on the board. Please checkout the latest tag v1.0.1.

Best regards,
PIrmin

@speedzheng
Copy link
Author

speedzheng commented Sep 18, 2018 via email

@vogelpi vogelpi closed this as completed Sep 19, 2018
@speedzheng
Copy link
Author

Hi,

I just clone the rel-v1.0.1-rc2 but met another similar error and did I clone the wrong branch?

git checkout -b

HEAD is now at 34a9206 Fixed bug, register 0 was sometimes written

sdk:vp:checkout: make checkout

MODULE pulp-rt:pulp-rt command checkout
HEAD is now at e923766 Few fixes for software implementation of barriers and locks

MODULE pc_analyzer:pc_analyzer command checkout
Cloning into '/home/speedzheng/hero/hero-sdk/pulp-sdk/tools/pc_analyzer'...
git@iis-git.ee.ethz.ch's password:

@vogelpi vogelpi reopened this Sep 19, 2018
@vogelpi
Copy link
Contributor

vogelpi commented Sep 19, 2018

Hi,

it seems that the builder wants to build more modules than actually needed. Unfortunately, I cannot fix it right now. Can you please try the following:

In hero-sdk, open the file hero-z-7045-builder and then you comment Line 83, i.e.,
make env all
turns into
#make env all

Then you do again hero-z-7045-builder -A. Please let me know if it works.

Best regards,
Pirmin

@alessandrocapotondi alessandrocapotondi self-assigned this Sep 19, 2018
@alessandrocapotondi alessandrocapotondi added the bug Something isn't working label Sep 19, 2018
@alessandrocapotondi
Copy link
Contributor

Dear @speedzheng,
thanks for your reports. I see the problem and I am fixing right now. I will push the fix as soon as I finish to test it.

Best,
Alessandro

@speedzheng
Copy link
Author

Hi, just tried but same issue. Do you need to sync to rel-v1.0.1-rc2 again? Thanks!

Reached EOF with exit status 0
git submodule update --init
./pulp-tools/bin/plpbuild --p sdk checkout build --stdout

MODULE pc_analyzer:pc_analyzer command checkout
Cloning into '/home/speedzheng/hero/hero-sdk/pulp-sdk/tools/pc_analyzer'...
git@iis-git.ee.ethz.ch's password:

@alessandrocapotondi
Copy link
Contributor

It is not yet merged. I will merge it into the master and I will let you know.

@speedzheng
Copy link
Author

Now it works. Thanks for great support.

@andreaskurth
Copy link
Contributor

Thanks for your feedback!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants