2017-02-26 21:30:46 +00:00
+++
title = "Submitting the Package"
+++
2017-02-10 17:12:23 +00:00
# Submitting the Package
2017-01-05 02:56:10 +00:00
2017-02-10 17:12:23 +00:00
This article will walk you through submitting a patch for a package for review and inclusion in the Solus repository.
2017-01-05 02:56:10 +00:00
2017-05-06 18:45:04 +00:00
When contributing patches to Solus, please note that we do not accept the upload of binary files (i.e. `.eopkg` ).
All repository packages are built by a dedicated build server, from the source files provided in your patch upload.
2017-01-05 02:56:10 +00:00
2017-05-06 18:45:04 +00:00
At minimum, your patch will include changes for the following:
* `package.yml`
* `pspec_*.xml`
If any additional files were required for the build, then you must also include the `files/` directory.
If you are creating a new package, you will also need to include a `Makefile` containing the following text:
```
include ../Makefile.common
```
Lastly, many package builds may result in the generation of an ABI report. These files start with `abi_*` and must also
be included, as they allow simple tracking of changes to symbols and dependencies.
For all patch submissions you must be using the `arcanist` utility to communicate with the [Solus Developer Portal ](https://dev.solus-project.com/ )
## Setting up Arcanist
In three easy steps, you can set up arcanist for the first time:
2017-05-14 07:56:41 +00:00
``` bash
2017-05-06 18:45:04 +00:00
sudo eopkg it arcanist
arc set-config default https://dev.solus-project.com
arc install-certificate
```
On the third step you will be given a unique link to log into the Developer Portal, to create a `Conduit API Token` . This
token will be used to allow the CLI `arc` utility to communicate with Phabricator.
## Creating the patch
2017-05-14 07:56:41 +00:00
For every file you change or add, you must let git know about them: `git add someFile`
2017-05-06 18:45:04 +00:00
2017-05-14 07:56:41 +00:00
For files that must be removed, you must do so using git: `git rm someFile`
2017-05-06 18:45:04 +00:00
2017-05-14 07:56:41 +00:00
Likewise, for renaming a file, you must do so via git: `git mv someFile someFileName2`
2017-05-06 18:45:04 +00:00
Once you're happy with your change, and you have verified locally that it works by having first built and
2017-05-14 07:56:41 +00:00
installed it, it's time to commit your changes with `git commit` .
2017-05-06 18:45:04 +00:00
Make sure you provide a meaningful summary and a separate body to your commit message. For more information
on suitable commit messages, please check the [tooling central documentation ](https://github.com/solus-project/tooling-central/blob/master/README.rst#using-git ).
2017-05-14 07:56:41 +00:00
- If you want to link this patch to an issue on the Developer portal, simply mention it in your commit message: `The inclusion of <somepackage> fixes T1234`
- If you need a change to depend on another change, mention it in the commit message too: `Depends on D5`
2017-05-06 18:45:04 +00:00
2017-05-14 07:56:41 +00:00
Now you have your git commit, it's time to send it to us for review. Using the CLI again, simply issue: `arc diff`
2017-05-06 18:45:04 +00:00
2017-05-14 07:56:41 +00:00
A new editor session will open, where you can provide optional details. Note that the default reviewer will be assigned after you submit, so it is not necessary to specify anyone here. Once you're finished, save and exit the editor (`CTRL+O` + `CTRL+X` for nano), and the patch will then be uploaded. You'll be presented with the Differential URL, and a review will happen as soon as possible.
2017-05-06 18:45:04 +00:00
2017-05-14 07:53:08 +00:00
## Updating a patch that needs changes
### Updating files
2017-05-06 18:45:04 +00:00
That's easy. Don't make a new commit, just make any relevant changes to your local tree, adding + removing as
2017-05-14 07:56:41 +00:00
before, but this time run: `git commit --amend`
2017-05-06 18:45:04 +00:00
2017-05-06 23:29:07 +00:00
This will amend your original changes, and you can submit the patch once more with `arc diff` .
A new editor session will open, where you can provide details about the changes you've made between the last
revision and the newly amended one. This comment will help reviewers to see what you've changed, to streamline
the process of getting your patch into Solus.
The web UI will automatically update with the latest patch, without having to create any new tasks. Once accepted, your patch
2017-05-06 18:45:04 +00:00
will be merged, and a build will be issued.
2017-01-05 02:56:10 +00:00
2017-05-14 07:53:08 +00:00
### Updating Task Information
If you forgot information such as a test plan, you can run resolve this by:
1. Running `arc diff --edit` . This will open up an editor session, where you can edit the information.
2. Next, you will see an editor session for updating the commit message. You can choose a new one or opt for the same one.
After this editor session, the updated patch will be automatically uploaded, the web UI will automatically update, and no new tasks will be created.
2017-01-05 02:56:10 +00:00
2017-02-10 17:12:23 +00:00
## Maintainership
2017-01-05 02:56:10 +00:00
2017-02-10 17:12:23 +00:00
Submission directly to a repository is only possible for maintainers. As a maintainer you may freely push to your package(s) and initiate builds for them, which will be pushed to the unstable repository. You can watch
builds [here ](https://build.solus-project.com/ ).
2017-01-05 02:56:10 +00:00
Pushing changes is not possible unless you have maintainer access. The same is also true of `make publish` .
2017-02-10 17:12:23 +00:00
To request maintainer rights for a repository, it is expected that some level of contribution/maintenance has already happened by way of testing/patching, and there is reasonable trust demonstrated to "hand the keys"
over to a repository.
2017-01-05 02:56:10 +00:00
2017-02-10 14:56:35 +00:00
Currently, the request mechanism [contact Ikey on IRC ](/articles/contributing/getting-involved/en ). It is far easier to grant access to active community members than those unknown to the project.
2017-01-05 02:56:10 +00:00
2017-05-06 18:45:04 +00:00
Finally, note that the management reserve the right to revoke access at any time, in order to preserve project safety and integrity.