<pre style='margin:0'>
Ryan Schmidt (ryandesign) pushed a commit to branch master
in repository macports-ports.

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/d5b295abf6f04f0b895b0f22b33bbaf64df56d32">https://github.com/macports/macports-ports/commit/d5b295abf6f04f0b895b0f22b33bbaf64df56d32</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new d5b295a  app-1.0.tcl: Update comments
</span>d5b295a is described below

<span style='display:block; white-space:pre;color:#808000;'>commit d5b295abf6f04f0b895b0f22b33bbaf64df56d32
</span>Author: Ryan Schmidt <ryandesign@macports.org>
AuthorDate: Sat Aug 25 18:16:20 2018 -0500

<span style='display:block; white-space:pre;color:#404040;'>    app-1.0.tcl: Update comments
</span>---
 _resources/port1.0/group/app-1.0.tcl | 56 ++++++++++++++++++++----------------
 1 file changed, 32 insertions(+), 24 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/app-1.0.tcl b/_resources/port1.0/group/app-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 7fb6007..fa8ef98 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/app-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/app-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -7,7 +7,7 @@
</span> # app bundle to easily launch it.
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.create: whether to create the app bundle at all.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.create: whether to create the app bundle at all
</span> #
 # The default is yes.
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -15,7 +15,7 @@ options app.create
</span> default app.create yes
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.name: the name of the app that users will see in the Finder.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.name: the name of the app that users will see in the Finder
</span> #
 # The default is based on ${name}: if ${name} contains any uppercase letters,
 # ${name} is used, otherwise the first character of ${name} is uppercased.
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -34,14 +34,14 @@ proc app.get_default_name {} {
</span> }
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.executable: the program the app will run.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.executable: the program the app will run
</span> #
 # The default is ${name}; relative paths are relative to ${prefix}/bin. If you
 # specify a relative or absolute path to a program that exists in ${destroot},
 # the app will contain a symlink to that program. If you specify an absolute
 # path in ${workpath} or ${filespath} it will be copied into the app. This is
 # useful if you need to write a wrapper script, for example to set environment
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# variables. If your wrapper script can be used as is, leave it in ${filespath}
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# variables. If your wrapper script can be used as-is, leave it in ${filespath}
</span> # and let it be copied from there. If the wrapper needs placeholders to be
 # reinplaced first, copy it into ${workpath}, do your reinplacing, then let it
 # be copied from there.
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -52,14 +52,14 @@ options app.executable
</span> default app.executable {${name}}
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.icon: the icon the app will have.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.icon: the icon the app will have
</span> #
 # The default is empty; if no icon graphic is available for this software, this
 # is fine. You can supply the path to an existing .icns file, or the path to a
 # .png or other graphic file that the makeicns program can convert. A build
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# dependency on makeicns will be automatically added if needed. You can also
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# dependency on makeicns will be added automatically if needed. You can also
</span> # supply the path to a .svg file and it will be rasterized to the different icon
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# formats. A build dependency on librsvg will be automatically added if needed.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# formats. A build dependency on librsvg will be added automatically if needed.
</span> # Paths may be absolute or relative to ${worksrcpath}.
 #
 # Relates to Info.plist key CFBundleIconFile.
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -68,7 +68,7 @@ options app.icon
</span> default app.icon ""
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.short_version_string: the version number.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.short_version_string: the version number
</span> #
 # The default is ${version}. This is fine for most ports, but ports that list
 # both version and build number in ${version} may wish to separate these here.
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -79,7 +79,7 @@ options app.short_version_string
</span> default app.short_version_string {${version}}
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.version: the build number.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.version: the build number
</span> #
 # The default is ${version}. This is fine for most ports, but ports that list
 # both version and build number in ${version} may wish to separate these here.
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -90,10 +90,11 @@ options app.version
</span> default app.version {${version}}
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.identifier: the app's unique bundle identifier.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.identifier: the app's unique bundle identifier
</span> #
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# The default is computed based on ${homepage} and ${app.name}. For almost all
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# ports this does not need to be overridden.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# The default is computed based on ${homepage} and ${app.name}. For most ports
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# this does not need to be overridden, but for software that already has an
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# established bundle identifier outside of MacPorts, you can set it here.
</span> #
 # Info.plist key CFBundleIdentifier.
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -120,17 +121,23 @@ proc app.get_default_identifier {} {
</span> #
 # The default is no.
 #
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# Info.plist key NSHighResolutionCapable
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Info.plist key NSHighResolutionCapable.
</span> 
 options app.retina
 default app.retina no
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.hide_dock_icon: hide the dock icon
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.hide_dock_icon: hide the Dock icon
</span> #
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# x11 apps do not receive a proper indication that application has successfully
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# launched, and so the icon keeps bouncing in the dock. Until this is properly
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# fixed, just hide the the dock icon for now
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# SDKs like SDL and Qt use the necessary macOS APIs to implement proper Dock
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# icon functionality, including stopping the icon from bounding when the app
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# has finished launching and bringing the app to the front when the icon is
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# clicked. Other SDKs like X11 don't use those macOS APIs, so Dock icons for
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# apps using those SDKs would not function correctly and should be hidden. The
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# default is based on whether the port has an x11 variant and the user has
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# enabled it.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Info.plist key LSUIElement.
</span> 
 options app.hide_dock_icon
 default app.hide_dock_icon  {[app.get_default_hide_dock_icon]}
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -140,12 +147,13 @@ proc app.get_default_hide_dock_icon {} {
</span> }
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# app.use_launch_script: use a bash launch script instead of a symlink to the executable
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# app.use_launch_script: use a Bash launch script instead of a symlink to the executable
</span> #
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# the default behaviour is to symlink the executable into the bundle.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# However, this has two issues -- it passes -psn to the executable,
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# which some ports can't handle. Also, it doesn't set up the path to ${prefix}/bin. The launch
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# script option solves both these issues.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# The default behaviour is to symlink the executable into the bundle. However,
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# this has two issues: OS X 10.8 and earlier pass a `-psn` argument (the process
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# serial number) to the executable, which some programs can't handle. Also, it
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# doesn't modify the PATH, e.g. to add ${prefix}/bin to it. Using a launch
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# script solves both of these issues.
</span> 
 options app.use_launch_script
 default app.use_launch_script  no
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -328,8 +336,8 @@ proc app._resolve_symlink {path destroot} {
</span> }
 
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# Write a default launch script for the executable into the bundle,
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# setting the default PATH as would be expected by the binary
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Write a launch script for the executable into the bundle, modifying PATH to
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# allow the executable to find other executables installed with MacPorts.
</span> proc app._write_launch_script  {executable app_destination} {
     global prefix
     set launch_script [open ${app_destination} w]
</pre><pre style='margin:0'>

</pre>