I am reviewing bug 2244378.  Fedora-review says:

- systemd_user_post is invoked in %post and systemd_user_preun in %preun
  for Systemd user units service files.
  Note: Systemd user unit service file(s) in xdg-desktop-portal-hyprland
  See: https://docs.fedoraproject.org/en-US/packaging-
  guidelines/Scriptlets/#_user_units

which is wrong, because the spec file has exactly what it is asking for:

%post
%systemd_user_post %{name}.service

%preun
%systemd_user_preun %{name}.service

I stepped through the python code in /usr/share/fedora-review/plugins/generic.py, CheckSystemdUserunitdirScriplets.run.  At the point where the regular expression searches are tried, str(rpmpkg.post) is:

'\n \nif [ $1 -eq 1 ] && [ -x "/usr/lib/systemd/systemd-update-helper" ]; then \n    # Initial installation \n    /usr/lib/systemd/systemd-update-helper install-user-units xdg-desktop-portal-hyprland.service || : \nfi'

The call to rpm.expandMacro("%systemd_user_post .*.service") on the other hand, produces this:

'\n \nif [ $1 -eq 1 ] && [ -x "/usr/lib/systemd/systemd-update-helper" ]; then \n    # Initial installation \n    /usr/lib/systemd/systemd-update-helper install-user-units .*.service || : \nfi \n'

Notice that has an extra ' \n' at the end.  I see that systemd_user_post_re is altered by taking a slice: [2:].  Should that remove the *final* 2 characters instead of the *initial* 2 characters?

Reproducible: Always

Steps to Reproduce:
1. fedora-review -b 2244378 -m fedora-rawhide-x86_64
2.
3.
Actual Results:  
Fedora-review complains about missing systemd scriptlets that are not, in fact, missing.

Expected Results:  
No complaints, since the systemd scripts are present.