Re: CVS commit: src/tools/host-mkdep

2013-03-15 Thread Christos Zoulas
In article <20130315185454.gb22...@apb-laptoy.apb.alt.za>,
Alan Barrett   wrote:
>On Fri, 15 Mar 2013, Christos Zoulas wrote:
>> I still don't understand how a missing include does not cause 
>> an error in the compilation phase, but it can be ignore in the 
>> dependency generation.
>
>tools/compat creates a dummy include file, if necessary.  This is 
>good enough for the compilation phase.  I am not sure why it's not 
>good enough for host-mkdep, but two ideas spring to mind:  perhaps 
>host-mkdep is run before the dummy include file is created, or 
>perhaps host-mkdep is run without the -I flags that would allow it 
>to find the dummy include file.

So, let's put back the changes reverted and find and fix the real
problem. I'll try my cross-build on windows tonight.

christos



Re: CVS commit: src/tools/host-mkdep

2013-03-15 Thread Alan Barrett

On Fri, 15 Mar 2013, Christos Zoulas wrote:
I still don't understand how a missing include does not cause 
an error in the compilation phase, but it can be ignore in the 
dependency generation.


tools/compat creates a dummy include file, if necessary.  This is 
good enough for the compilation phase.  I am not sure why it's not 
good enough for host-mkdep, but two ideas spring to mind:  perhaps 
host-mkdep is run before the dummy include file is created, or 
perhaps host-mkdep is run without the -I flags that would allow it 
to find the dummy include file.


--apb (Alan Barrett)


Re: CVS commit: src/tools/host-mkdep

2013-03-15 Thread Christos Zoulas
In article <20130315130901.gd...@pony.stderr.spb.ru>,
Valery Ushakov   wrote:
>On Fri, Mar 15, 2013 at 16:56:53 +0400, Valery Ushakov wrote:
>
>> On Thu, Mar 14, 2013 at 21:02:06 -0700, Tom Spindler (moof) wrote:
>> 
>> > > > >> >Log Message:
>> > > > >> >Revert 1.25 and 1.26: Breaks build on !NetBSD
>> > > > >> 
>> > > > >> Can you explain how? The shell does not work?
>> > > > >
>> > > > >E.g. util.h not found in binstall and a lot of other header issues.
>> > > > 
>> > > > On what os?
>> > > 
>> > > I've seen it on OSX ( 10.5 PowerPC if it matters ).
>> > 
>> > And I've seen it on Debian.
>> 
>> That begins to sound like "I've got it on MacOS", err, sorry, "I've
>> got it from Agnes" song. :)
>> 
>> Was there an error message, perhaps?
>
>Ah, as Martin pointed out privately, the problem is not some bug in
>the change ("Can you explain how? The shell does not work?"), but the
>fact that a missing include is now fatal, which is a wrong thing to do
>for mkdep.

I still don't understand how a missing include does not cause an error
in the compilation phase, but it can be ignore in the dependency generation.

christos



Re: CVS commit: src/tools/host-mkdep

2013-03-15 Thread Valery Ushakov
On Fri, Mar 15, 2013 at 16:56:53 +0400, Valery Ushakov wrote:

> On Thu, Mar 14, 2013 at 21:02:06 -0700, Tom Spindler (moof) wrote:
> 
> > > > >> >Log Message:
> > > > >> >Revert 1.25 and 1.26: Breaks build on !NetBSD
> > > > >> 
> > > > >> Can you explain how? The shell does not work?
> > > > >
> > > > >E.g. util.h not found in binstall and a lot of other header issues.
> > > > 
> > > > On what os?
> > > 
> > > I've seen it on OSX ( 10.5 PowerPC if it matters ).
> > 
> > And I've seen it on Debian.
> 
> That begins to sound like "I've got it on MacOS", err, sorry, "I've
> got it from Agnes" song. :)
> 
> Was there an error message, perhaps?

Ah, as Martin pointed out privately, the problem is not some bug in
the change ("Can you explain how? The shell does not work?"), but the
fact that a missing include is now fatal, which is a wrong thing to do
for mkdep.

-uwe


Re: CVS commit: src/tools/host-mkdep

2013-03-15 Thread Valery Ushakov
On Thu, Mar 14, 2013 at 21:02:06 -0700, Tom Spindler (moof) wrote:

> > > >> >Log Message:
> > > >> >Revert 1.25 and 1.26: Breaks build on !NetBSD
> > > >> 
> > > >> Can you explain how? The shell does not work?
> > > >
> > > >E.g. util.h not found in binstall and a lot of other header issues.
> > > 
> > > On what os?
> > 
> > I've seen it on OSX ( 10.5 PowerPC if it matters ).
> 
> And I've seen it on Debian.

That begins to sound like "I've got it on MacOS", err, sorry, "I've
got it from Agnes" song. :)

Was there an error message, perhaps?

-uwe