Request 769057 (accepted)

Overview

Request 769057 accepted

Drop llvm6-devel requirement for ARM

Created by Guillaume_G over 4 years ago
In state accepted

Submit ghc-bootstrap

Comments for request 769057 8

Guillaume GARDET (Guillaume_G) - over 4 years ago

author

How do you fix ghc in Factory then? https://build.opensuse.org/package/show/openSUSE:Factory:ARM/ghc

Dominique Leuenberger (dimstar) - over 4 years ago

reviewer

@ptrommler

Then you have bigger issues: ghc.spec has:

%ifarch aarch64 %{arm} %{ix86} x86_64
%if 0%{?suse_version} >= 1550
BuildRequires:  llvm7-devel
%else
BuildRequires:  llvm-devel
%endif
%endif

so you BuildRequire llvm7-devel - and ghc-bootstrap requires llvm6-devel: the two conflict (which is why ghc is unresolvable in :ARM at the moment)

Peter Trommler (ptrommler) - over 4 years ago

True. We have a big issue here. We need llvm6 to run ghc 8.6.5 which is our bootstrap compiler and llvm7 to run stage1 of ghc 8.8.x to build stage 2.

I could update the ghc-bootstrap to ghc 8.8.2 then the versions would match but that will only offload the issue onto our users. When LLVM is updated to the next major version GHC is likely to break.

A clean solution would be to have an llvm package for version 6 and one for version 7 that can live in parallel. All GHC needs is opt and llc and if they were available as opt-<major> and llc-<major> GHC would even work after an upgrade of LLVM.

This requires a change in the packaging of LLVM. Should we raise a ticket for that?

Dominique Leuenberger (dimstar) - over 4 years ago

reviewer

Not sure if having multiple llvm-devel packages in parallel is going to be a lot of fun.

The libraries for example don't contain a version, i.e.

/usr/bin/llvm-config
/usr/lib64/libLLVM.so

then there is /usr/include/llvm, and /usr/lib64/cmake/llvm that conflict in the -devel packages

You can try with a ticket - not sure if there is already some though to have this parallel installable or if the maintainer would not want this extra complexity

Peter Trommler (ptrommler) - over 4 years ago

I think, requiring llvm<major> would be sufficient. I don't remember why I required llvm,major>-devel. The all that would be required are versioned links as described above. I hope, I can convince the maintainers to add that :-)

I will file a ticket and continue discussion there.

Dominique Leuenberger (dimstar) - over 4 years ago

reviewer

llvm is already parallel installable:

> rpm -qa llvm*
llvm9-polly-9.0.1-4.1.x86_64
llvm9-vim-plugins-9.0.1-4.1.noarch
llvm9-gold-9.0.1-4.1.x86_64
llvm9-LTO-devel-9.0.1-4.1.x86_64
llvm9-devel-9.0.1-4.1.x86_64
llvm-gold-9.0.1-1.1.x86_64
llvm9-doc-9.0.1-4.1.noarch
llvm9-polly-devel-9.0.1-4.1.x86_64
llvm9-9.0.1-4.1.x86_64
llvm7-7.0.1-13.5.x86_64
llvm-devel-9.0.1-1.1.x86_64

> llvm-config --version
9.0.1

> ls -la /usr/bin/clang
lrwxrwxrwx 1 root root 23 Jan 23 20:00 /usr/bin/clang -> /etc/alternatives/clang

> clang --version
clang version 9.0.1

Dominique Leuenberger (dimstar) - over 4 years ago

reviewer

and for opt:

> ls -la /usr/bin//opt*
lrwxrwxrwx 1 root root      21 Jan 23 20:00 /usr/bin//opt -> /etc/alternatives/opt
-rwxr-xr-x 1 root root 3746704 Jan 23 20:00 /usr/bin//opt-7.0.1
-rwxr-xr-x 1 root root 4664656 Jan 24 16:16 /usr/bin//opt-9.0.1

> opt --version
LLVM (http://llvm.org/):
  LLVM version 9.0.1
  Optimized build.
  Default target: x86_64-unknown-linux-gnu
  Host CPU: haswell

Staging Bot (staging-bot) - about 4 years ago

reviewer

@mimi_vx, @psimons, @sbahling, @spanne: review reminder

Request History

Guillaume_G created request over 4 years ago

Drop llvm6-devel requirement for ARM

factory-auto added opensuse-review-team as a reviewer over 4 years ago

Please review sources

factory-auto accepted review over 4 years ago

Check script succeeded

licensedigger accepted review over 4 years ago

staging-bot added as a reviewer over 4 years ago

Being evaluated by staging project "openSUSE:Factory:Staging:adi:60"

staging-bot accepted review over 4 years ago

Picked "openSUSE:Factory:Staging:adi:60"

dimstar accepted review over 4 years ago

ptrommler declined review over 4 years ago

This would break ghc-bootstrap-helpers in Factory ARM. Package ghc-bootstrap needs opt and llc at version 6.* to work and that is the reason ghc-bootstrap requires llvm6-devel.

ptrommler declined request over 4 years ago

This would break ghc-bootstrap-helpers in Factory ARM. Package ghc-bootstrap needs opt and llc at version 6.* to work and that is the reason ghc-bootstrap requires llvm6-devel.