samba-bugs at samba.org
2016-Jul-20 07:12 UTC
[Bug 12029] New: Makefile missing dep gives parallel race for rounding.h
https://bugzilla.samba.org/show_bug.cgi?id=12029 Bug ID: 12029 Summary: Makefile missing dep gives parallel race for rounding.h Product: rsync Version: 3.1.2 Hardware: All OS: All Status: NEW Severity: minor Priority: P5 Component: core Assignee: wayned at samba.org Reporter: greg.whiteley at gmail.com QA Contact: rsync-qa at samba.org I've observed a make race when building in parallel against 3.1.1 and 3.1.2. Reproducibility is low - I've had a CI machine building ~once a week for over a year and just now its hit twice in a row - never before. Can't reproduce it on my own machine.> $ git clone ... > ./configure --prefix=/path/to/sysroot/rpi/usr --disable-ipv6 --disable-locale --disable-iconv-open --disable-iconv --disable-acl-support --disable-xattr-support --host=arm-linux-gnueabihf > $ make -j8 > perl ./mkproto.pl ./*.c ./lib/compat.c > sed 's;\@bindir\@;/home/buildbot/slaves/enterprise-master/enterprise-rootfs-master/build/sysroot/rpi/usr/bin;g' <./rsync-ssl.in >rsync-ssl > sed 's;\@stunnel4\@;stunnel;g' <./stunnel-rsync.in >stunnel-rsync > sed 's;\@bindir\@;/home/buildbot/slaves/enterprise-master/enterprise-rootfs-master/build/sysroot/rpi/usr/bin;g' <./stunnel-rsyncd.conf.in >stunnel-rsyncd.conf > Copying srcdir rsync.1 > cp: cannot stat ‘./rsync.1’: No such file or directory > make: [man-copy] Error 1 (ignored) > Copying srcdir rsyncd.conf.5 > cp: cannot stat ‘./rsyncd.conf.5’: No such file or directory > make: [man-copy] Error 1 (ignored) > arm-linux-gnueabihf-gcc -std=gnu99 -I. -I. -I./zlib -I./popt -g -O2 -DHAVE_CONFIG_H -Wall -W -Wno-unused-parameter -c rsync.c -o rsync.o > In file included from ./rounding.c:20:0: > ./rsync.h:1006:19: fatal error: proto.h: No such file or directory > #include "proto.h" > ^ > compilation terminated. > Failed to create rounding.h! > make: *** [rounding.h] Error 1 > make: *** Waiting for unfinished jobs....-- You are receiving this mail because: You are the QA Contact for the bug.
samba-bugs at samba.org
2016-Jul-20 07:22 UTC
[Bug 12029] Makefile missing dep gives parallel race for rounding.h
https://bugzilla.samba.org/show_bug.cgi?id=12029 --- Comment #1 from Greg Whiteley <greg.whiteley at gmail.com> --- Created attachment 12280 --> https://bugzilla.samba.org/attachment.cgi?id=12280&action=edit Patch to depend on generated proto.h for rounding.h The temporary `rounding` requires rsync.h to be includable, which requires proto.h to have been pre-generated Generation of rounding.h seems to be the only exception to the proto.h dependencies applied by line 96:> $(OBJS): $(HEADERS)This is almost certainly a duplicate of bug 10910, however without context in that error report I cannot be sure. If they are duplicate then the patch attached to bug 10910 is not correct as proto.h rsync.h is source and not generated by make, and thus its "up-to-date" status is dependent on checkout ordering. -- You are receiving this mail because: You are the QA Contact for the bug.
samba-bugs at samba.org
2016-Jul-20 15:52 UTC
[Bug 12029] Makefile missing dep gives parallel race for rounding.h
https://bugzilla.samba.org/show_bug.cgi?id=12029 Wayne Davison <wayned at samba.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #2 from Wayne Davison <wayned at samba.org> --- I've committed your patch to git for the next release. Thanks! -- You are receiving this mail because: You are the QA Contact for the bug.
samba-bugs at samba.org
2016-Jul-20 15:54 UTC
[Bug 12029] Makefile missing dep gives parallel race for rounding.h
https://bugzilla.samba.org/show_bug.cgi?id=12029 Wayne Davison <wayned at samba.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |philippe.coval at open.eurogic | |iel.org --- Comment #3 from Wayne Davison <wayned at samba.org> --- *** Bug 10910 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the QA Contact for the bug.
Possibly Parallel Threads
- [Bug 10910] New: build: FTBFS paralel build start before proto.h generation (patch)
- [PATCH server] quote freeipa password for special chars
- [LLVMdev] Undefined symbol when loading pass
- Starting stunnel on boot with CentOS7
- Starting stunnel on boot with CentOS7