Bug 1203847 - Updating to systemd-251.4-3.1 throws additional rpm output "Too few arguments".
Summary: Updating to systemd-251.4-3.1 throws additional rpm output "Too few arguments".
Status: NEW
Alias: None
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: Basesystem (show other bugs)
Version: Current
Hardware: x86-64 Other
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: Michael Schröder
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-09-28 13:27 UTC by Paul Tannington
Modified: 2023-11-24 16:41 UTC (History)
2 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Tannington 2022-09-28 13:27:20 UTC 
On both of my two TW systems when performing a "zypper dup" 20220921 -> 20220926 when installing "systemd-251.4-3.1" additional rpm output indicates "Too few arguments"

# 2022-09-28 13:59:21 systemd-251.4-3.1.x86_64.rpm installed ok
# Additional rpm output:
# Too few arguments.
# 
2022-09-28 13:59:21|install|systemd|251.4-3.1|x86_64||repo-oss|7c696173a1c08d2ae8d9e5db3b3ef9c5073da27d2c2d2a0326ea682c07c3b00a|


paul@Orion-15:~$ rpm -q rpm-config-SUSE
rpm-config-SUSE-20220421-2.1.noarch
paul@Orion-15:~$
Comment 1 Franck Bui 2022-10-04 08:08:00 UTC 
In systemd.spec:

%postun                                                                                                                                                                                                                                                                                    
# daemon-reload is implied by %%systemd_postun_with_restart                                                                                                                                                                                                                                
%systemd_postun_with_restart systemd-journald.service                                                                                                                                                                                                                                      
%systemd_postun_with_restart systemd-timesyncd.service                                                                                                                                                                                                                                     
# Avoid restarting logind until fixed upstream (issue #1163)

# rpm -q systemd
systemd-251.4-2.1.x86_64

# rpm -q --scripts systemd
[...]
postuninstall scriptlet (using /bin/sh):
# daemon-reload is implied by :
if [ -x /usr/bin/systemctl ]; then                                                      
        /usr/bin/systemctl daemon-reload || :                                           
        if [ $1 -ge 1 ]; then                                                           
                # Package upgrade, not uninstall                                        
                
        if [ -e /etc/sysconfig/services ]; then                                         
                DISABLE_RESTART_ON_UPDATE=                                              
                . /etc/sysconfig/services                                               
                case "$DISABLE_RESTART_ON_UPDATE" in                                    
                yes|1)  ;;                                                              
                *)      /usr/bin/systemctl try-restart  || :                    
                esac                                                                    
        fi                                                                              
        
        fi                                                                              
fi
[...]

Apparently rpmbuild (?) expanded "%%systemd_postun_with_restart" despite the fact that '%systemd_postun_with_restart' (included in a comment) was escaped (by a second '%').

@Michael, am I right in thinking that this issue belongs to rpmbuild ?
Comment 2 Franck Bui 2022-10-10 15:03:35 UTC 
Re-assigning, apparently rpmbuild (or whatever) doesn't understand escaped "%" in comments...
Comment 3 OBSbugzilla Bot 2022-10-21 10:25:06 UTC 
This is an autogenerated message for OBS integration:
This bug (1203847) was mentioned in
https://build.opensuse.org/request/show/1030349 Factory / systemd