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

Bluetooth PAN and Terminal servers #1716

Merged
merged 10 commits into from Jun 15, 2019
Merged

Bluetooth PAN and Terminal servers #1716

merged 10 commits into from Jun 15, 2019

Conversation

tim-moody
Copy link
Contributor

No description provided.

@holta holta added this to the 7.0 milestone Jun 2, 2019
@tim-moody
Copy link
Contributor Author

passed smoke test on non-rpi
bt-term not working on rpi

@tim-moody
Copy link
Contributor Author

passed smoke test on rpi
win 10 can connect to pan

@holta holta requested a review from georgejhunt June 11, 2019 20:30
@jvonau
Copy link
Contributor

jvonau commented Jun 12, 2019

To ensure updates don't undue the change please copy the Bluetooth service file to /etc/systemd/system/ and edit that copy.

@tim-moody
Copy link
Contributor Author

good point. thanks for catching it.

@georgejhunt
Copy link
Contributor

What a pain to have to add "| bool" to all our flags. Some are missing in this PR

@tim-moody
Copy link
Contributor Author

It's my impression that when there is a Boolean operation the cast is not required. so when: is_rpi and bluetooth_install but when: is_rpi |bool

@georgejhunt
Copy link
Contributor

georgejhunt commented Jun 12, 2019 via email

Copy link
Contributor

@jvonau jvonau left a comment

Choose a reason for hiding this comment

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

Let's go with it..

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

Successfully merging this pull request may close these issues.

None yet

4 participants