diff --git a/vsftpd.changes b/vsftpd.changes
index e985217..5f443bd 100644
--- a/vsftpd.changes
+++ b/vsftpd.changes
@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Tue Oct 23 22:05:43 UTC 2018 - Christian Boltz <suse-beta@cboltz.de>
+
+- firewall-macros should be BuildRequires, not Requires(post)
+  (the macro gets expanded during package build)
+
 -------------------------------------------------------------------
 Thu Jun 28 15:57:13 UTC 2018 - psimons@suse.com
 
diff --git a/vsftpd.spec b/vsftpd.spec
index 2ace60e..87a04ed 100644
--- a/vsftpd.spec
+++ b/vsftpd.spec
@@ -12,7 +12,7 @@
 # license that conforms to the Open Source Definition (Version 1.9)
 # published by the Open Source Initiative.
 
-# Please submit bugfixes or comments via http://bugs.opensuse.org/
+# Please submit bugfixes or comments via https://bugs.opensuse.org/
 #
 
 
@@ -91,7 +91,7 @@ Requires:       logrotate
 Requires(pre):  shadow
 Provides:       ftp-server
 %if %{use_firewalld}
-Requires(post): firewall-macros
+BuildRequires:  firewall-macros
 %endif
 %if 0%{?suse_version} >= 1330
 Requires:       group(nobody)