Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Sage build with clang (3.7+) and make it the default on OS X #12426

Closed
ohanar opened this issue Feb 4, 2012 · 556 comments
Closed

Make Sage build with clang (3.7+) and make it the default on OS X #12426

ohanar opened this issue Feb 4, 2012 · 556 comments
Assignees
Milestone

Comments

@ohanar
Copy link
Member

ohanar commented Feb 4, 2012

Apple stopped supporting GCC in favour of clang. This is to track the status of building Sage with clang with the aim of making it the default compiler used on OS X and other OS where it is the default compiler.


Packages failing their tests


Supporting clang not only on OSX:

Depends on #24579

CC: @nexttime @jpflori @dimpase @mkoeppe @isuruf

Component: build

Keywords: clang porting

Author: François Bissey

Branch: ac7816d

Reviewer: Jeroen Demeyer, John Palmieri, Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/12426

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@jdemeyer
Copy link

jdemeyer commented Feb 6, 2012

comment:7

I guess many of these tickets are useful for porting in general.

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@ohanar

This comment has been minimized.

@nexttime

This comment has been minimized.

@nexttime

This comment has been minimized.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 8, 2012

Changed keywords from clang to clang C++11 porting

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 8, 2012

comment:13

Added further ticket references.

@nexttime

This comment has been minimized.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Jun 3, 2013

comment:15

Description presumably needs some updating...

@nexttime nexttime mannequin changed the title make sage build with clang (3.0+) Make Sage build with clang (3.0+) Jun 3, 2013
@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Jun 3, 2013

comment:22

eclib has been fixed upstream last year (IIRC) as well; currently re-checking whether our current version still builds...

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@vbraun
Copy link
Member

vbraun commented Jan 21, 2018

comment:483

Is installed and matches version

buildbot@sagebd07_64s02:~$ gcc --version
gcc (Debian 4.7.2-5) 4.7.2
[...]
buildbot@sagebd07_64s02:~$ gfortran --version
GNU Fortran (Debian 4.7.2-5) 4.7.2

@vbraun
Copy link
Member

vbraun commented Jan 21, 2018

comment:484
  • The error happend with both incremental and full build
  • Only happens on Debian 7, not other distros

@kiwifb
Copy link
Member

kiwifb commented Jan 21, 2018

comment:485

The bit that's supposed to prevent this from happening in configure is

# if GCC needs to be installed GFORTRAN shouldn't be installed
# GCC will provide GFORTRAN in that case
if test $need_to_install_gcc = yes ; then
    need_to_install_gfortran=no
fi

so this test must be failing with the shell used debian 7. It will to be converted into something more universally reliable.

@dimpase
Copy link
Member

dimpase commented Jan 22, 2018

comment:486

Could it be that on that Debian 7 system /bin/sh is not bash? This might explain this singularity.

@jdemeyer
Copy link

comment:487

Please read [comment:479] for a possible explanation and see #24579 for a possible fix...

@dimpase
Copy link
Member

dimpase commented Jan 27, 2018

comment:488

Why is this in sage-pending?

@jdemeyer jdemeyer added this to the sage-8.2 milestone Jan 28, 2018
@jdemeyer jdemeyer removed the pending label Jan 28, 2018
@vbraun
Copy link
Member

vbraun commented Feb 1, 2018

Changed branch from u/jdemeyer/clang-progress to ac7816d

@kiwifb
Copy link
Member

kiwifb commented Feb 1, 2018

comment:491

Thanks everyone!

@kiwifb
Copy link
Member

kiwifb commented Feb 1, 2018

Changed commit from ac7816d to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests