Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

diff.exe を取り込みたい。 #477

Open
KENCHjp opened this issue Sep 24, 2018 · 8 comments
Open

diff.exe を取り込みたい。 #477

KENCHjp opened this issue Sep 24, 2018 · 8 comments
Assignees
Labels
enhancement ■機能追加 installer installer 関連

Comments

@KENCHjp
Copy link
Member

KENCHjp commented Sep 24, 2018

diffはよく使います。

ヘルプに記載のダウンロード先がちょっと怪しい感じなので取り込みたいです。
手っ取り早くインストーラーにdiff.exeを包含する(ctags.exe方式)でもいいかと思ってますが、
(ちなみに、diff.exeはGPL2みたいですね。exe内包しても伝搬するんでしたっけ、GPL・・・)
もしC++のえらい人にかかると、diffのライブラリ(?)みたいなのから、エディタ内に実装できたりしますでしょうか?

sakuraの機能でファイル内容比較ってのがありますが、この機能がそのままdiff形式になってくれるのが希望です。

@berryzplus
Copy link
Contributor

GPLアプリとしては既にcppcheckが取り込まれています。(ビルド用ツールですが。)

diff.exeをexeのまま取り込む分には何の問題もないです。
改変してサクラエディタ仕様にしたいだと話変わってきますが、そういう需要はない認識です。

最近のGPLは感染力弱まってて、
ソースコードを取り込まない限りパンデミックしないらしいです。

sakuraの機能でファイル内容比較ってのがありますが、この機能がそのままdiff形式になってくれるのが希望です。

ファイル内容比較は確か、完全独自実装だった気がします。

個人的には差分比較は WinMerge を主に使っていて、
WinMergeのテキストビューア設定にサクラエディタをいれて連携させています。
アプリの得意分野を活かした連携をさせてるというか・・・

@k-takata
Copy link
Member

もしC++のえらい人にかかると、diffのライブラリ(?)みたいなのから、エディタ内に実装できたりしますでしょうか?

LibXDiff というものがあって Git にも使われており、この前 Vim にも取り込まれました。ただしLGPLですが。

@KENCHjp
Copy link
Member Author

KENCHjp commented Sep 24, 2018

超個人的には、いろんな先人のものを組み合わせて"要件"をみたせれば十分なのですが、
平成の次の人たちは、「はいどぞ」って時に揃ってないと、見向きもしてくれなかったりするので、
内包してくれると、うれしいっす・・・

@m-tmatma
Copy link
Member

こんなの発見
koron/vim-kaoriya#10

@k-takata
Copy link
Member

そこで話題になっている https://github.com/koron/diffutilsmsvc/master ブランチでは、diff 2.7 (かなり古い)がVC2010でコンパイルが通るようになっています。ビルド済みのdiff.exeは最新の香り屋Vimに同梱されていますが、今のところ単体配布はされてないようですね。

@KENCHjp KENCHjp added the enhancement ■機能追加 label Sep 25, 2018
@KageShiron
Copy link
Member

ヘルプ( http://sakura-editor.sourceforge.net/htmlhelp/HLP000251.html )で推奨されているW32Texにあるdiff.exeとhttps://github.com/koron/diffutils では何か異なるんですかね?

@berryzplus
Copy link
Contributor

直前のこの文章が関係あるのでは?

GNU DIFF 2.5/2.7が必要です.WinCVS 1.2付属のdiffはパス中のスペースを正しく扱えないので使わないでください.

古いユーティリティは C:\Documents and Settings\ みたいな空白を含むパスをうまく扱えないです。
古いユーティリティは ASCII 前提で作られてるので、マルチバイト環境で使うとダメ文字問題などの分かりづらい不具合が発生します。現代の diffutils がマルチバイトに対応しとらんとも思えんのですが、utix系ツールの日本向けカスタム版が作られてる経緯はだいたいそんな感じだと思います。

カスタム版を作る場合、かならず動機があるはずです。
・・・readmeには「本家より速いっす!」としか書いてないですね。

う~む。

@KageShiron
Copy link
Member

とりあえず、W32Texのものをctagsと同様にpostBuild.batで埋め込む方針で作業中です。
postBuild.batの中身もついでにリファクタリング

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ■機能追加 installer installer 関連
Projects
None yet
Development

No branches or pull requests

5 participants