Re: [release-3.6] compile error: 'GF_REPLACE_OP_START' undeclared

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

 



On Mon, Jul 06, 2015 at 02:19:07PM +0530, Raghavendra Bhat wrote:
> On 07/06/2015 01:39 PM, Niels de Vos wrote:
> >On Mon, Jul 06, 2015 at 12:09:28PM +0530, Raghavendra Bhat wrote:
> >>On 07/06/2015 09:52 AM, Kaushal M wrote:
> >>>I checked on NetBSD-7.0_BETA and FreeBSD-10.1. I couldn't reproduce
> >>>this. I'll try on NetBSD-6 next.
> >>>
> >>>~kaushal
> >>I think it has to be included before 3.6.4 is made G.A. I can wait till the
> >>fix for this issue is merged before making 3.6.4. Does it sound ok? Or
> >>should I go ahead with 3.6.4 and make a quick 3.6.5 with this fix?
> >I only care about getting http://review.gluster.org/11335 merged :-)
> >
> >This is a patch I promised to take into release-3.5. It would be nicer
> >to have this change included in the release-3.6 branch before I merge
> >the 3.5 backport. At the moment, 3.5.5 is waiting on this patch. But I
> >do not think you really need to delay 3.6.4 off for that one. It should
> >be fine if it lands in 3.6.5. (The compile error looks more like a 3.6.4
> >blocker.)
> >
> >Niels
> 
> Niels,
> 
> The patch you mentioned has received the acks and also has passed the linux
> regression tests. But it seem to have failed netbsd regression tests.

Yes, at least the smoke tests on NetBSD and FreeBSD fail with the
compile error mentioned in the subject of this email :)

Thanks,
Niels


> Regards,
> Raghavendra Bhat
> 
> >>Regards,
> >>Raghavendra Bhat
> >>
> >>>On Mon, Jul 6, 2015 at 8:38 AM, Kaushal M <kshlmster@xxxxxxxxx> wrote:
> >>>>Krutika hit this last week, and let us (GlusterD maintiners) know of
> >>>>it. I volunteered to look into this, but couldn't find time. I'll do
> >>>>it now.
> >>>>
> >>>>~kaushal
> >>>>
> >>>>On Sun, Jul 5, 2015 at 10:43 PM, Atin Mukherjee
> >>>><atin.mukherjee83@xxxxxxxxx> wrote:
> >>>>>I remember Krutika reporting it few days back. So it seems like its not
> >>>>>fixed yet. If there is no taker I will send a patch tomorrow.
> >>>>>
> >>>>>-Atin
> >>>>>Sent from one plus one
> >>>>>
> >>>>>On Jul 5, 2015 9:58 PM, "Niels de Vos" <ndevos@xxxxxxxxxx> wrote:
> >>>>>>Hi,
> >>>>>>
> >>>>>>it seems that the current release-3.6 branch does not compile on
> >>>>>>FreedBSD and NetBSD (not sure why it compiles on CentOS-6). These errors
> >>>>>>are thrown:
> >>>>>>
> >>>>>>   --- glusterd_la-glusterd-op-sm.lo ---
> >>>>>>     CC       glusterd_la-glusterd-op-sm.lo
> >>>>>>
> >>>>>>/home/jenkins/root/workspace/netbsd6-smoke/xlators/mgmt/glusterd/src/glusterd-op-sm.c:
> >>>>>>In function 'glusterd_op_start_rb_timer':
> >>>>>>
> >>>>>>/home/jenkins/root/workspace/netbsd6-smoke/xlators/mgmt/glusterd/src/glusterd-op-sm.c:3685:19:
> >>>>>>error: 'GF_REPLACE_OP_START' undeclared (first use in this function)
> >>>>>>
> >>>>>>/home/jenkins/root/workspace/netbsd6-smoke/xlators/mgmt/glusterd/src/glusterd-op-sm.c:3685:19:
> >>>>>>note: each undeclared identifier is reported only once for each function it
> >>>>>>appears in
> >>>>>>
> >>>>>>/home/jenkins/root/workspace/netbsd6-smoke/xlators/mgmt/glusterd/src/glusterd-op-sm.c:
> >>>>>>In function 'glusterd_bricks_select_status_volume':
> >>>>>>
> >>>>>>/home/jenkins/root/workspace/netbsd6-smoke/xlators/mgmt/glusterd/src/glusterd-op-sm.c:5800:34:
> >>>>>>warning: unused variable 'snapd'
> >>>>>>   *** [glusterd_la-glusterd-op-sm.lo] Error code 1
> >>>>>>
> >>>>>>
> >>>>>>Could someone send a (pointer to the) backport that addresses this?
> >>>>>>
> >>>>>>Thanks,
> >>>>>>Niels
> >>>>>>
> >>>>>>
> >>>>>>On Sun, Jul 05, 2015 at 08:59:32AM -0700, Gluster Build System (Code
> >>>>>>Review) wrote:
> >>>>>>>Gluster Build System has posted comments on this change.
> >>>>>>>
> >>>>>>>Change subject: nfs: make it possible to disable nfs.mount-rmtab
> >>>>>>>......................................................................
> >>>>>>>
> >>>>>>>
> >>>>>>>Patch Set 1: -Verified
> >>>>>>>
> >>>>>>>Build Failed
> >>>>>>>
> >>>>>>>http://build.gluster.org/job/compare-bug-version-and-git-branch/9953/ :
> >>>>>>>SUCCESS
> >>>>>>>
> >>>>>>>http://build.gluster.org/job/freebsd-smoke/8551/ : FAILURE
> >>>>>>>
> >>>>>>>http://build.gluster.org/job/smoke/19820/ : SUCCESS
> >>>>>>>
> >>>>>>>http://build.gluster.org/job/netbsd6-smoke/7808/ : FAILURE
> >>>>>>>
> >>>>>>>--
> >>>>>>>To view, visit http://review.gluster.org/11335
> >>>>>>>To unsubscribe, visit http://review.gluster.org/settings
> >>>>>>>
> >>>>>>>Gerrit-MessageType: comment
> >>>>>>>Gerrit-Change-Id: I40c4d8d754932f86fb2b1b2588843390464c773d
> >>>>>>>Gerrit-PatchSet: 1
> >>>>>>>Gerrit-Project: glusterfs
> >>>>>>>Gerrit-Branch: release-3.6
> >>>>>>>Gerrit-Owner: Niels de Vos <ndevos@xxxxxxxxxx>
> >>>>>>>Gerrit-Reviewer: Gluster Build System <jenkins@xxxxxxxxxxxxxxxxx>
> >>>>>>>Gerrit-Reviewer: Kaleb KEITHLEY <kkeithle@xxxxxxxxxx>
> >>>>>>>Gerrit-Reviewer: NetBSD Build System <jenkins@xxxxxxxxxxxxxxxxx>
> >>>>>>>Gerrit-Reviewer: Niels de Vos <ndevos@xxxxxxxxxx>
> >>>>>>>Gerrit-Reviewer: Raghavendra Bhat <rabhat@xxxxxxxxxx>
> >>>>>>>Gerrit-Reviewer: jiffin tony Thottan <jthottan@xxxxxxxxxx>
> >>>>>>>Gerrit-HasComments: No
> >>>>>>_______________________________________________
> >>>>>>Gluster-devel mailing list
> >>>>>>Gluster-devel@xxxxxxxxxxx
> >>>>>>http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>>_______________________________________________
> >>>>>Gluster-devel mailing list
> >>>>>Gluster-devel@xxxxxxxxxxx
> >>>>>http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>>
> >>>_______________________________________________
> >>>Gluster-devel mailing list
> >>>Gluster-devel@xxxxxxxxxxx
> >>>http://www.gluster.org/mailman/listinfo/gluster-devel
> 
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.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