clang 3.8 and 3.9 added

[InderKaur]

New versions of clang are added to .travis.yml file. #105

[jmmut]

why both 3.8 and 3.9? each compiler we add makes the build slower, so we have to justify each addition. I didn't have time to check which version is used by default in newer linuxes and newer OSXs.

[cyenyxe]

Clang 4 and 5 are now supported in Travis, see travis-ci/apt-package-safelist#3297 and travis-ci/apt-package-safelist#4034

What I think we should do is to deprecate some of the oldest versions by checking what is the last Ubuntu LTS onwards, and add these more modern ones to the build. I would also keep at least one 3.x version.

[InderKaur]

I have done the requested changes. please have a look

[cyenyxe]

Any particular reason why you have left the oldest Clang 3.x version?

[jmmut]

thanks for contributing to the project, but there are some things that have to be done before we can merge it:

• first of all, travis has to pass (look at the red "X"). It's failing now because clang 4 and 5 are not available in apt-get repositories (https://travis-ci.org/EBIvariation/vcf-validator/builds/337655482?utm_source=github_status&utm_medium=notification). try googling on how to install those versions in ubuntu trusty (the OS used by travis).
• keep only the newest version for each major release (e.g. keep 3.9 as the only 3.x release)

[Anishka0107]

The latest version of clang I can see in ubuntu trusty is 3.9 (https://packages.ubuntu.com/trusty/devel/)

Maybe this could help you in installing clang 4.0 in trusty: https://superuser.com/questions/1280068/how-do-i-install-clang-4-0-on-ubuntu-14-04

See this too, for further information: https://apt.llvm.org

But I'm not sure, I'm not very familiar with Ubuntu.

[cyenyxe]

Closed due to lack of activity.