all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eric Bavier <bavier@cray.com>
To: "Ludovic Courtès" <ludovic.courtes@inria.fr>,
	"Ricardo Wurmus" <rekado@elephly.net>
Cc: "35666@debbugs.gnu.org" <35666@debbugs.gnu.org>,
	"pgarlick@tourbillion-technology.com"
	<pgarlick@tourbillion-technology.com>
Subject: [bug#35666] [PATCH 0/2] Build a thread-safe hdf5 library
Date: Fri, 10 May 2019 15:09:25 +0000	[thread overview]
Message-ID: <CY4PR1101MB2325973B0EC24AB6A7371D9AB70C0@CY4PR1101MB2325.namprd11.prod.outlook.com> (raw)
In-Reply-To: <87d0kq1mr2.fsf@gnu.org>

I think this should be fine, though I've not heard of anyone who has relied on this feature.  The "unsupported" part here is that the posix lock used for thread-safety is not hoisted into the higher-level API calls.  So if your colleague is using the C++ interface and expecting thread-safety, they are out of luck.  So the disclaimer is that only the low-level C interface gains thread-safety, and the rest are no better.

Eric Bavier, Scientific Libraries, Cray Inc.

________________________________________
From: Ludovic Courtès <ludovic.courtes@inria.fr>
Sent: Friday, May 10, 2019 08:07
To: Ricardo Wurmus
Cc: Eric Bavier; pgarlick@tourbillion-technology.com; 35666@debbugs.gnu.org
Subject: Re: [bug#35666] [PATCH 0/2] Build a thread-safe hdf5 library

Hi!

Ricardo Wurmus <rekado@elephly.net> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:

[…]

>> It also tells you that, if you insist, you can go ahead and pass
>> ‘--enable-unsupported’, but you’re on your own.
>>
>> We found that Debian chose to pass ‘--enable-unsupported’, and indeed
>> that seems to be saner than providing a variant that does very little,
>> but does it in a thread-safe way.
>
> What other effects does “--enable-unsupported” have?  I see that in
> Fedora “--enable-threadsafe” was removed in 2008 because it’s
> “incompatible with --enable-cxx and --enable-fortran”.

“--enable-unsupported” allows you to force a build that combines C++,
Fortran, and thread-safety.  If you don’t pass that flag, you have to
choose between thread-safety and C++/Fortran¹.  A tough choice!

> Instead they seem to be building different flavours: one with
> --enable-fortran, another with --enable-cxx, yet another with MPI and
> --enable-parallel.

Problem is, my colleagues have code that expects both C++ and
thread-safety (as crazy as it might seem).  They were using the Debian
package until now and hadn’t realized about this.

> Do we have contact to the hdf5 developers to ask what the implications
> of “enable-unsupported” are?

I think it’s a warranty-void kind of flag: by passing it, the user
asserts they understand they’re using a configuration not “officially
supported” by the HDF Group, meaning that if it’s buggy, we’re on our
own.

Thoughts?

Ludo’.

¹ You would think it’s an April fool’s day prank, but it’s not!  We’re
  in May, at least in my timezone.

  reply	other threads:[~2019-05-10 15:10 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-10  9:56 [bug#35666] [PATCH 0/2] Build a thread-safe hdf5 library Ludovic Courtès
2019-05-10 10:05 ` [bug#35666] [PATCH 1/2] gnu: hdf5: Build a thread-safe library Ludovic Courtès
2019-05-10 10:05   ` [bug#35666] [PATCH 2/2] gnu: hdf5: Add dependency on Perl Ludovic Courtès
2019-05-10 11:52 ` [bug#35666] [PATCH 0/2] Build a thread-safe hdf5 library Ricardo Wurmus
2019-05-10 13:07   ` Ludovic Courtès
2019-05-10 15:09     ` Eric Bavier [this message]
2019-05-14  7:28       ` Ludovic Courtès
2019-05-14 14:40         ` Eric Bavier
2019-05-10 15:27     ` Paul Garlick
2019-05-14 10:21       ` bug#35666: " Ludovic Courtès
2019-05-14 12:02     ` [bug#35666] " Ricardo Wurmus

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=CY4PR1101MB2325973B0EC24AB6A7371D9AB70C0@CY4PR1101MB2325.namprd11.prod.outlook.com \
    --to=bavier@cray.com \
    --cc=35666@debbugs.gnu.org \
    --cc=ludovic.courtes@inria.fr \
    --cc=pgarlick@tourbillion-technology.com \
    --cc=rekado@elephly.net \
    /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.