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

target 重新构建会执行不必要的构建步骤 #129

Closed
llxwj opened this Issue Jun 29, 2017 · 5 comments

Comments

Projects
None yet
2 participants
@llxwj

llxwj commented Jun 29, 2017

一个xmake.lua文件中指定了多个target, 第一次全部构建完成后,如果后续只对某个target所依赖的源码文件进行了修改,那么再次使用xmake构建时, 其他target的源码确实没有重新编译,但是target最终生成步骤(ar或ld)又执行了一次,重复构建应该是不必要的.

@waruqi waruqi added the improvement label Jun 29, 2017

@waruqi waruqi modified the milestones: todo, v2.1.5 Jun 30, 2017

@waruqi

This comment has been minimized.

Show comment
Hide comment
@waruqi

waruqi Jul 3, 2017

Member

dev分支上已修复这个问题。。

Member

waruqi commented Jul 3, 2017

dev分支上已修复这个问题。。

@waruqi waruqi added the need response label Jul 3, 2017

@llxwj

This comment has been minimized.

Show comment
Hide comment
@llxwj

llxwj Jul 4, 2017

经测试表现已经很好了,但仍然存在问题,故障描述如下:

  • xmale.lua中包含5个target
  • 只修改了一个target所依赖的源码文件
  • xmake构建后,被修改的源码文件执行编译并且重新生成了它所归属的target
  • 同时另外一个无关的target也被重新生成,此target和被修改的源码文件没有任何依赖关系
  • 重复修改,构建这个步骤,无关的target是同一个target.
  • 其余3个target没有重新生成.

llxwj commented Jul 4, 2017

经测试表现已经很好了,但仍然存在问题,故障描述如下:

  • xmale.lua中包含5个target
  • 只修改了一个target所依赖的源码文件
  • xmake构建后,被修改的源码文件执行编译并且重新生成了它所归属的target
  • 同时另外一个无关的target也被重新生成,此target和被修改的源码文件没有任何依赖关系
  • 重复修改,构建这个步骤,无关的target是同一个target.
  • 其余3个target没有重新生成.
@waruqi

This comment has been minimized.

Show comment
Hide comment
@waruqi

waruqi Jul 4, 2017

Member

好的。。我后两天继续改进下。。

Member

waruqi commented Jul 4, 2017

好的。。我后两天继续改进下。。

@waruqi

This comment has been minimized.

Show comment
Hide comment
@waruqi

waruqi Jul 5, 2017

Member

@llxwj 我改进了下。。你再试试

Member

waruqi commented Jul 5, 2017

@llxwj 我改进了下。。你再试试

@llxwj

This comment has been minimized.

Show comment
Hide comment
@llxwj

llxwj Jul 5, 2017

@waruqi 刚才测试了一下,正常了.

llxwj commented Jul 5, 2017

@waruqi 刚才测试了一下,正常了.

@waruqi waruqi closed this Jul 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment