Bug#833667: freemat: FTBFS with LLVM 3.8

2016-10-07 Thread Gianfranco Costamagna
control: tags -1 patch pending

Hi, based on Graham little testing (sorry but I can't provide testing, I'll 
trust
his wordings), I uploaded on deferred/1 and git committed the patch on repo.

cheers

G.

-- 
debian-science-maintainers mailing list
debian-science-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/debian-science-maintainers


Bug#833667: freemat: FTBFS with LLVM 3.8

2016-10-05 Thread Emilio Pozuelo Monfort
Control: severity -1 serious

On Sun, 7 Aug 2016 18:39:29 +0200 Graham Inggs  wrote:
> Source: freemat
> Version: 4.2+dfsg1-1
> Severity: wishlist
> Tags: patch
> 
> 
> Hi Maintainer
> 
> Freemat currently FTBFS with LLVM 3.8, although LLVM 3.8 is not yet
> the default in Sid.
> I made the following minor changes:
> 
> In libs/libMatC/CJitFuncClang.cpp, include llvm/Config/llvm-config.h
> instead of llvm/Config/config.h and remove the second include of
> llvm/Config/config.h.
> 
> In CMakeLists.txt, link with clangTidyReadabilityModule instead of
> clangTidyReadability.
> 
> I attempted to produce a patch against
> debian/patches/12_update_clang_deps.patch but the result was not very
> clear and the differing line endings made it difficult.  Attached is a
> patch that should be applied after applying
> debian/patches/12_update_clang_deps.patch.
> 
> Not being familiar with freemat, I was not able to do extensive
> testing, but the package built and installed and I was able to open
> the freemat gui and perform some basic operations.
> 
> LLVM 3.8 is already the default in Ubuntu and I would like to apply
> this patch there, but I would appreciate it if you would please review
> it first.

LLVM 3.8 is now the default in Debian too. Your package failed to build, see:

https://buildd.debian.org/status/package.php?p=freemat

Cheers,
Emilio

-- 
debian-science-maintainers mailing list
debian-science-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/debian-science-maintainers