mvhub-dev team mailing list archive

[Dan MacNeil <dan@xxxxxxxxxx>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


> ConfigSimple opens a config file, it'll create it if
> it doesn't already exist. I tested it when I originally
> wrote that functionality.
	
Sadly no.

# steps to reproduce:

	# mv_set_active trunk
	cdw
	mv_setup acarvalh
	sudo more /var/www/mvhub/acarvalh/conf/nsp.conf

This branch also has fixes for:

	--keys without values in template.conf
	--keys removed in template.conf removed from nsp.conf

> If you run mv_setup lgoodrich, it does some actions,
> but if you run sudo mv_setup lgoodrich, it does additional actions.

If you aren't lgoodrich, it tells you

	need to run under sudo if you aren't lgoodrich

IMNSHO it isn't completely wacko to expect to need sudo to configure
apache or dns, or that configuring dns & apache are one shot deals.

> Why not add a simple message stating that apache and
> dns configuration is being skipped and to run the
> command via sudo if you wish those actions to be performed?

Right now the sudo functionality is 90% used by me to create new
developer setups.

The 2nd use case is pushing to release, I've not thought about this, but
I'd guess we'd both remember to run sudo

The 3rd and by far the most common use case is people running mv_setup
to make dir , db and template file changes to their developer setup so
trunk works.

Here the:

	 "re-run with sudo to configure apache"

..is just noise

The solution may be to move subs to a library and create:

	mv_release
	mv_install
	mv_update_dev
	mv_setup_dev
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkvM63AACgkQLzI3mETyffy7VgCglDRsQZ7aHkM+mzOnniU0JBRv
XfwAnjnZrpP469ChAf15U8gd2aPa41c9
=WwO3
-----END PGP SIGNATURE-----
-- 
https://code.launchpad.net/~omacneil/mvhub/improve_brave_add_dev_2010-04-16/+merge/23633
Your team mvhub-dev is subscribed to branch lp:mvhub.