all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Ludovic Courtès" <ludo@gnu.org>
To: Greg Hogan <code@greghogan.com>
Cc: 46728@debbugs.gnu.org, Leo Famulari <leo@famulari.name>
Subject: [bug#46728] [PATCH 0/7] gnu: aws-sdk-cpp: Update to 1.8.148.
Date: Mon, 08 Mar 2021 14:49:38 +0100	[thread overview]
Message-ID: <87h7lleowd.fsf@gnu.org> (raw)
In-Reply-To: <CA+3U0ZkBiDs10Z-GYSejt8sug+7s2B531i-+5mjU6QZ-B1gkvQ@mail.gmail.com> (Greg Hogan's message of "Tue, 2 Mar 2021 15:13:29 -0500")

Hi Greg,

Greg Hogan <code@greghogan.com> skribis:

> On Tue, Mar 2, 2021 at 2:37 PM Ludovic Courtès <ludo@gnu.org> wrote:

[...]

>> In general, it’s best to build only shared libraries.  That allows for
>> reduced memory usage, fast security updates via grafts, etc.  So if you
>> could make the AWS packages build as shared libraries only, that’d be
>> great.
>>
>> It’s OK to optionally add static libraries, preferably in a separate
>> output or even a separate package, but that’s not what should be used by
>> default.
>>
>> Does that make sense?
>
>
> Yes, I would prefer to provide shared as the default output and static as a
> separate output to the same package.

One option is to not provide static libraries at all, at least for now.
They make little sense from a Guix perspective anyway.

Would that be an option for you?

> There look to be three options:
>
> 1) separate packages (as with boost and boost-static). This is simple but
> not as elegant (boost-static is the eighth most relevant package when guix
> search'ing for "boost").
>
> 2) duplicate CMake phases in the package definition (much more verbose than
> with gnu-build-system). This results in considerable duplication of build
> system code and is unmaintainable.

Can’t CMake build both static and “shared” (-fPIC) libraries in one go?
I think it can do that.

> 3) extend the cmake-build-system to optionally configure, build, and
> install static libraries with additional phases. The ideal result but
> requires modification of the cmake-build-system.

In the vast majority of cases, we don’t provide static libraries at
all.  The only exceptions are when we need statically linked binaries
for bootstrapping purposes or within the initrd.

HTH,
Ludo’.




  reply	other threads:[~2021-03-08 13:58 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-23 23:11 [bug#46728] [PATCH 0/7] gnu: aws-sdk-cpp: Update to 1.8.148 Greg Hogan
     [not found] ` <handler.46728.B.161412192713427.ack@debbugs.gnu.org>
2021-02-23 23:14   ` [bug#46728] [PATCH 1/7] gnu: aws-c-common: Update to 0.5.1 Greg Hogan
2021-02-23 23:15   ` [bug#46728] [PATCH 2/7] gnu: aws-checksums: Update to 0.1.11 Greg Hogan
2021-02-23 23:16   ` [bug#46728] [PATCH 3/7] gnu: Add aws-c-cal Greg Hogan
2021-02-23 23:17   ` [bug#46728] [PATCH 4/7] gnu: Add s2n Greg Hogan
2021-02-24 16:34     ` Leo Famulari
2021-02-25 14:19       ` Greg Hogan
2021-03-02 19:37         ` [bug#46728] [PATCH 0/7] gnu: aws-sdk-cpp: Update to 1.8.148 Ludovic Courtès
2021-03-02 20:13           ` Greg Hogan
2021-03-08 13:49             ` Ludovic Courtès [this message]
2021-03-11 21:37               ` Greg Hogan
2021-03-12 23:01                 ` bug#46728: " Ludovic Courtès
2021-02-23 23:18   ` [bug#46728] [PATCH 5/7] gnu: Add aws-c-io Greg Hogan
2021-02-23 23:18   ` [bug#46728] [PATCH 6/7] gnu: aws-c-event-stream: Update to 0.2.6 Greg Hogan
2021-02-23 23:19   ` [bug#46728] [PATCH 7/7] gnu: aws-sdk-cpp: Update to 1.8.148 Greg Hogan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h7lleowd.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=46728@debbugs.gnu.org \
    --cc=code@greghogan.com \
    --cc=leo@famulari.name \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.