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

[Marvell-arm64]: Fix SYNCD_RPC build #17266

Merged
merged 1 commit into from Feb 12, 2024
Merged

Conversation

pavannaregundi
Copy link
Contributor

Change-Id: I0bd4932d03141f3f7bc523b49a1bf3d1809817a8

Why I did it

With 'ENABLE_SYNCD_RPC' flag enabled, compilation for marvell-arm64 was failing. Failure was due to bookworm now being default docker and absence of python2 support in it.

dh: error: unable to load addon python2: Can't locate Debian/Debhelper/Sequence/python2.pm in @inc (you may need to install the Debian::Debhelper::Sequence::python2 module) (@inc contains: /etc/perl /usr/local/lib/aarch64-linux-gnu/perl/5.36.0 /usr/local/share/perl/5.36.0 /usr/lib/aarch64-linux-gnu/perl5/5.36 /usr/share/perl5 /usr/lib/aarch64-linux-gnu/perl-base /usr/lib/aarch64-linux-gnu/perl/5.36 /usr/share/perl/5.36 /usr/local/lib/site_perl) at (eval 11) line 1.
BEGIN failed--compilation aborted at (eval 11) line 1.
make[2]: *** [debian/rules:91: clean] Error 25
make[2]: Leaving directory '/sonic/src/thrift/thrift-0.11.0'
dpkg-buildpackage: error: fakeroot debian/rules clean subprocess returned exit status 2
make[1]: *** [Makefile:16: /sonic/target/debs/bookworm/libthrift-0.11.0_0.11.0-4_arm64.deb] Error 2
make[1]: Leaving directory '/sonic/src/thrift'

Work item tracking
  • Microsoft ADO (number only):

How I did it

Marking syncd_rpc components also to be built with bullsyeye.

How to verify it

Completion of build with 'ENABLE_SYNCD_RPC' flag enabled and checked for "show interface up" by loading the image.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Change-Id: I0bd4932d03141f3f7bc523b49a1bf3d1809817a8
Signed-off-by: Pavan Naregundi <pnaregundi@marvell.com>
@pavannaregundi
Copy link
Contributor Author

@lguohan Can we merge this?

@pavannaregundi
Copy link
Contributor Author

@lguohan Can we merge this PR

@yxieca yxieca merged commit c6602c9 into sonic-net:master Feb 12, 2024
12 checks passed
sonic-otn pushed a commit to Weitang-Zheng/sonic-buildimage that referenced this pull request Mar 11, 2024
Change-Id: I0bd4932d03141f3f7bc523b49a1bf3d1809817a8

Signed-off-by: Pavan Naregundi <pnaregundi@marvell.com>
saksarav-nokia pushed a commit to saksarav-nokia/sonic-buildimage that referenced this pull request Mar 12, 2024
Change-Id: I0bd4932d03141f3f7bc523b49a1bf3d1809817a8

Signed-off-by: Pavan Naregundi <pnaregundi@marvell.com>
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.

None yet

3 participants