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

LICENSE #5

Closed
zieone opened this issue Sep 26, 2018 · 2 comments
Closed

LICENSE #5

zieone opened this issue Sep 26, 2018 · 2 comments

Comments

@zieone
Copy link

zieone commented Sep 26, 2018

What license is this code under?

I want to make it work on the BSD's via standard POSIX shell, instead of the unsafe bash that it is now.

@GordStephen
Copy link
Owner

First, let me say it would be great to have this properly POSIX compliant!

Good question about the license, I'll make it BSD three-clause.

@zieone
Copy link
Author

zieone commented Sep 27, 2018

Awesome. Without a license, I didn't want to make any changes to the code, as I didn't want to break any laws. Next time I get around to messing with my websites again I'll play with making this POSIX compliant.

May I ask why you picked YAML for config, when a simple shell sourced script with variables would have been way easier and less error-prone(since you don't need all those lines to drag config out of a YAML file..)

The config stuff as is doesn't work properly under BSD, since sed behaves differently and you get ' default' instead of 'default' (i.e. a space at the beginning). that's what got me upset with this code ;)

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