Merge branch 'dev' into dev-stable

.github/workflows/shellcheck.yml

@@ -0,0 +1,34 @@
+# This is a basic workflow to help you get started with Actions
+
+name: ShellCheck CL
+
+# Controls when the action will run. 
+on:
+  # Triggers the workflow on push or pull request events but only for the dev branch
+  push:
+    branches: [ dev ]
+  pull_request:
+    branches: [ dev ]
+
+  # Allows you to run this workflow manually from the Actions tab
+  workflow_dispatch:
+
+# A workflow run is made up of one or more jobs that can run sequentially or in parallel
+jobs:
+  # This workflow contains a single job called "build"
+  build:
+    # The type of runner that the job will run on
+    runs-on: ubuntu-latest
+
+    # Steps represent a sequence of tasks that will be executed as part of the job
+    steps:
+      # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
+      - uses: actions/checkout@v2
+
+      # Runs a single command using the runners shell
+      - name: Run ShellCheck
+        uses: ludeeus/action-shellcheck@master
+        with:
+          severity: error
+        env:
+          SHELLCHECK_OPTS: --shell=bash

build.sh

@@ -32,7 +32,7 @@ alteriso_version="3.1"
 # Load config file
 [[ ! -f "${defaultconfig}" ]] && "${tools_dir}/msg.sh" -a 'build.sh' error "${defaultconfig} was not found." && exit 1
 for config in "${defaultconfig}" "${script_path}/custom.conf"; do
-    [[ -f "${script_path}/${config}.conf" ]] && source "${script_path}/${config}.conf"
+    [[ -f "${config}" ]] && source "${config}"
 done
 
 umask 0022
@@ -424,7 +424,7 @@ prepare_build() {
     local module_check
     module_check(){
         msg_debug "Checking ${1} module ..."
-        ! bash "${tools_dir}/module.sh" check "${1}" && msg_error "Module ${1} is not available." "1";
+        bash "${tools_dir}/module.sh" check "${1}" || msg_error "Module ${1} is not available." "1";
     }
     modules=($(printf "%s\n" "${modules[@]}" | awk '!a[$0]++'))
     for_module "module_check {}"
@@ -600,10 +600,10 @@ make_pkgbuild() {
 # Customize installation (airootfs)
 make_customize_airootfs() {
     # Overwrite airootfs with customize_airootfs.
-    local _airootfs _airootfs_script_options _script _script_list _airootfs_list _main_script
+    local _airootfs _airootfs_script_options _script _script_list _airootfs_list=() _main_script
 
-    _airootfs_list=("${channel_dir}/airootfs.any" "${channel_dir}/airootfs.${arch}")
-    for_module '_airootfs_list=("${_airootfs_list[@]}" "${module_dir}/{}/airootfs.any" "${module_dir}/{}/airootfs.${arch}")'
+    for_module '_airootfs_list+=("${module_dir}/{}/airootfs.any" "${module_dir}/{}/airootfs.${arch}")'
+    _airootfs_list+=("${channel_dir}/airootfs.any" "${channel_dir}/airootfs.${arch}")
 
     for _airootfs in "${_airootfs_list[@]}";do
         if [[ -d "${_airootfs}" ]]; then

channels/cinnamon/airootfs.any/etc/lightdm/lightdm.conf.d/02-autologin-session.conf

@@ -0,0 +1,2 @@
+[Seat:*]
+autologin-session=cinnamon2d

channels/i3/airootfs.any/etc/lightdm/lightdm.conf.d/02-autologin-session.conf

@@ -0,0 +1,2 @@
+[Seat:*]
+autologin-session=i3

channels/i3/airootfs.any/etc/skel/.config/rofi/power.sh

@@ -22,7 +22,7 @@ declare -A menu_list=(
 
 function main() {
   local -r IFS=$'\n'
-  [[ $# -ne 0 ]] && eval "${menu_list[$1]}" || echo "${!menu_list[*]}"
+  [[ "${#}" -ne 0 ]] && eval "${menu_list[$1]}" || echo "${!menu_list[*]}"
 }
 
-main $@
+main "${@}"

modules/lightdm/airootfs.any/root/customize_airootfs_lightdm.sh

@@ -20,7 +20,7 @@ sed -i "s|%USERNAME%|${username}|g" "/etc/lightdm/lightdm.conf.d/02-autologin.co
 
 
 # Session list
-if cat "/etc/lightdm/lightdm.conf.d/02-autologin-session.conf" | grep "%SESSION%" 1> /dev/null 2>&1; then
+if [[ -f "/etc/lightdm/lightdm.conf.d/02-autologin-session.conf" ]] && cat "/etc/lightdm/lightdm.conf.d/02-autologin-session.conf" | grep "%SESSION%" 1> /dev/null 2>&1; then
     session_list=()
     while read -r session; do
         session_list+=("${session}")
@@ -32,6 +32,10 @@ if cat "/etc/lightdm/lightdm.conf.d/02-autologin-session.conf" | grep "%SESSION%
     elif (( "${#session_list[@]}" == 0)); then
         echo "Warining: Auto login session was not found"
     else
-        remove "/etc/lightdm/lightdm.conf.d/02-autologin-session.conf"
+        echo "Failed to set the session.Multiple sessions were found." >&2
+        echo "Please set the session of automatic login in /etc/lightdm/lightdm.conf.d/02-autologin-session.conf"
+        echo "Found session: $(printf "%s " "${session_list[@]}")"
+        sleep 0.5
+        exit 1
     fi
 fi

system/aur.sh

@@ -9,7 +9,7 @@
 set -e -u
 
 aur_username="aurbuild"
-pacman_debug=true
+pacman_debug=false
 pacman_args=()
 
 trap 'exit 1' 1 2 3 15

system/pkgbuild.sh

@@ -9,7 +9,7 @@
 set -e
 
 build_username="pkgbuild"
-pacman_debug=true
+pacman_debug=false
 pacman_args=()
 
 _help() {
@@ -66,7 +66,7 @@ function user_check () {
 
 # 一般ユーザーで実行します
 function run_user () {
-    sudo -u "${build_username}" ${@}
+    sudo -u "${build_username}" "${@}"
 }
 
 # 引数を確認
@@ -106,12 +106,12 @@ pacman -Syy "${pacman_args[@]}"
 cd "${pkgbuild_dir}"
 pkgbuild_dirs=($(ls "${pkgbuild_dir}" 2> /dev/null))
 if (( "${#pkgbuild_dirs[@]}" != 0 )); then
-    for _dir in ${pkgbuild_dirs[@]}; do
+    for _dir in "${pkgbuild_dirs[@]}"; do
         cd "${_dir}"
         depends=($(source "${pkgbuild_dir}/${_dir}/PKGBUILD"; echo "${depends[@]}"))
         makedepends=($(source "${pkgbuild_dir}/${_dir}/PKGBUILD"; echo "${makedepends[@]}"))
         if (( ${#depends[@]} + ${#makedepends[@]} != 0 )); then
-            for _pkg in ${depends[@]} ${makedepends[@]}; do
+            for _pkg in "${depends[@]}" "${makedepends[@]}"; do
                 if pacman -Ssq "${_pkg}" | grep -x "${_pkg}" 1> /dev/null; then
                     pacman -S --asdeps --needed "${pacman_args[@]}" "${_pkg}"
                 fi

tools/README.md

@@ -7,6 +7,9 @@ Alter Linuxのビルドに使用する処理をコマンド化したものです
 全てのチャンネルのパッケージの一覧を表示します。  
 `channel.sh`でチャンネルの一覧を取得後、`pkglist.sh`でパッケージ一覧を取得します。
 
+## alteriso-info.sh
+ビルド情報を書いたテキストファイルを出力します。詳細は`alteiso-info.sh`を実行してください。  
+
 ## build_helper.py
 GUIのビルドヘルパーです。PyGobjectが必要です。  
 
@@ -43,6 +46,10 @@ menuconfigで使用されるスクリプトです。
 ## menuconf-to-alterconf.sh
 menuconfigで使用されるスクリプトです。  
 
+## module.sh
+使用可能なモジュールの一覧や確認を行います。  
+詳細は`module.sh -h`を実行してください。  
+
 ## msg.sh
 ラベルと色がついたメッセージを出力します。このスクリプトは様々な場所から呼び出されます。  
 詳細は`msg.sh -h`を実行してください。  
@@ -52,14 +59,19 @@ menuconfigで使用されるスクリプトです。
 `build.sh`の依存関係チェックに使用されています。  
 詳細は`package.py -h`を実行してください。
 
-## pkglist.py
+## pkglist.sh
 指定されたチャンネルのパッケージ一覧を取得します。  
 詳細は`pkglist.sh`を実行してください。
 
 ## testpkg.sh
 `allpkglist.sh`で全てのパッケージ一覧を取得後、そのパッケージが公式リポジトリから利用可能かどうかを調べます。  
 詳細は`testpkg.sh`を実行してください。  
 
+## umount.sh
+指定されたディレクトリ以下のマウントポイントを検索してアンマウントします。  
+`build.sh`や`clean.sh`から呼び出されます。  
+詳細は`umount.sh`を実行してください。  
+
 ## wizard.sh
 CLIの対話型のビルド設定ツールです。英語と日本語に対応しています。  
 依存パッケージの自動インストールやキーリングの自動インストールを行います。  

tools/allpkglist.sh

@@ -4,7 +4,7 @@ set -e
 
 load_config() {
     local _file
-    for _file in ${@}; do
+    for _file in "${@}"; do
         if [[ -f "${_file}" ]]; then
             source "${_file}"
         fi
@@ -79,7 +79,7 @@ fi
 
 load_config() {
     local _file
-    for _file in ${@}; do
+    for _file in "${@}"; do
         if [[ -f "${_file}" ]]; then
             source "${_file}"
         fi
@@ -88,12 +88,12 @@ load_config() {
 
 for_module(){
     local module
-    for module in ${modules[@]}; do
-        eval $(echo ${@} | sed "s|{}|${module}|g")
+    for module in "${modules[@]}"; do
+        eval $(echo "${@}" | sed "s|{}|${module}|g")
     done
 }
 
-for arch in ${archs[@]}; do
+for arch in "${archs[@]}"; do
     for channel in $("${tools_dir}/channel.sh" show -a "${arch}" -b -d -k zen -f); do
         modules=($(
             load_config "${script_path}/default.conf" "${script_path}/custom.conf"
@@ -105,7 +105,7 @@ for arch in ${archs[@]}; do
                     modules=("share")
                 fi
             fi
-            for module in ${modules[@]}; do
+            for module in "${modules[@]}"; do
                 dependent="${module_dir}/${module}/dependent"
                 if [[ -f "${dependent}" ]]; then
                     modules+=($(grep -h -v ^'#' "${dependent}" | tr -d "\n" ))

tools/alteriso-info.sh

@@ -97,7 +97,7 @@ variable_list=(
     "iso_version"
 )
 
-for var in ${variable_list[@]}; do
+for var in "${variable_list[@]}"; do
     if [[ -z "$(eval echo '$'${var})" ]]; then
         echo "${var} is empty" >&2
         exit 1

tools/channel.sh

@@ -237,10 +237,10 @@ else
 fi
 
 case "${mode}" in
-    "check" ) check ${@}                  ;;
+    "check" ) check "${@}"                ;;
     "show"  ) show                        ;;
-    "desc"  ) desc ${@}                   ;;
-    "ver"   ) get_alteriso_version ${@}   ;;
+    "desc"  ) desc "${@}"                 ;;
+    "ver"   ) get_alteriso_version "${@}" ;;
     "help"  ) _help; exit 0               ;;
     *       ) _help; exit 1               ;;
 esac