[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 20, 2013 at 02:15:15PM +1000, Michael Ellerman wrote:
 > On Wed, 2013-06-19 at 22:40 -0400, Dave Jones wrote:
 > > Nothing wrong with this patch, but it's highlighted a bug that's been in trinity
 > > for a while.  Changes to the syscall struct should cause everything in syscalls/*
 > > to be rebuilt. But somehow the dependency magic in the Makefile doesn't pick it up.
 > > 
 > > Anyone with better make-fu than me want to take a stab at that ?
 > 
 > I'm not a make expert, but I think it's because it's looking for .deps/syscalls/foo.d
 > whereas we generate .deps/foo.d

Bah, I knew it would be something that simple that I was overlooking.

 > This patch works for me, touching include/syscall.h rebuilds a few
 > things and then everything under syscalls.

Looks good to me.

 > I guess the other approach would be to flatten the directory structure
 > where we include the .d files.
 
That sounds like it would be a mess if we're unfortunate enough to have the same
name file in 2 dirs.  Your patch seems to do the right thing, so I'll
just apply that and move on.

thanks

	Dave
--
To unsubscribe from this list: send the line "unsubscribe trinity" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SCSI]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux