unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#50778] [PATCH] m4: Don't require courage on unsupported systems.
@ 2021-09-24 13:39 Efraim Flashner
  2021-10-13  9:00 ` Ludovic Courtès
  0 siblings, 1 reply; 7+ messages in thread
From: Efraim Flashner @ 2021-09-24 13:39 UTC (permalink / raw)
  To: 50778; +Cc: ludo

[-- Attachment #1: Type: text/plain, Size: 3003 bytes --]

* m4/guix.m4 (GUIX_ASSERT_SUPPORTED_SYSTEM): Don't error if building on
an unsupported system.
* doc/guix.texi (GNU Distribution): Relabel mips64el-linux as
unsupported.
---

As we discussed previously with the powerpc-linux patches, I've gotten
around to changing the m4 error to a warning. Also, I didn't change
i586-gnu to say unsupported next to it in GNU Distribution in the
manual, although I suppose the other option would be to add i586-gnu to
supported in m4/guix.m4.

---
 doc/guix.texi |  2 +-
 m4/guix.m4    | 17 +++--------------
 2 files changed, 4 insertions(+), 15 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 9bb91b94fd..5cec6c0645 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -525,7 +525,7 @@ way for you to give it a try is by setting up an instance of
 (@pxref{transparent-emulation-qemu, @code{hurd-vm-service-type}}).
 @xref{Contributing}, on how to help!
 
-@item mips64el-linux (deprecated)
+@item mips64el-linux (unsupported)
 little-endian 64-bit MIPS processors, specifically the Loongson series,
 n32 ABI, and Linux-Libre kernel.  This configuration is no longer fully
 supported; in particular, there is no ongoing work to ensure that this
diff --git a/m4/guix.m4 b/m4/guix.m4
index 05d409a674..7a5eaa47f9 100644
--- a/m4/guix.m4
+++ b/m4/guix.m4
@@ -1,7 +1,7 @@
 dnl GNU Guix --- Functional package management for GNU
 dnl Copyright © 2012, 2013, 2014, 2015, 2016, 2018, 2019, 2020, 2021 Ludovic Courtès <ludo@gnu.org>
 dnl Copyright © 2014 Mark H Weaver <mhw@netris.org>
-dnl Copyright © 2017 Efraim Flashner <efraim@flashner.co.il>
+dnl Copyright © 2017, 2021 Efraim Flashner <efraim@flashner.co.il>
 dnl Copyright © 2021 Chris Marusich <cmmarusich@gmail.com>
 dnl
 dnl This file is part of GNU Guix.
@@ -79,25 +79,14 @@ dnl Assert that this is a system to which the distro is ported.
 AC_DEFUN([GUIX_ASSERT_SUPPORTED_SYSTEM], [
   AC_REQUIRE([GUIX_SYSTEM_TYPE])
 
-  AC_ARG_WITH([courage], [AS_HELP_STRING([--with-courage],
-    [Assert that even if this platform is unsupported, you will be
-courageous and port the GNU System distribution to it (see
-"GNU Distribution" in the manual.)])],
-    [guix_courageous="$withval"],
-    [guix_courageous="no"])
-
   # Currently only Linux-based systems are supported, and only on some
   # platforms.
   case "$guix_system" in
     x86_64-linux|i686-linux|armhf-linux|aarch64-linux|powerpc64le-linux)
       ;;
     *)
-      if test "x$guix_courageous" = "xyes"; then
-        AC_MSG_WARN([building Guix on `$guix_system', which is not supported])
-      else
-        AC_MSG_ERROR([`$guix_system' is not a supported platform.
-See "GNU Distribution" in the manual, or try `--with-courage'.])
-      fi
+      AC_MSG_WARN([`$guix_system' IS NOT A SUPPORTED PLATFORM.
+See "GNU Distribution" in the manual, for more information.])
       ;;
   esac
 ])

base-commit: a7b6c5a1270e52385a70b4192ee40ac6eb3b7c27
-- 
2.33.0


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [bug#50778] [PATCH] m4: Don't require courage on unsupported systems.
  2021-09-24 13:39 [bug#50778] [PATCH] m4: Don't require courage on unsupported systems Efraim Flashner
@ 2021-10-13  9:00 ` Ludovic Courtès
  2021-10-17  7:20   ` Efraim Flashner
  0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2021-10-13  9:00 UTC (permalink / raw)
  To: 50778

Efraim Flashner <efraim@flashner.co.il> skribis:

> * m4/guix.m4 (GUIX_ASSERT_SUPPORTED_SYSTEM): Don't error if building on
> an unsupported system.
> * doc/guix.texi (GNU Distribution): Relabel mips64el-linux as
> unsupported.
> ---
>
> As we discussed previously with the powerpc-linux patches, I've gotten
> around to changing the m4 error to a warning. Also, I didn't change
> i586-gnu to say unsupported next to it in GNU Distribution in the
> manual, although I suppose the other option would be to add i586-gnu to
> supported in m4/guix.m4.

Yes, I’d rather mark i586-gnu as supported in m4/guix.m4 (it should have
been the case long ago) and keep the error and ‘--with-courage’ flag.

WDYT?

> -@item mips64el-linux (deprecated)
> +@item mips64el-linux (unsupported)

Good idea.  :-)

Thanks!

Ludo’.




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [bug#50778] [PATCH] m4: Don't require courage on unsupported systems.
  2021-10-13  9:00 ` Ludovic Courtès
@ 2021-10-17  7:20   ` Efraim Flashner
  2021-10-18  8:43     ` Ludovic Courtès
  0 siblings, 1 reply; 7+ messages in thread
From: Efraim Flashner @ 2021-10-17  7:20 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 50778

[-- Attachment #1: Type: text/plain, Size: 1851 bytes --]

On Wed, Oct 13, 2021 at 11:00:16AM +0200, Ludovic Courtès wrote:
> Efraim Flashner <efraim@flashner.co.il> skribis:
> 
> > * m4/guix.m4 (GUIX_ASSERT_SUPPORTED_SYSTEM): Don't error if building on
> > an unsupported system.
> > * doc/guix.texi (GNU Distribution): Relabel mips64el-linux as
> > unsupported.
> > ---
> >
> > As we discussed previously with the powerpc-linux patches, I've gotten
> > around to changing the m4 error to a warning. Also, I didn't change
> > i586-gnu to say unsupported next to it in GNU Distribution in the
> > manual, although I suppose the other option would be to add i586-gnu to
> > supported in m4/guix.m4.
> 
> Yes, I’d rather mark i586-gnu as supported in m4/guix.m4 (it should have
> been the case long ago) and keep the error and ‘--with-courage’ flag.
> 
> WDYT?

I'm OK with changing i586-gnu to supported in m4/guix.m4. However, part
of the plan was to make it so that we didn't have to have the
--with-courage flag when building for mips64el, and so we could remove
powerpc-linux so it would also say that it is unsupported but we
wouldn't need people to make special versions of the guix package so
they can try to build it. I can make it bigger and flashier, so it looks
more like

****************************************

WARNING: YOU ARE ATTEMPTING TO BUILD GUIX ON AN UNSUPPORTED PLATFORM!
PROCEED AT YOUR OWN RISK!!! YOU HAVE BEEN WARNED! HERE BE DRAGONS!!!!

****************************************

> > -@item mips64el-linux (deprecated)
> > +@item mips64el-linux (unsupported)
> 
> Good idea.  :-)
> 
> Thanks!
> 
> Ludo’.
> 
> 
> 

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [bug#50778] [PATCH] m4: Don't require courage on unsupported systems.
  2021-10-17  7:20   ` Efraim Flashner
@ 2021-10-18  8:43     ` Ludovic Courtès
  2021-10-18 12:15       ` Efraim Flashner
  0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2021-10-18  8:43 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: 50778

Hello!

Efraim Flashner <efraim@flashner.co.il> skribis:

> On Wed, Oct 13, 2021 at 11:00:16AM +0200, Ludovic Courtès wrote:

[...]

>> Yes, I’d rather mark i586-gnu as supported in m4/guix.m4 (it should have
>> been the case long ago) and keep the error and ‘--with-courage’ flag.
>> 
>> WDYT?
>
> I'm OK with changing i586-gnu to supported in m4/guix.m4. However, part
> of the plan was to make it so that we didn't have to have the
> --with-courage flag when building for mips64el, and so we could remove
> powerpc-linux so it would also say that it is unsupported but we
> wouldn't need people to make special versions of the guix package so
> they can try to build it. I can make it bigger and flashier, so it looks
> more like

Ah!  So I think you’re asking that there be three levels:

  1. Fully supported (should include i586-gnu).

  2. “Half supported” or “in the works”, like mips64el-linux and
     powerpc-linux: in that case, maybe just emit a warning with
     AC_MSG_WARN?

  3. Unsupported: error asking users to pass ‘--with-courage’.

How does that sound?

Ludo’.




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [bug#50778] [PATCH] m4: Don't require courage on unsupported systems.
  2021-10-18  8:43     ` Ludovic Courtès
@ 2021-10-18 12:15       ` Efraim Flashner
  2021-10-19 10:15         ` Ludovic Courtès
  2021-12-14 16:36         ` bug#50778: " Efraim Flashner
  0 siblings, 2 replies; 7+ messages in thread
From: Efraim Flashner @ 2021-10-18 12:15 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 50778

[-- Attachment #1: Type: text/plain, Size: 1981 bytes --]

On Mon, Oct 18, 2021 at 10:43:54AM +0200, Ludovic Courtès wrote:
> Hello!
> 
> Efraim Flashner <efraim@flashner.co.il> skribis:
> 
> > On Wed, Oct 13, 2021 at 11:00:16AM +0200, Ludovic Courtès wrote:
> 
> [...]
> 
> >> Yes, I’d rather mark i586-gnu as supported in m4/guix.m4 (it should have
> >> been the case long ago) and keep the error and ‘--with-courage’ flag.
> >> 
> >> WDYT?
> >
> > I'm OK with changing i586-gnu to supported in m4/guix.m4. However, part
> > of the plan was to make it so that we didn't have to have the
> > --with-courage flag when building for mips64el, and so we could remove
> > powerpc-linux so it would also say that it is unsupported but we
> > wouldn't need people to make special versions of the guix package so
> > they can try to build it. I can make it bigger and flashier, so it looks
> > more like
> 
> Ah!  So I think you’re asking that there be three levels:
> 
>   1. Fully supported (should include i586-gnu).
> 
>   2. “Half supported” or “in the works”, like mips64el-linux and
>      powerpc-linux: in that case, maybe just emit a warning with
>      AC_MSG_WARN?
> 
>   3. Unsupported: error asking users to pass ‘--with-courage’.
> 
> How does that sound?

I hadn't really planned on keeping the third category, but thinking
about it more it would make sense to keep it for architectures that we
don't even have bootstrap binaries for. Vagrant said he was able to
build the guix package for riscv64-linux, I think without even adding my
patches for adding the bootstrap binaries. By forcing people to pass
--with-courage it goes past "it doesn't really work" and all the way
into "really really unsupported"

So yes, I like the idea of the three levels.

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [bug#50778] [PATCH] m4: Don't require courage on unsupported systems.
  2021-10-18 12:15       ` Efraim Flashner
@ 2021-10-19 10:15         ` Ludovic Courtès
  2021-12-14 16:36         ` bug#50778: " Efraim Flashner
  1 sibling, 0 replies; 7+ messages in thread
From: Ludovic Courtès @ 2021-10-19 10:15 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: 50778

Howdy!

Efraim Flashner <efraim@flashner.co.il> skribis:

> On Mon, Oct 18, 2021 at 10:43:54AM +0200, Ludovic Courtès wrote:

[...]

>> Ah!  So I think you’re asking that there be three levels:
>> 
>>   1. Fully supported (should include i586-gnu).
>> 
>>   2. “Half supported” or “in the works”, like mips64el-linux and
>>      powerpc-linux: in that case, maybe just emit a warning with
>>      AC_MSG_WARN?
>> 
>>   3. Unsupported: error asking users to pass ‘--with-courage’.
>> 
>> How does that sound?
>
> I hadn't really planned on keeping the third category, but thinking
> about it more it would make sense to keep it for architectures that we
> don't even have bootstrap binaries for. Vagrant said he was able to
> build the guix package for riscv64-linux, I think without even adding my
> patches for adding the bootstrap binaries. By forcing people to pass
> --with-courage it goes past "it doesn't really work" and all the way
> into "really really unsupported"
>
> So yes, I like the idea of the three levels.

Alright.  Can you send (or push) an updated patch?

Thanks!

Ludo’.




^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#50778: [PATCH] m4: Don't require courage on unsupported systems.
  2021-10-18 12:15       ` Efraim Flashner
  2021-10-19 10:15         ` Ludovic Courtès
@ 2021-12-14 16:36         ` Efraim Flashner
  1 sibling, 0 replies; 7+ messages in thread
From: Efraim Flashner @ 2021-12-14 16:36 UTC (permalink / raw)
  To: Ludovic Courtès, 50778-done

[-- Attachment #1: Type: text/plain, Size: 2396 bytes --]

Finally took care of this today.

On Mon, Oct 18, 2021 at 03:15:54PM +0300, Efraim Flashner wrote:
> On Mon, Oct 18, 2021 at 10:43:54AM +0200, Ludovic Courtès wrote:
> > Hello!
> > 
> > Efraim Flashner <efraim@flashner.co.il> skribis:
> > 
> > > On Wed, Oct 13, 2021 at 11:00:16AM +0200, Ludovic Courtès wrote:
> > 
> > [...]
> > 
> > >> Yes, I’d rather mark i586-gnu as supported in m4/guix.m4 (it should have
> > >> been the case long ago) and keep the error and ‘--with-courage’ flag.
> > >> 
> > >> WDYT?
> > >
> > > I'm OK with changing i586-gnu to supported in m4/guix.m4. However, part
> > > of the plan was to make it so that we didn't have to have the
> > > --with-courage flag when building for mips64el, and so we could remove
> > > powerpc-linux so it would also say that it is unsupported but we
> > > wouldn't need people to make special versions of the guix package so
> > > they can try to build it. I can make it bigger and flashier, so it looks
> > > more like
> > 
> > Ah!  So I think you’re asking that there be three levels:
> > 
> >   1. Fully supported (should include i586-gnu).
> > 
> >   2. “Half supported” or “in the works”, like mips64el-linux and
> >      powerpc-linux: in that case, maybe just emit a warning with
> >      AC_MSG_WARN?
> > 
> >   3. Unsupported: error asking users to pass ‘--with-courage’.
> > 
> > How does that sound?
> 
> I hadn't really planned on keeping the third category, but thinking
> about it more it would make sense to keep it for architectures that we
> don't even have bootstrap binaries for. Vagrant said he was able to
> build the guix package for riscv64-linux, I think without even adding my
> patches for adding the bootstrap binaries. By forcing people to pass
> --with-courage it goes past "it doesn't really work" and all the way
> into "really really unsupported"
> 
> So yes, I like the idea of the three levels.
> 
> -- 
> Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
> GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
> Confidentiality cannot be guaranteed on emails sent or received unencrypted



-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-12-14 16:38 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-24 13:39 [bug#50778] [PATCH] m4: Don't require courage on unsupported systems Efraim Flashner
2021-10-13  9:00 ` Ludovic Courtès
2021-10-17  7:20   ` Efraim Flashner
2021-10-18  8:43     ` Ludovic Courtès
2021-10-18 12:15       ` Efraim Flashner
2021-10-19 10:15         ` Ludovic Courtès
2021-12-14 16:36         ` bug#50778: " Efraim Flashner

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).