]> git.puffer.fish Git - mirror/frr.git/commitdiff
packaging: Redirect stderr to /dev/null in %post action (frr.spec.in) 5632/head
authorDonatas Abraitis <donatas.abraitis@gmail.com>
Sun, 5 Jan 2020 14:34:40 +0000 (16:34 +0200)
committerDonatas Abraitis <donatas.abraitis@gmail.com>
Sun, 5 Jan 2020 17:33:21 +0000 (19:33 +0200)
Solves:
```
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
```

This happens because `grep -q` returns immediately after first match
and closes the pipe while sed has more output to write.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
redhat/frr.spec.in

index be3b83bf80b74d5f306a39b2166cf82bcc37dc80..670bc6f4c91060b7c6f1bb4dd8d721866be7149d 100644 (file)
@@ -444,7 +444,7 @@ zebra_spec_add_service ()
 {
     # Add port /etc/services entry if it isn't already there
     if [ -f %{_sysconfdir}/services ] && \
-        ! %__sed -e 's/#.*$//' %{_sysconfdir}/services | %__grep -wq $1 ; then
+        ! %__sed -e 's/#.*$//' %{_sysconfdir}/services 2>/dev/null | %__grep -wq $1 ; then
         echo "$1        $2          # $3"  >> %{_sysconfdir}/services
     fi
 }