Skip to content

Commit cd95f76

Browse files
authored
Merge pull request #1111 from bloomberg/default_command_lines
stream line clean-world, make-world, allow user to override package-specs
2 parents 098ab37 + f623362 commit cd95f76

File tree

12 files changed

+434
-185
lines changed

12 files changed

+434
-185
lines changed

bin/bsb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ if (fs.existsSync(bsconfig)) {
1414
try {
1515
child_process.execFileSync(bsb_exe, delegate_args, { stdio: 'inherit' })
1616
} catch (e) {
17-
console.error('Error happend when running command', bsb_exe, 'with args', delete_args)
17+
console.error('Error happend when running command', bsb_exe, 'with args', delegate_args)
1818
}
1919
} else {
2020
var path = require('path')
@@ -32,7 +32,7 @@ if (fs.existsSync(bsconfig)) {
3232
try {
3333
child_process.execFileSync(bsb_exe, delegate_args, { stdio: 'inherit', cwd: search_dir })
3434
} catch (e) {
35-
console.error('Error happend when running command', bsb_exe, 'with args', delete_args)
35+
console.error('Error happend when running command', bsb_exe, 'with args', delegate_args)
3636
}
3737
}
3838
}

docs/docson/build-schema.json

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -177,15 +177,15 @@
177177
},
178178
"ppx-flags": {
179179
"$ref": "#/definitions/ppx-specs",
180-
"description": "TODO: ppx flags"
180+
"description": "syntax is package_name/binary, for example: reason/reactjs_jsx_ppx.native"
181181
},
182182
"bsc-flags": {
183183
"$ref": "#/definitions/stringArray",
184184
"description": "A list of flags for bsc.exe"
185185
},
186186
"package-specs": {
187187
"$ref": "#/definitions/package-specs",
188-
"description": "TODO: currently only support commonjs"
188+
"description": "currently support commonjs, amdjs and google module"
189189
},
190190
"ocamllex": {
191191
"type": "string",
@@ -201,22 +201,24 @@
201201
},
202202
"refmt": {
203203
"type": "string",
204-
"description": "Path to refmt"
204+
"description": "Path to refmt for Reason syntax, for example: reason/refmt_impl.native"
205205
},
206206
"bs-external-includes": {
207207
"$ref": "#/definitions/stringArray",
208-
"description": "external include directories, which will be applied -I to all compilation units, it is not needed in most cases"
208+
"description": "external include directories, which will be applied -I to all compilation units, it is *not needed* in most cases"
209209
},
210210
"sources": {
211211
"oneOf": [
212212
{
213213
"type": "array",
214214
"items": {
215215
"$ref": "#/definitions/sourceItem"
216-
}
216+
},
217+
"description" : "A list of source items"
217218
},
218219
{
219-
"$ref": "#/definitions/sourceItem"
220+
"$ref": "#/definitions/sourceItem",
221+
"description": "A single source item"
220222
}
221223
]
222224
},

jscomp/Makefile

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -251,8 +251,10 @@ OTHER_CORE_SRCS= bsppx_main bspack_main jsoo_main bspp_main js_cmi_datasets \
251251
js_main
252252
OTHER_CORE_CMXS= $(addprefix core/, $(addsuffix .cmx, $(OTHER_CORE_SRCS)))
253253
BSB_SRCS= bsb_config bsb_build_schemas bsb_build_util \
254-
bsb_dep_infos bsb_dir bsb_ninja \
255-
bsb_build_ui bsb_default bsb_gen\
254+
bsb_dir bsb_ninja \
255+
bsb_build_ui bsb_default\
256+
bsb_dep_infos\
257+
bsb_gen\
256258
bsb_unix
257259

258260
BSB_CMXS=$(addprefix bsb/, $(addsuffix .cmx, $(BSB_SRCS)))

jscomp/all.depend

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -661,11 +661,12 @@ bsb/bsb_gen.cmx : ext/string_map.cmx ext/literals.cmx ext/ext_filename.cmx \
661661
bsb/bsb_gen.cmi
662662
bsb/bsb_helper_main.cmx : depends/depends_post_process.cmx \
663663
bsb/bsb_helper_main.cmi
664-
bsb/bsb_main.cmx : ext/string_vec.cmx ext/literals.cmx ext/ext_string.cmx \
665-
ext/ext_json.cmx ext/ext_filename.cmx ext/ext_file_pp.cmx \
666-
ext/ext_array.cmx bsb/bsb_unix.cmx bsb/bsb_gen.cmx bsb/bsb_dep_infos.cmx \
667-
bsb/bsb_default.cmx bsb/bsb_config.cmx bsb/bsb_build_util.cmx \
668-
bsb/bsb_build_ui.cmx bsb/bsb_build_schemas.cmx bsb/bsb_main.cmi
664+
bsb/bsb_main.cmx : ext/string_vec.cmx ext/string_set.cmx ext/literals.cmx \
665+
ext/ext_string.cmx ext/ext_json.cmx ext/ext_filename.cmx \
666+
ext/ext_file_pp.cmx ext/ext_array.cmx bsb/bsb_unix.cmx bsb/bsb_gen.cmx \
667+
bsb/bsb_dep_infos.cmx bsb/bsb_default.cmx bsb/bsb_config.cmx \
668+
bsb/bsb_build_util.cmx bsb/bsb_build_ui.cmx bsb/bsb_build_schemas.cmx \
669+
bsb/bsb_main.cmi
669670
bsb/bsb_ninja.cmx : ext/string_set.cmx ext/string_map.cmx ext/literals.cmx \
670671
ext/ext_filename.cmx bsb/bsb_config.cmx bsb/bsb_build_util.cmx \
671672
bsb/bsb_build_ui.cmx bsb/bsb_build_schemas.cmx common/binary_cache.cmx \

0 commit comments

Comments
 (0)