Bug 14133 - "waf install", will do a full rebuild even when "waf build" did run.
Summary: "waf install", will do a full rebuild even when "waf build" did run.
Status: NEW
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: Build (show other bugs)
Version: 4.4.0
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Samba QA Contact
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-09-18 13:17 UTC by andieq
Modified: 2019-12-25 02:13 UTC (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description andieq 2019-09-18 13:17:54 UTC
What we observed for a very long time when cross-compiling samba4 for openwrt is that "waf install" without --targets will always do a full rebuild, ignoring any previously build "waf build --targets" run. 

Actually "waf install" seems to always repeat what "waf build" did, so if we invoke "waf build" and than "waf install" with the exact same --targets set, we see 2 identical build runs.

What we would expect is that "waf install" only installs previously build targets. 

So to fix this we skip the "waf build" step altogether and directly use "waf install --targets" to ensure only the targets we want are build/installed and prevent doing the same build twice in a row.
Comment 1 Uri Simchoni 2019-10-22 04:15:45 UTC
AFAIK it's been that way ever since Samba moved to waf build. I don't know the details but there's something different about install vs build - build is designed to run the binaries from their build dir, and load all dependent shared libs (which are also built) from their respective build dir, and install is designed to make binaries that load shared libs from their prefix location.
Comment 2 Andrew Bartlett 2019-10-22 05:19:57 UTC
Exactly, we have some magic to allow running in the source directory.

If you try to run the binaries from the source after a 'make install' then they (typically) don't work, you need to run 'make' again for in-tree work.  

Thankfully 'make test' also triggers a rebuild.
Comment 3 andieq 2019-10-22 08:32:58 UTC
"Exactly, we have some magic to allow running in the source directory."

So "waf install" for samba is actually designed to do the build + install, while "waf build" is only designed to get bins that can run from the build dir?

I checked the build scripts of some distros that cross-compile samba and all of them basically do a waf build and waf install, without ever running anything from the build dir. This is because any other build system works this way, yet for samba this means they do one unnecessary step and double the compile time + resource usage. Before i discovered this quirk our automated testing often would fail samba because it took like 30+ minutes to compile and failed some maximum timeout threshold.

So maybe make it more clear whats actually going on there and that in many cases you don't actually need to do the "waf build" step, while "waf install" is not the same as a "make install".

So some section that explicitly states this difference in the documentation.
Comment 4 Andrew Bartlett 2019-10-22 08:54:14 UTC
Generally only a few (not sure exactly, say a couple of dozen, but certainly not every) files get re-compiled, and then the resulting re-link is done. 

Either way, a 30min limit on Samba is always going to be tight, a typical compile on modern cloud hardware is ~20mins looking over our recnet build pipelines. 

https://gitlab.com/samba-team/devel/samba/-/jobs/327663264
Comment 5 andieq 2019-10-22 18:03:32 UTC
Without the extra "waf build" we are fine now, so please consider makeing this more clear.

As noted in many cases the "waf build" is a unnecessary waste of resources, mainly because its not clearly documented and somewhat misleading named. That's why i filed a bug, since i actually thought "waf install" should just install and not do a full rebuild, which i have never seen on any other build system, that's why this is confusing.
Comment 6 Ralph Böhme 2019-10-22 19:19:32 UTC
(In reply to andieq from comment #5)
waf install is not a full rebuild, it's just relinking. Other build systems with support for running binaries from the build directory are doing the same, eg automake.
Comment 7 andieq 2019-10-22 20:05:46 UTC
If i cross-compile via "waf build" and without any changes run a "waf install" right after, it takes the same amount of time, so its not just re-linking. The output also clearly shows it literally does the same steps that "waf build" does.

This was always the case, since i started creating our samba4 package.
Comment 8 Ralph Böhme 2019-10-23 12:39:55 UTC
(In reply to andieq from comment #7)
yes, I just wanted to point out the general feature. If this is not working for cross-compilation, that's likely a deficiency.
Comment 9 andieq 2019-10-23 12:45:10 UTC
Sure i now understand what is supposed to happen, need to retest what happens without cross-compiling. 
Was around samba4.3-4 where i first encountered this issue, mainly because i did "waf build --targets=" and than if you use "waf install" it will ignore all previously defined/build targets and do a full rebuild anyway. So i had to also specify "waf install --targets=" and then noticed that the install step looked nearly identical to the build step.