Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

cli 工具对文件夹快捷方式无法扫描,产生报错进而无法编译。 #4431

Closed
tangbc opened this issue Sep 9, 2019 · 3 comments
Assignees

Comments

@tangbc
Copy link

tangbc commented Sep 9, 2019

问题描述

在 src 源码目录下存在快捷方式的文件夹时,跑 run dev 时会报错。

复现步骤

我们的项目采用了 lerna 来管理,而 taro 的源码目录就是一个用 lerna 管理的 package,所以就生成了如下目录:

├── actions
├── app.less
├── app.tsx
├── index.html
├── node_modules
├───── @utils (这个是 lerna 包的快捷方式,由 lerna 自动生成)
├──────── index.ts
├── package.json
├── pages
├── reducers
└── store

显然 src/node_modules/@utils 被当成了一个文件,实际上是文件夹的快捷方式!

报错信息

image

问题就出在 node_modules/@utils 这个快捷方式的判断上:

创建  发现文件  src/pages/index/index.less
创建  发现文件  src/pages/index/index.tsx
创建  发现文件  src/node_modules/@utils

系统信息

  • 操作系统: macOS
  • Taro 版本 1.3.17
  • Node.js 版本 v10.16.0
  • 报错平台 h5 weapp

补充信息

初步定位报错点在源码这里:
https://github.com/NervJS/taro/blob/master/packages/taro-cli/src/h5/index.ts#L162

if (!file.stats.isDirectory()) {
  util_1.printLog("create" /* CREATE */, '发现文件', relativePath);
  this.processFiles(file.path);
}

显然这里的 if 判断有问题, node_modules/@utils 这个是个文件夹的快捷方式,也被判断成了非文件夹,再进行 fs.readFileSync 就报错了。

再往前面翻一下逻辑,问题应该就出在 klaw(sourcePath) 这里缺少对文件夹快捷方式的判断和 walker,最终放到根目录 .temp 文件夹后 src/node_modules 就是一个空文件夹。

希望 taro 团队尽快解决一下,谢谢!

@taro-bot
Copy link

taro-bot bot commented Sep 9, 2019

欢迎提交 Issue~

如果你提交的是 bug 报告,请务必遵循 Issue 模板的规范,尽量用简洁的语言描述你的问题,最好能提供一个稳定简单的复现。🙏🙏🙏

如果你的信息提供过于模糊或不足,或者已经其他 issue 已经存在相关内容,你的 issue 有可能会被关闭。

Good luck and happy coding~

@tangbc tangbc changed the title cli 工具对快捷方式的文件夹无法扫描,产生报错进而无法编译。 cli 工具对文件夹快捷方式无法扫描,产生报错进而无法编译。 Sep 9, 2019
@luckyadam luckyadam added the CLI label Sep 9, 2019
@taro-bot
Copy link

taro-bot bot commented Sep 9, 2019

CC @luckyadam

@CntChen
Copy link

CntChen commented Sep 10, 2019

解决软链问题外, 用lerna可能有另一个问题.
taro 的打包是对入口目录进行扫描, 并复制一份到 .temp 目录进行构建.
使用 lerna bootstrap 后, 会在 src 目录下安装 learn 的依赖, src/node_modules, 这个文件夹同样会被扫描, 可能导致扫描文件规模很大.

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

No branches or pull requests

3 participants