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.

Sign up for GitHub

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

Jump to bottom

[202211][buildsystem]: Fix hiredis package version: 0.14.0-3. #15476

Merged
merged 1 commit into from
Jun 19, 2023
Merged

[202211][buildsystem]: Fix hiredis package version: 0.14.0-3. #15476

merged 1 commit into from
Jun 19, 2023

Conversation

nazariig
Copy link
Collaborator

@nazariig nazariig commented Jun 15, 2023 •
edited
Loading

Build error:

18:20:27  [ finished ] [ target/debs/buster/libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb ]
18:20:27  [ FAIL LOG START ] [ target/debs/buster/libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb ]
18:20:27  [ REASON ] :      target/debs/buster/libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb does not exist  
18:20:27  [ FLAGS  FILE    ] : [mellanox amd64 buster y buster]
18:20:27  [ FLAGS  DEPENDS ] : [mellanox amd64 buster y buster]
18:20:27  [ FLAGS  DIFF    ] : []
18:20:27  make[1]: warning: -j30 forced in submake: resetting jobserver mode.
18:20:27  make[1]: Entering directory '/sonic/src/hiredis'
18:20:27  rm -rf hiredis-0.14.0
18:20:27  wget -O hiredis_0.14.0.orig.tar.gz http://http.debian.net/debian/pool/main/h/hiredis/hiredis_0.14.0.orig.tar.gz
18:20:27  wget -O hiredis_0.14.0-3~bpo9+1.debian.tar.xz http://http.debian.net/debian/pool/main/h/hiredis/hiredis_0.14.0-3~bpo9+1.debian.tar.xz
18:20:27  wget -O hiredis_0.14.0-3~bpo9+1.dsc http://http.debian.net/debian/pool/main/h/hiredis/hiredis_0.14.0-3~bpo9+1.dsc
18:20:27  dpkg-source -x hiredis_0.14.0-3~bpo9+1.dsc
18:20:27  pushd hiredis-0.14.0
18:20:27  dpkg-buildpackage -rfake******** -d -b -us -uc -j30 --********dir /sonic/dpkg/tmp.ALm1Mqoiea
18:20:27  popd
18:20:27  mv libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb libhiredis0.14-dbgsym_0.14.0-3~bpo9+1_amd64.deb libhiredis-dev_0.14.0-3~bpo9+1_amd64.deb /sonic/target/debs/buster/
18:20:27  /usr/local/share/buildinfo/scripts/buildinfo_base.sh: line 165: /usr/local/share/buildinfo/build-versions/versions-web: Permission denied
18:20:27  --2023-06-14 15:20:25--  http://http.debian.net/debian/pool/main/h/hiredis/hiredis_0.14.0.orig.tar.gz
18:20:27  Resolving http.debian.net (http.debian.net)... 130.89.148.77, 128.31.0.62, 149.20.4.15, ...
18:20:27  Connecting to http.debian.net (http.debian.net)|130.89.148.77|:80... connected.
18:20:27  HTTP request sent, awaiting response... 302 Found
18:20:27  Location: http://cdn-fastly.deb.debian.org/debian/pool/main/h/hiredis/hiredis_0.14.0.orig.tar.gz [following]
18:20:27  --2023-06-14 15:20:25--  http://cdn-fastly.deb.debian.org/debian/pool/main/h/hiredis/hiredis_0.14.0.orig.tar.gz
18:20:27  Resolving cdn-fastly.deb.debian.org (cdn-fastly.deb.debian.org)... 146.75.122.132, 2a04:4e42:8e::644
18:20:27  Connecting to cdn-fastly.deb.debian.org (cdn-fastly.deb.debian.org)|146.75.122.132|:80... connected.
18:20:27  HTTP request sent, awaiting response... 200 OK
18:20:27  Length: 63061 (62K) [application/x-gzip]
18:20:27  Saving to: 'hiredis_0.14.0.orig.tar.gz'
18:20:27  
18:20:27       0K .......... .......... .......... .......... .......... 81%  716K 0s
18:20:27      50K .......... .                                          100% 1.79M=0.08s
18:20:27  
18:20:27  2023-06-14 15:20:26 (809 KB/s) - 'hiredis_0.14.0.orig.tar.gz' saved [63061/63061]
18:20:27  
18:20:27  /usr/local/share/buildinfo/scripts/buildinfo_base.sh: line 165: /usr/local/share/buildinfo/build-versions/versions-web: Permission denied
18:20:27  --2023-06-14 15:20:26--  http://http.debian.net/debian/pool/main/h/hiredis/hiredis_0.14.0-3~bpo9+1.debian.tar.xz
18:20:27  Resolving http.debian.net (http.debian.net)... 149.20.4.15, 130.89.148.77, 128.31.0.62, ...
18:20:27  Connecting to http.debian.net (http.debian.net)|149.20.4.15|:80... connected.
18:20:27  HTTP request sent, awaiting response... 302 Found
18:20:27  Location: http://cdn-fastly.deb.debian.org/debian/pool/main/h/hiredis/hiredis_0.14.0-3~bpo9+1.debian.tar.xz [following]
18:20:27  --2023-06-14 15:20:27--  http://cdn-fastly.deb.debian.org/debian/pool/main/h/hiredis/hiredis_0.14.0-3~bpo9+1.debian.tar.xz
18:20:27  Resolving cdn-fastly.deb.debian.org (cdn-fastly.deb.debian.org)... 146.75.122.132, 2a04:4e42:8e::644
18:20:27  Connecting to cdn-fastly.deb.debian.org (cdn-fastly.deb.debian.org)|146.75.122.132|:80... connected.
18:20:27  HTTP request sent, awaiting response... 404 Not Found
18:20:27  2023-06-14 15:20:27 ERROR 404: Not Found.
18:20:27  
18:20:27  make[1]: *** [Makefile:9: /sonic/target/debs/buster/libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb] Error 8
18:20:27  make[1]: Leaving directory '/sonic/src/hiredis'
18:20:27  [  FAIL LOG END  ] [ target/debs/buster/libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb ]
18:20:27  make: *** [slave.mk:658: target/debs/buster/libhiredis0.14_0.14.0-3~bpo9+1_amd64.deb] Error 1
18:20:27  make: *** Waiting for unfinished jobs....

Backport of: #15461

Why I did it

  • To fix hiredis compilation
Work item tracking
  • N/A

How I did it

  • Changed package version: 0.14.0-3~bpo9+1 -> 0.14.0-3

How to verify it

  1. make configure PLATFORM=mellanox
  2. make target/sonic-mellanox.bin

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

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

Tested branch (Please provide the tested image version)

  • 202211

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

  • N/A

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

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

@liushilongbuaa
Copy link
Contributor

liushilongbuaa commented Jun 16, 2023 •
edited
Loading

@nazariig , files/build/versions will be refreshed automatically.
If you want to pin a package version, just change rules and Makefile.

@nazariig
[buildsystem]: Fix hiredis package version: 0.14.0-3.
791ca15 
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@nazariig nazariig force-pushed the 202211-hiredis-fix branch from 0a0f587 to 791ca15 Compare June 16, 2023 08:36
@nazariig
Copy link
Collaborator Author

@nazariig , files/build/versions will be refreshed automatically. If you want to pin a package version, just change rules and Makefile.

@liushilongbuaa fixed

@nazariig
Copy link
Collaborator Author

@liushilongbuaa would you please help to merge?

liushilongbuaa
liushilongbuaa approved these changes Jun 19, 2023
View reviewed changes
Copy link
Contributor

@liushilongbuaa liushilongbuaa left a comment

Choose a reason for hiding this comment

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

LGTM. I don't have permission to merge.

@keboliu
Copy link
Collaborator

keboliu commented Jun 19, 2023

@StormLiangMS would you please help to merge?

StormLiangMS
StormLiangMS approved these changes Jun 19, 2023
View reviewed changes
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit f593323 into sonic-net:202211 Jun 19, 2023
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-buildimage that referenced this pull request Jun 21, 2023
@nazariig @Junchao-Mellanox
[buildsystem]: Fix hiredis package version: 0.14.0-3. (sonic-net#15476)
3cfd570 
Backport of: sonic-net#15461

Why I did it
To fix hiredis compilation
Work item tracking
N/A
How I did it
Changed package version: 0.14.0-3~bpo9+1 -> 0.14.0-3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StormLiangMS StormLiangMS StormLiangMS approved these changes

@liushilongbuaa liushilongbuaa liushilongbuaa approved these changes

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft is a code owner

@xumia xumia Awaiting requested review from xumia xumia is a code owner

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

@liat-grozovik liat-grozovik Awaiting requested review from liat-grozovik

@volodymyrsamotiy volodymyrsamotiy Awaiting requested review from volodymyrsamotiy

Assignees
No one assigned
Labels
Bug 🐛 Build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nazariig @liushilongbuaa @keboliu @StormLiangMS