github/workflows (build): rename directory in "ble-nightly.tar.xz" to…

… "ble-nightly"

.github/workflows/build.yml

@@ -24,7 +24,8 @@ jobs:
         run: |
           make install INSDIR="./ble-${{ env.blesh_build_pkgver }}"
           tar -cJf "ble-${{ env.blesh_build_pkgver }}.tar.xz" "ble-${{ env.blesh_build_pkgver }}"
-          cp "ble-${{ env.blesh_build_pkgver }}.tar.xz" ble-nightly.tar.xz
+          mv "ble-${{ env.blesh_build_pkgver }}" ble-nightly
+          tar -cJf ble-nightly.tar.xz ble-nightly
       - name: Release
         uses: softprops/action-gh-release@v1
         with:

README-ja_JP.md

@@ -57,13 +57,14 @@ echo 'source ~/.local/share/blesh/ble.sh' >> ~/.bashrc
 # 簡単お試し (インストールせずにお試しいただけます)
 
 curl -L https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
-source ble-nightly*/ble.sh
+source ble-nightly/ble.sh
 
 # インストール & .bashrc 簡単設定 (動かない場合は節1.3を御参照下さい)
 
 curl -L https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
 mkdir -p ~/.local/share/blesh
-mv ble-nightly* ~/.local/share/blesh
+mv -f ble-nightly/* ~/.local/share/blesh/
+rmdir ble-nightly
 echo 'source ~/.local/share/blesh/ble.sh' >> ~/.bashrc
 ```
 </details>
@@ -76,13 +77,14 @@ echo 'source ~/.local/share/blesh/ble.sh' >> ~/.bashrc
 # 簡単お試し (インストールせずにお試しいただけます)
 
 wget -O - https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
-source ble-nightly*/ble.sh
+source ble-nightly/ble.sh
 
 # インストール & .bashrc 簡単設定 (動かない場合は節1.3を御参照下さい)
 
 wget -O - https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
 mkdir -p ~/.local/share/blesh
-mv ble-nightly* ~/.local/share/blesh
+mv ble-nightly/* ~/.local/share/blesh/
+rmdir ble-nightly
 echo 'source ~/.local/share/blesh/ble.sh' >> ~/.bashrc
 ```
 </details>

README.md

@@ -60,14 +60,14 @@ This requires the commands `curl`, `tar` (with the support for the `J` flag), an
 # TRIAL without installation
 
 curl -L https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
-source ble-nightly*/ble.sh
+source ble-nightly/ble.sh
 
 # Quick INSTALL to BASHRC (If this doesn't work, please follow Sec 1.3)
 
 curl -L https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
 mkdir -p ~/.local/share/blesh
-mv -f ble-nightly*/* ~/.local/share/blesh/
-rmdir ble-nightly*
+cp -Rf ble-nightly/* ~/.local/share/blesh/
+rm -rf ble-nightly
 echo 'source ~/.local/share/blesh/ble.sh' >> ~/.bashrc
 ```
 </details>
@@ -80,13 +80,14 @@ This requires the commands `wget`, `tar` (with the support for the `J` flag), an
 # TRIAL without installation
 
 wget -O - https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
-source ble-nightly*/ble.sh
+source ble-nightly/ble.sh
 
 # Quick INSTALL to BASHRC (If this doesn't work, please follow Sec 1.3)
 
 wget -O - https://github.com/akinomyoga/ble.sh/releases/download/nightly/ble-nightly.tar.xz | tar xJf -
 mkdir -p ~/.local/share/blesh
-mv ble-nightly*/* ~/.local/share/blesh/
+cp -Rf ble-nightly/* ~/.local/share/blesh/
+rm -rf ble-nightly
 echo 'source ~/.local/share/blesh/ble.sh' >> ~/.bashrc
 ```
 </details>

ble.pp

@@ -1585,8 +1585,8 @@ function ble-update/.download-nightly-build {
   local tarname=ble-nightly.tar.xz
   local url_tar=$_ble_base_repository_url/releases/download/nightly/$tarname
   (
-    shopt -u failglob
-    shopt -s nullglob
+    set +f
+    shopt -u failglob nullglob
 
     # mkcd "$_ble_base/src"
     if ! ble/bin/mkdir -p "$_ble_base/src"; then
@@ -1639,15 +1639,9 @@ function ble-update/.download-nightly-build {
     fi
 
     # cp -T ble-nightly* "$_ble_base"
-    local extracted_dir
-    extracted_dir=(ble-nightly*/)
-    if ((${#extracted_dir[@]}!=1)); then
-      if ((${#extracted_dir[@]}==0)); then
-        ble/util/print 'ble-update (nightly): the directory "ble-nightly*" not found in the tarball.' >&2
-      else
-        ble/util/print 'ble-update (nightly): multiple directories "ble-nightly*" found in the tarball.' >&2
-        ble/bin/rm -rf ble-nightly*/*
-      fi
+    local extracted_dir=ble-nightly
+    if [[ ! -d $extracted_dir ]]; then
+      ble/util/print "ble-update (nightly): the directory 'ble-nightly' not found in the tarball '$PWD/$tarname'." >&2
       return 1
     fi
     ble/bin/cp -Rf "$extracted_dir"/* "$_ble_base/" || return 1

docs/ChangeLog.md

@@ -313,7 +313,6 @@
 - main. util: fix problems of readlink etc. found by test in macOS (reported by aiotter) `#D1849` fa955c1 `#D1855` a22e145
 - progcomp: fix a bug that `COMP_WORDBREAKS` is ignored `#D1872` 4d2dd35
 - global: quote `return $?` `#D1884` 801d14a
-- benchmark (zsh): fix for `KSH_ARRAYS` `#D1886` a144ffa 8cb9b84
 
 ## Documentation
 
@@ -444,7 +443,7 @@
     - util (ble/util/s2bytes): clear locale cache `#D1881` 2e1a7c1
   - complete: fix syntax error for bash-3.0 `#D1881` 0b3e611
 - test (ble/util/writearray): use `ble/file#hash` instead of `sha256sum` `#D1882` b76e21e
-- test (ble/util/readlink): work around external aliases `#D1890` xxxxxxx
+- test (ble/util/readlink): work around external aliases `#D1890` 0c6291f
 
 ## Internal changes and fixes
 
@@ -483,6 +482,9 @@
 - history: refactor hooks `history_{{delete,clear,insert} => change}` `#D1860` c393c93
 - history: rename the hook `history_{on => }leave` `#D1860` c393c93
 - make: check necessary `.git` `#D1887` 0f7c04b
+- benchmark (zsh): fix for `KSH_ARRAYS` `#D1886` a144ffa 8cb9b84
+- benchmark: support for ksh as `benchmark.ksh` `#D1886` 5dae4da
+- github/workflows (build): rename directory in `ble-nightly.tar.xz` to `ble-nightly` (reported by Harduex) `#D1891` xxxxxxx
 
 ## Contrib
 
@@ -491,7 +493,7 @@
 - config/execmark: show exit status in a separate line `#D1828` 4d24f84
 - prompt-git: ignore untracked files in submodules `#D1829` 4d24f84
 - contrib/fzf-completion: fix integration (reported by ferdinandyb) `#D1837` 12c022b
-- contrib/fzf-completion: remove `noquote` (reported by MK-Alias) `#D1889` xxxxxxx
+- contrib/fzf-completion: remove `noquote` (reported by MK-Alias) `#D1889` 0c6291f
 
 <!---------------------------------------------------------------------------->
 # ble-0.4.0-devel2

note.txt

@@ -6638,6 +6638,73 @@ bash_tips
     Done (実装ログ)
 -------------------------------------------------------------------------------
 
+2022-11-28
+
+  * 2022-10-13 github/workflows について (reported by Harduex) [#D1891]
+    https://github.com/akinomyoga/ble.sh/pull/240
+
+    ? cpio -pd を使えば良いのではないか。と思ったが cpio がどのシステム
+      でも使えるとは限らないし、そもそも cpio のオプションとして何が使え
+      るかもシステムに依存するのかもしれない。という事でやはり cpio に依
+      存するというのは避けたい。結局 POSIX cpio には -p はないみたいなの
+      で (むしろ異なる意味のオプションの様である) GNU cpio の non-POSIX
+      option を使うのだったら、そもそも GNU tar の -x を使うところなので
+      ある。
+
+      →改めて ble.pp を見てみたら cp -Rf を使っている。これに倣う事にした。
+
+    * それとは別に現在 ble-nightly* としているディレクトリ名を何とかした
+      い。
+
+      a done: うーん。そもそも ble-nightly でダウンロードする人は細かい
+        version 等を気にするとは思えないし、そうでなくとも標準的な手順に
+        従うと結局ディレクトリの version 名は移動した時に失われてしまう。
+        なので、初めから単に ble-nightly というディレクトリにしてしまえ
+        ば良い気がする。→ 変える事にした。
+
+      b ユーザーに直接ディレクトリ名を指定させる? しかしそうするとしても
+        ディレクトリ名は毎回変わるし commit id が含まれているので README
+        に載せられない。なのでユーザーにその都度 ls などで確認してもらう
+        必要があって、その為には結局 ble-nightly* などとしてもらう事にな
+        る気がする。
+
+    後は README, wiki 等を更新する。というか wiki に nightly version の
+    記述はあっただろうか。なかった気がする。追加する必要はあるだろうか。
+    追加しなくて良い気がする。
+
+    % 2022-11-28 ble-nightly.tar.xz 展開後のディレクトリ名について毎回
+    % 全てを入力するのは大変である。なので、ble-nightly に生成する事に
+    % する。ディレクトリ名に version が記録されない事になるが、そもそ
+    % も ble-nightly をダウンロードしている時点で細かい version は気に
+    % していないのだし、直後に削除する事を想定しているのだから気にして
+    % も仕方がない。何れにしても ble.sh 本体の内部には記録される。
+    %
+    % →これは実質的に 2022-10-13 の議論と同じである。
+
+    2022-11-28 所で、これに関しては master に直接変更を push するので
+    はなく、向こうの branch に push して確認するしかないのでは。と思っ
+    たが、別に独立に変更をしてから向こうの branch を調整しても良い気が
+    する。どちらでも良い。取り敢えず dev の上で作業して後で移し替えて
+    すぐに merge してしまえば良い。
+
+    と思ったが既にその様に変更を実施済みである。うーん。後は README を
+    更新すると書かれている。
+
+    * done: README を更新した。PR は何故か curl 版だけしか更新していな
+      い。関連する物は全て更新するべきである。また日本語の README も更
+      新した。
+
+    * done: ble.pp でも ble-nightly で検索を実施している。これについて
+      も修正を行う。
+
+      というか今までのこのコードは set -f に対して対策はしていたのだっ
+      たか? failglob と nullglob しか設定していない気がする。<del>何れ
+      にしても最早パス名展開は使わなくなったので気にしなくて良い。
+      </del> と思ったがよく見たら昔に展開した結果の始末や cp -Rf 本体
+      でもパス名展開を使っているのでやはり set +f, -u failglob, -s
+      nullglob はつけておく必要がある。後、cp -Rf で変な事が起こらない
+      様に -u nullglob に変更する事にする。
+
 2022-11-12
 
   * test: テストで用いている rm が alias によって rm -i になって失敗する [#D1890]
@@ -6786,6 +6853,10 @@ bash_tips
     2022-11-11 実際に zsh の上で使おうとしたらたくさんの修正が必要になっ
     た。取り敢えず現状で分かっている分を追加修正した。
 
+    2022-11-28 ksh で動かそうとしたら全然駄目だ。ksh は local を始めとして違い
+    が大きすぎる。取り敢えず動く様にはしたが、色々違いが大きいので mwg_pp で別
+    途 ksh 用に修正した物を生成する事にした。
+
 2022-09-26
 
   * vte で unsupported modifyOtherKeys のシーケンスが画面に表示されてしまう (reported by dongxi8) [#D1885]