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

[WIP] installerリポジトリ再取り込み(履歴表示用) #68

Closed
wants to merge 3 commits into from

Conversation

kobake
Copy link
Member

@kobake kobake commented Jun 6, 2018

検証中。

変更内容

#60, #62 で取り込んだ installer リポジトリのログ内容がうまく参照できていない
https://github.com/sakura-editor/sakura/commits/master/installer
ので、merge ではなく subtree コマンドにより取り込み直す。

参考 subtree 手順

上記 PR では subtree による取り込みを行っており、履歴も正常に取り込めている。
https://github.com/sakura-editor/sakura/commits/master/help

手順

git fetch github
git checkout -b reimport-installer-with-log github/master

git revert 8bd3214e02277ade57917569b6d82f11714db9a8
git revert -m 1 42e7bcfa81abd9e86f4ba5f5a72603f5cc39b76b

git remote add github-installer git@github.com:sakura-editor/installer.git
git subtree add --prefix=installer/ github-installer master

git push github reimport-installer-with-log

取り込んだ installer のログ確認

berryzplus
berryzplus previously approved these changes Jun 6, 2018
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okだと思います。

ヘルプ系とやり方違うのは気になっていました。
関係ないけどcommitsで見るとreadme.txtが文字化けして見えるのは何故・・・
(ブランチに入った状態でちゃんと読めるのを確認してるので問題ないと思っています。

@kobake
Copy link
Member Author

kobake commented Jun 7, 2018

readme.txtが文字化け

たぶん SJIS だからですね。
最近 GitHub でも SJIS 表示できるようになってきましたが、まだ全画面対応しきってないのかもしれません。

@kobake kobake force-pushed the reimport-installer-with-log branch from 807cdf4 to d8e54e5 Compare June 7, 2018 05:31
@kobake
Copy link
Member Author

kobake commented Jun 7, 2018

うーん、なんかダメっぽい気がしてきました。

ヘルプ群と同じ手順でやっても
https://github.com/sakura-editor/sakura/commits/reimport-installer-with-log/installer
ここのログに過去のログが出ません。

今回は諦める、かもです。

@berryzplus
Copy link
Contributor

前にローカルで実験したときは、ここの手順通りで取り込めたような。

https://git-scm.com/book/ja/v1/Git-%E3%81%AE%E3%81%95%E3%81%BE%E3%81%96%E3%81%BE%E3%81%AA%E3%83%84%E3%83%BC%E3%83%AB-%E3%82%B5%E3%83%96%E3%83%84%E3%83%AA%E3%83%BC%E3%83%9E%E3%83%BC%E3%82%B8
※サブツリーの歴史を取り込まない方法の説明の中に、チラッと書いてある。

gitはほぼ分かっていないので確かなことは言えませんが、公式マニュアルでは取り込み対象リポジトリを別ブランチのルートに取り込んでから作業してるような気がします。ここでやろうとしてる取り込はサブツリー化+階層合わせなのでよりややこしいものだろうとは思ってます。

@kobake
Copy link
Member Author

kobake commented Jul 11, 2018

情報ありがとうございます。
が、今やるには難しそうなので一旦取り下げます。今後の時間あるときに個人領域のほうで実験してみようと思ってます

@kobake kobake closed this Jul 11, 2018
@kobake kobake deleted the reimport-installer-with-log branch July 11, 2018 06:26
@KENCHjp KENCHjp added the CI appveyor など CI 関連 【ChangeLog除外】 label Dec 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants