Re: CVS commit: src/usr.sbin/crash

2012-02-18 Thread Izumi Tsutsui
 In article 20120217122414.7254417...@cvs.netbsd.org,
 Martin Husemann source-changes-d@NetBSD.org wrote:
 -=-=-=-=-=-
 
 Module Name: src
 Committed By:martin
 Date:Fri Feb 17 12:24:14 UTC 2012
 
 Modified Files:
  src/usr.sbin/crash: Makefile
 
 Log Message:
 Fix build for sparc64
 
 I'd rather re-organize the contents of the files so that sparc64 is not
 different than everyone else.

m68k and arm didn't have db_machdep.c in old Makefile and
they are also broken, so sparc64 is not alone ;-p
http://releng.netbsd.org/builds/HEAD/201202171310Z/

What checks did you do before committing your refactoring?
---
Izumi Tsutsui


Re: CVS commit: src/usr.sbin/crash

2012-02-18 Thread Izumi Tsutsui
 http://releng.netbsd.org/builds/HEAD/201202171310Z/
 
 What checks did you do before committing your refactoring?
 
 None, this is why I waited after the branch. I am fixing sparc64 now.

Can you also fix arm and m68k? Or should I put kludge?
On m68k only sun3 (which has different pmap) has db_machdep.c.

---
Izumi Tsutsui


Re: CVS commit: src/usr.sbin/crash

2012-02-18 Thread Christos Zoulas
On Feb 19,  1:08am, tsut...@ceres.dti.ne.jp (Izumi Tsutsui) wrote:
-- Subject: Re: CVS commit: src/usr.sbin/crash

|  http://releng.netbsd.org/builds/HEAD/201202171310Z/
|  
|  What checks did you do before committing your refactoring?
|  
|  None, this is why I waited after the branch. I am fixing sparc64 now.
| 
| Can you also fix arm and m68k? Or should I put kludge?
| On m68k only sun3 (which has different pmap) has db_machdep.c.

I thought arm works. I am fixing mvme68k now. I am trying to make things
work without the kludge that is why I left it broken.

christos


Re: CVS commit: src/usr.sbin/crash

2012-02-17 Thread Martin Husemann
On Fri, Feb 17, 2012 at 02:15:33PM +, Christos Zoulas wrote:
 I'd rather re-organize the contents of the files so that sparc64 is not
 different than everyone else.

I just needed a quick build fix - feel free to rename/move...

Martin