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

[build] libyang2 fails to build in master #13504

Open
dgsudharsan opened this issue Jan 25, 2023 · 3 comments
Open

[build] libyang2 fails to build in master #13504

dgsudharsan opened this issue Jan 25, 2023 · 3 comments
Labels
Triaged this issue has been triaged

Comments

@dgsudharsan
Copy link
Collaborator

Description

We are noticing the following build failure in the master branch. I believe this PR introduces dget #13394 .

[2023-01-25T03:16:57.941Z] # download debian libyang
[2023-01-25T03:16:57.941Z] dget https://deb.debian.org/debian/pool/main/liby/libyang2/libyang2_2.0.112-6.dsc
[2023-01-25T03:16:57.941Z] pushd libyang2-2.0.112
[2023-01-25T03:16:57.941Z] #sed -i 's/set(LIBYANG_MAJOR_SOVERSION 1)/set(LIBYANG_MAJOR_SOVERSION 2)/' CMakeLists.txt
[2023-01-25T03:16:57.941Z] #sed -i 's/libyang2/libyang2/' debian/libyang2.install
[2023-01-25T03:16:57.941Z] # Enable large file support for 32-bit arch
[2023-01-25T03:16:57.941Z] echo 'add_definitions(-D_FILE_OFFSET_BITS=64)' >> CMakeLists.txt
[2023-01-25T03:16:57.941Z] dpkg-buildpackage -rfakeroot -b -us -uc -j30 --admindir /sonic/dpkg/tmp.cmQpw3g5UU
[2023-01-25T03:16:57.941Z] popd
[2023-01-25T03:16:57.941Z] # Move the newly-built .deb packages to the destination directory
[2023-01-25T03:16:57.941Z] mv libyang2_2.0.112-6_amd64.deb libyang2-dev_2.0.112-6_amd64.deb libyang2-dbgsym_2.0.112-6_amd64.deb libyang2-tools_2.0.112-6_amd64.deb libyang2-tools-dbgsym_2.0.112-6_amd64.deb /sonic/target/debs/buster/
[2023-01-25T03:16:57.942Z] dget: retrieving https://deb.debian.org/debian/pool/main/liby/libyang2/libyang2_2.0.112-6.dsc
[2023-01-25T03:16:57.942Z] chown: changing ownership of '/vcache': Operation not permitted
[2023-01-25T03:16:57.942Z] get_url_version https://deb.debian.org/debian/pool/main/liby/libyang2/libyang2_2.0.112-6.dsc failed
[2023-01-25T03:16:57.942Z] dget: curl libyang2_2.0.112-6.dsc https://deb.debian.org/debian/pool/main/liby/libyang2/libyang2_2.0.112-6.dsc failed
[2023-01-25T03:16:57.942Z] make[1]: *** [Makefile:10: /sonic/target/debs/buster/libyang2_2.0.112-6_amd64.deb] Error 1
[2023-01-25T03:16:57.942Z] make[1]: Leaving directory '/sonic/src/libyang2'
[2023-01-25T03:16:57.942Z] [ FAIL LOG END ] [ target/debs/buster/libyang2_2.0.112-6_amd64.deb ]

Steps to reproduce the issue:

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@guillaumelambert
Copy link
Contributor

guillaumelambert commented Jan 25, 2023

Libyang2 is only available in debian testing.
The version 2.0.112 of libyang2 was removed from Debian official mirror.
Corresponding sources are no more available there.
But they are still available for the moment in SONiC mirror at https://sonicstorage.blob.core.windows.net/debian/pool/main/liby/libyang/libyang2_2.0.112-6.dsc
PR #13394 shifted from wget to dget but also from SONiC mirror to Debian ones.
I propose a new PR to only shift to dget and keep the SONiC mirror.
#13517

Ideally, this package should be upgraded to version 2.1.30 which is currently the only one available in debian official mirrors.
https://deb.debian.org/debian/pool/main/liby/libyang2/

@qiluo-msft
Copy link
Collaborator

@guillaumelambert Could you share the new PR link? The above link 13504 is an issue link.

@guillaumelambert
Copy link
Contributor

guillaumelambert commented Jan 26, 2023

@guillaumelambert Could you share the new PR link? The above link 13504 is an issue link.

Right, my mistake. I''ve updated it directly in the original message.

@gechiang gechiang added the Triaged this issue has been triaged label Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triaged this issue has been triaged
Projects
None yet
Development

No branches or pull requests

4 participants