Package: rspamd
Version: 1.9.4-2
Severity: normal
Control: fixed -1 1.8.1-2

Dear Maintainer,

It seems rspamd package in stable ships a native systemd service
(provided by upstream) masking the init script, however the version
in testing/unstable does not!

I tried to figure out why such a change might have been made, but
I didn't find anything suggesting this as a deliberate change.

The problem seems to be caused by:
commit 03941b9d783c0593415744af259b54a27290d565
Author: Sergio Durigan Junior <sergi...@debian.org>
Date:   Sun May 26 17:32:12 2019 -0400

  d/{control,rules}: Enable build with hyperscan (amd64 only), and add myself 
to Uploaders.

That commit incorrectly tries to use make conditionals inside a
line-broken string rather than starting on a (separate) new line.

This can be seen in the build log eg.
https://buildd.debian.org/status/fetch.php?pkg=rspamd&arch=amd64&ver=1.9.4-2%2Bb1&stamp=1573695788&raw=0
as
[...]
CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_EXPORT_NO_PACKAGE_REGISTRY
    CMAKE_INSTALL_LIBDIR
    CMAKE_INSTALL_LOCALSTATEDIR
    CMAKE_INSTALL_RUNSTATEDIR
    CMAKE_INSTALL_SYSCONFDIR
    FORCE_GMIME24


-- Build files have been written to: /<<PKGBUILDDIR>>/obj-x86_64-linux-gnu
DENABLE_HYPERSCAN=ON \
endif
/bin/sh: 1: endif: not found
make[1]: [debian/rules:21: override_dh_auto_configure] Error 127 (ignored)
DWANT_SYSTEMD_UNITS=ON
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
   dh_auto_build -a
[...]

I'd provide a patch fixing this, but it seems this is already fixed in
https://salsa.debian.org/debian/rspamd/merge_requests/2/diffs?commit_id=f5b63c708e57b7803bb8553b62d4e7a7f85fc2ca

I assume !2 is going to be merged and uploaded soon, but if not then
please cherry-pick that particular commit.

Regards,
Andreas Henriksson

Reply via email to