Project

General

Profile

Actions

バグ #1284

closed

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

Added by Izumi Tsutsui about 6 years ago. Updated about 6 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
クラッシュする:
No

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

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

Related issues

Related to 機能 #823: develディレクトリの削除終了ふぇのまー 葱野2016-02-28

Actions
Actions #1

Updated by Izumi Tsutsui about 6 years ago

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

Actions #2

Updated by toshi_a 初音 about 6 years ago

  • Status changed from 分類待ち to パッチ適用待ち
  • Assignee set to toshi_a 初音

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

Actions #3

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しました。そちらでも確認お願いします。

Actions #4

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

Actions #5

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 で削除することが決まっています。

Actions #6

Updated by toshi_a 初音 about 6 years ago

  • Related to 機能 #823: develディレクトリの削除 added
Actions #7

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と実行フラグを取りました。

Actions #8

Updated by Izumi Tsutsui about 6 years ago

  • Status changed from レビュー待ち to マージ待ち

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

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

Actions #9

Updated by toshi_a 初音 about 6 years ago

  • Status changed from マージ待ち to 終了

merged.

Actions

Also available in: Atom PDF