Re: CVS import: src/external/bsd/llvm/dist/llvm

2017-08-01 Thread Joerg Sonnenberger
On Tue, Aug 01, 2017 at 10:36:09AM +, Joerg Sonnenberger wrote:
> Module Name:  src
> Committed By: joerg
> Date: Tue Aug  1 10:36:09 UTC 2017
> 
> Update of /cvsroot/src/external/bsd/llvm/dist/llvm
> In directory ivanova.netbsd.org:/tmp/cvs-serv21315
> 
> Log Message:
> Import Clang 5.0rc1 r309604.

So this one obviously went into the wrong directory. I've asked Petra to
restore the files from the last backup and recommitted both the LLVM and
clang import, now in the correct location. Sorry about that.

Joerg


Re: CVS commit: src

2017-08-01 Thread coypu
On Tue, Aug 01, 2017 at 06:28:39PM +0200, Joerg Sonnenberger wrote:
> Correct, do not remove obsolete entries from the set files without a
> very good reason. This one is plain wrong.

Thanks, I reverted that part.


Re: CVS commit: src

2017-08-01 Thread Martin Husemann
On Tue, Aug 01, 2017 at 07:13:22PM +0200, Maxime Villard wrote:
> Mmh, didn't know that. What is the policy to remove entries then? For
> a release old enough? Several svr4_machdep.h entries have been in the other
> md.* files for years.

Never remove obsolete entries unless you added them eroneously (i.e. the
file is actually still installed but accidently marked as obsolete).

The obsolete entries are used to remove files no longer needed on system
updates. We do not expect users to update to every major release, so
if, say, someone updates from NetBSD 0.9 to NetBSD 8, the file should
still be removed.

Martin


Re: CVS commit: src

2017-08-01 Thread coypu
On Tue, Aug 01, 2017 at 01:57:03PM +, Maxime Villard wrote:
> Module Name:  src
> Committed By: maxv
> Date: Tue Aug  1 13:57:03 UTC 2017
> 
> Modified Files:
>   src/distrib/sets/lists/comp: md.amd64 md.i386
> Removed Files:
>   src/sys/arch/i386/include: svr4_machdep.h
> 
> Log Message:
> Remove svr4_machdep.h right away, no one should include it.

I think obsolete means it will be removed by postinstall if asked to
remove obsolete files