Re: cwrappers (Was: shell wrappers: merge -l nameoflib into a single -lnameoflib)

[Jonathan Perkin <jperkin%joyent.com@localhost>]

* On 2016-09-19 at 00:30 BST, David Brownlee wrote:

> On 12 September 2016 at 14:26, Jonathan Perkin <jperkin%joyent.com@localhost> wrote:
> > * On 2016-09-12 at 14:20 BST, Ryo ONODERA wrote:
> >
> >> I am working on pkgsrc/lang/rust on NetBSD and Linux.
> >> The older shell wrappers cannot handle "-l m" (whitespace between -l
> >> and library name, m) and I cannot build some libraries in lang/rust.
> >>
> >> I suggest the following patch.
> >> It does not break
> >>
> >> BUILDLINK_TRANSFORM=  l:execinfo:execinfo:elf
> >>
> >> like BUILDLINK_TRANSFORM.
> >>
> >> What should be confirmed to adopt this change?
> >
> > Looks fine to me.  For the record cwrappers already handles this
> > construct, which is why I didn't notice this issue with lang/rust
> > prior to importing.
> >
> > I'd like to get this integrated before 2016Q3, so I will perform a
> > bulk build just to check that there are no regressions, but I don't
> > expect any.
> >
> > Results should hopefully be available tomorrow, though it's been a
> > long time since I did a non-cwrappers build!
> 
> Does anyone know what was outstanding for cutting across to using
> cwrappers by default? A lot of people have been using it for quite a
> while now, and it would be nice to have only one copy of all that
> infrastructure :)

No, so I propose we make it default after 2016Q3 is cut to expose any
remaining issues and fix them ready for 2016Q4.

-- 
Jonathan Perkin  -  Joyent, Inc.  -  www.joyent.com