Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[arm] [build_debian] arm build fix #6289

Merged
merged 1 commit into from
Dec 24, 2020

Conversation

Sabareesh-Kumar-Anandan
Copy link
Contributor

@Sabareesh-Kumar-Anandan Sabareesh-Kumar-Anandan commented Dec 24, 2020

Clean up qemu-arm-static bin after collecting post-versions info

Signed-off-by: Sabareesh Kumar Anandan sanandan@marvell.com

- Why I did it
scripts/collect_host_image_version_files.sh fails with below error:

  • scripts/collect_host_image_version_files.sh target ./fsroot
    /usr/sbin/chroot: failed to run command 'post_run_buildinfo': No such file or directory
    /bin/cp: cannot stat './fsroot/usr/local/share/buildinfo/post-versions': No such file or directory

- How I did it
Issues is because qemu-arm-static is removed before this step. So, I moved the cleanup step to the end.

- How to verify it
Compiled and verified for marvell-armhf

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Clean up qemu-arm bins after collecting post-versions info

Signed-off-by: Sabareesh Kumar Anandan <sanandan@marvell.com>
@lguohan lguohan requested a review from xumia December 24, 2020 09:09
build_debian.sh Show resolved Hide resolved
Copy link
Collaborator

@xumia xumia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@lguohan lguohan merged commit 9aae9c9 into sonic-net:master Dec 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants