バグ #1284
closedいくつかの .rb ファイルに実行属性がついている
Description
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 としておいたほうがよいのではないかと思います。
Files
Related issues
Updated by Izumi Tsutsui about 6 years ago
chmod だけの差分で git-format-patch が効くのかどうかわかりませんが一応添付します
0001-executable-bit.patch
Updated by toshi_a 初音 about 6 years ago
- Status changed from 分類待ち to パッチ適用待ち
- Assignee set to toshi_a 初音
間違ってついているだけなので、mergeしようと思います
Updated by toshi_a 初音 about 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee changed from toshi_a 初音 to Izumi Tsutsui
- ブランチ set to topic/1284-disable-unexpected-executable-bit
commitしました。そちらでも確認お願いします。
Updated by Izumi Tsutsui about 6 years ago
- Status changed from レビュー待ち to マージ待ち
git pull して ls で確認してOKです。
以下は shbang あるので executable bit は意図通りということですよね。
mikutter.rb
test.rb
core/lib/piapro.rb
core/miku/miku.rb
devel/makechi.rb
Updated by toshi_a 初音 about 6 years ago
- Status changed from マージ待ち to まだダメ
- Assignee changed from Izumi Tsutsui to toshi_a 初音
- mikutter.rb
- test.rb
- core/miku/miku.rb
これは意図してます。
- core/lib/piapro.rb
これは多分バグなので調査します。
- devel/makechi.rb
このディレクトリの内容は、 #823 で削除することが決まっています。
Updated by toshi_a 初音 about 6 years ago
- Related to 機能 #823: develディレクトリの削除 added
Updated by toshi_a 初音 about 6 years ago
- Status changed from まだダメ to レビュー待ち
- Assignee changed from toshi_a 初音 to Izumi Tsutsui
#823 を解決したため、develディレクトリはなくなりました。piapro.rbも、shebangと実行フラグを取りました。
Updated by Izumi Tsutsui about 6 years ago
- Status changed from レビュー待ち to マージ待ち
再度 git clone/checkout して確認しました。問題ないと思います。
最初から全部チェックしておくべきでした……。
pkgsrc のエラーメッセージで抽出できている分だけと思ってチケットを入れて、
レビュー待ちでの確認で総チェックしたら他にも見つかったけれど、
それらは shbang 付きなので警告が出ていなかった、という流れでした。