tech-pkg archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: [PATCH] Fix bootstrap on FreeBSD 12.1 (needs -lmd when linking libarchive statically)
* On 2020-10-20 at 04:24 BST, David Holland wrote:
> On Sun, Oct 18, 2020 at 03:15:48PM +0000, coypu%sdf.org@localhost wrote:
> > Hi all,
> >
> > I tried my best to do this as properly as possible, but I have a feeling
> > that I'm not meeting the expected style for this file.
> >
> > Any suggestions, or else I will commit it soon because bootstrap being
> > broken is bad.
>
> Seems reasonable to me, except for one carp: STATIC_LIBARCHIVE_DEPENDS
> sounds like it's something you should use like this:
>
> foo: $(STATIC_LIBARCHIVE_DEPENDS)
> $(CC) $(LDFLAGS) $(OBJS) $(STATIC_LIBARCHIVE) $(LIBS) -o foo
>
> and since that's not what it is, it might be better to use some other
> name. STATIC_LIBARCHIVE_MORELIBS comes to mind as one possible
> choice...
STATIC_LIBARCHIVE_LDADD would be more in line with pkgsrc (and
automake) naming convention.
I don't like the patch, but I can't think of a better way of doing it
right now, so I'd say go for it and we can clean up later if
necessary. It's more important that we aren't broken on platforms.
--
Jonathan Perkin - Joyent, Inc. - www.joyent.com
Home |
Main Index |
Thread Index |
Old Index