Add https port to gen_dev and rebar overlay #416

Open
wants to merge 1 commit into
from

Projects

None yet

4 participants

@andrelaszlo

Changing the https port manually for all my devs each rebuild was
getting annoying, so I added a new variable https_port to
dev_vars.config and changed gen_dev accordingly. For consistency, I
also added the variable to vars.config.

@andrelaszlo andrelaszlo Add https port to gen_dev and rebar overlay
Changing the https port manually for all my devs each rebuild was
getting annoying, so I added a new variable `https_port` to
dev_vars.config and changed gen_dev accordingly. For consistency, I
also added the variable to vars.config.
faf7a7c
@jaredmorrow
Contributor

We are in the middle of our 2.0 development / test cycle where our config is going to change dramatically from having an app.config to having a generated config based around a simple configuration file. Because of this, we probably won't merge this PR.

In addition, it will probably be best to use your branch as we have changed our development branch from 'master' to 'develop', so we aren't merging PR's into 'master' for now.

I'll /cc @joedevivo (who is the primary person reworking config) to make sure he knows to look out for this in 2.0.

Thanks for the PR, sorry it is not going to work with timing for us at the moment.

@andrelaszlo

@jaredmorrow No problem, just thought I should share :) I noticed that you changed the whole config system in 2.0 but I didn't know if you were still adding stuff to the 1.4 branch. Looking forward to see the final result!

@jaredmorrow
Contributor

We are possibly releasing another rev on 1.4, but sadly 2.0 is keeping me too busy to give this a good look at the moment.

@slfritchie
Contributor

Howdy. I don't believe that this kind of feature will be added to the 1.4 branch, and as Jared said, the Cuttlefish-based configuration scheme for Riak 2.0 changes your problem. (Hopefully in a good way!)

@andrelaszlo

Cuttlefish looks really promising, looking forward to seeing it in action. Thank you both for having a look!

@jaredmorrow
Contributor

I don't think the presence of cuttlefish affects this issue at all. An https port still needs to be set as far as I know. Handing this off to @joedevivo so he can make a call.

@jaredmorrow jaredmorrow added this to the 2.0.1 milestone Mar 24, 2014
@joedevivo joedevivo was unassigned by coderoshi Jul 7, 2014
@andrelaszlo

🔔 ping 🔔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment