[haiku-development] Re: Proposal for removing "ftp" from default builtin tools

  • From: Kacper Kasper <kacperkasper@xxxxxxxxx>
  • To: haiku-development@xxxxxxxxxxxxx
  • Date: Mon, 22 Jul 2019 10:34:30 +0200

niedz., 21 lip 2019 o 20:07 Adrien Destugues
<pulkomandy@xxxxxxxxxxxxx> napisał(a):


Hi,

As you know if you're following the list our sourcecode was recently
scanned again by PVS studio. I'm currently going through their report
and fixing some of the issues.

A lot of the problems are raised in the code for our FTP client
implementation. The code looks quite messy and I don't feel very
motivated to fix it. I would suggest we simply remove it and instead
ship another FTP client instead. I would suggest tnftp, which is the one
used by all the *BSD currently. It is already packaged at haikuports and
at 144K, wouldn't be an heavy addition to our default install if we
choose to put it there, and it is under a BSD license.

This would allow us to remove a lot of the warnings from the next PVS
studio analysis.

If there is general agreement on this I will remove the ftp client from
our sources, and add an alias "ftp" to the tnftp package (currently the
binary there is named tnftp).

--
Adrien.


+1 from me. The less code we have to maintain in the repo the better.

- Kacper

Other related posts: