unofficial mirror of guix-patches@gnu.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

  List information: https://guix.gnu.org/

* 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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).