LorbusChris / rspamd-rpm

:package: RPM packaging for Rspamd in Fedora
https://copr.fedorainfracloud.org/coprs/lorbus/rspamd/
Apache License 2.0
13 stars 10 forks source link

Update to 2.4 #17

Closed judemille closed 4 years ago

LorbusChris commented 4 years ago

Thank you for your contribution!

The version is 2.4, not 2.4.2, correct?

Maybe @kaechele has some suggestions on what else to include here from https://github.com/LorbusChris/rspamd-rpm/pull/15

judemille commented 4 years ago

The version in https://rspamd.com/rpm-stable/centos-8/x86_64/ is 2.4.2.

kaechele commented 4 years ago

It's actually 2.4-2 (dash, not dot). So Version 2.4 and Release 2. You will need the replxx patch, otherwise it won't build on F32 and newer.

judemille commented 4 years ago

Ah, I missed that. I'm not familiar with the replxx patch, care to send it?

kaechele commented 4 years ago

Here's the patch: https://github.com/LorbusChris/rspamd-rpm/pull/15/files#diff-a3809a42b00ce1bcd001bc81654ec292

Check out https://github.com/LorbusChris/rspamd-rpm/pull/15 for other changes that I made that haven't been merged yet. I closed that pull request because I wanted to re-do it with 2.4 but haven't gotten around to it yet.

LorbusChris commented 4 years ago

:+1: It does seem the replxx patch has merged upstream in https://github.com/AmokHuginnsson/replxx/pull/65

If anybody could PR that update to https://github.com/rspamd/rspamd/tree/master/contrib/replxx too, that'd be awesome! Patching it downstream in the RPM is fine in the meantime.