all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Why is GCL built with gcc@4.9?
@ 2018-12-13  5:03 Mark H Weaver
  2018-12-13 20:33 ` Efraim Flashner
  0 siblings, 1 reply; 4+ messages in thread
From: Mark H Weaver @ 2018-12-13  5:03 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: guix-devel

Hi Efraim,

I'm curious about this commit of yours from April 2017:

--8<---------------cut here---------------start------------->8---
commit 5c7815f205e9164d4b82378de91bee7a65bcfbcb
Author: Efraim Flashner <efraim@flashner.co.il>
Date:   Mon Apr 10 05:20:09 2017 +0300

    gnu: gcl: Build with gcc@4.9.
    
    * gnu/packages/lisp.scm (gcl)[native-inputs]: Add gcc@4.9.
--8<---------------cut here---------------end--------------->8---

Do you remember why you did this?  There's no explanation in the
comments, nor in the commit log, nor in the 'bug-guix' or 'guix-devel'
email archives from around that time.

I'd like to remove it, and I've verified that on x86_64-linux, GCL
builds successfully with the default compiler.

In general, it would be good to include comments with rationale for
workarounds like this, so that we have some idea of when the workaround
can be removed, and what tests we must do to ensure that the original
problem has been addressed.

     Thanks,
       Mark

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

* Re: Why is GCL built with gcc@4.9?
  2018-12-13  5:03 Why is GCL built with gcc@4.9? Mark H Weaver
@ 2018-12-13 20:33 ` Efraim Flashner
  2018-12-16  7:37   ` Efraim Flashner
  2018-12-16  8:02   ` Mark H Weaver
  0 siblings, 2 replies; 4+ messages in thread
From: Efraim Flashner @ 2018-12-13 20:33 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

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

On Thu, Dec 13, 2018 at 12:03:59AM -0500, Mark H Weaver wrote:
> Hi Efraim,
> 
> I'm curious about this commit of yours from April 2017:
> 
> --8<---------------cut here---------------start------------->8---
> commit 5c7815f205e9164d4b82378de91bee7a65bcfbcb
> Author: Efraim Flashner <efraim@flashner.co.il>
> Date:   Mon Apr 10 05:20:09 2017 +0300
> 
>     gnu: gcl: Build with gcc@4.9.
>     
>     * gnu/packages/lisp.scm (gcl)[native-inputs]: Add gcc@4.9.
> --8<---------------cut here---------------end--------------->8---
> 
> Do you remember why you did this?  There's no explanation in the
> comments, nor in the commit log, nor in the 'bug-guix' or 'guix-devel'
> email archives from around that time.
> 
> I'd like to remove it, and I've verified that on x86_64-linux, GCL
> builds successfully with the default compiler.
> 
> In general, it would be good to include comments with rationale for
> workarounds like this, so that we have some idea of when the workaround
> can be removed, and what tests we must do to ensure that the original
> problem has been addressed.
> 
>      Thanks,
>        Mark

I looked through the commits and I'm not sure why I added gcc@4.9. When
did we change our default gcc from 4.9 to 5? I've made one attempt so
far at building on aarch64-linux without gcc@4.9 and I got a core-dump
but I haven't built it recently to see if it does that as-is.

I'll take a closer look at it and try to see what's up.


-- 
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] 4+ messages in thread

* Re: Why is GCL built with gcc@4.9?
  2018-12-13 20:33 ` Efraim Flashner
@ 2018-12-16  7:37   ` Efraim Flashner
  2018-12-16  8:02   ` Mark H Weaver
  1 sibling, 0 replies; 4+ messages in thread
From: Efraim Flashner @ 2018-12-16  7:37 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

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

On Thu, Dec 13, 2018 at 10:33:48PM +0200, Efraim Flashner wrote:
> On Thu, Dec 13, 2018 at 12:03:59AM -0500, Mark H Weaver wrote:
> > Hi Efraim,
> > 
> > I'm curious about this commit of yours from April 2017:
> > 
> > --8<---------------cut here---------------start------------->8---
> > commit 5c7815f205e9164d4b82378de91bee7a65bcfbcb
> > Author: Efraim Flashner <efraim@flashner.co.il>
> > Date:   Mon Apr 10 05:20:09 2017 +0300
> > 
> >     gnu: gcl: Build with gcc@4.9.
> >     
> >     * gnu/packages/lisp.scm (gcl)[native-inputs]: Add gcc@4.9.
> > --8<---------------cut here---------------end--------------->8---
> > 
> > Do you remember why you did this?  There's no explanation in the
> > comments, nor in the commit log, nor in the 'bug-guix' or 'guix-devel'
> > email archives from around that time.
> > 
> > I'd like to remove it, and I've verified that on x86_64-linux, GCL
> > builds successfully with the default compiler.
> > 
> > In general, it would be good to include comments with rationale for
> > workarounds like this, so that we have some idea of when the workaround
> > can be removed, and what tests we must do to ensure that the original
> > problem has been addressed.
> > 
> >      Thanks,
> >        Mark
> 
> I looked through the commits and I'm not sure why I added gcc@4.9. When
> did we change our default gcc from 4.9 to 5? I've made one attempt so
> far at building on aarch64-linux without gcc@4.9 and I got a core-dump
> but I haven't built it recently to see if it does that as-is.
> 
> I'll take a closer look at it and try to see what's up.
> 

I tried compiling gcl with gcc@7 also and it still failed on aarch64.
The closest I have to a clue is that Debian's rule file has commented
out to use gcc@4.6, so I'm guessing that following our upgrade to
building with gcc@5 the package broke and it worked for a time with
gcc@4.9. Since it fails to build in any case on aarch64 and building
with gcc@5 doesn't cause any problems I'll go ahead and remove it.


-- 
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] 4+ messages in thread

* Re: Why is GCL built with gcc@4.9?
  2018-12-13 20:33 ` Efraim Flashner
  2018-12-16  7:37   ` Efraim Flashner
@ 2018-12-16  8:02   ` Mark H Weaver
  1 sibling, 0 replies; 4+ messages in thread
From: Mark H Weaver @ 2018-12-16  8:02 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: guix-devel

Hi Efraim,

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

> I looked through the commits and I'm not sure why I added gcc@4.9. When
> did we change our default gcc from 4.9 to 5?

The change was made on core-updates on 12 December 2016 (commit
b810a85019ab3c4ee1f889d0751b8eb06157dadc), and merged into master on
3 April 2017 (commit 2a0d5de5a9decd785b22dafa69aae5320231f1b7).  You
applied the workaround a week later.

> I'll take a closer look at it and try to see what's up.

I see that you recently removed the workaround.
Thanks for taking care of it.

    Regards,
      Mark

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

end of thread, other threads:[~2018-12-16  8:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-13  5:03 Why is GCL built with gcc@4.9? Mark H Weaver
2018-12-13 20:33 ` Efraim Flashner
2018-12-16  7:37   ` Efraim Flashner
2018-12-16  8:02   ` Mark H Weaver

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.