748344 - xplat: use something other than AVM for reference avmshell (used for e.g. tracer generation)

Status: RESOLVED FIXED

(Tamarin Graveyard :: Build Config, defect)

Description

[Felix S. Klock II [:pnkfelix, :fklock]]

Bug 632857, comment 19, put in a change that was a slight change to our old semantics if you ran an xplat make in a context where the AVM environment variable is defined.

Ingo has been running into problems with that, since he had set AVM to some value for separate testing and his build was failing.

He recommended that we either (or both):

* Use a different environment variable (I am currently thinking AVMSHELL_TOOL) so that we do not introduce a new interpretation for AVM in the xplat build.

* Issue a message when we do the tracer generation that explicitly relates its running to the presence of the environment variable, so that someone who is trying to understand the build output has a better chance of understanding what has gone wrong.

Comment 1

[Felix S. Klock II [:pnkfelix, :fklock]]

Attachment: patch A: fix it!

Comment 2

[Felix S. Klock II [:pnkfelix, :fklock]]

Attachment: patch A v2: fix it!

(aesthetic fix to make similarity of both rules more explicit)

Comment 3

[Felix S. Klock II [:pnkfelix, :fklock]]

Comment on attachment 617863 [details] [diff] [review]
patch A v2: fix it!

R?

(I had been skipping code-reviews on a lot of my xplat revisions lately; but maybe the problems Ingo encountered would have been addressed earlier if I had followed the process properly.  So I am trying to make good on that now.)

Comment 4

[Tamarin Bot]

changeset: 7371:c8e201cf8268
user:      Felix Klock II <[e-mail]>
summary:   Bug 748344: [xplat] dont overload AVM; use AVMSHELL_TOOL env var for tracer-gen (r=brbaker).

http://hg.mozilla.org/tamarin-redux/rev/c8e201cf8268

Comment 5

[Tamarin Bot]

changeset: 7379:9ef7bb33fc2f
user:      Felix Klock II <[e-mail]>
summary:   Bug 748344: [xplat] fix syntactic mistake in previous patch (r=fklockii).

http://hg.mozilla.org/tamarin-redux/rev/9ef7bb33fc2f