(no longer active) Reporter
	Description • 11 years ago

This could be one reason for clobbering.

	(no longer active) Reporter
	Comment 1 • 11 years ago

Gregory, can you please fix this or tell me how to fix this?  This will break the patch for bug 967167 when it gets checked in.

	Gregory Szorc [:gps]
	Comment 2 • 11 years ago

We may have this on file already. The make dependency file for IPDL isn't pulling in imported .py files in addition to self.

	Nathan Froyd [:froydnj] Assignee
	Comment 3 • 11 years ago

This is not the absolute best solution, but it fits in well with what we
already have.

	(no longer active) Reporter
	Comment 4 • 11 years ago

This is a gross hack.  Why can't we just encode the dependency information in the build system instead?

	Mike Hommey [:glandium]
	Comment 5 • 11 years ago

(In reply to :Ehsan Akhgari (needinfo? me!) (slow responsiveness, emailacopolypse) from comment #4)
> This is a gross hack.  Why can't we just encode the dependency information
> in the build system instead?

Can you guarantee that dependency information won't get outdated?

	Gregory Szorc [:gps]
	Comment 6 • 11 years ago

(In reply to :Ehsan Akhgari (needinfo? me!) (slow responsiveness, emailacopolypse) from comment #4)
> This is a gross hack.  Why can't we just encode the dependency information
> in the build system instead?

This is a similar approach we use elsewhere.

Next step is to write out the Python dependencies into a .pp file somewhere so make won't invoke the IPDL processor if nothing has changed.

	(no longer active) Reporter
	Comment 7 • 11 years ago

(In reply to comment #5)
> (In reply to :Ehsan Akhgari (needinfo? me!) (slow responsiveness,
> emailacopolypse) from comment #4)
> > This is a gross hack.  Why can't we just encode the dependency information
> > in the build system instead?
> 
> Can you guarantee that dependency information won't get outdated?

I take it that this is a rhetorical question.

	(no longer active) Reporter
	Comment 8 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/9185c4a586df

	Carsten Book [:Tomcat]
	Comment 9 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/9185c4a586df