Re: ``make dist'' and friends

[Thomas Schwinge]

Hello!

On Wed, Nov 29, 2006 at 12:30:13AM -0500, Barry deFreese wrote:
> Barry deFreese wrote:
> >Thomas Schwinge wrote:
> >>On Tue, Nov 28, 2006 at 12:13:20AM -0500, Barry deFreese wrote:
> >>>Barry deFreese wrote:
> >>>   
> >>>>[...] I have added a few dist_libkernel_a_SOURCE to include all the 
> >>>>header files, .sym, .srv, and .cli files.
> >>>>      
> >>
> >>That seems to be correct.  I hope that doing so doesn't break anything.
> >>(Thinking about the rules in `Makerules.am' that deal with those files.)
> >>But you should have noticed if it had...

I decided to add the `.srv' and `.cli' files to `EXTRA_DIST' instead.


> >OK gang, I've gotten pretty far but now I'm a little confused.  I had 
> >to include linux/src/drivers/scsi/FlashPoint.c because BusLogic.c 
> >includes.  But now it is attempting to build FlashPoint.c and failing.
> >
> >[...]
> >
> >But afaict, in a normal (i.e. not from make dist's tarball), 
> >FlashPoint.c is not build.

Correct.

> Also, I meant to mention.  I suppose one way around this may be to add
> FlashPoint.c to EXTRA_DIST instead of inside of a
> dist_liblinux_a_SOURCES target?

Correct.  This and similar other files should also be added to
`EXTRA_DIST' instead of some `*_SOURCES'.


> >I am attaching a diff of what I have done so far.

Barry, thanks for the patch, it was of great help!

I integrated it into a local tree here at home, changed and added some
bits, began testing it to eventually commit it into the cvs repository,
but before I was able to finish the testing, I hit another Automake bug,
I believe, which I then reported to <bug-automake@gnu.org>.  So we now
have to wait until that is resolved.


Regards,
 Thomas

signature.asc
Description: Digital signature