Project

General

Profile

バグ #1284

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

Added by Izumi Tsutsui 10 months ago. Updated 9 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
topic/1284-disable-unexpected-executable-bit
クラッシュする:
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

Associated revisions

Revision a8b3e9ee (diff)
Added by Izumi Tsutsui 10 months ago

executable bit を削除 refs #1284

Revision cac0c1a6 (diff)
Added by toshi_a 初音 10 months ago

不要な実行属性の削除 refs #1284

History

#1

Updated by Izumi Tsutsui 10 months ago

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

#2

Updated by toshi_a 初音 10 months ago

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

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

#3

Updated by toshi_a 初音 10 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Izumi Tsutsui
  • ブランチ set to topic/1284-disable-unexpected-executable-bit

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

#4

Updated by Izumi Tsutsui 10 months 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

#5

Updated by toshi_a 初音 10 months 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 で削除することが決まっています。

#6

Updated by toshi_a 初音 10 months ago

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

Updated by toshi_a 初音 10 months ago

  • Status changed from まだダメ to レビュー待ち
  • Assignee changed from toshi_a 初音 to Izumi Tsutsui

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

#8

Updated by Izumi Tsutsui 10 months ago

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

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

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

#9

Updated by toshi_a 初音 9 months ago

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

merged.

Also available in: Atom PDF