siemens / kas

Setup tool for bitbake based projects
MIT License
353 stars 144 forks source link

Silently fails if permission denied when setting up repositories #26

Closed rossburton closed 3 years ago

rossburton commented 3 years ago

My kas in CI just failed with the following mysterious errors:

1202020-10-13 10:53:46 - INFO     - /builds/poky$ git remote set-url origin https://git.yoctoproject.org/git/poky
1222020-10-13 10:53:46 - INFO     - Changing remote URL to https://git.yoctoproject.org/git/poky failed with error: 

Note the absence of error messages.

The problem was that kas was running as a normal user but the poky checkout was owned by root.

jan-kiszka commented 3 years ago

Do we get a proper message when that command is run in the shell? I do not spot yet where we should lose output.

jan-kiszka commented 3 years ago

Sent a patch to the list.

I suspect there are more cases of improper "fail=False" settings, but one after the other.

jan-kiszka commented 3 years ago

merged to master