[CRIU] failing to build in launchpad upload environment
Cyrill Gorcunov
gorcunov at gmail.com
Thu Jun 30 08:26:55 PDT 2016
On Thu, Jun 30, 2016 at 09:18:44AM -0600, Tycho Andersen wrote:
> On Thu, Jun 30, 2016 at 06:08:20PM +0300, Cyrill Gorcunov wrote:
> > On Thu, Jun 30, 2016 at 08:08:55AM -0600, Tycho Andersen wrote:
> > >
> > > Well, we'd like to avoid any build-script specific hacks to keep in
> > > line with debian.
> > >
> > > The build system passes its own LDFLAGS, but it passes them with the
> > > gcc wrappers "-Wl". The problem is that CRIU's makefiles use ld
> > > directly, and don't invoke it through gcc, and ld doesn't recognize
> > > the -Wl bits.
> > >
> > > I'm not sure how/why this works in the debian builds, because
> > > presumably they're passing the same flags. Perhaps Salvatore can
> > > elaborate?
> >
> > Could you give the patch a shot?
>
> Yep, works for me,
>
> Tested-by: Tycho Andersen <tycho.andersen at canonical.com>
I believe we can merge such thing. I'll prepare complete
patch and resend. Thanks!
>
> > ---
> > diff --git a/scripts/nmk/scripts/build.mk b/scripts/nmk/scripts/build.mk
> > index bd409441293e..cee9a93c53b2 100644
> > --- a/scripts/nmk/scripts/build.mk
> > +++ b/scripts/nmk/scripts/build.mk
> > @@ -25,6 +25,13 @@ ifndef obj
> > endif
> >
> > #
> > +# Filter out any -Wl,XXX option: some of build farms
> > +# assumes that we're using $(CC) for building built-in
> > +# targets.
> > +LDFLAGS-MASK := -Wl,%
> > +LDFLAGS := $(filter-out $(LDFLAGS-MASK),$(LDFLAGS))
> > +
> > +#
> > # Accumulate common flags.
> > define nmk-ccflags
> > $(CFLAGS) $(ccflags-y) $(CFLAGS_$(@F))
>
Cyrill
More information about the CRIU
mailing list