search

ldc-developers / ldc2.snap

Snap package definition for LDC, the LLVM-based D compiler
11 stars 4 forks source link

Explicitly specify the LLVM version to use #3

Closed WebDrake closed 7 years ago

WebDrake commented 7 years ago

It seems saner to be explicit about the LLVM version than to rely on a distro-dependent version that may change. Ubuntu 16.04 does not provide a package for LLVM 3.9, so 3.8 is the most recent version that can be used unless the snap builds LLVM itself. This may be worth implementing in future, but it would add heavily to the build time and so has been avoided for now.

WebDrake commented 7 years ago

@JohanEngelen does this seem an improvement to you ... ?

JohanEngelen commented 7 years ago

I think it's better to use the system LLVM version. The snap package is not linked to any particular version of a distro, right? So other distro versions may use an LLVM version different from 3.8, and you'd again have the long build time. I'm no packaging/distro expert by any measure, so don't count on my judgement.

WebDrake commented 7 years ago

Yes, fair enough. I'll drop this, then, and look towards building LLVM in the snap at some point in future.