coreutils - pr: headers ignore the --indent option

Bug #331817 reported by William Shotts
6
Affects Status Importance Assigned to Milestone
coreutils
Fix Released
Undecided
Unassigned
coreutils (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: coreutils

While the pr command recognizes the --indent (-o) option, page headers are printed without indentation.

Expected results (for example, version 5.2.1 from CentOS 4):
$ ls /usr/bin | pr --indent 5 | head

     2009-02-19 17:43 Page 1

     [
     4odb
     4rdf
     4ss
     4ssd

Actual results (in 8.04/ pr version 6.10):
$ ls /usr/bin | pr --indent 5 | head

2009-02-19 17:45 Page 1

     [
     411toppm
     a2p
     a2ps
     a2ps-lpr-wrapper

Revision history for this message
Arnaud Soyez (weboide) wrote :

Confirmed for pr 6.10.

BUT, in the man page, for --indent, it doesn't say it should apply to headers (it doesn't it shouldn't either). So I don't know if that's the intended behaviour or not.

Changed in coreutils:
status: New → Confirmed
Revision history for this message
C de-Avillez (hggdh2) wrote :

Confirmed with git (7.0.188-0e40e-dirty) pr, so marking as triaged.

I have sent an email to the coreutils mailing list about that, and will post a response when I receive it.

Changed in coreutils:
status: Confirmed → Triaged
importance: Undecided → Low
Revision history for this message
C de-Avillez (hggdh2) wrote :

I proposed a patch for it upstream (attached), and I am waiting to hear back, if at all. This patch applies against the upstream trunk, and I have not verified yet if it would apply cleanly against coreutils 6.10.

Revision history for this message
C de-Avillez (hggdh2) wrote :

The patch was adjusted and accepted in coreutils GIT, and should make it into Ubuntu when coreutils 7.1x is sync-ed/merged, perhaps on Karmic.

C de-Avillez (hggdh2)
Changed in coreutils:
status: New → Fix Released
Revision history for this message
C de-Avillez (hggdh2) wrote :

This bug was fixed in the package coreutils - 7.2.1

coreutils (7.2.1) karmic; urgency=low

  * new upstream version (Closes: #517558)
  * [78] fix kfreebsd build problem (Closes: #520368)

coreutils (7.1-2) unstable; urgency=low

  * remove timeout binary, conflicts with package "timeout" (Closes: #516652)

coreutils (7.1-1) unstable; urgency=low

  * New upstream version

coreutils (6.12-2) unstable; urgency=low

  * [77] fall back if utimensat doesn't exist (Closes: #515731)

coreutils (6.12-1) unstable; urgency=low

  * New upstream version
  * Switch to dpatch

Changed in coreutils (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.