<pre style='margin:0'>
Rainer Müller (raimue) pushed a commit to branch vcs-fetch
in repository macports-base.

</pre>
<p><a href="https://github.com/macports/macports-base/commit/600c6b2c7e03d208eefda5ae25d1f1d0ac58933e">https://github.com/macports/macports-base/commit/600c6b2c7e03d208eefda5ae25d1f1d0ac58933e</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 600c6b2c7e03d208eefda5ae25d1f1d0ac58933e
</span>Author: Rainer Müller <raimue@macports.org>
AuthorDate: Sun Apr 15 18:34:48 2018 +0200

<span style='display:block; white-space:pre;color:#404040;'>    fetch: Do not redirect stderr
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    There is no need to redirect stderr to stdout with 2>&1, because the
</span><span style='display:block; white-space:pre;color:#404040;'>    system command will already do this by default for us.
</span>---
 src/port1.0/portfetch.tcl | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/port1.0/portfetch.tcl b/src/port1.0/portfetch.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 2affe27..f1ab690 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/port1.0/portfetch.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/port1.0/portfetch.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -373,7 +373,7 @@ proc portfetch::mktar {tarfile dir mtime {excludes {}}} {
</span>         close $mtreefd
 
         set tar [findBinary bsdtar ${portutil::autoconf::bsdtar_path}]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${tar} -cf $tarfile @$mtreefile 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${tar} -cf $tarfile @$mtreefile"
</span>         if {[catch {system -W $dir $cmdstring} result]} {
             error [msgcat::mc "tarball creation failed"]
         }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -425,7 +425,7 @@ proc portfetch::bzrfetch {args} {
</span>         ui_info "$UI_PREFIX Checking out ${fetch.type} repository"
         set exportpath [file join ${workpath} export]
         file mkdir ${exportpath}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${bzr.cmd} --builtin --no-aliases checkout --lightweight --verbose -r ${bzr.revision} ${bzr.url} ${exportpath}/${bzr.file_prefix} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${bzr.cmd} --builtin --no-aliases checkout --lightweight --verbose -r ${bzr.revision} ${bzr.url} ${exportpath}/${bzr.file_prefix}"
</span>         if {[catch {system $cmdstring} result]} {
             error [msgcat::mc "Bazaar checkout failed"]
         }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -512,7 +512,7 @@ proc portfetch::cvsfetch {args} {
</span> 
     try -pass_signal {
         if {[regexp ^:pserver: ${cvs.root}]} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            set cmdstring "echo ${cvs.password} | ${cvs.cmd} -z9 -f -d ${cvs.root} login 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            set cmdstring "echo ${cvs.password} | ${cvs.cmd} -z9 -f -d ${cvs.root} login"
</span>             if {[catch {system -notty $cmdstring} result]} {
                 error [msgcat::mc "CVS login failed: $result"]
             }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -522,7 +522,7 @@ proc portfetch::cvsfetch {args} {
</span> 
         set exportpath [file join ${workpath} export]
         file mkdir ${exportpath}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${cvs.cmd} -z9 -f -d ${cvs.root} export -d ${cvs.file_prefix} ${cvs.args} ${cvs.module} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${cvs.cmd} -z9 -f -d ${cvs.root} export -d ${cvs.file_prefix} ${cvs.args} ${cvs.module}"
</span>         if {[catch {system -notty -W ${exportpath} $cmdstring} result]} {
             error [msgcat::mc "CVS checkout failed"]
         }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -634,17 +634,17 @@ proc portfetch::svnfetch {args} {
</span>     ui_info "$UI_PREFIX Checking out ${fetch.type} repository"
     set exportpath [file join ${workpath} export]
     if {${svn.subdirs} eq ""} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${svn.cmd} --non-interactive ${connectargs} export ${exportargs} ${svn.url} ${exportpath}/${svn.file_prefix} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${svn.cmd} --non-interactive ${connectargs} export ${exportargs} ${svn.url} ${exportpath}/${svn.file_prefix}"
</span>         if {[catch {system $cmdstring} result]} {
             return -code error [msgcat::mc "Subversion checkout failed"]
         }
     } else {
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${svn.cmd} --non-interactive ${connectargs} checkout --depth empty ${svn.url} ${exportpath}/${svn.file_prefix} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${svn.cmd} --non-interactive ${connectargs} checkout --depth empty ${svn.url} ${exportpath}/${svn.file_prefix}"
</span>         if {[catch {system $cmdstring} result]} {
             return -code error [msgcat::mc "Subversion checkout failed"]
         }
         foreach dir ${svn.subdirs} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            set cmdstring "${svn.cmd} --non-interactive ${connectargs} update --depth infinity ${exportpath}/${svn.file_prefix}/${dir} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            set cmdstring "${svn.cmd} --non-interactive ${connectargs} update --depth infinity ${exportpath}/${svn.file_prefix}/${dir}"
</span>             if {[catch {system $cmdstring} result]} {
                 return -code error [msgcat::mc "Subversion update for subdir $dir failed"]
             }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -803,7 +803,7 @@ proc portfetch::gitfetch {args} {
</span> 
     ui_info "$UI_PREFIX Cloning ${fetch.type} repository"
     set exportpath [file join ${workpath} export]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    set cmdstring "${git.cmd} clone $options ${git.url} ${exportpath} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    set cmdstring "${git.cmd} clone $options ${git.url} ${exportpath}"
</span>     if {[catch {system $cmdstring} result]} {
         return -code error [msgcat::mc "Git clone failed"]
     }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -812,7 +812,7 @@ proc portfetch::gitfetch {args} {
</span>     # required to have the right version of .gitmodules
     if {${git.branch} ne ""} {
         ui_debug "Checking out branch ${git.branch}"
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${git.cmd} checkout -q ${git.branch} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${git.cmd} checkout -q ${git.branch}"
</span>         if {[catch {system -W $exportpath $cmdstring} result]} {
             return -code error [msgcat::mc "Git checkout failed"]
         }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -821,7 +821,7 @@ proc portfetch::gitfetch {args} {
</span>     # fetch all submodules
     if {[file isfile "$exportpath/.gitmodules"] && [tbool git.fetch_submodules]} {
         ui_info "$UI_PREFIX Cloning git submodules"
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        set cmdstring "${git.cmd} submodule -q update --init --recursive 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set cmdstring "${git.cmd} submodule -q update --init --recursive"
</span>         if {[catch {system -W $exportpath $cmdstring} result]} {
             return -code error [msgcat::mc "Git submodule init failed"]
         }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -836,7 +836,7 @@ proc portfetch::gitfetch {args} {
</span> 
     # generate main tarball
     set tardst "${distpath}/${distname}.${fetch.type}.TMP.tar"
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    set cmdstring "${git.cmd} archive --format=tar --prefix=\"${git.file_prefix}/\" --output=${tardst} ${git.branch} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    set cmdstring "${git.cmd} archive --format=tar --prefix=\"${git.file_prefix}/\" --output=${tardst} ${git.branch}"
</span>     if {[catch {system -W $exportpath $cmdstring} result]} {
         delete $tardst
         return -code error [msgcat::mc "Git archive creation failed"]
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -853,7 +853,7 @@ proc portfetch::gitfetch {args} {
</span>             "${git.cmd} submodule -q foreach --recursive '" \
             "${git.cmd} archive --format=tar --prefix=\"${git.file_prefix}/\${PWD#\$MPTOPDIR/}/\" \$sha1 " \
             "| ${tar} -uf ${tardst} @-" \
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            "' 2>&1"] ""]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            "'"] ""]
</span>         if {[catch {system -W $exportpath $cmdstring} result]} {
             delete $tardst
             return -code error [msgcat::mc "Git submodule archive creation failed"]
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -895,7 +895,7 @@ proc portfetch::hgfetch {args} {
</span>     }
 
     set exportpath [file join ${workpath} export]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    set cmdstring "${hg.cmd} clone${insecureflag} --rev \"${hg.tag}\" ${hg.url} ${exportpath}/${hg.file_prefix} 2>&1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    set cmdstring "${hg.cmd} clone${insecureflag} --rev \"${hg.tag}\" ${hg.url} ${exportpath}/${hg.file_prefix}"
</span>     if {[catch {system $cmdstring} result]} {
         return -code error [msgcat::mc "Mercurial clone failed"]
     }
</pre><pre style='margin:0'>

</pre>