lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master 0713a8f 14/28: Don't put NORMAL_UID in the en


From: Greg Chicares
Subject: [lmi-commits] [lmi] master 0713a8f 14/28: Don't put NORMAL_UID in the environment unnecessarily
Date: Wed, 12 May 2021 18:14:44 -0400 (EDT)

branch: master
commit 0713a8ff4e5d592c14b5226ada250e38c108d7ac
Author: Vadim Zeitlin <vadim@tt-solutions.com>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Don't put NORMAL_UID in the environment unnecessarily
    
    Just a tiny simplification: this variable is only used locally in a
    single step, so there is no need to put it into the global job
    environment.
---
 .github/workflows/ci.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 7c54c24..4493d6e 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -100,7 +100,6 @@ jobs:
         run: |
           sudo sed -i'' -e 's/^int _putenv/_CRTIMP int _putenv/' 
/usr/share/libtool/build-aux/ltmain.sh
 
-          echo "NORMAL_UID=`id --user`" >> $GITHUB_ENV
           echo "coefficiency=--jobs=`nproc`" >> $GITHUB_ENV
           if ${{ matrix.mingw }}
           then
@@ -149,9 +148,10 @@ jobs:
 
       - name: Create lmi directories
         run: |
+          normal_uid=`id --user`
           for d in /opt/lmi /opt/lmi/print /etc/opt/lmi /srv/cache_for_lmi; do
             sudo mkdir --parents $d
-            sudo --preserve-env chown $NORMAL_UID $d
+            sudo --preserve-env chown $normal_uid $d
           done
           mkdir /srv/cache_for_lmi/downloads
           mkdir /opt/lmi/gui_test



reply via email to

[Prev in Thread] Current Thread [Next in Thread]