Re: CVS commit: src/sys/external/isc/atheros_hal/dist

[christos%zoulas.com@localhost (Christos Zoulas)]

On Dec 31,  1:14pm, lacombar%gmail.com@localhost ("Arnaud Lacombe") wrote:
-- Subject: Re: CVS commit: src/sys/external/isc/atheros_hal/dist

| On Wed, Dec 31, 2008 at 9:08 AM, Christos Zoulas 
<christos%netbsd.org@localhost> wrote:
| > NB: These were not imported properly. The Id RCS keyword lines were not
| > cleaned up,
| Shouldn't our CVS only interpret $NetBSD$ tag and not every $Id$.

Well, historically it has. Plus users (non-netbsd-developers) use "our CVS"
for their own repositories.

| > and there were no NetBSD keywords added.
| >
| I hesitated to do add /* $NetBSD$ */ on top of every file. I'll add
| them later today.

It's ok either way.
| 
| > Perhaps Re-import correctly?
| >
| I don't see any reason to modify the vendor branch with this. This
| should be done on HEAD.

Adding $NetBSD$ is fine, fixing the $Id$ I don't see how you can do it
only on head.

|  - Arnaud
| 
| ps: in all case, couldn't RCS ID change not be included in commit
| diff, that very noisy :/

I know, but this is CVS, remember?

christos