search

kernsuite / packaging

Use this repository to report issues with packages or request new packages
13 stars 4 forks source link

presto package needs modifcations to use python module #145

Closed gijzelaerr closed 4 years ago

gijzelaerr commented 5 years ago

since there were a lot of upstream modifications, we disabled this patch (for now):

https://github.com/kernsuite-debian/presto/blob/master/debian/patches/series#L3

which breaks all python code. Long term solution is to get all python code into a python module upstream. Issue has been raised upstream:

https://github.com/scottransom/presto/issues/69

gijzelaerr commented 5 years ago

PR is waiting approval https://github.com/scottransom/presto/pull/94

gijzelaerr commented 4 years ago

The currently released presto still fails a default python install:

    x86_64-linux-gnu-gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-Bsymbolic-functions -Wl,-z,relro -g -fwrapv -O2 -Wl,-Bsymbolic-functions -Wl,-z,relro -g -fwrapv -O2 -march=native -O2 -pipe -Wdate-time -D_FORTIFY_SOURCE=2 build/temp.linux-x86_64-3.8/python/presto_src/presto_wrap.o -lpresto -lfftw3f -lm -o _presto.cpython-38-x86_64-linux-gnu.so
    /usr/bin/ld: cannot find -lpresto

Due to time and budget restrictions for KERN I think we just have to drop presto.