Merge branch 'hatred_45-master-patch-05095' into 'master'
Update Redox chat link, cleanup old stuff, add "make rebuild" command and update contribute link. See merge request redox-os/redox!1300
This commit is contained in:
commit
fea33aaec5
|
@ -13,7 +13,6 @@ _**Thank you for your interest in contributing to Redox!** This document will ou
|
||||||
* [News](#news)
|
* [News](#news)
|
||||||
* [Code Contributions](#code-contributions)
|
* [Code Contributions](#code-contributions)
|
||||||
* [Low-Hanging Fruit - Easy Targets for Newbies](#easy-targets)
|
* [Low-Hanging Fruit - Easy Targets for Newbies](#easy-targets)
|
||||||
* [Creating a Pull Request](#creating-a-pr)
|
|
||||||
* [Best Practices/Guidelines](#best-practices)
|
* [Best Practices/Guidelines](#best-practices)
|
||||||
* [General](#general)
|
* [General](#general)
|
||||||
* [Kernel](#kernel)
|
* [Kernel](#kernel)
|
||||||
|
@ -35,7 +34,11 @@ _**Thank you for your interest in contributing to Redox!** This document will ou
|
||||||
|
|
||||||
### <a name="chat"> Chat </a>
|
### <a name="chat"> Chat </a>
|
||||||
|
|
||||||
The quickest and most open way to **communicate with the Redox team** is on our **chat server**. Currently, you can only get an invite by sending an email request to [info@redox-os.org](mailto:info@redox-os.org), which might take a little while, since it's not automated. Simply say you'd like to join the chat. We're working on an better way to do this, but this is the best way right now.
|
- [Redox Dev room]
|
||||||
|
- [Redox Support room]
|
||||||
|
|
||||||
|
[Redox Dev room]: https://matrix.to/#/#redox-dev:matrix.org
|
||||||
|
[Redox Support room]: https://matrix.to/#/#redox-support:matrix.org
|
||||||
|
|
||||||
### <a name="issues"> GitLab Issues </a>
|
### <a name="issues"> GitLab Issues </a>
|
||||||
|
|
||||||
|
@ -43,7 +46,7 @@ A bit more formal way of communication with fellow Redox devs, but a little less
|
||||||
|
|
||||||
### <a name="prs"> Pull Requests </a>
|
### <a name="prs"> Pull Requests </a>
|
||||||
|
|
||||||
It's fine to just submit a small pull request without first making an issue or asking in the chat, **unless** it's a significant change that will require a lot of planning and reviewing. Also see [Creating a Pull Request](#creating-a-pr) and [Git Style Guidelines](#git-style-guidelines).
|
[How to make pull requests properly]: https://doc.redox-os.org/book/ch12-04-creating-proper-pull-requests.html
|
||||||
|
|
||||||
### <a name="discourse"> Discourse </a>
|
### <a name="discourse"> Discourse </a>
|
||||||
|
|
||||||
|
@ -81,47 +84,6 @@ News and updates for Redox are posted at [redox-os.org/news](https://redox-os.or
|
||||||
* Grep for `TODO`, `FIXME`, `BUG`, `UNOPTIMIZED`, `REWRITEME`, `DOCME`, and `PRETTYFYME` and fix the code you find.
|
* Grep for `TODO`, `FIXME`, `BUG`, `UNOPTIMIZED`, `REWRITEME`, `DOCME`, and `PRETTYFYME` and fix the code you find.
|
||||||
* **Improve and optimize code, especially in the kernel**
|
* **Improve and optimize code, especially in the kernel**
|
||||||
|
|
||||||
### <a name="creating-a-pr"> Creating a Pull Request </a>
|
|
||||||
|
|
||||||
**1**. _**Fork**_ the repository
|
|
||||||
|
|
||||||
**2**. Clone the _original repository_ to your local PC using one of the following commands based on the protocol you are using:
|
|
||||||
* HTTPS:`git clone https://gitlab.redox-os.org/redox-os/redox.git --origin upstream --recursive`
|
|
||||||
* SSH:`git clone git@gitlab.redox-os.org:redox-os/redox.git --origin upstream --recursive`
|
|
||||||
* Then rebase: `git rebase upstream master`
|
|
||||||
If you use HTTPS, you will have to log in each time when pushing to your fork. (Recommended: learn about git SSH support, it logs in automatically using SSH keys)
|
|
||||||
|
|
||||||
**3**. **Add** your fork with
|
|
||||||
* HTTPS:`git remote add origin https://gitlab.redox-os.org/your-username/redox.git`
|
|
||||||
* SSH:`git remote add origin git@gitlab.redox-os.org:your-username/redox.git`
|
|
||||||
|
|
||||||
**4**. Alternatively, if you already have a fork and copy of the repo, you can simply check to **make sure you're up-to-date**
|
|
||||||
* Pull the upstream:`git pull upstream --rebase`
|
|
||||||
* Update the submodules:`git submodule update --recursive --init`
|
|
||||||
|
|
||||||
**5**. Create a _**separate branch**_ (recommended if you're making multiple changes simultaneously) (`git checkout -b my-branch`)
|
|
||||||
|
|
||||||
**6**. _Make changes_
|
|
||||||
|
|
||||||
**7**. **Commit** (`git add <item(s) you changed>; git commit`) and write your commit message
|
|
||||||
|
|
||||||
**8**. Optionally run [rustfmt](https://github.com/rust-lang-nursery/rustfmt) on the _files you changed_ and commit again if it did anything (check with `git diff` first)
|
|
||||||
|
|
||||||
**9**. Test your changes by **cleaning** (`make clean; git clean -Xfd`) and **building** with `make qemu` (you might have to use `make qemu kvm=no`) or `make virtualbox`.
|
|
||||||
(see [Best Practices and Guidelines](#best-practices))
|
|
||||||
|
|
||||||
**10**. _**Pull**_ from upstream (`git pull upstream --rebase`) (Note: Make sure to include `--rebase`, as it will apply your changes on top of the changes you just pulled, allowing for a much cleaner merge)
|
|
||||||
|
|
||||||
**11**. Repeat step **9** to make sure the rebase still builds and starts
|
|
||||||
|
|
||||||
**12**. Push to **your fork** (`git push origin <branch>`), `<branch>` being the branch you created earlier
|
|
||||||
|
|
||||||
**13**. Create a _pull request_
|
|
||||||
|
|
||||||
**14**. If your changes are _minor_, you can just describe them in a paragraph or less. If they're _major_, please fill out the provided form.
|
|
||||||
|
|
||||||
**15. Submit!**
|
|
||||||
|
|
||||||
## <a name="best-practices"> Best Practices and Guidelines </a>
|
## <a name="best-practices"> Best Practices and Guidelines </a>
|
||||||
|
|
||||||
### <a name="general"> General </a>
|
### <a name="general"> General </a>
|
||||||
|
|
18
README.md
18
README.md
|
@ -78,17 +78,16 @@ The ecosystem and software Redox OS provides is listed below.
|
||||||
Sometimes things go wrong when compiling. Try the following before opening an issue:
|
Sometimes things go wrong when compiling. Try the following before opening an issue:
|
||||||
|
|
||||||
1. Run `rustup update`
|
1. Run `rustup update`
|
||||||
1. Run `make clean pull`.
|
1. Run `make clean pull`
|
||||||
1. Make sure you have **the latest version of Rust nightly!** ([rustup.rs](https://www.rustup.rs) is recommended for managing Rust versions. If you already have it, run `rustup`).
|
1. Make sure you have **the latest version of Rust nightly!** ([rustup.rs](https://www.rustup.rs) is recommended for managing Rust versions. If you already have it, run `rustup`).
|
||||||
1. Update **GNU Make**, **NASM** and **QEMU/VirtualBox**.
|
1. Update **GNU Make**, **NASM** and **QEMU/VirtualBox**.
|
||||||
1. Pull the upstream master branch (`git remote add upstream git@gitlab.redox-os.org:redox-os/redox.git; git pull upstream master`).
|
1. Pull the upstream master branch (`git remote add upstream git@gitlab.redox-os.org:redox-os/redox.git; git pull upstream master`).
|
||||||
1. Update submodules (`git submodule update --recursive --init`).
|
1. Update submodules (`git submodule update --recursive --init`).
|
||||||
|
1. Rebuild with `make rebuild`
|
||||||
and then rebuild!
|
|
||||||
|
|
||||||
## <a name="contributing"> Contributing to Redox </a>
|
## <a name="contributing"> Contributing to Redox </a>
|
||||||
|
|
||||||
If you're interested in this project, and you'd like to help us out, [here](CONTRIBUTING.md) is a list of ways you can do just that.
|
If you're interested in this project, and you'd like to help us out, [here](https://doc.redox-os.org/book/ch10-02-low-hanging-fruit.html) is a list of ways you can do just that.
|
||||||
|
|
||||||
## <a name="cloning-building-running"> Cloning, Building and Running </a>
|
## <a name="cloning-building-running"> Cloning, Building and Running </a>
|
||||||
|
|
||||||
|
@ -162,11 +161,14 @@ make qemu kvm=no vga=no
|
||||||
### <a name="setup-using-docker"> Setup using Docker </a>
|
### <a name="setup-using-docker"> Setup using Docker </a>
|
||||||
We also provide docker image. After cloning this repository, please follow README under the `docker` directory.
|
We also provide docker image. After cloning this repository, please follow README under the `docker` directory.
|
||||||
|
|
||||||
### Updating the codebase using the Makefile
|
### Update the codebase using the Makefile
|
||||||
To update the codebase run:
|
To update the codebase run:
|
||||||
|
|
||||||
```
|
`make pull; make fetch`
|
||||||
make pull; make fetch
|
|
||||||
```
|
|
||||||
|
|
||||||
`make pull` pulls and updates the submodules, and `make fetch` updates the sources for cookbook recipes.
|
`make pull` pulls and updates the submodules, and `make fetch` updates the sources for cookbook recipes.
|
||||||
|
|
||||||
|
### Update the codebase and compile new changes
|
||||||
|
Run:
|
||||||
|
|
||||||
|
`make rebuild`
|
||||||
|
|
Loading…
Reference in a new issue