From 4ea2e2333fe6696321a481059d5068b4c6f7484a Mon Sep 17 00:00:00 2001 From: Koichi Murase Date: Wed, 12 Oct 2022 01:35:35 +0900 Subject: [PATCH] github/workflows (build): rename directory in "ble-nightly.tar.xz" to "ble-nightly" --- .github/workflows/build.yml | 3 +- README-ja_JP.md | 10 +++--- README.md | 11 +++--- ble.pp | 16 +++------ docs/ChangeLog.md | 8 +++-- note.txt | 71 +++++++++++++++++++++++++++++++++++++ 6 files changed, 95 insertions(+), 24 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index bfd6e1ae..ff7e6307 100644 --- a/.github/workflows/build.yml +++ b/.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: diff --git a/README-ja_JP.md b/README-ja_JP.md index 932f0850..b8660a24 100644 --- a/README-ja_JP.md +++ b/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 ``` @@ -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 ``` diff --git a/README.md b/README.md index 9cc4046f..6029d600 100644 --- a/README.md +++ b/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 ``` @@ -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 ``` diff --git a/ble.pp b/ble.pp index e2473896..d16cb756 100644 --- a/ble.pp +++ b/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 diff --git a/docs/ChangeLog.md b/docs/ChangeLog.md index b40b1b7d..b4fc5ab5 100644 --- a/docs/ChangeLog.md +++ b/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 diff --git a/note.txt b/note.txt index bc3e78b2..45586a4a 100644 --- a/note.txt +++ b/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 しか設定していない気がする。何れ + にしても最早パス名展開は使わなくなったので気にしなくて良い。 + と思ったがよく見たら昔に展開した結果の始末や 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]