519884 - NJ merge: nanojit-central configure.in and Makefile.in

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Graydon Hoare :graydon]

Attachment: Yes, it contains autoconf. Shield your eyes.

The attached patch provides nanojit-central with its own configure.in and Makefile.in, as well as supporting files. It requires bug 519856 to work (at least when deriving from the TM head of n-c), and needs to be in a repository with both nanojit/ and vprof/, which I've added to the knitting script but haven't yet had a cset trigger.

Otherwise it lets you do out-of-tree, standard autoconf-y builds of nanojit and lirasm, and executes the lirasm tests, outputting results in the way our tinderbox parsers will want to see them. No dependence on TM or TR. Yay!

Comments welcome from people who know how to do this kind of stuff better than me. I'm always kinda flailing when it comes to configury.

Comment 1

[Nicholas Nethercote [inactive]]

For anyone reading the patch, config.guess, config.sub and install-sh are all autogenerated by automake, so you can presumably skip over them.

Comment 2

[Edwin Smith]

Comment on attachment 403934 [details] [diff] [review]
Yes, it contains autoconf. Shield your eyes.

nothing obviously wrong.  i'll try it out for real when the patch's dependencies land

Comment 3

[Edwin Smith]

oh, does Makefile.in need a copyright header?

Comment 4

[Graydon Hoare :graydon]

Sure, yeah. I'll call it part of Nanojit Build System, boilerplate it up. Just waiting on a bit of review from jimb regarding the kludging of VMPI this depends on.

Comment 5

[Graydon Hoare :graydon]

Not the world's most stable revision ID, I'll grant. But assuming no more repository-rebuilds (tenuous assumption) it landed here:
 
http://hg.mozilla.org/users/graydon_mozilla.com/nanojit-central/rev/968bafe874d9