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

Init script documentation/repo/deployed don't match #7

Closed
amcewen opened this issue Jun 12, 2015 · 2 comments
Closed

Init script documentation/repo/deployed don't match #7

amcewen opened this issue Jun 12, 2015 · 2 comments

Comments

@amcewen
Copy link
Member

amcewen commented Jun 12, 2015

The readme.md lists "Copy the startup script into /etc/init" in the "To be done" section.

The remote-setup.rb script includes the line:
ssh.exec!("sudo cp /home/pi/logcards/init/logcards /etc/init.d/") do |channel, stream, data|
which looks like it's copying the script across.

However, none of the doorbots up and running have the logcards script in /etc/init.d

It looks like that's been deprecated, and some script called startup-rfid.sh - which doesn't seem to be checked in anywhere? - is in use instead?

@johnmckerrell
Copy link
Member

It's quite possible that none of the live doorbots are built from the setup script, if any it will just be doorbot3 but I don't think that one is, so this would be a new rather than old step.

On 12 Jun 2015, at 10:43, Adrian McEwen notifications@github.com wrote:

The readme.md lists "Copy the startup script into /etc/init" in the "To be done" section.

The remote-setup.rb script includes the line:
ssh.exec!("sudo cp /home/pi/logcards/init/logcards /etc/init.d/") do |channel, stream, data|
which looks like it's copying the script across.

However, none of the doorbots up and running have the logcards script in /etc/init.d

It looks like that's been deprecated, and some script called startup-rfid.sh - which doesn't seem to be checked in anywhere? - is in use instead?


Reply to this email directly or view it on GitHub #7.

@amcewen
Copy link
Member Author

amcewen commented Jun 16, 2015

The README and code now match, and I've run through it on the new Doorbot1. I guess we can leave Doorbot2 (and 3) for now, and see how reliable Doorbot1 is, but it would be good to update them in the medium-term.

@amcewen amcewen closed this as completed Jun 16, 2015
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

No branches or pull requests

2 participants