バグ #1284
完了
いくつかの .rb ファイルに実行属性がついている
説明
git clone した場合、以下のファイルに実行属性が付いているため
pkgsrc のビルド時に警告が出ています。
core/miku/yamlisp.rb: executable bit is set on non-executable file.
core/mui/gtk_crud.rb: executable bit is set on non-executable file.
core/mui/gtk_selectbox.rb: executable bit is set on non-executable file.
core/plugin/extract/edit_window.rb: executable bit is set on non-executable file.
特に不都合はないのですが、 shbang もないようですし、意図せず実行属性が付いているのであれば
chmod 644 としておいたほうがよいのではないかと思います。
ファイル
関連するチケット
Izumi Tsutsui さんが6年以上前に更新
chmod だけの差分で git-format-patch が効くのかどうかわかりませんが一応添付します
0001-executable-bit.patch
toshi_a 初音 さんが6年以上前に更新
- ステータス を 分類待ち から パッチ適用待ち に変更
- 担当者 を toshi_a 初音 にセット
間違ってついているだけなので、mergeしようと思います
toshi_a 初音 さんが6年以上前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Izumi Tsutsui に変更
- ブランチ を topic/1284-disable-unexpected-executable-bit にセット
commitしました。そちらでも確認お願いします。
Izumi Tsutsui さんが6年以上前に更新
- ステータス を レビュー待ち から マージ待ち に変更
git pull して ls で確認してOKです。
以下は shbang あるので executable bit は意図通りということですよね。
mikutter.rb
test.rb
core/lib/piapro.rb
core/miku/miku.rb
devel/makechi.rb
toshi_a 初音 さんが6年以上前に更新
- ステータス を マージ待ち から まだダメ に変更
- 担当者 を Izumi Tsutsui から toshi_a 初音 に変更
- mikutter.rb
- test.rb
- core/miku/miku.rb
これは意図してます。
- core/lib/piapro.rb
これは多分バグなので調査します。
- devel/makechi.rb
このディレクトリの内容は、 #823 で削除することが決まっています。
toshi_a 初音 さんが6年以上前に更新
- ステータス を まだダメ から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Izumi Tsutsui に変更
#823 を解決したため、develディレクトリはなくなりました。piapro.rbも、shebangと実行フラグを取りました。
Izumi Tsutsui さんが6年以上前に更新
- ステータス を レビュー待ち から マージ待ち に変更
再度 git clone/checkout して確認しました。問題ないと思います。
最初から全部チェックしておくべきでした……。
pkgsrc のエラーメッセージで抽出できている分だけと思ってチケットを入れて、
レビュー待ちでの確認で総チェックしたら他にも見つかったけれど、
それらは shbang 付きなので警告が出ていなかった、という流れでした。