Re: CVS commit: src/external/bsd/top/dist

[Pierre Pronchery <khorben%defora.org@localhost>]

On 03/06/2018 05:10, Christos Zoulas wrote:
> On Jun 2,  7:36pm, lists%eitanadler.com@localhost (Eitan Adler) wrote:
> -- Subject: Re: CVS commit: src/external/bsd/top/dist
> 
> | I don't mind reverting this: it came out of a discussion on IRC. That
> | said, perhaps "cd /tmp" would resolve both issues?
> 
> This is a 3rd party program that behaves everywhere the same way;
> changing it for NetBSD will just produce unexpected results for
> the user:  How will I even know to cd to /tmp to look for the core?
> As I said, if you want it to run from a different directory, wrap
> it in a shell script or use an alias. These user-specific customizations
> don't belong in a general purpose program. It is a slippery slope
> to start making programs behave in random ways just because it is
> convenient for some uses. For example, I've never needed top to
> chdir before it grew a 'machine' subdirectory to fix the mess of
> ifdefs it required to run on multiple OS's. Are you planning to
> change vmstat to do the same? iostat? systat? Why not? These changes
> don't make sense (to me).

I agree with Christos here.

Cheers,
-- 
khorben