On Mon, Dec 22, 2008 at 06:53:36PM +0100, Sam Ravnborg wrote:
> On Mon, Dec 22, 2008 at 09:26:43AM -0800, Inaky Perez-Gonzalez wrote:
> > On Sunday 21 December 2008, Sam Ravnborg wrote:
> > > On Mon, Dec 22, 2008 at 01:42:21PM +1100, Stephen Rothwell wrote:
> > > > (Just cc'ing Sam)
> > > >
> > > > On Sun, 21 Dec 2008 17:31:10 -0800 Greg KH <greg@xxxxxxxxx> wrote:
> > > > > On Mon, Dec 22, 2008 at 11:33:35AM +1100, Stephen Rothwell wrote:
> > > > > > Hi Greg,
> > > > > >
> > > > > > Today's linux-next build (x86_64 allmodconfig) failed like this:
> > > > > >
> > > > > > x86_64-linux-ld: drivers/net/wimax/built-in.o: No such file: No such
> > > > > > file or directory
> > > > > >
> > > > > > I reverted commit 66aa675719ab82a03a66b75210fd40be6541f8fb ("wimax:
> > > > > > Makefile, Kconfig and docbook linkage for the stack") just to make
> > > > > > the build work.
> > > > >
> > > > > Inaky, any ideas? This really looks like a build system error that I
> > > > > ran into with the staging tree in the past. I had to add a "dummy"
> > > > > file to the empty directory to get it to build properly.
> > > > >
> > > > > I tried to duplicate the problem with a sample patch, but never could
> > > > > :(
> > >
> > > Incidentally I sent some review comments to said patch yesterday
> > > but missed this one.
> > >
> > > We have:
> > > diff --git a/drivers/net/wimax/Makefile b/drivers/net/wimax/Makefile
> > > new file mode 100644
> > > index 0000000..bda9430
> > > --- /dev/null
> > > +++ b/drivers/net/wimax/Makefile
> > > @@ -0,0 +1,2 @@
> > > +
> > > +obj-$(CONFIG_WIMAX_I2400M) += i2400m/
> > >
> > >
> > >
> > > But the problem here is that because we do not even pretend to
> > > build some files in this dir thus kbuild does not create build-in.o
> > >
> > > The fix is a bit ugly but simple. Add a dummy statement to
> > > the drivers/net/wimax/Makefile like this:
> > >
> > > # force kbuild to create built-in.o
> > > obj- := dummy.o
> >
> > ops -- ok, I will. I guess I have to add this in all the similar ones, right?
> You need this in all cases where you have a Makefile that
> only specify sub directories. I assume this is the only Makefile
> where you have this.
Ah great, I'll do this for the drivers/staging/ directory as well, I had
the same error there and couldn't figure it out.
thanks,
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
