Navigation Menu

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

iproute: 5.8.0 -> 5.9.0 #102475

Closed
wants to merge 1 commit into from
Closed

Conversation

r-ryantm
Copy link
Contributor

@r-ryantm r-ryantm commented Nov 2, 2020

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/iproute2/versions.

meta.description for iproute is: "A collection of utilities for controlling TCP/IP networking and traffic control in Linux"

meta.homepage for iproute is: "https://wiki.linuxfoundation.org/networking/iproute2"

meta.changelog for iproute is: ""

Updates performed
  • Version update
To inspect upstream changes
Impact
Checks done (click to expand)

Rebuild report (if merged into master) (click to expand)
2792 total rebuild path(s)

1012 package rebuild(s)

1012 x86_64-linux rebuild(s)
884 i686-linux rebuild(s)
29 x86_64-darwin rebuild(s)
867 aarch64-linux rebuild(s)


First fifty rebuilds by attrpath
Sylk
acme-sh
adapta-gtk-theme
almanah
apostrophe
appimage-run
arc-theme
areca
arx-libertatis
asciidoc-full
asciidoc-full-with-plugins
astroid
azureus
balsa
bareos
baresip
bashSnippets
bibata-cursors
bibata-cursors-translucent
bibata-extra-cursors
birdfont
blueman
bolt
bomi
bookworm
btrbk
calligra
calls
capitaine-cursors
cawbird
ccache
ccacheStdenv
ccacheWrapper
ceph
ceph-client
ceph-dev
charliecloud
chrome-gnome-shell
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.iso-flags-png-320x420
cinnamon.iso-flags-svg
citrix_workspace
citrix_workspace_20_04_0
citrix_workspace_20_06_0
citrix_workspace_20_09_0
citrix_workspace_20_10_0
claws-mail
Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/bii3q5q1sj6g2hr6isalhj5hjfxxs1c2-iproute2-5.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A iproute https://github.com/r-ryantm/nixpkgs/archive/084ec06860ad0b437842ecba53c2f7405578916d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/bii3q5q1sj6g2hr6isalhj5hjfxxs1c2-iproute2-5.9.0
ls -la /nix/store/bii3q5q1sj6g2hr6isalhj5hjfxxs1c2-iproute2-5.9.0/bin


Pre-merge build results

NixPkgs review skipped


Maintainer pings

cc @primeos @edolstra @fpletz @globin for testing.


src = fetchurl {
url = "mirror://kernel/linux/utils/net/${pname}/${pname}-${version}.tar.xz";
sha256 = "0vk4vickrpahdhl3zazr2qn2bf99v5549ncirjpwiy4h0a4izkfg";
sha256 = "1kys6dmhrl43iaq95n5sh02p39d7bq8i5y672qrzgwnwpjaaqpd2";
};

preConfigure = ''
# Don't try to create /var/lib/arpd:
sed -e '/ARPDDIR/d' -i Makefile
# TODO: Drop temporary version fix for 5.8 (53159d81) once 5.9 is out:
Copy link
Member

Choose a reason for hiding this comment

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

This todo needs to be addressed and the merge conflict.

@primeos
Copy link
Member

primeos commented Nov 2, 2020

This change is already in staging: 01ec8c0
Does @r-ryantm maybe only consider commits in master? Nvm, it's already in master (at least now), not sure what happened here.

@primeos primeos closed this Nov 2, 2020
@ryantm
Copy link
Member

ryantm commented Nov 2, 2020

@primeos it does check staging too so I'm not sure what happened.

@primeos
Copy link
Member

primeos commented Nov 3, 2020

Oh, forgot to update my comment, it seems to be due to #101207 (comment). Basically: Staging was accidentally merged into master, the merge commit was than reverted, and that revert is also in staging now. So both master and staging contain 01ec8c0 but the actual patch was reverted. So until #101207 is merged that is unfortunately a bit of a problem :o

@r-ryantm r-ryantm deleted the auto-update/iproute2 branch November 10, 2020 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants