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

Added noninteractive for apt-get installs #18

Closed
wants to merge 5 commits into from

Conversation

BlackthornYugen
Copy link
Contributor

This prevents a lot of the error messages during install that begin with "debconf: unable to initialize frontend: Dialog".
screen shot 2015-05-10 at 1 30 43 pm
This likely doesn't make any functional difference but it would make it easier to spot actual problems during a docker build.

BlackthornYugen and others added 5 commits May 10, 2015 14:03
This prevents a lot of the error messages during install that begin with "debconf: unable to initialize frontend: Dialog". This likely doesn't make any functional difference but it would make it easier to spot actual problems during a docker build.
Changed the repo name
Changed nginx site config to redirect http to https and to serve https
traffic. In this scenario ssl key and cert could be stored in the
gitignored data folder.
@fideloper
Copy link
Owner

Howdy!
Looks like a lot of your commits are coming through here, some specific to your repository instead of this one!

(However I'd love to pull in the noninteractive bit!)

@BlackthornYugen
Copy link
Contributor Author

Sorry about that, this was my first cross-repo pull request. I guess I did something wrong. Let me try again.

@fideloper
Copy link
Owner

No problem! (I think it keeps adding commits as you make them to the branch
used for the PR, I forget if there's a way to stop that other than making a
branch specifically to PR from)

On Mon, May 11, 2015 at 7:09 AM John Steel notifications@github.com wrote:

Sorry about that, this was my first cross-repo pull request. I guess I did
something wrong. Let me try again.


Reply to this email directly or view it on GitHub
#18 (comment)
.

@BlackthornYugen
Copy link
Contributor Author

Right, thats what it looks like. I wonder if I do that and then deleted the pulled branch, would it mess anything up?

@fideloper
Copy link
Owner

I'm ~90% sure that'll work fine

@BlackthornYugen
Copy link
Contributor Author

Yup! #19 still shows that it was pulled from the branch even after it was deleted. I thought maybe it might say that it pulled from master or possibly stop tracking the history after deleting the branch.

@fideloper
Copy link
Owner

Glad it didn't, github is pretty snazzy !

On Mon, May 11, 2015 at 10:18 AM John Steel notifications@github.com
wrote:

Yup! #19 #19 still
shows that it was pulled from the branch even after it was deleted. I
thought maybe it might say that it pulled from master or possibly stop
tracking the history after deleting the branch.


Reply to this email directly or view it on GitHub
#18 (comment)
.

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