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