Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Port to Plan 9#604

Closed
lufia wants to merge 3 commits intogit:mastergit/git:masterfrom
lufia:plan9lufia/git:plan9Copy head branch name to clipboard
Closed

Port to Plan 9#604
lufia wants to merge 3 commits intogit:mastergit/git:masterfrom
lufia:plan9lufia/git:plan9Copy head branch name to clipboard

Conversation

@lufia
Copy link
Contributor

@lufia lufia commented Apr 30, 2019

I ported git, and git subcommands only written in C to Plan 9.
This pull request contains patches for existing codes, and new files to build git in Plan 9.

All build options such as NO_PERL are not supported yet.

Whole installing process is published at https://medium.com/@lufia/14fee2ad7493


Thanks for taking the time to contribute to Git! Please be advised that the
Git community does not use github.com for their contributions. Instead, we use
a mailing list (git@vger.kernel.org) for code submissions, code reviews, and
bug reports. Nevertheless, you can use GitGitGadget (https://gitgitgadget.github.io/)
to conveniently send your Pull Requests commits to our mailing list.

Please read the "guidelines for contributing" linked above!

@lufia lufia force-pushed the plan9 branch 3 times, most recently from 56ac0c1 to 6561dda Compare May 3, 2019 11:55
@dscho
Copy link
Member

dscho commented Jun 18, 2019

Wow! Plan 9, eh? How's this effort going?

@lufia
Copy link
Contributor Author

lufia commented Jun 18, 2019

@dscho Thank you.
Git works well on my Plan 9 box.

After replying to reviews for other dependent libraries, such as LibreSSL, I will tidy this pull request and submit it to ML.

@lufia lufia force-pushed the plan9 branch 3 times, most recently from 038bbec to b2dc51f Compare August 5, 2019 17:00
@lufia lufia marked this pull request as ready for review August 15, 2019 15:35
@lufia lufia force-pushed the plan9 branch 6 times, most recently from e3a4dc5 to 055c340 Compare August 24, 2019 04:06
lufia added 3 commits May 31, 2020 21:51
In Plan 9, almost environment variables are not capitalized.

Signed-off-by: lufia <lufia@lufia.org>
Signed-off-by: lufia <lufia@lufia.org>
Plan 9 ANSI/POSIX environment is:
* No expr(1)
* sed(1) limits max length of label to 7 characters
* pcc ignores object files that has incorrect extension,
  so should use underlying loader directly.
* No ln(1). Instead use bind(1), but bind isn't persisted to the disk.
  However it isn't efficient to copy git to git subcommands such as git-add.
  Therefore Plan 9 needs exec-wrapper to switch behavior by executable name.
* tar(1) don't have -o option

Signed-off-by: lufia <lufia@lufia.org>
@lufia
Copy link
Contributor Author

lufia commented Jan 14, 2021

Continuation: gitgitgadget#305

@lufia lufia closed this Jan 14, 2021
@dscho
Copy link
Member

dscho commented Jan 14, 2021

Continuation: gitgitgadget#305

Why a new PR? You lose all of GitGitGadget's convenience like automatically generating the range-diff.

@lufia
Copy link
Contributor Author

lufia commented Jan 14, 2021

This PR is opened on git/git so I think it is used only for testing. Actual PR still be opened on gitgitgadget/git repo.

Is it correct? or is there better way?

@dscho
Copy link
Member

dscho commented Jan 14, 2021

This PR is opened on git/git so I think it is used only for testing.

These days, you can /submit directly from git/git.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.