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

Development mode #310

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Development mode #310

wants to merge 1 commit into from

Conversation

alangecker
Copy link
Collaborator

following #306 this implements a way for an instant dev setup with auto reloading for the html5 server/client 🙂

See instructions:
https://github.com/bigbluebutton/docker/blob/dev-mode/docs/development.md

Comment on lines +29 to +48
export BBB_DOCKER_USER="$(id -u):$(id -g)"

# also add it to ~/.zshrc and/or ~/.bashrc so
# that people can also use commands like `docker compose up`
# without that variable being missing
function add_permanent_env {
STR='export BBB_DOCKER_USER="$(id -u):$(id -g)"'
if [ -z "$(grep "$STR" "$1")" ]; then
echo "append"
echo "" >> $1
echo "# following line got added by bbb-docker" >> $1
echo "$STR" >> $1
fi
}
if [ -f "$(realpath ~/.zshrc)" ]; then
add_permanent_env "$(realpath ~/.zshrc)"
fi
if [ -f "$(realpath ~/.bashrc)" ]; then
add_permanent_env "$(realpath ~/.bashrc)"
fi

Choose a reason for hiding this comment

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

Modifying user shell config files without his explicit consent may be considered offensive.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good point! I'll add a prompt asking for consent :)

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