Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: automation/build.js

Issue 29901591: Issue 7020 - publish gecko with Node.js (Closed) Base URL: https://gitlab.com/eyeo/adblockplus/adblockpluschrome/tree/6323e74d580026e3bd1e8e631fcddaf0bbaa34df
Patch Set: Created Oct. 4, 2018, 3:21 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | automation/builds/gecko.js » ('j') | automation/builds/gecko.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: automation/build.js
diff --git a/ext/devtools.js b/automation/build.js
similarity index 77%
copy from ext/devtools.js
copy to automation/build.js
index 3f743e7fdce7d2c2dc01ad1d38a5183012fc89ce..983f065b25189d2d3137fc6cc283d05754384dd7 100644
--- a/ext/devtools.js
+++ b/automation/build.js
@@ -15,12 +15,12 @@
* along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
*/
+/* eslint-env node */
Sebastian Noack 2018/10/04 19:56:52 If we put these scripts into a separate directory
tlucas 2018/10/15 10:26:19 Done.
+
"use strict";
-{
- let inspectedTabId = browser.devtools.inspectedWindow.tabId;
- let port = browser.runtime.connect({name: "devtools-" + inspectedTabId});
+const path = require("path");
+
+let platform = process.argv[2];
Sebastian Noack 2018/10/04 19:56:52 This temporary variable seems unnecessary. You cou
tlucas 2018/10/15 10:26:19 Not relevant anymore.
- ext.onMessage = port.onMessage;
- ext.devtools = browser.devtools;
-}
+require(path.resolve(`automation/builds/${platform}`));
« no previous file with comments | « no previous file | automation/builds/gecko.js » ('j') | automation/builds/gecko.js » ('J')

Powered by Google App Engine
This is Rietveld