Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

execFile方法在win10x64执行doxmate-build出错,改了 #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MemoryNoodles
Copy link

./bin/doxmate这个文件的
// execFile(local, args, options, function (err, stdout) {
// if (err) {
// console.error(err);
// return;
// }
// console.log(stdout);
// });
改为
execFile("node", [local], function (err, stdout) {
if (err) {
console.error(err);
return;
}
console.log(stdout);
});在win10x64可以执行成功

@@ -61,10 +61,17 @@ if (fs.existsSync(local)) {

// execute command
var options = {maxBuffer: 1024 * 1024};
execFile(local, args, options, function (err, stdout) {
if (err) {
execFile("node", [local], function (err, stdout) {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你把 args 和 options 丢了。

Copy link
Owner

@JacksonTian JacksonTian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SMC

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants