Re: Change #18681 has broken build on master

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Tue, Nov 7, 2017 at 5:32 PM, Nigel Babu <nigelb@xxxxxxxxxx> wrote:

Thanks Nigel. I am wondering how did the build for #18681 succeed? Any insights on this?


On Tue, Nov 7, 2017 at 5:23 PM, Nigel Babu <nigelb@xxxxxxxxxx> wrote:
Rafi has a fix[1]. I'm going to make it skip regressions and land it directly.


On Tue, Nov 7, 2017 at 4:42 PM, Raghavendra Gowdappa <rgowdapp@xxxxxxxxxx> wrote:
Please check [1].

Build on master branch on my laptop failed too:

[raghu@unused server]$ make > /dev/null
server.c: In function 'init':
server.c:1205:9: error: too few arguments to function 'rpcsvc_program_register'
In file included from server.h:17:0,
                 from server.c:16:
../../../../rpc/rpc-lib/src/rpcsvc.h:426:1: note: declared here
make[1]: *** [server.lo] Error 1
make: *** [all-recursive] Error 1

The change was introduced by [2]. However, the puzzling thing is [2] itself was built successfully and has passed all tests. Wondering how did that happen.

[1] https://build.gluster.org/job/centos6-regression/7281/console
[2] review.gluster.org/18681

regards,
Raghavendra



--
nigelb



--
nigelb

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel



--
Raghavendra G
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux