440068 - Pork/Piglet: package and document

Status: RESOLVED FIXED

(Developer Infrastructure :: Source Code Analysis, defect)

Description

[David Mandelin [:dmandelin]]

Piglet is a lightweight system for using Elsa as a static library. It should be packaged up, added to Pork, and documented on MDC.

Comment 1

[(dormant account)]

packaged:

http://hg.mozilla.org/users/tglek_mozilla.com/piglet

Now it's a matter of 
a) moving my tools out of oink into piglet
b) nuking oink
c) sticking all of elsa deps into elsa dir so pork consists of elsa/ + piglet/ dirs. I think in a checkout elsa/ would be a subdir of piglet.

Comment 2

[(dormant account)]

Attachment: configure

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 372473 [details] [diff] [review]
configure

>diff --git a/configure b/configure
>new file mode 100755
>--- /dev/null
>+++ b/configure
>@@ -0,0 +1,60 @@
>+#!/usr/bin/env python

Need a license block here?

>+# Author: Taras Glek
>+import getopt, sys, os.path, os, platform, subprocess
>+
>+def usage():
>+    print """
>+Usage: ./configure ...
>+--with-rewriting-cxx=compiler with MCPP(used for rewriting)
>+--pork-dir=Directory with pork, ie one that contains elsa/ dir
>+""".strip()
>+    pass
>+
>+def error(msg):
>+    print >> sys.stderr, "Error: " + msg
>+    sys.exit(1)
>+
>+def checkdir(path,opt):
>+    print "Checking " + opt + ": " + path
>+    if not os.path.isdir (path):
>+        error("directory '"+path+"' doesn't exist, specify corect directory with --" + opt)
>+    else:
>+        return os.path.abspath(path)
>+
>+if __name__ == "__main__":
>+    try:
>+        opts, args = getopt.getopt(sys.argv[1:], "h",
>+                                   ["with-rewriting-cxx=","pork-dir="])
>+    except getopt.GetoptError, err:
>+        # print help information and exit:
>+        print str(err) # will print something like "option -a not recognized"
>+        usage()
>+        sys.exit(2)
>+    rewriting_cxx = "g++"
>+    pork_dir = None
>+    for o, val in opts:
>+        if o == "--with-rewriting-cxx":
>+            rewriting_cxx = val
>+        elif o == "--pork-dir":
>+            pork_dir = val
>+        elif o in ("-h", "--help"):
>+            usage()
>+            exit(0)
>+        else:
>+            error("unhandled option " +  o)
>+    if (pork_dir):
>+        pork_dir = checkdir(pork_dir  +"/elsa", "pork-dir")

It'd be nice to also have sanity checks for the other pork tools: piglet, pork-barrel, and porky.

>+    else:
>+        error("run configure with --pork-dir=");
>+    config = dict(REWRITING_CXX=rewriting_cxx,
>+                  PORK_DIR=pork_dir)

... and similarly generate PIGLET_DIR, etc.  Not hugely important, though, since they can be constructed from PORK_DIR.  It's also be great to have paths here for LIBELSA and LIBPORK/LIBPIGLET.

This suits my needs.

Comment 4

[(dormant account)]

added a license, pushed http://hg.mozilla.org/users/tglek_mozilla.com/piglet/rev/8aa25be7b739

Comment 5

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Attachment: updated build system to better support porky

Comment 6

[(dormant account)]

Comment on attachment 372822 [details] [diff] [review]
updated build system to better support porky

Good stuff

Comment 7

[(dormant account)]

pushed http://hg.mozilla.org/users/tglek_mozilla.com/piglet/rev/9c016ea0e1f1