Opened 7 years ago

Closed 7 years ago

#39621 closed defect (fixed)

miriad: let MacPorts do the word wrapping in the notes

Reported by: ryandesign (Ryan Schmidt) Owned by: pkgw (Peter Williams)
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc:
Port: miriad

Description

The miriad port hardcodes newlines into its notes:

$ port notes miriad
miriad has the following notes:
  MIRIAD programs are not in your $PATH by default.
  You need to run 'source /opt/local/lib/miriad/automiriad.sh' to set up
  your shell environment to use MIRIAD. (Source /opt/local/lib/miriad/automiriad.csh
  if you use csh or tcsh.) *** The 'source' keyword is important here! ***
  You can put this command in your .profile or other shell initialization 
  files to have your shells automatically set up to use MIRIAD.

miriad should allow MacPorts to do the word wrapping at the width of the user's terminal. This is accomplished by placing the entire notes into a single Tcl string. If it is desired to wrap lines for better readability in the portfile, end such lines with a backslash, without any preceding whitespace. When actually wanting to start a newline in the displayed notes, simply insert an actual newline character. Here is an example:

notes "
Here is the first paragraph of the\
notes. MacPorts will automatically\
wrap them to the width of the user's\
terminal.

Here is a second paragraph of the\
notes.
"

Attachments (1)

miriad-linebreaks.diff (1.2 KB) - added by pkgw (Peter Williams) 7 years ago.
Fix linebreaking of notes message.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by pkgw (Peter Williams)

Attachment: miriad-linebreaks.diff added

Fix linebreaking of notes message.

comment:1 Changed 7 years ago by pkgw (Peter Williams)

Attached is a patch to fix this. Does the automatic linebreaking also happen for ui_msg commands? That's another case where we have some manual linebreaks in the Portfile.

comment:2 Changed 7 years ago by ryandesign (Ryan Schmidt)

No, only for notes. And the long description in port info.

comment:3 Changed 7 years ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: newclosed

Thanks for the patch! Committed in r107644.

Note: See TracTickets for help on using tickets.