Skip to content

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

Merged
StormLiangMS merged 1 commit intosonic-net:202211from
nazariig:202211-hiredis-fix
Jun 19, 2023
Merged

[202211][buildsystem]: Fix hiredis package version: 0.14.0-3.#15476
StormLiangMS merged 1 commit intosonic-net:202211from
nazariig:202211-hiredis-fix

Conversation

@nazariig
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator Author

nazariig commented Jun 16, 2023

@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
Copy Markdown
Collaborator Author

nazariig commented Jun 16, 2023

@liushilongbuaa would you please help to merge?

liushilongbuaa
liushilongbuaa approved these changes Jun 19, 2023
View reviewed changes
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

@liushilongbuaa liushilongbuaa liushilongbuaa approved these changes

@StormLiangMS StormLiangMS StormLiangMS 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