OpenPNE 3

更新履歴 b0284f83bd4f9631f7e54033bfe76ab20d857d25 で適用されました。

レビューが通っていますが、以下の点について確認してください。修正する必要がないと判断された場合はテスト待ちにしていただいて結構です。

指摘内容¶

• note-2 の修正後の記述
  

  CentOS や Ubuntu など、-uオプションがある場合。
  
    $ cp -au /path/to/old/OpenPNE3/plugins/. /path/to/new/OpenPNE3/plugins
  
  MacOSX や FreeBSD など、-nオプションがある場合。
  
    $ cp -an /path/to/old/OpenPNE3/plugins/. /path/to/new/OpenPNE3/plugins
  
  * どちらのコマンドを実行可能かについては「man cp」コマンドを実行してチェックしてください。
  

• 手元の GNU coreutils に含まれる cp コマンド
  

  $ gcp --version
  cp (GNU coreutils) 8.10
  Copyright (C) 2011 Free Software Foundation, Inc.
  ライセンス GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>.
  
  $ man gcp
  
    -n, --no-clobber
        do not overwrite an existing file (overrides a previous -i option)
  
    -u, --update
        copy only when the SOURCE file is newer than the destination file or when the destination file is missing
  

cp コマンドが -n オプションと -u オプションの両方を含む場合があり、その場合は冒頭に示している文章ではどちらを実行すべきか判断できません。

• このコマンドを実行することで行いたい処理についての説明を加える
• -n と -u オプションの意味を踏まえた上でどちらを実行すべきか優先度を付けて手順を示す（ -u が実行できない場合は -n を試してください、のように記述する）
  • 併せて -n と -u オプションが何を意味するものなのかについての説明を記述したほうが親切かもしれません。

このいずれかの対応を行うべきではないでしょうか。

また、 -n と -u のオプションが両方無い場合について考慮されていませんが、もし考慮するのであれば、上記に示した対応のうち、前者の対応を行うことが好ましいのではないでしょうか。

とはいっても、元の文章は、どちらかさえ実行できれば良いとも読めるのであまり問題ではないかもしれません。

考えられる修正¶

次のような文章にするだけでもよいかもしれません。

• 案1
  

  CentOS や Ubuntu など、 cp コマンドに -u オプションがある場合。
  
    $ cp -au /path/to/old/OpenPNE3/plugins/. /path/to/new/OpenPNE3/plugins
  
  MacOSX や FreeBSD など、 cp コマンドに -u オプションがなく -n オプションがある場合。
  
    $ cp -an /path/to/old/OpenPNE3/plugins/. /path/to/new/OpenPNE3/plugins
  
  * cp コマンドで使用可能なオプションについては「man cp」コマンドを実行して確認してください。
  

• 案2
  

    $ cp -au /path/to/old/OpenPNE3/plugins/. /path/to/new/OpenPNE3/plugins
  
  cp コマンドに -u オプションがない場合は代わりに -n オプションを用いてください。
  
    $ cp -an /path/to/old/OpenPNE3/plugins/. /path/to/new/OpenPNE3/plugins
  
  * cp コマンドで使用可能なオプションについては「man cp」コマンドを実行して確認してください。
  

• このコマンドを実行することで行いたい処理についての説明を加える
• -n と -u オプションの意味を踏まえた上でどちらを実行すべきか優先度を付けて手順を示す（ -u が実行できない場合は -n を試してください、のように記述する）
  • 併せて -n と -u オプションが何を意味するものなのかについての説明を記述したほうが親切かもしれません。

提案内容について両方共もっともであると感じたため両方共を考慮した記述を検討します．

更新履歴 952350437f455c6d436605c63f8aa1f96c042f2c で適用されました。

テスト無事終了しました。