Bug 209613

Summary: Change of behaviour breaks psset
Product: [Fedora] Fedora Reporter: Tim Waugh <twaugh>
Component: a2psAssignee: Tim Waugh <twaugh>
Status: CLOSED ERRATA QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: tao
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 4.13b-57.fc6.3 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-10-31 10:27:47 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 203697    
Bug Blocks: 207681    
Attachments:
Description Flags
a2ps-psset.patch none

Description Tim Waugh 2006-10-06 12:37:48 UTC
+++ This bug was initially created as a clone of Bug #203697 +++

+++ This bug was initially created as a clone of Bug #203339 +++

psset will generate broken PS output when used on RHEL4.

1. Run
ls | a2ps -o - | psset -t > test.ps
2. Notice the broken ps on RHEL4 (where it works on RHEL3)

The bug (control characters where there should be text) is due to a change in
sed's behaviour.

RHEL3, sed-4.0.7-3:
$ echo ''|sed -e ' i\\co' | hexdump -C
00000000  5c 63 6f 0a 0a                                    |\co..|
00000005

RHEL4, sed-4.1.2-4:
$ echo ''|sed -e ' i\\co' | hexdump -C
00000000  0f 0a 0a                                          |...|
00000003

RHEL4, with RHEL3's sed:
$ echo ''|./sed -e ' i\\co' | hexdump -C
00000000  5c 63 6f 0a 0a                                    |\co..|
00000005

-- Additional comment from pmachata on 2006-08-22 13:36 EST --
Hmm... yes, there indeed is a change in behavior.  However it seems to me that
the new version is correct in handling backslashes, and that the right way to
write the sequence '\co' is with 'i\\\co'.  First '\' is the delimiter (which
can be missing, but if it's here, it has to be the delimiter), following '\\' is
escaped backslash, and then 'co'.  In contrast, 'i\\co' interprets \c as escape
and turns 'o' into control character.
If you retarget this bug against EL3, I will adjust the behavior of sed there.

-- Additional comment from twaugh on 2006-08-23 06:06 EST --
So no a2ps change needed?

-- Additional comment from bnocera on 2006-08-23 07:07 EST --
No, psset is broken in both RHEL3 and RHEL4, wrt. its usage of sed. Just that it
doesn't look like it on RHEL3 because sed is misbehaving.

Comment 1 Tim Waugh 2006-10-06 12:47:08 UTC
Created attachment 137911 [details]
a2ps-psset.patch

Comment 2 Tim Waugh 2006-10-17 12:18:47 UTC
Fixed in CVS for devel (4.13b-58).

Comment 3 Tim Waugh 2006-10-17 12:59:58 UTC
Fixed in CVS for FC-6 (4.13b-57.1 built).

Comment 4 Fedora Update System 2006-10-30 21:37:25 UTC
Fixed in update: a2ps-4.13b-57.fc6.3