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

📝 music server docs improvements #511

Merged
merged 9 commits into from
Nov 11, 2018

Conversation

CenoX
Copy link
Contributor

@CenoX CenoX commented Nov 10, 2018

📝 Bemuse's pack feature is not supported properly by AAC codec on Linux, but still can pack and run. added prerequisites for linux.

📝 I experienced when install bemuse-tools via yarn can cause no command error, added the way how export yarn path.

📝 Current docs describe xampp-apache based local machine, so added nginx based way for macOS and Linux.

@codecov-io
Copy link

codecov-io commented Nov 10, 2018

Codecov Report

Merging #511 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #511   +/-   ##
======================================
  Coverage    84.1%   84.1%           
======================================
  Files         171     171           
  Lines        5387    5387           
  Branches        1       1           
======================================
  Hits         4531    4531           
  Misses        856     856

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f6512e...e0c9214. Read the comment docs.

@dtinth
Copy link
Member

dtinth commented Nov 10, 2018

docs/music-server.md Outdated Show resolved Hide resolved
Co-Authored-By: CenoX <dev@cenox.co>
Copy link
Member

@dtinth dtinth left a comment

Choose a reason for hiding this comment

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

Thanks!!!

Since it is not officially supported yet...
Dropbox deprecated public folders long ago
Then, create a symlink to enable the configured site:

```bash
$ sudo ln -s /etc/nginx/site-available/bemuse site-enabled/bemuse
Copy link
Member

Choose a reason for hiding this comment

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

Are you sure it’s called site-available and site-enabled, and not sites-available and sites-enabled?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh you right, it is site's' available and enabled. I'll commit changes.

📝 Fix typo of linux nginx folder.
📝 Fix typo of open command on linux
@dtinth dtinth merged commit b81a112 into bemusic:master Nov 11, 2018
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.

3 participants