Comment 1 for bug 405379

Revision history for this message
Loïc Minier (lool) wrote :

Packaging looks good overall.

Random minor things:
- copyright still mention the SVN for the download location but looking at the version and watch files seems to imply that it's not a SVN snapshot anymore
- DEB_DH_STRIP_ARGS is probably not needed anymore with recent cdbs (should pick up proper name automatically)
- some explanation for -O3 would be welcome, this looks like just a regular toolkit which we usually build with the default -O2
- the whole SONAME/package name handling is quite ugly but is a consequence from upstream's handling
- doc installation could use dh_install
- shlibs would better be set without hardcoding the SONAME in the shlibs file, by using the CDBS var for the dh_mkshlibdeps calls

FYI this is actually not the latest upstream version (0.0.2.062).

I'm a bit scared that it detects MMX/SSE/altivec CFlags on the buildd to optimize the package; it means that people without MMX or SSE or Altivec will get SIGILL when using the binaries. It seems to be the flags aren't used in the package, but would you mind double checking? It would be best to remove the checks if they are useless or use an override to disable this support even if it's detected.

Is it also normal that the package is built without libevas support?

Tests aren't run during build.

Doc is built twice during build on i386.

Code looks ok; only did a quick check.

So I'd just like some clarifications on the SIMD checks and libevas and woud love if some of the build issues (doc built twice and testsuite not run) would be addressed; thanks!