[PATCH v3 0/5] Waf: Semi-automatic generation of results for cross-answers files
Alexander Bokovoy
ab at samba.org
Mon May 18 08:02:01 MDT 2015
On Mon, May 18, 2015 at 10:00:03AM +0300, Uri Simchoni wrote:
> OK here goes v4 of this patchset - only diff from v3 is for autobuild
> to output the diff in resulting configurations in case they differ.
Thanks for the patches.
Patch v4 1/5:
- please split it into two separate patches: one handling the missing
f.close() and retstring cleanup, the other for actually adding
--cross-answers and --crose-execute options.
Patch v4 2/5: ACK
Patch v4 3/5: ACK
Patch v4 4/5:
- please add --verbos to rsync options in xfer_files and add a comment
to mention that remote files will be removed on target. Perhaps split
the cmd string into some parts to allow people a bit easier to
manipulate it? For those who wouldn't be careful enough, removal of
files on the target might be a surprise.
- also would be good to add a separate directory than just a homedir as
a target.
Patch v4 5/5:
- please split it into two (or three) separate patches: one for adding
'samba-xc' task in autobuild.py, and other adding the scripts.
--
/ Alexander Bokovoy
More information about the samba-technical
mailing list