Review: Approve Looks good to me
Diff comments: > diff --git a/Makefile b/Makefile > index 9d48829..22955cf 100644 > --- a/Makefile > +++ b/Makefile > @@ -1,10 +1,22 @@ > #!/usr/bin/make > PYTHON := /usr/bin/python3 > export PYTHONPATH := hooks > +PROJECTPATH = $(dir $(realpath $(MAKEFILE_LIST))) > +CHARM_NAME = $(notdir $(PROJECTPATH:%/=%)) > +ifndef CHARM_BUILD_DIR > + CHARM_BUILD_DIR := /tmp/$(CHARM_NAME)-builds > + $(warning Warning CHARM_BUILD_DIR was not set, defaulting to > $(CHARM_BUILD_DIR)) > +endif I don't think we should avoid this functionality due to lack of current standard. If a standard is added later updating this would be easier than adding it fresh. > > default: > echo Nothing to do > > +build: > + @echo "Building charm to base directory > $(CHARM_BUILD_DIR)/$(CHARM_NAME)" > + @-git describe --tags > ./repo-info > + @mkdir -p $(CHARM_BUILD_DIR)/$(CHARM_NAME) > + @cp -r * $(CHARM_BUILD_DIR)/$(CHARM_NAME) > + > # Primitive test runner. Someone please fix this. > test: > tests/00-setup -- https://code.launchpad.net/~addyess/charm-nagios/+git/charm-nagios/+merge/387090 Your team Nagios Charm developers is subscribed to branch charm-nagios:master. -- Mailing list: https://launchpad.net/~nagios-charmers Post to : [email protected] Unsubscribe : https://launchpad.net/~nagios-charmers More help : https://help.launchpad.net/ListHelp
