Skip to content

Conversation

pevik
Copy link
Contributor

@pevik pevik commented Jul 28, 2021

Lets drop software which does not make much sense to carry out.

NOTE: later something else to remove? I'd suggest tftpd (dnsmasq has much better alternative) and consider also rarpd, rdisc.

@pevik pevik requested review from kerolasa, nmeyerhans, okias and a team July 28, 2021 08:03
@pevik
Copy link
Contributor Author

pevik commented Jul 28, 2021

@zenczykowski
Copy link
Contributor

Probably ifenslave too - ip should be used instead.

@pevik
Copy link
Contributor Author

pevik commented Jul 28, 2021

Probably `ifenslave; too - ip should be used instead.

ifenslave is not from iputils.

@zenczykowski
Copy link
Contributor

zenczykowski commented Jul 28, 2021

# rpm -q --filesbypkg iputils-20210202-2.fc34.x86_64 | egrep bin
iputils /usr/bin/arping
iputils /usr/bin/clockdiff
iputils /usr/bin/ping
iputils /usr/bin/tracepath
iputils /usr/sbin/arping
iputils /usr/sbin/ifenslave
iputils /usr/sbin/ping
iputils /usr/sbin/ping6
iputils /usr/sbin/rdisc
iputils /usr/sbin/tracepath
iputils /usr/sbin/tracepath6

@pevik
Copy link
Contributor Author

pevik commented Jul 28, 2021

@nmeyerhans
Copy link
Contributor

nmeyerhans commented Jul 28, 2021

I'm in favor of dropping traceroute6.

I'd also be in favor of dropping tftpd and ninfod. Possibly rarpd and rdisc as well, but I'm not familiar with the quality of the alternatives.

@okias
Copy link
Member

okias commented Jul 28, 2021

I would propose link to suitable replacement solution from build process, when traceroute6 will be requested

@neheb
Copy link
Contributor

neheb commented Jul 28, 2021

Fine with me.

pevik added 2 commits July 28, 2021 18:08
traceroute6 requires inetd to run, that is quite a rarity nowadays.
Due this it has been disabled since 2615a52 ("build-sys: change build
defaults") (released in s20190324) because there are much better
implementation from Dmitry Butskoy [1]. Also it can be replaced by
other tools, e.g. tracepath -6 or mtr.

[1] http://traceroute.sourceforge.net/

Signed-off-by: Petr Vorel <pvorel@suse.cz>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
@pevik pevik force-pushed the remove/traceroute6 branch from f04637a to 309a8c1 Compare July 28, 2021 16:09
@pevik pevik mentioned this pull request Jul 28, 2021
@pevik
Copy link
Contributor Author

pevik commented Jul 28, 2021

Thanks! I'll wait few more days before removing it. I also created separated ticket to move the discussion there.

@pevik pevik closed this in a139421 Aug 4, 2021
@pevik pevik deleted the remove/traceroute6 branch August 30, 2021 11:15
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this pull request Dec 16, 2021
tftp and traceroute6 has been removed in this release [1][2].
Due that LICENSE file has been changed.

Also remove patch from this release.

[1] iputils/iputils#369
[2] iputils/iputils#362

Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
algitbot pushed a commit to alpinelinux/aports that referenced this pull request Apr 15, 2022
tftp and traceroute6 has been removed in this release [1][2].

[1] iputils/iputils#369
[2] iputils/iputils#362

Signed-off-by: Petr Vorel <petr.vorel@gmail.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.

5 participants