Skip to content
This repository was archived by the owner on Feb 26, 2020. It is now read-only.

Conversation

w3st3ry
Copy link
Contributor

@w3st3ry w3st3ry commented Aug 23, 2016

What I did

SVFS now works using https://github.com/spf13/viper

How to verify it

Mount container(s) using config file or ENV variables.

Description for changelog

  • Can mount container(s) using config file or ENV variables.

Signed-off-by: Valentin Pichard valentin.pichard@corp.ovh.com

if ARGV[3].nil?
opts = "rw".split(',')
else
opts = ARGV[3].spli(',')
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo : spli(.. -> split(..

@xlucas
Copy link
Contributor

xlucas commented Aug 24, 2016

Your commit contains two Signed-Off statement in your commit message, can you edit it ?

@w3st3ry w3st3ry force-pushed the viper branch 2 times, most recently from 13fc6b7 to 1b50b45 Compare August 24, 2016 20:13
@w3st3ry
Copy link
Contributor Author

w3st3ry commented Aug 24, 2016

Oops, fixed.

Signed-off-by: Valentin Pichard <valentin.pichard@corp.ovh.com>
@xlucas
Copy link
Contributor

xlucas commented Aug 25, 2016

This will fix #3

@xlucas
Copy link
Contributor

xlucas commented Aug 30, 2016

I Did some changes to support all authentication variables and merged. Thanks !

@xlucas xlucas closed this Aug 30, 2016
@xlucas xlucas added this to the v0.8.0 milestone Aug 30, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants