プロジェクト

全般

プロフィール

バグ #1284

完了

いくつかの .rb ファイルに実行属性がついている

Izumi Tsutsui さんが5年以上前に追加. 5年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
クラッシュする:
いいえ

説明

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 としておいたほうがよいのではないかと思います。


ファイル

0001-executable-bit.patch (1.14 KB) 0001-executable-bit.patch Izumi Tsutsui, 2018-07-28 18:31

関連するチケット

関連している 機能 #823: develディレクトリの削除終了ふぇのまー 葱野2016-02-28

操作

Izumi Tsutsui さんが5年以上前に更新

chmod だけの差分で git-format-patch が効くのかどうかわかりませんが一応添付します
0001-executable-bit.patch

toshi_a 初音 さんが5年以上前に更新

  • ステータス分類待ち から パッチ適用待ち に変更
  • 担当者toshi_a 初音 にセット

間違ってついているだけなので、mergeしようと思います

toshi_a 初音 さんが5年以上前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Izumi Tsutsui に変更
  • ブランチtopic/1284-disable-unexpected-executable-bit にセット

commitしました。そちらでも確認お願いします。

Izumi Tsutsui さんが5年以上前に更新

  • ステータスレビュー待ち から マージ待ち に変更

git pull して ls で確認してOKです。

以下は shbang あるので executable bit は意図通りということですよね。
mikutter.rb
test.rb
core/lib/piapro.rb
core/miku/miku.rb
devel/makechi.rb

toshi_a 初音 さんが5年以上前に更新

  • ステータスマージ待ち から まだダメ に変更
  • 担当者Izumi Tsutsui から toshi_a 初音 に変更
  • mikutter.rb
  • test.rb
  • core/miku/miku.rb

これは意図してます。

  • core/lib/piapro.rb

これは多分バグなので調査します。

  • devel/makechi.rb

このディレクトリの内容は、 #823 で削除することが決まっています。

toshi_a 初音 さんが5年以上前に更新

  • 関連している 機能 #823: develディレクトリの削除 を追加

toshi_a 初音 さんが5年以上前に更新

  • ステータスまだダメ から レビュー待ち に変更
  • 担当者toshi_a 初音 から Izumi Tsutsui に変更

#823 を解決したため、develディレクトリはなくなりました。piapro.rbも、shebangと実行フラグを取りました。

Izumi Tsutsui さんが5年以上前に更新

  • ステータスレビュー待ち から マージ待ち に変更

再度 git clone/checkout して確認しました。問題ないと思います。
最初から全部チェックしておくべきでした……。

pkgsrc のエラーメッセージで抽出できている分だけと思ってチケットを入れて、
レビュー待ちでの確認で総チェックしたら他にも見つかったけれど、
それらは shbang 付きなので警告が出ていなかった、という流れでした。

toshi_a 初音 さんが5年以上前に更新

  • ステータスマージ待ち から 終了 に変更

merged.

他の形式にエクスポート: Atom PDF