Skip to content

Commit 83c0a4c

Browse files
committed
reviewer notes
1 parent f2357ab commit 83c0a4c

File tree

1 file changed

+35
-1
lines changed

1 file changed

+35
-1
lines changed

README.md

Lines changed: 35 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ Install
3030

3131
### Source
3232

33-
Requirements:
33+
Requirements (tested on macOS):
3434

3535
* [bash 4](https://www.gnu.org/software/bash/) - GNU Bash
3636
* [nvm](https://github.com/creationix/nvm) - Node Version Manager
@@ -46,6 +46,40 @@ Build from source:
4646

4747
*Load Temporary Addon* at `dist/firephp.build/` into Firefox or Chrome.
4848

49+
### Notes for build reviewers
50+
51+
* Releases for this extension are built using [github.com/pinf-to/to.pinf.org.mozilla.web-ext](https://github.com/pinf-to/to.pinf.org.mozilla.web-ext). Build instructions for this tool are in `./workspace.sh`.
52+
* The tool provides a runtime framework for browser extensions as well as a build tool.
53+
* The following `./dist/firephp.build` files belong to the runtime framework and are copied into the build by the build tool:
54+
* `./devtools-page.html`
55+
* `./devtools-page.js`
56+
* `./background.html`
57+
* `./background.js`
58+
* `./lib.js`
59+
* `./default.css`
60+
* `./lib/github.com~pinf~pinf-for-mozilla-web-ext/*`
61+
* https://github.com/pinf/pinf-for-mozilla-web-ext
62+
* https://github.com/pinf/pinf-loader-js
63+
* `./lib/eventemitter3.js`
64+
* https://github.com/primus/EventEmitter3
65+
* The following `./dist/firephp.build` files are generated by the build tool by processing source files from `./src` using instructions from `./workspace.sh`:
66+
* `./skin`
67+
* `./scripts/background.js`
68+
* `./scripts/devtools/index.loader.js`
69+
* `./scripts/devtools/index.html`
70+
* `./scripts/devtools/dist/riot.csp.js`
71+
* https://github.com/riot/riot
72+
* `./scripts/devtools/dist/insight.rep.js`
73+
* https://github.com/insight/insight.domplate.reps
74+
* `./scripts/devtools/dist/insight.domplate.reps/*`
75+
* https://github.com/insight/insight.domplate.reps
76+
* `./scripts/devtools/lib/jsonrep.js`
77+
* https://github.com/jsonrep/jsonrep
78+
* `./scripts/devtools/*.rep.js`
79+
* `./scripts/devtools/*.rep.css`
80+
* `./manifest.json`
81+
82+
4983
Preview
5084
-------
5185

0 commit comments

Comments
 (0)