[91597] trunk/dports/x11/xinit

[Ryan Schmidt]

On Apr 6, 2012, at 12:24, jeremyhu at macports.org wrote:

> Revision: 91597
>          https://trac.macports.org/changeset/91597
> Author:   jeremyhu at macports.org
> Date:     2012-04-06 10:24:08 -0700 (Fri, 06 Apr 2012)
> Log Message:
> -----------
> xinit: Provide instructions for setting up the LaunchAgent, and make it disabled by default.  Also unload the LaunchDaemon on deactivate.
> 
> Modified Paths:
> --------------
>    trunk/dports/x11/xinit/Portfile

> +platform darwin {
> +    post-activate {
> +        system "launchctl load /Library/LaunchDaemons/org.macports.privileged_startx.plist"
> +
> +        ui_msg "###############################################################################"
> +        ui_msg "# To choose MacPorts' X11 as the default server, you must install xorg-server,"
> +        ui_msg "# load the LaunchAgent, logout, and log back in.  To load the LaunchAgent,"
> +        ui_msg "# please execute the following:"
> +
> +        if { ${os.major} < 10 } {
> +            ui_msg "# sudo launchctl load -w /Library/LaunchAgents/org.macports.startx.plist"
> +            ui_msg "# "
> +            ui_msg "# This will be required after every upgrade of the xinit port on Leopard"
> +        } else {
> +            ui_msg "# launchctl load -w /Library/LaunchAgents/org.macports.startx.plist"
> +            ui_msg "# "
> +            ui_msg "# This will be remembered across port updates on Snow Leopard and later."
> +        }
> +
> +        # OS X 10.6.3 was the first version whose native X11 allowed the modern socket naming
> +        # for better X11 server coexistence, so we require it for installing the LaunchAgent
> +        if {[vercmp ${os.version} 10.3.0] < 0} {
> +            ui_msg "# "
> +            ui_msg "# Note that doing so on your OS version may make the Apple-provided version"
> +            ui_msg "# fail to launch."
> +        }
> +
> +        ui_msg "###############################################################################"
> +    }
> }

Could these messages be presented using the "notes" feature instead of "ui_msg"?