Skip to content

Make it clear that bash is required #47

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

Closed
wants to merge 42 commits into from
Closed

Conversation

xrmx
Copy link
Contributor

@xrmx xrmx commented Jan 12, 2016

The script contains some bashims like echo -e so better make it
explicit.

Signed-off-by: Riccardo Magliocchetti riccardo.magliocchetti@gmail.com

facchinm and others added 30 commits October 22, 2015 17:50
Windows is smart enough to add trailing .exe, so we'll be able to add different commandlines depending on host OS
this should solve problems with slower (or Win based) systems
will be used during installation to trigger a new certificate creation
Add configurable CORS allowed origins
Be sure we have a comma + space separated list
add generateCert commandline switch
Give the correct port for upload
@xrmx
Copy link
Contributor Author

xrmx commented Jan 12, 2016

Should i rebase this to devel?

@mastrolinux
Copy link
Contributor

yes please, make the PR to devel and we will merge it

The script contains some bashims like echo -e so better make it
explicit.

Signed-off-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
@xrmx
Copy link
Contributor Author

xrmx commented Jan 13, 2016

It looks i cannot change target branch of a pull request, closing and opening a new one.

@xrmx xrmx closed this Jan 13, 2016
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.

4 participants