Skip to content

Conversation

@ann0see
Copy link
Member

@ann0see ann0see commented Jul 27, 2023

Short description of changes
This adds documentation for the Debian repository.

Context: Fixes an issue? Related issues

Fixes: #937

Status of this Pull Request
Ready for review

What is missing until this pull request can be merged?

Review and update of the GPG key by Volker on the repo.

Does this need translation?
YES

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I'm sure that this Pull Request goes to the correct branch

@ann0see ann0see added this to the Release 3.10.0 milestone Jul 27, 2023
@ann0see ann0see linked an issue Jul 27, 2023 that may be closed by this pull request
Co-authored-by: Peter L Jones <pljones@users.noreply.github.com>
@ann0see ann0see merged commit 791f966 into jamulussoftware:next-release Jul 29, 2023
@ann0see ann0see deleted the deb/documentDebRepo branch July 29, 2023 10:16
1. Once installed, you can delete the file and close any console windows.

**Upgrades:** If you need to upgrade Jamulus to a newer version, just download the new .deb file and re-install as above.
**Upgrades:** If you need to upgrade Jamulus to a newer version and install Jamulus manually, just download the new .deb file and re-install as above.
Copy link
Contributor

Choose a reason for hiding this comment

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

or install Jamulus manually?

Copy link
Member Author

@ann0see ann0see Jul 29, 2023

Choose a reason for hiding this comment

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

The repository handles automatic upgrades so it should be an and


1. Download the setup script: `curl https://raw.githubusercontent.com/jamulussoftware/jamulus/main/linux/setup_repo.sh > setup_repo.sh`
1. Make the script executable: `chmod +x setup_repo.sh`
1. Run the script and install the headless server: `sudo ./setup_repo.sh && sudo apt install jamulus-headless`
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd prefer to see each command on an indented second line, which I think requires a newline and 4 spaces. A command on a newline is easier to visually understand (in particular, avoiding newlines mid-command) and copy from the browser with a mouse, which is the way most readers will use these.

Copy link
Member Author

Choose a reason for hiding this comment

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

True. Feel free to open a PR to next_release

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I think this looks better:
grafik

Copy link
Contributor

Choose a reason for hiding this comment

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

do we have an option to add a Copy clickable link?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure. I also thought about that, but that's something for research (preferably it would work without JS)

@ann0see ann0see mentioned this pull request Jul 30, 2023
3 tasks
@ann0see
Copy link
Member Author

ann0see commented Aug 5, 2023

In the /linux folder.

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add documentation for Debian repository

3 participants