Merge pull request #23 from nodenv/chakracore

Scrape chakracore defs

Author: jasonkarns

bin/nodenv-update-version-defs

@@ -2,7 +2,9 @@
 #
 # Summary: Create build definitions from nodejs.org
 #
-# Usage: nodenv update-version-defs [-f] [-d <dir>] [-n] [--nodejs|--nodejs-pre]
+# Usage: nodenv update-version-defs [-f] [-d <dir>] [-n]
+#        [--nodejs] [--nodejs-pre]
+#        [--chakracore] [--chakracore-pre]
 #
 # Scrapes nodejs.org to create build definitions for node
 # versions not yet available to node-build
@@ -15,7 +17,10 @@
 #
 #   --nodejs           Scrape nodejs.org for node definitions;
 #   --nodejs-pre       Scrape nodejs.org for node pre-release definitions;
-#                      If none of --nodejs,--nodejs-pre are supplied, defaults to --nodejs
+#   --chakracore       Scrape nodejs.org for chakracore definitions;
+#   --chakracore-pre   Scrape nodejs.org for chakracore pre-release definitions;
+#                      Defaults to --nodejs, if none of:
+#                      --nodejs, --nodejs-pre, --chakracore, --chakracore-pre
 #
 # Notes: If this plugin is not installed directly into "$NODENV_ROOT/plugins",
 # (ie, homebrew, npm, etc) then 'share/node-build' must be added to the
@@ -68,15 +73,17 @@ while [ $# -gt 0 ]; do
       echo --force
       echo --nodejs
       echo --nodejs-pre
+      echo --chakracore
+      echo --chakracore-pre
       exit ;;
     -d | --destination )
       shift
       # overwrite the defs write target
       NODE_BUILD_DEFINITIONS="$(abs_dirname "${1%/}/"):${NODE_BUILD_DEFINITIONS}" ;;
     -f | --force | \
     -n | --dry-run | \
-    --nodejs  | \
-    --nodejs-pre )
+    --nodejs | --nodejs-pre | \
+    --chakracore | --chakracore-pre )
       SCRAPE_OPTS[${#SCRAPE_OPTS[*]}]="$1" ;;
     * )
       nodenv-help --usage update-version-defs >&2

lib/scraper-nodejs_org.js

@@ -4,7 +4,7 @@ const fetch = require('./fetch')
 const IterableRegExp = require('./iterable-regexp')
 
 class Scraper {
-  constructor ({ name, displayName, baseUri, listing }) {
+  constructor ({ name, displayName = name, baseUri, listing }) {
     this.name = name
     this.displayName = displayName
     this.baseUri = baseUri
@@ -30,6 +30,15 @@ class Scraper {
 }
 
 module.exports = class NodejsOrgScraper extends Scraper {
+  constructor ({
+    name = 'node',
+    displayName,
+    baseUri,
+    listing = baseUri + 'index.json'
+  }) {
+    super({ name, displayName, baseUri, listing })
+  }
+
   fetch (listing) {
     return fetch(listing)
   }

libexec/scrape

@@ -3,21 +3,31 @@
 const DefinitionFile = require('../lib/definition-file')
 const NodejsOrgScraper = require('../lib/scraper-nodejs_org')
 
-const nodejs = new NodejsOrgScraper({
-  name: 'node',
-  displayName: 'nodejs',
-  listing: 'https://nodejs.org/dist/index.json',
-  baseUri: 'https://nodejs.org/dist/'
-})
+const scrapers = {
+  '--nodejs': new NodejsOrgScraper({
+    displayName: 'nodejs',
+    baseUri: 'https://nodejs.org/dist/'
+  }),
 
-const nodejsPre = new NodejsOrgScraper({
-  name: 'node',
-  displayName: 'nodejs release candidate',
-  listing: 'https://nodejs.org/download/rc/index.json',
-  baseUri: 'https://nodejs.org/download/rc/'
-})
+  '--nodejs-pre': new NodejsOrgScraper({
+    displayName: 'nodejs release candidate',
+    baseUri: 'https://nodejs.org/download/rc/'
+  }),
+
+  '--chakracore': new NodejsOrgScraper({
+    name: 'chakracore',
+    baseUri: 'https://nodejs.org/download/chakracore-release/'
+  }),
+
+  '--chakracore-pre': new NodejsOrgScraper({
+    name: 'chakracore',
+    displayName: 'chakracore release candidate',
+    baseUri: 'https://nodejs.org/download/chakracore-rc/'
+  })
+}
+
+const scrapersToRun = []
 
-const scrapers = []
 const options = {
   dryRun: false,
   overwrite: false,
@@ -26,7 +36,7 @@ const options = {
   ).split(':')
 }
 
-process.argv.forEach(function (arg) {
+process.argv.forEach(arg => {
   switch (arg) {
     case '-f':
     case '--force':
@@ -37,18 +47,18 @@ process.argv.forEach(function (arg) {
       options.dryRun = true
       break
     case '--nodejs':
-      scrapers.push(nodejs)
-      break
     case '--nodejs-pre':
-      scrapers.push(nodejsPre)
+    case '--chakracore':
+    case '--chakracore-pre':
+      scrapersToRun.push(arg)
       break
   }
 })
 
-if (!scrapers.length) {
-  scrapers.push(nodejs)
+if (!scrapersToRun.length) {
+  scrapersToRun.push('nodejs')
 }
 
 DefinitionFile.configure(options)
 
-scrapers.forEach(scraper => scraper.scrape())
+scrapersToRun.forEach(scraper => scrapers[scraper].scrape())