[Bug 12029] New: Makefile missing dep gives parallel race for rounding.h

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 12029] New: Makefile missing dep gives parallel race for rounding.h

samba-bugs
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: [hidden email]
          Reporter: [hidden email]
        QA Contact: [hidden email]

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.

--
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
Reply | Threaded
Open this post in threaded view
|

[Bug 12029] Makefile missing dep gives parallel race for rounding.h

samba-bugs
https://bugzilla.samba.org/show_bug.cgi?id=12029

--- Comment #1 from Greg Whiteley <[hidden email]> ---
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.

--
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
Reply | Threaded
Open this post in threaded view
|

[Bug 12029] Makefile missing dep gives parallel race for rounding.h

samba-bugs
In reply to this post by samba-bugs
https://bugzilla.samba.org/show_bug.cgi?id=12029

Wayne Davison <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
                   |                            |iel.org

--- Comment #3 from Wayne Davison <[hidden email]> ---
*** 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.

--
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
Reply | Threaded
Open this post in threaded view
|

[Bug 12029] Makefile missing dep gives parallel race for rounding.h

samba-bugs
In reply to this post by samba-bugs
https://bugzilla.samba.org/show_bug.cgi?id=12029

Wayne Davison <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Wayne Davison <[hidden email]> ---
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.

--
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html